[Simon]

Re-ordering the execution of code so it "doesn't matter" is not correct! You are still silent crashing the server, you are still possibly corrupting memory with your out-of-bounds array write. You should be checking if the killerid is invalid by using:

pawn Код:

if (killerid != INVALID_PLAYER_ID)

or

pawn Код:

if (IsPlayerConnected(killerid))

The correct way this code should be written is like so (also used the increment/decrement operators to shorten it),

pawn Код:

public OnPlayerDeath(playerid, killerid, reason)
{
    SendDeathMessage(killerid, playerid, reason);

    PlayerInfo[playerid][pDeaths]++;
    PlayerInfo[playerid][pScore]--;

    SetPlayerScore(playerid, GetPlayerScore(playerid) - 1);

    if (killerid != INVALID_PLAYER_ID)
        PlayerInfo[killerid][pScore]++;
        PlayerInfo[killerid][pKills]++;
        PlayerInfo[killerid][pGameKills]++;

        SetPlayerScore(killerid, GetPlayerScore(killerid) + 1);
    }

    return 1;
}