Scripting HelpersARCHIVE
A little piece of Roblox history. Preserved for everyone, one question at a time.Read about the archive →
All questions/Question #70376
✓ ANSWER ACCEPTED

If all bools are true then make visible green tick?

Asked 283 views1 answers

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 


Target1 = game.ReplicatedStorage["Shot-1Letter"]
Target2 = game.ReplicatedStorage["Shot-2Letter"]
Target3 = game.ReplicatedStorage["Shot-3Letter"]
Target4 = game.ReplicatedStorage["Shot-4Letter"]
Target5 = game.ReplicatedStorage["The-1Letter"]
Target6 = game.ReplicatedStorage["The-2Letter"]
Target7 = game.ReplicatedStorage["The-3Letter"]
Target8 = game.ReplicatedStorage["Gun-1Letter"]
Target9 = game.ReplicatedStorage["Gun-2Letter"]
Target10 = game.ReplicatedStorage["Gun-3Letter"]
repeat
wait()
if Target1 == true then
if Target2 == true then
if Target3 == true then
if Target4 == true then
if Target5 == true then
if Target6 == true then
if Target7 == true then
if Target8 == true then
if Target9 == true then
if Target10 == true then
	game.StarterGui.s.GreenTick.Visible = true

end
end

end
end

end
end

end
end

end
end



wait()
until false
Anonymous

2 comments

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.

Anonymous#
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

Anonymous#

1 answer

✓ Accepted answer

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 

	local Target1 = game.ReplicatedStorage:FindFirstChild("Shot-1Letter")
	local Target2 = game.ReplicatedStorage:FindFirstChild("Shot-2Letter")
	local Target3 = game.ReplicatedStorage:FindFirstChild("Shot-3Letter")
	local Target4 = game.ReplicatedStorage:FindFirstChild("Shot-4Letter")
	local Target5 = game.ReplicatedStorage:FindFirstChild("The-1Letter")
	local Target6 = game.ReplicatedStorage:FindFirstChild("The-2Letter")
	local Target7 = game.ReplicatedStorage:FindFirstChild("The-3Letter")
	local Target8 = game.ReplicatedStorage:FindFirstChild("Gun-1Letter")
	local Target9 = game.ReplicatedStorage:FindFirstChild("Gun-2Letter")
	local Target10 = game.ReplicatedStorage:FindFirstChild("Gun-3Letter")
	repeat
	wait()	
if Target1 == nil then
	if Target2 == nil then
	if Target3 == nil then
	if Target4 == nil then
	if Target5 == nil then
	if Target6 == nil then
	if Target7 == nil then	
if Target8 == nil then
	if Target9 == nil then
	if Target10 == nil then
	    game.StarterGui.s.GreenTick.Visible = true
	 
	end
	end
	 
	end
	end	 
	end
	end
	 
	end
	end
	 
	end
	end
	 
	 
	 
	wait(0,500)
	until false

tell me if i forgot to explain something

AnonymousPermalink

6 comments

0

i dont know if u wanted a script like this so im srry if not lol pls dont downvote it

Anonymous#
0

Why are you comparing (assumably) ValueBase objects to nil?

Anonymous#
0

oh they values?

Anonymous#
0

so i have to do like if Target1 == 0 then tho we dont know what the value is

Anonymous#
0

oh i mean if Target1.Value == 0 then

Anonymous#
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

Anonymous#

This discussion is preserved as it was. New answers, comments, and votes are closed.