Scripting Helpers is winding down operations and is now read-only. More info→

Using ".PlayerAdded" event. What am I doing wrong?

Asked by

Necrorave 560

10 years ago

I am trying to experiment with events and methods I have yet to use, and this was one of them.

I am not really sure what I did wrong with this, although. If somone could explain to me what I am missing that would be fantastic.

1function spawn(player)
2    player = game.Workspace:FindFirstChild(player.Name)
3        if player ~= nil then
4            player:MoveTo(Vector3.new(1, 1, 1))
5    end
6end
7 
8game.Players.PlayerAdded:connect(spawn)

Thanks

EDIT: No errors are being displayed in the Output. Probably due to my if statement protecting my script from having any errors.

EDIT2: I would prefer if you did not rewrite something entirely different. Unless you explain why my method was incorrect to even attempt.

i dont know if the if statements is actually protecting your code or not OR just all if statements dont work normally in a playerAdded event proROBLOXkiller5 112 — 4y

1 answer

Answered by

Spongocardo 1991

10 years ago

My recommended way:

1game.Players.PlayerAdded:connect(function(player) --Anonymous function for when the player is added, keep an open parenthesis (bracket).
2    player.CharacterAdded:connect(function(char) --Anonymous function for when the player's character has been added/respawned. Keep an open parenthesis (bracket).
3        repeat wait() until char --Waits until the char variable is available
4            char:MoveTo(Vector3.new(1, 1, 1)) --Moves the character to Vector3 point 1,1,1.
5    end) --Ends character added anonymous function with a closing parenthesis to make up for the open parenthesis at line 2.
6end) --Ends player added anonymous function with a closing parenthesis to make up for the open parenthesis at line 1.

More info:

http://wiki.roblox.com/index.php?title=Anonymous_function

What you did wrong:

You're replacing the player variable, when you can go and use the player's Character property. I also recommend linking that function to the player's CharacterAdded event in a separate function for the Player Added event.

01function Spawn(char)
02    repeat wait() until char
03    char:MoveTo(Vector3.new(1, 1, 1))
04end
05 
06function PAdded(player)
07    player.CharacterAdded:connect(Spawn)
08end
09 
10game.Players.PlayerAdded:connect(PAdded)

Sorry, but this was not the answer I was looking for. Thanks for giving me something, but I want to know what I am doing wrong and why its wrong. Necrorave 560 — 10y

Also, I despise anonymous functions :p Necrorave 560 — 10y

Alright, I'll edit my answer. Spongocardo 1991 — 10y

Answer edited. Spongocardo 1991 — 10y

View all comments (3 more)

Thanks. I will accept yours since you were the first to answer. Necrorave 560 — 10y

Wait, why a repair wait until the character exists? Isnt that the entire point of the CharacterAdded event? iaz3 190 — 10y

@Crazy It's a safety measure in case the character hasn't loaded yet. Spongocardo 1991 — 10y

Using ".PlayerAdded" event. What am I doing wrong?

1 answer

Answer this question