Lugia319's Site - Spawn Issue

ID:139120

Jun 23 2011, 5:38 pm

Code:

        Start() // Start the game
            if(length(Players) >= 1) // At least 2 to start a game
                world << "A new battle has begun!"
                while(length(Players) > 0) // While the Players list still has people in it
                    var/mob/A = pick(Players)
                    for(var/mob/M in world)
                        if(M == A)
                            Players -= A
                            M.loc = locate(13,13,2) // Move Player to map (Later 13,13,GameStage)
                            var/list/L = list()
                            for(var/turf/T in locate(/area/Spawn) in view(M))
                                L += T
                            M.Move(locate(pick(L)))
                            M.icon = 'Boats.dmi'
                            M.HUDAdd()
                Turn = 1 // Set the Turn = 1
                return
            else // If there are not enough people to start a game
                src << "There must be at least 2 People to start!"
                return

Problem description:

For some reason, the M.Move(locate(pick(L))) doesn't work. I think I may have coded the list or adding things to the list wrong, but I can't see the error.

#1 Jun 23 2011, 6:46 pm

DarkCampainger

It helps immensely if you tell us "how" something isn't working. Is it throwing an error? Is it doing nothing? Is it doing something, but not what you expected?

If you're storing the mob references into the list directly, you don't need the for() loop anymore. You can just act on 'A' (although you might want to check it's not null, in case someone logs out. Just make sure to remove it from the list either way to avoid infinite loops).

Also, you want to move the player to one of the turfs in L, so you don't need to run locate() on the turf returned by pick().

#2 Jun 23 2011, 6:55 pm In response to DarkCampainger (#1)
Lugia319	I'm sorry that I didn't explain myself clearly. Yes, the mob is not moving to the picked turf. I know the turfs are being put into the list. I just can't seem to get the pick to work the way I intend for it to. I'll try removing the locate and see what happens. Thanks for the tip on the for()