Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
If all bools are true then make visible green tick?
i tried to make it but its just a huge mess i tried to make all bool values and this script still doesn't work can someone fix it? Thanks. im bad at checking values
01 | Target1 = game.ReplicatedStorage["Shot-1Letter"] |
02 | Target2 = game.ReplicatedStorage["Shot-2Letter"] |
03 | Target3 = game.ReplicatedStorage["Shot-3Letter"] |
04 | Target4 = game.ReplicatedStorage["Shot-4Letter"] |
05 | Target5 = game.ReplicatedStorage["The-1Letter"] |
06 | Target6 = game.ReplicatedStorage["The-2Letter"] |
07 | Target7 = game.ReplicatedStorage["The-3Letter"] |
08 | Target8 = game.ReplicatedStorage["Gun-1Letter"] |
09 | Target9 = game.ReplicatedStorage["Gun-2Letter"] |
10 | Target10 = game.ReplicatedStorage["Gun-3Letter"] |
11 | repeat |
12 | wait() |
13 | if Target1 == true then |
14 | if Target2 == true then |
15 | if Target3 == true then |
View all 43 lines...
1
What are these values? Are they of the ValueBase class? If so you should be checking their .Value. You also have no reason to compare to true - just type "if a and b and c and d and e".. etc comparing values to true is redundant here. SummerEquinox 643 — 6y
0
^^^ If you're just putting == true, then that means you're just checking if the object is not nil, or it exists, not the value of it. Like SummerEquinox said, use Target1.Value, Target2.Value, etc, or you could use a for i loop User#22980 0 — 6y
1 answer
Log in to vote
0
FindFirstChild is useful if you want to verify an object something exists before continuing nill is nothing u can use nill to check if its there if Target1 is there i think is what u wanted then
01 | local Target1 = game.ReplicatedStorage:FindFirstChild("Shot-1Letter") |
02 | local Target2 = game.ReplicatedStorage:FindFirstChild("Shot-2Letter") |
03 | local Target3 = game.ReplicatedStorage:FindFirstChild("Shot-3Letter") |
04 | local Target4 = game.ReplicatedStorage:FindFirstChild("Shot-4Letter") |
05 | local Target5 = game.ReplicatedStorage:FindFirstChild("The-1Letter") |
06 | local Target6 = game.ReplicatedStorage:FindFirstChild("The-2Letter") |
07 | local Target7 = game.ReplicatedStorage:FindFirstChild("The-3Letter") |
08 | local Target8 = game.ReplicatedStorage:FindFirstChild("Gun-1Letter") |
09 | local Target9 = game.ReplicatedStorage:FindFirstChild("Gun-2Letter") |
10 | local Target10 = game.ReplicatedStorage:FindFirstChild("Gun-3Letter") |
11 | repeat |
12 | wait() |
13 | if Target1 == nil then |
14 | if Target2 == nil then |
15 | if Target3 == nil then |
View all 42 lines...
tell me if i forgot to explain something
0
i dont know if u wanted a script like this so im srry if not lol pls dont downvote it tacotown2 119 — 6y
0
Why are you comparing (assumably) ValueBase objects to nil? SummerEquinox 643 — 6y
0
oh they values? tacotown2 119 — 6y
0
so i have to do like if Target1 == 0 then tho we dont know what the value is tacotown2 119 — 6y
View all comments (2 more)
0
oh i mean if Target1.Value == 0 then tacotown2 119 — 6y
0
He provides that he is checking whether they are true or not - thus, we can assume these are bool ValueBase objects. IE .Value = true/false. It is also incredibly unneeded and redundant to create a new if statement for each check when you can do simply "if a.Value and b.Value and c.Value and d.Value SummerEquinox 643 — 6y