Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Why does my script clone a Handle into every part?
01 | local click = script.Parent.ClickDetector |
02 | local Number = 1 |
03 |
04 | click.MouseClick:Connect(function(clicked) |
05 | Number = 1 |
06 | local rand = math.random(1, 512) |
07 | for _, v in pairs(workspace.Mine:GetDescendants()) do |
08 | if v.Name == ("Handle") then |
09 | v:Destroy() |
10 | end |
11 | if v:IsA("Part") then |
12 | if Number == rand then |
13 | local Clone = game.ServerStorage.Handle:Clone() |
14 | Clone.Parent = workspace |
15 | Clone.Position = v.Position |
View all 21 lines...
For some reason this clones in every part, not just one.
0
i assume its because your doing it on a loop. what youve done is get every part from the workspace, and check if it is a part. If it is, then it clones to every part that is a part. Sorry if thats wrong, i dont understand lua too much. jdm4llfem8 94 — 6y
1 answer
Log in to vote
0
The main problem with this is that you are using a loop to first remove anything named "handle" then afterwards are just cloning a handle into anything that is a "Part" by its Position. If you want the script to add the new Handle to a specific part, you will have to define the name of v in the function to do so (as v represents all objects in the workspace).
If you only wanted one handle to be cloned, you'd have to add a line designating that if there is a handle, then to either return end (much like a debounce) or just not perform the clone, such as:
01 | local click = script.Parent.ClickDetector |
02 | local Number = 1 |
03 |
04 | click.MouseClick:Connect(function(clicked) |
05 | Number = 1 |
06 | local rand = math.random(1, 512) |
07 | for _, v in pairs(workspace.Mine:GetDescendants()) do |
08 | if v.Name == "Handle" then |
09 | v:Destroy() |
10 | elseif v:IsA("Part") and v.Name ~= "Handle" then |
11 | local check = workspace:FindFirstChild("Handle") |
12 | if Number == rand and check == nil then |
13 | local Clone = game.ServerStorage.Handle:Clone() |
14 | Clone.Parent = workspace |
15 | Clone.Position = v.Position |
View all 22 lines...
Edits: Moved a redundant if-then you didnt need, added a check, used break to stop the for loop.