I pasted your code and it says it is erroring. As much as we want to help you, we cannot because your formatting is unreadable.
Solution
A lot of your code could be shortened if you invest your time in making a system to create multiple commands easily. I highly recommend this tutorial made by EmeraldSlash. It'll show you how you should organize commands and more:
https://devforum.roblox.com/t/how-to-make-basic-admin-commands/59691
After reading this and understanding it, you can return arguments such as a Username and the new WalkSpeed and have them set from there.
If you want to connect things with a local script, use remote events.
Shortening your code
Now for your code, it is hard to read. You need to start making variables to stop calling the same family tree. It makes your code unpleasing to look at as well as making you waste time.
What you are doing:
game.Players.LocalPlayer.Character.Humanoid.WalkSpeed = 16
game.Players.LocalPlayer.Character.Humanoid.JumpPower = 50
game.Players.LocalPlayer.Character.Humanoid.MaxHealth = 100
game.Players.LocalPlayer.Character.Humanoid.Health= 100
game.Players.LocalPlayer.Chatted:Connect(function()
end)
if (game.Players.LocalPlayer.Character:FindFirstChild('Humanoid')) then
end
What you should be doing
local LocalPlayer = game.Players.LocalPlayer
local Character = LocalPlayer.Character
local Humanoid = Character:WaitForChild('Humanoid')
Humanoid.WalkSpeed = 16
Humanoid.JumpPower = 50
Humanoid.MaxHealth = 100
Humanoid.Health = 100
LocalPlayer.Chatted:Connect(function()
end)
if (Character:FindFirstChild('Humanoid')) then
end
See how much better that looks and how much faster you can type it?
Formatting
Fix your formatting. It will make your code easier to read for us as well as yourself. Your blocks should be aligned with each other.
Don't do
if (true) then
if (false) then
part.Touched:Connect(function()
part:Destroy()
end)
else
for i=1, 10 do
print(i)
end
end
end
That is really hard to read what is going on.
Do
if (true) then
if (false) then
part.Touched:Connect(function()
part:Destroy()
end)
else
for i=1, 10 do
print(i)
end
end
end
We all can easily read each block and catch errors easily as well.
Loops
Looking at your code, it seems like you are using useless for loops. If you want it to slowly rise up, you need to add a wait.
for i=1, 16 do
Humanoid.WalkSpeed = 16
end
for i=1, 16 do
Humanoid.WalkSpeed = i
end
Humanoid.WalkSpeed = 16
The loops run instantly because you don't make them wait. Don't use for loops if you aren't adding waits.
5 comments
You can only access 1 (not 16) character from local script. Even if you access other character (via game.Players:GetPlayers()), filtering enabled will prevent any changes to them.
nope, local scripts can change walkspeed and jumpspeed etc. done it several times
well, even if it changes the speed, it's only visible to the local player but not other player
The property change is, yes, but the speed effects still replicate. This is still very bad code, you have declared a
charactervariable but never used itye idk why i made that variable