[hectorz0r]

I'm trying to find an if inside of another if, yea I know it doesnt make any sense, but if you take a look at the exemple you'll get my point.

PHP Code:

get_pcount()
{
    new players

    for(new x = 1; x <= MaxClients; x++)
        if(IsClientInGame(x))
            players++

    return players
} 


I dont know, but a long time ago I've seen something with the "?" and ":" operators, in a case that looked like this.
I'm not that much experienced with operators, if somebody could tell me a bit more about those two, or atleast give me a link.

Trying to:

PHP Code:

get_pcount(bool:alive)
{
    new players

    for(new x = 1; x <= MaxClients; x++)
        if(/*alive = true, well you know.. include IsPlayerAlive(x) && or !IsPlayerAlive(x) && otherwise.*/ IsClientInGame(x))
            players++

    return players
} 


Note: I know I could do something like the this \/... But it wont help me since I'll need to check the "if inside of another if" more than once.

PHP Code:

get_pcount(bool:alive)
{
    new players

    for(new x = 1; x <= MaxClients; x++)
    {
        if(alive)
        {
            if(IsPlayerAlive(x) && IsClientInGame(x))
                players++
        }

        else
            if(!IsPlayerAlive(x) && IsClientInGame(x))
                players++
    }

    return players
} 


[DarkEnergy]

http://en.wikipedia.org/wiki/Ternary_operation

condition? statement if true : statement if false ;

nested operations
condition? (condition2? statement if true2 : statement if false2 ) : statement if false ;

[p3tsin]

IsPlayerAlive returns a boolean, 'alive' is a boolean... 1+1=?

PHP Code:

get_pcount(bool:alive)
{
    new players

    for(new x = 1; x <= MaxClients; x++)
    {
        if(IsClientInGame(x) && IsPlayerAlive(x) == alive)
        {
            players++
        }
    }

    return players
} 


EDIT: to actually answer your question, in case there was no clean way to do it, you'd have no choice but to repeat the checks. Although you could do the IsClientInGame check before anything else to avoid getting a runtime error from IsPlayerAlive.

[hectorz0r]

Originally Posted by p3tsin IsPlayerAlive returns a boolean, 'alive' is a boolean... 1+1=? PHP Code: get_pcount(bool:alive) {     new players     for(new x = 1; x <= MaxClients; x++)     {         if(IsClientInGame(x) && IsPlayerAlive(x) == alive)         {             players++         }     }     return players }  EDIT: to actually answer your question, in case there was no clean way to do it, you'd have no choice but to repeat the checks. Although you could do the IsClientInGame check before anything else to avoid getting a runtime error from IsPlayerAlive.

Erhm, I through there was a different method, so the ONLY way of doing this is with the following method, right?

PHP Code:

get_pcount(bool:this, bool:that, bool:something)
{
    new players

    for(new x = 1; x <= MaxClients; x++)
        if(IsClientInGame(x))
        {
            if(this)
            {
                if(bla1)
                    players++
            }

            if(that)
            {
                if(bla2)
                    players++
            }

            if(something)
            {
                if(bla3)
                    players++
            }
        }

    return players
} 


[p3tsin]

Your example code is a rather confusing without a real case situation. But surely you can combine the conditions, and make them if - else if (otherwise one player could increment the count multiple times)

PHP Code:

get_pcount(bool:this, bool:that, bool:something)
{
    new players

    for(new x = 1; x <= MaxClients; x++)
    {
        if(IsClientInGame(x))
        {
            if(this && bla1)
            {
                players++
            }
            else if(that && bla2)
            {
                players++
            }
            else if(something && bla3)
            {
                players++
            }
        }
    }

    return players
} 


Or even this, although it may be a little harder to read

PHP Code:

if(IsClientInGame(x) && ((this && bla1) || (that && bla2) || (something && bla3)))
{
    players++
} 


[atom0s]

Edit :: Nvm for some reason my compiler messed up.. -.-