PRoblems with maknig a mob apear once...

ID:176200
 
Feb 8 2003, 8:07 pm
In my game i kinda have a verb that makes a mob then puts it where the barracks are...but for some reaon if they is two barracks or more it creates one for each barrack. How could i edit this code so it only get created in one of the barracks location not both:

                    for(var/mob/units/buildings/Barracks/M in world)
                        if(M.owner == src)
                            var/mob/units/soldiers/Digger/T = new(M.loc)
                            T:owner = usr
                            T:team = usr.team


Thanks!
-Crio
Feb 8 2003, 8:13 pm
Koolguy900095 wrote:
In my game i kinda have a verb that makes a mob then puts it where the barracks are...but for some reaon if they is two barracks or more it creates one for each barrack. How could i edit this code so it only get created in one of the barracks location not both:

                    for(var/mob/units/buildings/Barracks/M in world)
>                       if(M.owner == src)
>                           var/mob/units/soldiers/Digger/T = new(M.loc)
>                           T:owner = usr
>                           T:team = usr.team

Thanks!
-Crio

Try putting a break right below T:team = user.team. Another thing, I'm pretty sure that you should use '.' in this case, not colon.

~>Volte
Feb 8 2003, 8:14 pm
In response to Volte
works alright for me...why change it?
Feb 8 2003, 8:15 pm
In response to Koolguy900095
Koolguy900095 wrote:
works alright for me...why change it?

Stability and compatability reasons, mostly. (I think.)

~>Volte
Feb 8 2003, 8:17 pm
From looking at just that bit of code, it looks like you're using usr in a case where src should be used. Replace usr with src in that snippet, unless you have a good reason for using usr that I don't know about. =)

T:owner = usr
T:team = usr.team

Argh! Use the . operator, not the : operator! You should always use . unless you have a good reason not to.

In answer to your actual question, put the <code>break</code> command after the <code>T.team = src.team</code> line.
Feb 8 2003, 8:17 pm
In response to Volte
k, any ideas of how i can make this only goto one barracks and not multiple?
Feb 8 2003, 8:18 pm
In response to Crispy
You looked at my post and stole my text! I'm telling Dantom! :]

~>Volte
Feb 8 2003, 8:19 pm
In response to Koolguy900095
I just hate it when I miss three posts while typing my own post ([link]). =P
Feb 8 2003, 8:20 pm
In response to Volte
I gave him a more comprehensive answer. =P
Feb 8 2003, 8:26 pm
In response to Crispy
i changed all the :'s to .'s and the code works with the break, thanks!