[Moron]

When I try to apply foreach, commands where you need to enter a players name say that there is no such player, what did I do wrong?

Original:

Код:

stock GetPlayeridMid(name[])
{
    for (new i = 0; i < MAX_PLAYERS; i++)
    {
        if (IsPlayerConnected(i))
        {
            new gPlayerName[MAX_PLAYER_NAME];
            GetPlayerName(i, gPlayerName, MAX_PLAYER_NAME);
            if(strfind(gPlayerName, name, true) != -1)
            {
                return i;
            }
        }
    }
return INVALID_PLAYER_ID;
}

With foreach:

Код:

stock GetPlayeridMid(name[])
{
        foreach(Player, i)
        {
            new gPlayerName[MAX_PLAYER_NAME];
            GetPlayerName(i, gPlayerName, MAX_PLAYER_NAME);
            if(strfind(gPlayerName, name, true) != -1)
            {
                return i;
            }
        }
return INVALID_PLAYER_ID;
}

And can I replace something like this with foreach, if yes, how?

Код:

new AutoSavet[MAX_PLAYERS];

[Threshold]

Try using this:

pawn Код:

stock GetPlayeridMid(name[])
{
        foreach(Player, i)
        {
            new gPlayerName[MAX_PLAYER_NAME];
            GetPlayerName(i, gPlayerName, MAX_PLAYER_NAME);
            if(strfind(gPlayerName, name, true) != -1)
            {
                return i;
                break;
            }
        }
    return INVALID_PLAYER_ID;
}

[Moron]

I get unreachable code error

Tried putting break; a bit lower, the error was gone, but it still says no such player.

PHP код:

stock GetPlayeridMid(name[])
{
        foreach(Player, i)
        {
            new gPlayerName[MAX_PLAYER_NAME];
            GetPlayerName(i, gPlayerName, MAX_PLAYER_NAME);
            if(strfind(gPlayerName, name, true) != -1)
            {
                return i;
            }
         break;
        }
    return INVALID_PLAYER_ID;
} 


[Threshold]

Try putting the break; ABOVE the return i; If that doesn't work, then I'm not sure what will :S

[Moron]

Quote:

Originally Posted by BenzoAMG Try putting the break; ABOVE the return i; If that doesn't work, then I'm not sure what will :S

Nope :S

[Vukilore]

Just delete the break, it stop the action of foreach.

[Moron]

Quote:

Originally Posted by Vukilore Just delete the break, it stop the action of foreach.

The original change had no break; ,but as I said in the first post, it says that there is no such player.

[Moron]

Oh how stupid of me :S I was testing the commands on NPCs and foreach only loops through players The original change works fine.

Though another question:

Is this replaceable with foreach, if yes, how:

pawn Код:

new AutoSavet[MAX_PLAYERS];