Browser Who Problem!

ID:263517
 
Mar 12 2007, 11:53 am
Code:
mob/verb/Who()
    var/tmp/PlayerCount
    for(var/mob/M in world)
        if(M.client)
            PlayerCount += 1
            usr << browse({"<font face="Comic Sans MS"><BODY bgcolor=black><font color=red><center><font size=4><b><u><font color=#00BBDD>.:Whos Online:.</u></b></center><br><br><font size=2><font color=green><b><u><font color=red>[M.name]</u><font color=green>[M.key]
<font size=3><font color=red><br><br><b>Player(s) Online:<u>[PlayerCount]</u>."})<


Shows only one person in the browser but the player count is right and when you have more then 1 person on a game it wont show the other people like its soposed to only shows 1 person adn that person is you >.<:

</<>
Mar 12 2007, 1:24 pm
You're repeatedly sending a who page (once for every client) which makes the last one override the others. What you need to do is build the page in the loop, then display it afterwards. You also may want to keep a list of players on-hand constantly.
Mar 12 2007, 3:20 pm
In response to YMIHere
so like what do you mean exactly I'm so confused
Mar 12 2007, 5:46 pm
Thats a noobish code, I thought you said you were a good coder.

- Wires
Mar 12 2007, 6:21 pm
In response to Wires
Your post is totally irrelevant.

a good way to do is add to the html.

mob/verb/Who()
    var/tmp/PlayerCount
    var/html = "<font face="Comic Sans MS"><BODY bgcolor=black><font color=red><center><font size=4><b><u><font color=#00BBDD>.:Whos Online:.</u></b></center><br><br>"
    for(var/mob/M in world)
        if(M.client)
            PlayerCount += 1
            html += "<font size=2><font color=green><b><u><font color=red>[M.name]</u><font color=green>[M.key]<font size=3><font color=red><br><br>"
    html += "<b>Player(s) Online:<u>[PlayerCount]</u>."
    usr << browse(html)
Mar 12 2007, 8:48 pm (Edited on Mar 12 2007, 8:54 pm)
In response to A.T.H.K
Duuuuude.

You can use the += operator on strings?

You know, I used to know that. Seriously, I just loaded up Silver Crescent (my old game) and noticed. However, I totally forgot until you pointed it out! I feel like an idiot now. Thanks. That really helps for what I'm currently working on. :)
Mar 13 2007, 2:52 am
In response to Zagreus
I figgured it out what you tell me and i get it and i am a good coder i never ever delt with browsers before only things in text that people want to see <_<
Mar 13 2007, 3:57 am
In response to The Unknown Ninja From Hell
Add a sleep in your for loop to see what I'm talking about. =) A.T.H.K's snippet is building the page then displaying it.
Mar 13 2007, 3:59 am
In response to YMIHere
okay
Mar 13 2007, 4:40 am
In response to Zagreus
Well, even if you couldn't, you could use copytext()+lentext()/length() anyway.