Подсчет ячеек

[DmitriyVasilev]

Здравствуйте, правильно ли я сделал тут всё или что-то нужно редактировать?

CMD:slap(playerid, params[]){    if(User[playerid][pAdmin] < 3 || dostup[playerid] == 0) return 1;    if(sscanf(params, "u",params[0])) return SCM(playerid, -1, !"Введите: /slap [id]");    new Float:shealth;    new Float:slx, Float:sly, Float:slz;    if(!IsPlayerConnected(params[0])) return true;    GetPlayerHealth(params[0], shealth);    SetPlayerHealthAC(params[0], shealth-5);    GetPlayerPos(params[0], slx, sly, slz);    SetPlayerPos(params[0], slx, sly, slz+5);    PlayerPlaySound(params[0], 1130, slx, sly, slz+5);    static const fmt_str[] = "Админ: %s дал поджопник %s";    static a_slap[sizeof fmt_str + ( MAX_PLAYER_NAME*2 - 4 )];    format(a_slap, sizeof a_slap, fmt_str, Name(playerid),Name(params[0]));    ABroadCast(COLOR_LIGHTRED,a_slap,1);    SCM(params[0],COLOR_LIGHTRED,a_slap);    return 1;}

[m1n1vv]

CMD:slap(playerid, params[]) 
{ 
    if (User[playerid][pAdmin] < 3 || dostup[playerid] == 0) 
        return 1; 
 
    if (isnull(params)) 
        return SCM(playerid, -1, !"Введите: /slap [id]"); 
 
    static const 
        fmt_str[] = "Админ: %s дал поджопник %s"; 
 
    static 
        id, 
        Float:shealth, 
        Float:slx, Float:sly, Float:slz, 
        a_slap[sizeof fmt_str + MAX_PLAYER_NAME*2 - 2*2]; 
 
    id = strval(params); 
 
    if (!IsPlayerConnected(id)) 
        return true; 
 
    GetPlayerHealth(id, shealth); 
    SetPlayerHealthAC(id, shealth-5); 
 
    GetPlayerPos(id, slx, sly, slz); 
    SetPlayerPos(id, slx, sly, slz+5); 
 
    PlayerPlaySound(id, 1130, slx, sly, slz+5); 
     
    format(a_slap, sizeof a_slap, fmt_str, Name(playerid), Name(id)); 
 
    ABroadCast(COLOR_LIGHTRED, a_slap, 1); 
    SCM(id, COLOR_LIGHTRED, a_slap); 
     
    return 1; 
}  

- - - Добавлено - - -

Еще можно так:

new
    a_slap[18] = !"Админ: ";

strcat(a_slap, Name(playerid));
strcat(a_slap, !" дал поджопник ");
strcat(a_slap, Name(id));  

[UnO]

Еще можно так:

new
    a_slap[18] = !"Админ: ";

strcat(a_slap, Name(playerid));
strcat(a_slap, !" дал поджопник ");
strcat(a_slap, Name(id));  

Это, по-моему, уже совсем извращения в стиле аптимизаторов3000

[m1n1vv]

Это, по-моему, уже совсем извращения в стиле аптимизаторов3000

Все ради скорости и экономии памяти

[UnO]

Все ради скорости и экономии памяти

Ага, но стоит всё-таки не выходить за рамки разумного. Читабельность от этого сильно страдает, имхо

[x86]

Ага, но стоит всё-таки не выходить за рамки разумного. Читабельность от этого сильно страдает, имхо

Не так уж и сильно. Из того блока кода понятно, что строка конкатенируются с двумя именами между которыми строка.

[UnO]

Смысл в том, что ощутимой разницы между форматом и этим способом на практике нет, но читабельность хуже; так зачем себе палки в колёса вставлять?

[m1n1vv]

Смысл в том, что ощутимой разницы между форматом и этим способом на практике нет, но читабельность хуже; так зачем себе палки в колёса вставлять?

1 format = 7 strcat
Перебор, это когда с strcat используешь valstr

[DmitriyVasilev]

а я вот не пойму, чем например тот же isnull и вот эти strtok сделают что-то лучше.

[x86]

а я вот не пойму, чем например тот же isnull и вот эти strtok сделают что-то лучше.

Зачем вызывать целую функцию sscanf, если можно просто проверить значение в ячейке через isnull. Где вы увидели функцию strtok?