Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
How do I make an onClicked Function work multiple times?
I'm clicking but it only rotates once and then it's stuck at a 45 degree angle.
1 | local chickenbutt = script.Parent |
2 |
3 | function onClicked(playerWhoClicked) |
4 | chickenbutt.CFrame = CFrame.new(38, 0.5, 34) * CFrame.Angles(45, math.pi, 0) |
5 | end |
6 |
7 | chickenbutt.ClickDetector.MouseClick:connect(onClicked) |
1 answer
Log in to vote
0
That's because you're redefining the CFrame absolutely, not relatively.
Compare this...
01 | local chickenbutt = script.Parent |
02 | -- Nice variable name |
03 |
04 | function onClicked(playerWhoClicked) |
05 | chickenbutt.CFrame = CFrame.new(38,.5,34) * CFrame.Angles(45, math.pi, 0) |
06 | -- CFrame.Angles assumes its constituents values as radian values, not degree values. |
07 | -- If you want correct angle values, use this: CFrame.Angles(math.rad(45), math.pi, 0) |
08 | -- BTW. math.pi is half of 360°, or 180°. |
09 | -- (math.pi / 4) == math.rad(45) |
10 | end |
11 |
12 | chickenbutt.ClickDetector.MouseClick:connect(onClicked) |
With this...
1 | local chickenbutt = script.Parent |
2 | chickenbutt.CFrame = CFrame.new(38,.5,34) -- If "chickenbutt" is already on this position, you don't need this line here. |
3 |
4 | function onClicked(playerWhoClicked) |
5 | chickenbutt.CFrame = chickenbutt.CFrame * CFrame.Angles(45, math.pi, 0) |
6 | end |
7 |
8 | chickenbutt.ClickDetector.MouseClick:connect(onClicked) |
See how it uses its own CFrame every time the onClicked function's fired? That's a good thing, because that's what you want.
Touch it one time, and it rotates the given angle. Touch it another time, and it will use the last CFrame, but apply it to the new one.