Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
Script assumes yes/no button is clicked?
My script (maybe) assumes that the yes/no button in a Gui in PlayerGui was clicked even before the gui appears. Script:
01 | for i = 1e+18, 1, -1 do |
02 | wait(.5) |
03 | print(i .. " Showing debts..") |
04 | if taxYes.MouseButton1Down or taxNo.MouseButton1Down then |
05 | break |
06 | end |
07 | end |
08 | if taxYes.MouseButton1Down or taxNo.MouseButton1Down then |
09 | wait(2) |
10 | unpaid_tax = unpaid_tax_datastore:GetAsync(player.UserId) |
11 | unpaid_tax_storage = unpaid_tax_datastorage:GetAsync(player.UserId) |
12 | debt = ((0 - plrInDebt.leaderstats.Cash.Value) + unpaid_tax or unpaid_tax_storage) |
13 |
14 | debtAmount.Text = "$" .. debt |
15 |
16 | end |
I'll edit the question if you have a problem concerning the other scripts which are very long.
0
Why do you need to use a for loop? Just use a while loop and put a for loop inside it. Dovydas1118 1495 — 4y
1 answer
Log in to vote
0
Okay so, I put a local script inside of a text button, and pasted this:
1 | if script.Parent.MouseButton1Click then |
2 | print("Clicked.") |
3 | end |
And, when I ran it, the game printed "Clicked" as soon as I joined if even if I clicked it or not.
To fix this, use:
1 | script.Parent.MouseButton1Click:Connect(function() |
2 | print("Clicked.") |
3 | end |
Another thing is, you don't need to do a loop at all.
Here's the final code:
01 | local function Clicked() |
02 | taxYes.MouseButton1Down:Connect(function() |
03 | return true |
04 | end) |
05 | taxNo.MouseButton1Down:Connect(function() |
06 | return true |
07 | end) |
08 | end |
09 | if Clicked() == true then |
10 | wait(2) |
11 | unpaid_tax = unpaid_tax_datastore:GetAsync(player.UserId) |
12 | unpaid_tax_storage = unpaid_tax_datastorage:GetAsync(player.UserId) |
13 | debt = ((0 - plrInDebt.leaderstats.Cash.Value) + unpaid_tax or unpaid_tax_storage) |
14 |
15 | debtAmount.Text = "$" .. debt |
16 | end |