Scripting Helpers is winding down operations and is now read-only. More info→

Script not taking away from a value?

Asked by

TheHospitalDev 1134

8 years ago

I'm trying to make an "Attempts" script, where you only have 5 attempts, but the script won't subtract the value.

01local UName = script.Parent:WaitForChild("UName")
02-------------------------------------
03local UPass = script.Parent:WaitForChild("UPass")
04-------------------------------------
05local Attempts = script.Parent:WaitForChild("Attempts").Value
06-------------------------------------
07local Button = script.Parent:WaitForChild("Login")
08-------------------------------------
09local pass = {
10    ["Herro"] = true;
11}
12-------------------------------------
13local transparency = 0
14-------------------------------------
15local debounce = false
16-------------------------------------
17local countdown = 10
18-------------------------------------
19local warning = "You have "..Attempts.." attempts left."
20----------------------
21local function checkAttempts()
22    if Attempts == 0 then
23        --Randomness
24    else
25        Attempts = Attempts - 1 --Here
26        script.Parent.Warning.Text = warning
27        wait(3)
28        script.Parent.Warning.Text = ""
29        debounce = false
30    end
31end
32-------------------------------------
33Button.MouseButton1Click:connect(function()
34    if debounce == false then
35        debounce = true
36        if pass[UPass.Text] then
37            --Randomness 
38        else
39            checkAttempts()
40        end
41    end
42end)
 View all 42 lines...

It will display the text, just not subtract the value. Any suggestions, because there aren't any errors.

Info:

`1`	`--No filtering Enabled`

`2`	`--It's a local script.`

Try assigning "Attempts" to the Attempts object itself instead of the value, then use Attempts.Value whenever you wish to refer to it through your script. Let me know if that works and I will provide more details afterwards Necrorave 560 — 8y

You are using the value of attempts and but not setting them remove the "value" part at the top and change line 25 to Attempts.Value = Attempts.Value -1 so that it updates User#5423 17 — 8y

1 answer

Answered by

Bubbles5610 217

8 years ago

Its a simple yet common mistake, on line 05, all you need to do is take away the ".Value" part and re-assign this to where you next use the variable. This is because when you assigned the variable you also assigned the value which means that you cant change it, where as now you can access the child of it:

01--(All errors should be fixed)
02local UName = script.Parent:WaitForChild("UName")
03-------------------------------------
04local UPass = script.Parent:WaitForChild("UPass")
05-------------------------------------
06local Attempts = script.Parent:WaitForChild("Attempts")
07-------------------------------------
08local Button = script.Parent:WaitForChild("Login")
09-------------------------------------
10local pass = {
11    ["Herro"] = true;
12}
13-------------------------------------
14local transparency = 0
15-------------------------------------
16local debounce = false
17-------------------------------------
18local countdown = 10
19-------------------------------------
20local warning = "You have "..Attempts.Value.." attempts left."
21----------------------
22local function checkAttempts()
23    if Attempts.Value  == 0 then
24        --Randomness
25    else
26        Attempts.Value  = Attempts.Value  - 1
27        script.Parent.Warning.Text = warning
28        wait(3)
29        script.Parent.Warning.Text = ""
30        debounce = false
31    end
32end
33-------------------------------------
34Button.MouseButton1Click:connect(function()
35    if debounce == false then
36        debounce = true
37        if pass[UPass.Text] then
38            --Randomness 
39        else
40            checkAttempts()
41        end
42    end
43end)
 View all 43 lines...

Script not taking away from a value?

1 answer

Answer this question