[MP2]

It's bad practise to write things all on one line. It looks very messy and can become confusing.

Compare:

pawn Code:

if(IsPlayerAdmin(playerid)) { pAdmin[playerid] = 1; } else { pAdmin[playerid] = 0; }

pawn Code:

if(IsPlayerAdmin(playerid))
{
    pAdmin[playerid] = 1;
}
else
{
    pAdmin[playerid] = 0;
}

Even though for this you should do:

pawn Code:

pAdmin[playerid] = IsPlayerAdmin(playerid);

But that's not relevant, I just couldn't think of another simple example.