This isn't working and i don't know why

[eager heart]

HELP ME

[pseudo kindle]

are the Clicked variables Values? What are they

[eager heart]

they are a bool value

[pseudo kindle]

Then you need to do Clicked1.Value to check the value of the BoolValue

[eager heart]

oh thanks

im dumb

thats so simple man how did i not noticed

[pseudo kindle]

It happens, don't worry

[wintry bay]

holy nested if statements

[eager heart]

it didnt work

i modified a little and still didnt work

is suppose to be working

[pseudo kindle]

Is this a server script or local script?

[eager heart]

server

[floral mauve]

Put them all in a table and make a for loop instead all of these if statements 😭

[eager heart]

srry i didnt understand it

im a newbie

[pseudo kindle]

Do you set the bool values on the server too? Like when they're clicked?

[eager heart]

yes

[floral mauve]

Local table = {clicked,clicked2 etc}
local allclicked = true
for I,v in pairs(table) do
if v.Value then allclicked= true
else allclicked = false return end
end

[eager heart]

oh

thats make sense now

[pseudo kindle]

Lets see if the Triggered event is running by adding a print("hello world") and see if you can find it in the output

[pseudo kindle]

Format that

[floral mauve]

Sure thing

[eager heart]

nothing below the if's is working

[floral mauve]

local allclicked = true
for I,v in pairs(table) do
if v.Value  then allclicked= true
else allclicked = false return end 
end```
Double check for mistypos

[pseudo kindle]

Are sure the BoolValues are being set? Check those values in the explorer and see what happens when you click them

[eager heart]

the value becomes true

for all of them

[eager heart]

i will try that

[floral mauve]

Do break instead of return

[eager heart]

like that?

[wintry kraken]

** You are now Level 5! **

[eager heart]

dang!

[floral mauve]

Ye

It loops thro every clicked variable and if one of them is not true then it'll set allclicked to false and not continue the for loop

[eager heart]

ok thanks

i will try it

play to see if works

[floral mauve]

My suggestion only makes the code smoother/easier to understand, don't think it'll fix anything

[eager heart]

oh thats why it didnt work

[floral mauve]

Also place the table and the allclicked boolean outside of the while loop

[eager heart]

thanks for all the help

[floral mauve]

local Collisions = script.Parent.Collisions

local GreenPart = script.Parent.GreenButton
local Prompt = GreenPart.ProximityPrompt

local stickmasterluke = script.Parent.Stickmasterluke
local sticknoid = stickmasterluke.Sticknoif

local AllClicked = false
task.Spawn(function()
while task.wait() do
for I,v in pairs(clickables) do
If v.Value == true then AllClicked = true 
else AllClicked =false break end
end
end)
prompt.Triggered:Connect(function()
if AllClicked then
sticknoid:TakeDamage(100)
end
end)```

Double check misstypos too sense all of this code is from memory

[eager heart]

the ```lua
task.Spawn

[floral mauve]

Not a capital s I think

[floral mauve]

Actually you could do

v.Changed:Connect(function()
for I,v in pairs(clickables) do
if v.Value then AllClicked = true 
else AllClicked = false break end
end
end
end

Instead of the entire spawned function

Less lag, only fires when a then value is changed instead of every ms

[shut quarry]

thats pretty bad, I think the objective of the one who created the post is do a thing when all are clicked right? @eager heart

[eager heart]

yeah sorry

i was on school