Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
why is my script completely ignoring these conditions?
I have a function where you click a sight attachment, and it attaches to your gun. However, the statements at lines 18 and 28 do not work, but do not create any errors. It seems like they are completely ignored. Is there a way to fix this or is there some sort of limit to how many conditions can be used in a function?
01 | attachments.Mini.Click:FindFirstChild("ClickDetector").MouseClick:connect(function() |
02 | local attpoint = self.current:FindFirstChild("optic") |
03 | local handle = self.current:FindFirstChild("Handle") |
04 | local origin = self.current:FindFirstChild("HandleOrigin") |
05 | if interaction == true and attpoint.Mini.Value == true then |
06 | deleteFrame() |
07 | if currentoptic == nil then |
08 | Mini() |
09 | elseif currentoptic ~= nil and currentoptic ~= Mini then |
10 | currentoptic:Destroy() |
11 | RISoff() |
12 | handle.Position = origin.Position |
13 | currentoptic = nil |
14 | attpoint:FindFirstChild("Motor6D"):Destroy() |
15 | self.config.aimFOV = 70 |
View all 35 lines...
2 answers
Log in to vote
1
I resolved the issue by giving the function Mini() and the variable Mini different names, since currentoptic == Mini was confusing Mini() and Mini without giving an error.
Log in to vote
0
Your indentation is off, misrepresenting the blocks in your conditional statements. Here it is properly indented.
01 | attachments.Mini.Click:FindFirstChild("ClickDetector").MouseClick:connect(function() |
02 | local attpoint = self.current:FindFirstChild("optic") |
03 | local handle = self.current:FindFirstChild("Handle") |
04 | local origin = self.current:FindFirstChild("HandleOrigin") |
05 | if interaction == true and attpoint.Mini.Value == true then |
06 | deleteFrame() |
07 | if currentoptic == nil then |
08 | Mini() |
09 | elseif currentoptic ~= nil and currentoptic ~= Mini then |
10 | currentoptic:Destroy() |
11 | RISoff() |
12 | handle.Position = origin.Position |
13 | currentoptic = nil |
14 | attpoint:FindFirstChild("Motor6D"):Destroy() |
15 | self.config.aimFOV = 70 |
View all 35 lines...
Is the above structured how you expected? If not, you might have misplaced an end somewhere.
1
anytime your if statements do not work, make sure your ends are correct and the variables you are comparing mean what you think they mean. ForeverBrown 356 — 5y