Read the paragraph below. This will explain a term that you might not know:
When I use the word method, it is another word for function. You will see the term method in other programming languages that are not LUA.
Alright, know on to the fixing.
I see a few problems in your script
First, the method, :children(), is deprecated. Roblox recommends to not use deprecated methods and syntaxes.
Next, you didn't include a click detector in your code. This is very essential for the click event. An event is a function that fires when something happens.
On line 6, you used a for loop. You could use this, however, a while loop uses the variable time in the condition (i.e in while true do, true is the condition; in while 5 == 5 do, 5 == 5 is the condition; the condition has to be true for the loop to fire)
On line 16 in your code, you made a huge mistake.
You have done player.human.WalkSpeed. "human" is not in the player.
The correction of this error is player.Character.Humanoid.WalkSpeed.
I stored player.Character in a variable called char.
You might be confused at player.Character or player.CharacterAdded:Wait(). To clear that up, if player.Character is nil,or not existing, it could wait until the character is added(see the method :Wait() ?) After that, it gets the humanoid(:WaitForChild() is another way to access a part; this will wait a few seconds(just in case the object just becomes not nil) until it will finally say the object is nil)
Then, you put 16 in between parenthesis. This is unneeded.
Finally, to make this event work when you click a part, you need to use your click detector that you stored (remember that the click detector is essential for the click event?). We will use an RBXScriptSignal known as Connect to connect the function to the event. In this event, when the part is clicked, the method, onMouseClick, will be fired.
local players = game.Players:GetChildren()
local clickDetector = script.Parent:WaitForChild("Your Click Detector Name Here")
function onMouseClick(player)
local char = player.Character or player.CharacterAdded:Wait()
local humanoid = char:WaitForChild("Humaonid")
local playerGui = player:WaitForChild("PlayerGui")
local textButton = playerGui:WaitForChild("CountdownGui"):FindFirstChild("TextButton")
local time = 60
wait(40)
playerGui.CountdownGui.TextButton.Visible = true
while time <= 60 and time >= 0 do
wait(1)
time = time - 1
workspace.Counter:Play()
textButton.Text = tostring(time)
textButton.Visible = false
workspace.Cannon:Play()
wait(0.1)
workspace.Bloodbath:Play()
player.Humanoid.WalkSpeed = 16
end
end
clickDetector.MouseClick:Connect(onMouseClick)
There two important things that you should know. You don't have to read this:
Please indent your code
Indenting your code makes it look more cleaner. This just makes coding be organized. This could be crucial when you are coming back.
Speaking of coming back to an old script, comment some confusing lines of code. When you come back, you could read what you wrote instead of reading a whole block of code and trying to understand it
Filtering Enabled or Experimental Mode off changes how your game will work. I won't go in depth here, but, this will be very important later on when you are learning. This script in Filtering Enabled might not work because of how Filtering Enabled works
Sorry about all of those coding terms. Sorry if that intimidated you.
1 comment
note that any code below the
for iloop will be paused until the loop is completed. No need to add a wait(60)