Ayuda con pNick

[godoy32]

Hola. Bueno desde hace ya mucho tiempo llevo usando la función pNick(playerid) (originalmente la saqué de garsinio), es una función donde se obtiene el nombre del jugador
Resulta que desde que actualicé (mas o menos) a la 0.3d, me está dando muchos problemas.
Por ej: con el comando /congelar

//Con pNick dice que el comando NO EXISTE
COMMAND:congelar(playerid,params[])
{
    if(Informacion[playerid][Admin] >= 1)
    {
        new id;
        if(sscanf(params,"d",id))
        {
            if(Informacion[id][Idioma]==0) SendClientMessage(playerid,-1,"Usa: {FF0000}/congelar {FFFFFF}[ID/Nombre]");
            if(Informacion[id][Idioma]==1) SendClientMessage(playerid,-1,"Use: {FF0000}/freeze {FFFFFF}[ID/Name]");
            return 1;
        }
        if(!IsPlayerConnected(id)) return SendClientMessage(playerid,0xFF0000FF,"Jugador no conectado");
        TogglePlayerControllable(id,0);
        new st[128 char];
        if(Informacion[id][Idioma]==0) format(st,128,"Has congelado a {FF0000}%s (%d)",pNick(id),id);
        if(Informacion[id][Idioma]==1) format(st,128,"You freeze {FF0000}%s (%d)",pNick(id),id);
        SendClientMessage(playerid,-1,st);
        if(Informacion[id][Idioma]==0) SendClientMessage(id,-1,"Has sido congelado por {FF0000}un admin");
        if(Informacion[id][Idioma]==1) SendClientMessage(id,-1,"You was freeze by{FF0000} admin");
        return 1;
    }
    return 1;
}
//Pero si saco el pNick y pongo por ej: "id"
COMMAND:congelar(playerid,params[])
{
    if(Informacion[playerid][Admin] >= 1)
    {
        new id;
        if(sscanf(params,"d",id))
        {
            if(Informacion[id][Idioma]==0) SendClientMessage(playerid,-1,"Usa: {FF0000}/congelar {FFFFFF}[ID/Nombre]");
            if(Informacion[id][Idioma]==1) SendClientMessage(playerid,-1,"Use: {FF0000}/freeze {FFFFFF}[ID/Name]");
            return 1;
        }
        if(!IsPlayerConnected(id)) return SendClientMessage(playerid,0xFF0000FF,"Jugador no conectado");
        TogglePlayerControllable(id,0);
        new st[128 char];
        if(Informacion[id][Idioma]==0) format(st,128,"Has congelado a {FF0000}%s (%d)",id,id);
        if(Informacion[id][Idioma]==1) format(st,128,"You freeze {FF0000}%s (%d)",id,id);
        SendClientMessage(playerid,-1,st);
        if(Informacion[id][Idioma]==0) SendClientMessage(id,-1,"Has sido congelado por {FF0000}un admin");
        if(Informacion[id][Idioma]==1) SendClientMessage(id,-1,"You was freeze by{FF0000} admin");
        return 1;
    }
    return 1;
}
//el comando anda PERFECTO (obviamente no muestra el nombre)
stock pNick(playerid)
{
    new nick[MAX_PLAYER_NAME];
    GetPlayerName(playerid, nick, MAX_PLAYER_NAME);
    return nick;
}

AGREGO:
Anduve probando con GetPlayerName

new name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,24);

Pero tampoco funciona :S

[TiNcH010]

Originally Posted by godoy32

Anduve probando con GetPlayerName

new name[MAX_PLAYER_NAME];
    GetPlayerName(playerid,name,24);

Pero tampoco funciona :S

Por que definis con MAX_PLAYER_NAME si después usas 24.
Podes probar de 2 formas:

new name[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, sizeof (name));

ó

new name[24];
GetPlayerName(playerid, name, 24);

[the_chaoz]

Estas causando un overflow, el tamaño de st deberia ser 128, no 128 chars.

Originally Posted by TiNcH010

Por que definis con MAX_PLAYER_NAME si después usas 24.
Podes probar de 2 formas:

new name[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, sizeof (name));

ó

new name[24];
GetPlayerName(playerid, name, 24);

Eso es porque si el día de mañana, se admiten nombres de 32 caracteres, deberas cambiar todo tu script o podrias causar un overflow, en cambio si utilizas MAX_PLAYER_NAME no tendras este problema.

[godoy32]

Originally Posted by the_chaoz

Estas causando un overflow, el tamaño de st deberia ser 128, no 128 chars.
(...)

Perfecto, era eso. Pero cuando se debería usar 'char'?

[the_chaoz]

se lo utiliza para strings comprimidos/empaquetados.

[godoy32]

Originally Posted by the_chaoz

se lo utiliza para strings comprimidos/empaquetados.

Perfecto. Muchas gracias por tu ayuda!