Помогите с командой!

[Sash.]

Всем привет. Изучаю язык pawn в целом, около месяца, дается тяжело.
При создании команды /setskin я представлял ее отображение в следующем виде: "Администратор Denis_Ivanov изменил Ваш скин" - это сообщение для игрока, которому был изменен скин.
Для админа, сменившего скин следующий текст: "Вы изменили скин игроку Ivan_Denisov".
Но в игре, ни одно имя не отображается.
Прошу помощи.
По поводу быдлокода не ругайте, я ж новичек.
Сам код:

CMD:setskin(playerid,params[])
{
new string[128], aName[MAX_PLAYER_NAME],str[128];
GetPlayerName(playerid,aName,sizeof(aName));
if(!IsPlayerAdmin(playerid)) return SCM(playerid,0x696969AA,"Неизвестная команда");
if(sscanf(params,"dd",params[0],params[1])) return SCM(playerid,-1,"Используйте: /setskin [ID игрока] [ID скина]");
if(!IsPlayerConnected(params[0])) return SCM(playerid,-1,"Игрока нет на сервере");
if((params[1] < 0) || (params[1] > 311)) return SCM(playerid,-1,"ID cкина не менее 0 и не более 311!");
format(string,sizeof(string),"Администратор %s изменил Ваш скин", aName);
format(str,sizeof(str),"Вы изменили скин игроку %s", params[0]);
SetPlayerSkin(params[0],params[1]);
if(IsPlayerAdmin(playerid)) return SCM(playerid,0x00C0FFAA,str);
SCM(params[0],0x00C0FFAA,string);
return 1;
}

[DeimoS]

Ты ник не получил второго игрока. Ну и есть другие проблемы. Вот так, в общем
CMD:setskin_id(playerid,params[])
{
if(!IsPlayerAdmin(playerid))
return SendClientMessage(playerid,0x696969AA,"Неизвестная команда");

new giveplayerid,
skin_id;
if(sscanf(params,"ud", giveplayerid, skin_id))
return SendClientMessage(playerid,-1,"Используйте: /setskin_id [ID игрока] [ID скина]");

if(giveplayerid == INVALID_PLAYER_ID)
return SendClientMessage(playerid,-1,"Игрока нет на сервере");

if((skin_id < 0) || (skin_id > 311))
return SendClientMessage(playerid,-1,"ID cкина не менее 0 и не более 311!");

new string[33+MAX_PLAYER_NAME+1];

GetPlayerName(giveplayerid, string, MAX_PLAYER_NAME);
format(string, sizeof(string), "Вы изменили скин игроку %s", string);
SendClientMessage(playerid, 0x00C0FFAA, string);

GetPlayerName(playerid, string, MAX_PLAYER_NAME);
format(string,sizeof(string),"Администратор %s изменил Ваш скин", string);
SendClientMessage(giveplayerid, 0x00C0FFAA, string);

SetPlayerSkin(giveplayerid, skin_id);
return 1;
}

[Sash.]

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

Спасибо! А какие другие проблемы? Укажите, приму во внимание

[Skyline]

1. К примеру ты юзаешь массив, вместо переменных, которые работают быстрее.
2. Указываешь индификатор, который получает только число а не ник/ID.
3. Сделать лучше в enum pInfo:

pName
в OnPlayerConnect

GetPlayerName(playerid, pInfo[playerid][pName], MAX_PLAYER_NAME);
Чтобы постоянно не узнавать имя игрока, а юзать pInfo[playerid][pName].
Размер массива, для этого 128 - слишком большой, выделяешь много лишнего и даешь нагрузку ненужную.

[DeimoS]

Ну, как видишь, я, например, перенёс вот этот код

new string[128], aName[MAX_PLAYER_NAME],str[128];
GetPlayerName(playerid,aName,sizeof(aName));
после всех условий, дабы он не срабатывал впустую тогда, когда игрок, например, не админ.

Так же, в твоём случае, можно было обойтись одним массивом, а не тремя, что тоже видно в моём примере.

Ну и лучше сразу привыкать писать нормальные имена функций/переменных, а не использовать SCM/params[0]. Это на производительность не влияет (ну не во всех случаях), но читаемость кода ухудшается, что тоже немаловажно.