Не работает команда /kick

Привет всем. Помогите с проблемой:

PHP код:

---

Richard_Moretz кикнул игрока 37. Причина: 1337 


---

Вот код:

PHP код:

---

CMD:kick(playerid,params[])
{
    if(pInfo[playerid][pAdmin]>=1){
        new string[128], name[MAX_PLAYER_NAME];
        new kickname[MAX_PLAYER_NAME], giveplayerid, reason;
        if (sscanf(params, "us", giveplayerid, reason)) return SendClientMessage(playerid, COLOR_WHITE, "Введите {FF0000}/kick [ID] [причина]{FFFFFF}");
        GetPlayerName(playerid,name,sizeof(name));
        GetPlayerName(giveplayerid,kickname,sizeof(kickname));
        format(string,sizeof(string),"%s кикнул игрока %s. Причина: %s", name, kickname, reason);
        SendClientMessageToAll(COLOR_RED, string);
        Kick(giveplayerid);
    }else{
        SendClientMessage(playerid,COLOR_RED,"У вас нету прав!");
    }
    return 1;
} 


---

Искал, искал, да так ошибку и не нашел. :mda:

Нужно указывать в sscanf размер строковой переменной, иначе функция не будет знать, до скольки символов можно считывать, и будет выводить предупреждающие сообщения в консоли сервера.
В вашем случае должна получиться строка "us[128]".

Цитата:

---

Сообщение от Daniel_Cortez

Нужно указывать в sscanf размер строковой переменной, иначе функция не будет знать, до скольки символов можно считывать, и будет выводить предупреждающие сообщения в консоли сервера.
В вашем случае должна получиться строка "us[128]".

---

Дело не в этом. При вводе команды /kick 0 test, в чате отображается только

PHP код:

---

Richard_Moretz кикнул игрока st . Причина: test 


---

и меня не кикает. Должно корректно отобразиться сообщение и меня должно кикнуть.

Только сейчас заметил. Да, дело не только в форматной строке. На объявление reason посмотрите, это должен быть массив.
Да и зачем создавать ещё один массив, когда можно считать причину кика в string?

Deimos подробно описал эту проблему, посмотрите.
http://pro-pawn.ru/showthread.php?93...C%D0%B0-sscanf

Цитата:

---

Сообщение от Mazzilla

Deimos подробно описал эту проблему, посмотрите.
http://pro-pawn.ru/showthread.php?93...C%D0%B0-sscanf

---

Это совсем другая проблема. Да и решение там, как оказалось, неправильное.
Недавно только помог разобрать похожий случай, гляньте решение:

http://pro-pawn.ru/showthread.php?12472

Цитата:

---

Сообщение от Daniel_Cortez

Это совсем другая проблема. Да и решение там, как оказалось, неправильное.
Недавно только помог разобрать похожий случай, гляньте решение:

http://pro-pawn.ru/showthread.php?12472

---

Можно поинтересоваться, почему там решение не правильное?
Была подобная проблема, объявил переменные > стало все нормально работать.

А переменная reason и вправду не правильно объявлена, Вы же вводите текст, нужен массив.

А разве так сделать нельзя?

PHP код:

---

CMD:kick(playerid,params[])
{
    if(pInfo[playerid][pAdmin] >= 1)
        return SendClientMessage(playerid,COLOR_RED,"У вас нету прав!");
    
    if(sscanf(params, "us[21]", params[0], params[1])) 
        return SendClientMessage(playerid, COLOR_WHITE, "Введите {FF0000}/kick [ID] [причина]");
    
    if(params[1] >= 21)
        return SendClientMessage(playerid, COLOR_WHITE, "Причина не может привышать 20 символов");
    
    new 
        string[128], 
        name[MAX_PLAYER_NAME],
        kickname[MAX_PLAYER_NAME]
    ;
    
    GetPlayerName(playerid, name, sizeof(name));
    GetPlayerName(params[0], kickname, sizeof(kickname));  

    format(string, sizeof(string), "%s кикнул игрока %s. Причина: %s", name, kickname, params[1]);
    SendClientMessageToAll(COLOR_RED, string);
    
    return 1;
} 


---

Цитата:

---

Сообщение от Bublik_Public

Можно поинтересоваться, почему там решение не правильное?

---

По поводу коллизии DeimoS был прав, но проблема была ещё и в том, что между "s" и следующим спецификатором нужно было поставить пробел.
Как результат, топикстартер так и не разобрался до конца и просто убрал второй параметр из команды -_-

Цитата:

---

Сообщение от Darge

А разве так сделать нельзя?

PHP код:

---

CMD:kick(playerid,params[])
{
    if(pInfo[playerid][pAdmin] >= 1)
        return SendClientMessage(playerid,COLOR_RED,"У вас нету прав!");
    
    if(sscanf(params, "us[21]", params[0], params[1])) 
        return SendClientMessage(playerid, COLOR_WHITE, "Введите {FF0000}/kick [ID] [причина]");
    
    if(params[1] >= 21)
        return SendClientMessage(playerid, COLOR_WHITE, "Причина не может привышать 20 символов");
    
    new 
        string[128], 
        name[MAX_PLAYER_NAME],
        kickname[MAX_PLAYER_NAME]
    ;
    
    GetPlayerName(playerid, name, sizeof(name));
    GetPlayerName(params[0], kickname, sizeof(kickname));  

    format(string, sizeof(string), "%s кикнул игрока %s. Причина: %s", name, kickname, params[1]);
    SendClientMessageToAll(COLOR_RED, string);
    
    return 1;
} 


---

---

ИМХО, юзать params[0/1] под одиночные переменные - редкостное извращение. Доступ к ячейкам массива медленнее, чем к обычным переменным, да и инструкций AMX от этого будет только больше => больше кода => больше памяти потребуется серверу для загрузки скрипта. Вообще использование массива params оправдано только если нужно использовать его вместо какого-то другого массива - но только под строковое значение (или под массив значений), считанное с помощью sscanf (оно всегда вместится в массив, т.к. размер params зависит от длины введённых параметров команды), иначе строковое значение может не вместиться в params => словите выход за пределы массива.

Цитата:

---

Сообщение от Daniel_Cortez

Только сейчас заметил. Да, дело не только в форматной строке. На объявление reason посмотрите, это должен быть массив.
Да и зачем создавать ещё один массив, когда можно считать причину кика в string?

---

Спасибо за совет, проблема решена. Удалил переменную reason, все записал в string.