more than one mob being transported(possible byond bug?) in Developer Help

ID:138924
 
Sep 10 2011, 12:20 am
Code:
                                                for(var/Buildings/Bed/A in world)
                                                    spawn(20)
                                                        var/obj/B=pick(A)
                                                        src.loc=locate(B.x,B.y-1,B.z)
                                                        return


Problem description: This may possibly be a BYOND bug.

For some strange reason, the player and anyone next to him/her gets teleported to the bed(B). What should be happening is just the player gets located to the bed. I've tested just setting a specific location(e.g. 1,1,1) and that works just fine. I don't know if I'm doing something wrong or if it's a BYOND bug.

#1 Sep 10 2011, 5:44 am
Show us the whole piece of code, it's indented as if in a larger block, and there's a lot of stuff missing.

Also, looping through every instance of one type of object is a very bad idea. Not to mention you're using pick() on a building/bed, pick only works on list.

simply using a list of all the beds and the world and picking from it would be easier; i.e:
bed
 New()
  .=..()
  beds+=src
//... later in the code
spawn(20)
 var/bed/bed = pick(beds)
 src.Move(locate(bed.x,bed.y,bed.z))

That would produce the same thing.

#2 Sep 10 2011, 8:07 am
My guess is you need to unindent the "return" one tab.

But if you want it to pick a random bed, you'd do something like this:

var/list/beds = list()
for(var/Buildings/Bed/A in world)
    beds += A

var/obj/B = pick(beds)
src.loc = locate(B.x, B.y - 1, B.z)
#3 Sep 11 2011, 4:29 am
In response to Forum_account (#2)
Ah thank you both :)
#4 Sep 11 2011, 4:42 am
In response to AbdelJN (#3)
nevermind.. same results.. and it's strange because I have a client check to check if the player being transported is connected to a client and isn't an npc, yet the npc gets transported too.. im starting to believe its a byond bug now..
#5 Sep 11 2011, 6:17 am
In response to AbdelJN (#4)
Give us all of the code. It is most likely not a BYOND bug.
#6 Sep 11 2011, 8:14 am
In response to ANiChowy (#5)
ANiChowy wrote:
Give us all of the code. It is most likely not a BYOND bug.

the rest is irrelevant, trust me.
#7 Sep 11 2011, 8:31 am
 
var/list/beds = list()
for(var/Buildings/Bed/A in world)
    beds += A
var/obj/A = pick(beds)
src.loc = locate(A.x,A.y-1,A.z)
return


if that produces the same results the rest is very relevant
#8 Sep 13 2011, 9:56 am
In response to AbdelJN (#6)
You'd be surprised, now hand it over! >:O