Script gives two of the same thing, not one?
My previous question was why a script that was supposed to give tools wasn't working. So I managed to create a better script, which uses loops and tables. It works! But wait, when I tested it out, with Squirtle, it gave two of the same move. Charmander works fine. The move that gets cloned twice is Tackle. I don't know what's wrong.
moves = {"Ember", "Tackle", "Bubble"}
charmander = {"Ember", "Tackle"}
squirtle = {"Bubble", "Tackle"}
movestorage = workspace.Moves
plr = game.Players.LocalPlayer
script.Parent.Charmander.MouseButton1Click:connect(function()
for i = 1,#charmander do
if moves[i] == charmander[i] then
local move = movestorage:FindFirstChild(moves[i]):Clone()
move.Parent = plr.Backpack
end
end
end)
-- MY PROBLEM IS HERE --
script.Parent.Squirtle.MouseButton1Click:connect(function()
for i = 1,#squirtle do
if moves[i] == squirtle[i] then
local move = movestorage:FindFirstChild(moves[i]):Clone()
move.Parent = plr.Backpack
end
end
end)
0
0
0
1 answer
I don't have a reason for why you're getting double, but I do have a reason you're not getting Bubble.
script.Parent.Squirtle.MouseButton1Click:connect(function()
for i = 1,#squirtle do
if moves[i] == squirtle[i] then
local move = movestorage:FindFirstChild(moves[i]):Clone()
move.Parent = plr.Backpack
end
end
end)
moves[i] will never be squirtle[i] when i is 1, nor even when it is 3, and therefore you just can't get Bubble. If you want to check if something is in a table, you'll have to iterate over the whole table in two parts. Not ideal.
The trick is that in your case, you don't need to check. Just use
script.Parent.Squirtle.MouseButton1Click:connect(function()
for i = 1,#squirtle do
local move = movestorage:FindFirstChild(squirtle[i]):Clone()
move.Parent = plr.Backpack
end
end)
Instead in your Squirtle loop. Likewise for the Charmander loop.
0
0