Scripting Helpers is winding down operations and is now read-only. More info→
Ad
Log in to vote
0

If, true, statements not working, why? [EDITED]

Asked by
TheHospitalDev 1134 Moderation Voter
9 years ago

Please provide more explanation in your question. If you explain exactly what you are trying to accomplish, it will be much easier to answer your question correctly.

So, I have a door which is MEANT to open when a button is clicked, but only ONE button. So, when OPEN is visible CLOSED is not, and when CLOSED is visible OPEN is not.

view source
01local LO = script.Parent.Parent.L.Open
02local LC = script.Parent.Parent.L.CLOSED
03local RO = script.Parent.Parent.R.OPEN
04local RC = script.Parent.Parent.R.CLOSED
05local L1O = script.Parent.Parent.L1.OPEN
06local L1C = script.Parent.Parent.L1.CLOSED
07local R1O = script.Parent.Parent.R1.OPEN
08local R1C = script.Parent.Parent.R1.CLOSED
09function onClicked()
10    if R1C.Transparency == 1 then
11            LC.Transparency = 1
12            RC.Transparency = 1
13            L1C.Transparency = 1
14            R1C.Transparency = 1
15            LC.CanCollide = false
View all 48 lines...
0
For properties, you must have correct capitalization. So, you need to capitalize the T in transparency and the 2 C's in cancollide. PreciseLogic 271 — 9y

1 answer

Log in to vote
1
Answered by
yoshi8080 445 Moderation Voter
9 years ago

Well I dunno if this is what you wanted but here are somethings that are wrong with your script.

  1. if open.transparency == 1 then will check if the parts Transparency is 0 which is will be 0 since there is no way to tell if the part will be changing. A way to do that is to add adebouncewhich would be named as opened.

  2. Transparency is lower cased, which is shouldn't be like that. it should be Transparency same for cancollide, it should be CanCollide

  3. It would be best to make a single button for this door from what the script says, so you only need one button named Open.

view source
01local open = script.Parent.Parent.Open
02local door = script.Parent.Parent.Door
03opened = false
04-- script
05function onClicked()
06    if opened == false then -- if value listed above is false, door is open and turns it true
07        door.Transparency = 1
08        door.CanCollide = false
09        open.BrickColor = BrickColor.new("Lime green") -- changes color if open
10        opened = true -- sets it back to true to open it
11 
12    elseif opened == true then -- if value listed above is true, door is closed and turns it false
13        door.Transparency = 0
14        door.CanCollide = true
15        open.BrickColor = BrickColor.new("Bright red") -- changes color if closed
16        opened = false -- sets it back to false to close it
17    end
18end
19script.Parent.ClickDetector.MouseClick:connect(onClicked)
0
What I mean, Open is the first door, and closed is the second door. TheHospitalDev 1134 — 9y
0
But, it works TheHospitalDev 1134 — 9y
0
There is a way to do that, just look at my example and you're able to add it to the script, if you learned how it works. yoshi8080 445 — 9y
Ad

Answer this question