[Slice]

Personally, I really don't like the way people use "return SendClientMessage". The return value of SendClientMessage has nothing to do with the return value of a command - it just happens to be the value you want to return anyway.

I'd say something like this is better:

pawn Код:

CMD:connected(playerid, params[]) {
    if (!IsPlayerConnected(playerid))
        SendClientMessage(playerid, -1, "For some reason, you are not connected.");
    else
        SendClientMessage(playerid, -1, "You are connected.");
    
    return 1;
}

CMD:kick(playerid, params[]) {
    new otherid = strval(params);
    
    if (!HasPermission(playerid))
        SendClientMessage(playerid, -1, "You don't have permission.");
    else if (!IsPlayerConnected(otherid))
        SendClientMessage(playerid, -1, "Invalid player id given.");
    else if (IsPlayerAdmin(otherid))
        SendClientMessage(playerid, -1, "Other player is admin.");
    else {
        Kick(otherid);
        
        SendClientMessage(playerid, -1, "KDONE");
    }
    
    return 1;
}