✓ ANSWER ACCEPTED

If statement not matching string with object name?

Asked Nov 11, 2018334 views2 answers

Hi, I'm currently having a problem where my 'if statement' doesn't detect the name of the map and instead just prints the error.

I'm relatively new to Lua (have been working on it for a couple of weeks now) and might just have overlooked something very simple here.

I would very much appreciate if anyone could fix this as it looks like it should work to me:

The error I get is as follows: bad argument #1 to 'find' (string expected, got Object)

I have tried doing if map == "Map1" but this does not work either

(my function MovePlayers does work as I've tested it outside of the if statement so that's not the issue)


local num = math.random(1,2)
local map = game.ServerStorage:WaitForChild("Map" ..num)
map:clone().Parent = game.Workspace

print(map)

if string.find(map, "Map1") then
	MovePlayers(Vector3.new(1129.8, 0.5, -6.1), Vector3.new(928.2, 0.5, 183.7), Vector3.new(784.1, 0.5, -86.3))
elseif string.find(map, "Map2") then
	MovePlayers(Vector3.new(1129.8, 0.5, -6.1), Vector3.new(928.2, 0.5, 183.7), Vector3.new(784.1, 0.5, -86.3))
else
	error("No map found")
end

Thanks for any help offered!

AnonymousNov 11, 2018

2 answers

✓ Accepted answer

I'm relatively new to scripting as well, not as new as you are, but still new enough that I don't have a clue what . . .

if string.find(map, "Map1") then

does, but you might have forgotten to ask if the name of the part is the same as the string. Here's an example:

if game.Workspace.Block.Name == "StringNameHere" then -- Make sure to always include that you're asking about the name of the part, and not the part.
-- Anything you want it to do in here.
end

Sorry that I don't know what that line necessarily means, but according to the error anyway, the problem is you're asking if the object itself is equal to a string, which results in an error.

Edit: Couldn't you just do . . .

if map.Name == "Map1" then
 -- Teleport to map1 code here
elseif map.Name == "Map2" then
-- Teleport to map2 code here
end

AnonymousNov 11, 2018Edited Nov 12, 2018Permalink

8 comments

Hope I helped at least a little bit!

AnonymousNov 11, 2018#

Hi, yeah I used that string.find because I've been trying to fix this for a while now and really can't seem to get it, the problem is I can't just use the block's name as it's cloned in and might be different each time, this is similar to what you are suggesting but still doesn't work, really out of ideas!: if game.Workspace["Map"..num] == "Map1" then

AnonymousNov 11, 2018#

I might be able to help you a little better if I knew what you're trying to do.

AnonymousNov 12, 2018#

Basically I have 2 models (maps) in the Server Storage and it selects one randomly by adding the random number to "Map", this is then cloned into the game and it is in the workspace as either Map1 or Map2, I'm basically trying to detect which one was selected and based on that it changes where the players are teleported into the maps, as they are different (right now the locations are the same)

AnonymousNov 12, 2018#

FIXED IT! Damn your post did actually help a lot! I hadn't put .Name after if game.Workspace["Map"..num] which meant it wasn't the name of the part it was comparing, I can't thank you enough man! I'd upvote you if I could!

AnonymousNov 12, 2018#

Oh, and I just put an edit showing what you could do xD.

AnonymousNov 12, 2018#

Yeah it's annoying when I don't know these little things in lua that make life so much easier haha, but I'm learning. Anyway thanks for your help again man and good luck with your own lua ability!

AnonymousNov 12, 2018#

Thanks, you too!

AnonymousNov 13, 2018#

Make a folder called maps instead, and make a models called spawns in the maps and you put parts in it.

local Maps = game.ServerStorage.Maps:GetChildren()
local RandomMap = Maps[math.random(1,#Maps)]

print(RandomMap.Name)

local Spawns = RandomMap:WaitForChild("Spawns"):GetChildren()

for _, v in pairs(game.Players:GetPlayers()) do
      if v.Character then
             local Spawn = Spawns[math.random(1,#Spawns)]
             local Character = v.Character:MoveTo(Spawn.Position)
      end
end

AnonymousNov 12, 2018Edited Nov 12, 2018Permalink

This discussion is preserved as it was. New answers, comments, and votes are closed.