Sword Setup

[odd bobcat]

I need some advice on my sword setup.
Which is better for scalability? And security.
Cuz idk if int value objects in replicated can be changed by exploiters.

Should I keep it text formatted or make it into object required.
Example:
replicated storage > ClassicSword
With child objects:

Damage int val 40
Roblox sound object
SlashEffect
Local "Ability Script"

or I make it like that other image where I have all the effects and just code the abilities in the general script.

[arctic oracle]

Exploiters can change int value on client

But it won't replicate

[odd bobcat]

so it wont affect the server and dmg to other people?

[arctic oracle]

No

[odd bobcat]

okay good. Also whats ur opinion on keeping it text based vs object

[arctic oracle]

Most of the value instances are completely useless

Almost all of them can be replaced by attributes

[odd bobcat]

u mean like the val instances?

or my code

[arctic oracle]

Val instances

I would keep the data for stuff like damage in a dict, like you have it

[odd bobcat]

how bout the vfx and like sound effects?

cuz ive seen like big games js keep em in a folder

but it might get messy idk

[arctic oracle]

Usually you'd wanna play those on client, so I'd keep them in a folder in replicated storage

[hybrid pasture]

** You are now Level 11! **

[arctic oracle]

Replicatedstorage.Assets[movename]

Or similar

[odd bobcat]

o i js got two folders sounds and effects

[arctic oracle]

And if you want scalability I recommend creating a type for the weapon data

And then casting the module return value to map<string, WeaponData>

Well

Doesn't directly make it more scalable but it makes you think about how you structure data

Which usually leads to better results

[odd bobcat]

ive never used roblox type functions

like typeof and stuff?

[arctic oracle]

Defining your own type

local a:string = ""

Or

type Map<t,k> = {[t]:k}

Local a:Map<string, number> = {
Example= 999
}```

[odd bobcat]

🤔 seems kinda abstract. Whats the benefit

[arctic oracle]

Better intellisense mostly

local function dosomemath(numbers:{number}, operator: "+" | "-" )
local total = table.remove(numbers, 1)

for _, num in numbers do 
     if operator == "+" then
    total += num
    continue
    end

    total -= num
end

return total
end

And then calling this function would prompt you with + or - for the second arg

Yk

Also I'm on mobile so

Too lazy for formatting

[odd bobcat]

🤔 oooo type annotations i forgot about those yeah i prob gotta do some

[hot crow]

i suggest only doing that for bigger things and function inputs, you don't need strict typing in fact it'll just slow you down if you're still learning

will say tho it helps build objects while they are incomplete

don't need constructor to write any implementation since the object type should already define everything the constructor will output