[dabv_astur]

Hi everyone, again me. This error seems to be very basic, but I do not understand what happens and I have the crazy head.

That's is my code:

pawn Code:

CMD:ir(playerid, params[])
{
    new idJugador;
    new Float:xJugador, Float:yJugador, Float:zJugador;
    new StringMen[64];
    if(pInfo[playerid][Admin] >= 1)
    {
        if(IsPlayerInAnyVehicle(playerid))
        {
              new vehiculo = GetPlayerVehicleID(playerid);
              new seat = GetPlayerVehicleSeat(playerid);
              GetPlayerPos(idJugador, xJugador, yJugador, zJugador);
              SetPlayerPos(playerid, xJugador, yJugador+2, zJugador);
              SetVehiclePos(vehiculo, xJugador, yJugador+2, zJugador);
              PutPlayerInVehicle(playerid, vehiculo, seat);
         }
         else
         {
            GetPlayerPos(idJugador, xJugador, yJugador, zJugador);
            SetPlayerPos(playerid, xJugador+1, yJugador+1, zJugador);
            
            format(StringMen, sizeof StringMen, "[IR] %s fue a %s", pInfo[playerid][Name], pInfo[idJugador][Name]);
            SendClientMessage(playerid, COLOR_RED, StringMen);
            
         }
    }
    else return SendClientMessage(playerid, COLOR_RED, "No eres staff o no tienes rango suficiente!");
    
    return 1;
}

The problem is when I try teleport with ID 0 to ID 1 I can't and I receive that SendClientMessage:

HTML Code:

[IR] DabvAsturDemo fue a DabvAsturDemo

My main account and also I am trying go to is named DabvAstur

Watch the video so understand me better.
https://youtube.com/watch?v=bdTVIX3_D2M

[OiLoL]

new vehiculo = GetPlayerVehicleID(playerid);
new seat = GetPlayerVehicleSeat(playerid);

You are doing wrong?

change it to :
idJugador

new vehiculo = GetPlayerVehicleID(idJugador);
new seat = GetPlayerVehicleSeat(idJugador);

You are getting player's vehicle ID and player's seat and placing the same player in that vehicle xD

[Rolux]

Don't listen to that guy.

You are not giving any value to the variable "idJugador",so it's value will be 0.
That causes you to teleport to ID 0.

[dabv_astur]

Quote:

Originally Posted by Rolux Don't listen to that guy. You are not giving any value to the variable "idJugador",so it's value will be 0. That causes you to teleport to ID 0.

Can you please tell me how I do that? With sscanf maybe?

[BiGuy]

Quote:

Originally Posted by dabv_astur Can you please tell me how I do that? With sscanf maybe?

Yes its easy check the examples https://sampforum.blast.hk/showthread.php?tid=570927

[dabv_astur]

Quote:

Originally Posted by BiGuy Yes its easy check the examples https://sampforum.blast.hk/showthread.php?tid=570927

if (sscanf(params, "u", idJugador)) return SendClientMessage(playerid, COLOR_RED, "Usa: /ir IDJUGADOR");

I add this line to my command but the result is same

EDIT:
Ok i change "u" parameter with "d" but with that I can't put the username on the command. Maybe because the variable is an integer and not string?

[TheToretto]

d and u are will work the same.

Show your new code.

[Flofey]

I don't know.. but it might work _:_ give it a try tho

PHP Code:

CMD:goto(playerid, params[])
{
    new idPlayer;
    new Float:xPlayer, Float:yPlayer, Float:zPlayer;
    new StringMen[64];
    new playername[MAX_PLAYER_NAME];
    new targetname[MAX_PLAYER_NAME];
    GetPlayerName(idPlayer,targetname,sizeof(targetname));
    GetPlayerName(playerid,playername,sizeof(playername));
    if(Info[playerid][Admin] >= 1)
    {
        if(sscanf(params,"u",idPlayer)) return SendClientMessage(playerid,-1, "/goto [ID]");
        if(!IsPlayerConnected(idPlayer)) return SendClientMessage(playerid, -1, "The ID is wrong or the player isn't online!");
        if(IsPlayerInAnyVehicle(playerid))
        {
              new vehicle = GetPlayerVehicleID(playerid);
              new seat = GetPlayerVehicleSeat(playerid);
              GetPlayerPos(idPlayer, xPlayer, yPlayer, zPlayer);
              SetPlayerPos(playerid, xPlayer, yPlayer+2, zPlayer);
              SetVehiclePos(vehicle, xPlayer, yPlayer+2, zPlayer);
              PutPlayerInVehicle(playerid, vehicle, seat);
         }
         else
         {
            GetPlayerPos(idPlayer, xPlayer, yPlayer, zPlayer);
            SetPlayerPos(playerid, xPlayer+1, yPlayer+1, zPlayer);

            format(StringMen, sizeof StringMen, "[GOTO] %s teleported to %s", playername, targetname);
            SendClientMessage(playerid, -1, StringMen);

         }
    }
    else return SendClientMessage(playerid, -1, "You're not allowed to use this command!");

    return 1;
} 


PS: sry for using English :P

[Mencent]

Quote:

Originally Posted by TheToretto d and u are will work the same.

That's wrong. d is only for integers and u is for strings and integers.

_______________________________________-


Try this:

PHP Code:

CMD:ir(playerid,params[])
{
    new idJugador,Float:xJugador,Float:yJugador,Float:zJugador,StringMen[64];
    if(pInfo[playerid][Admin] >= 1)
    {
        if(sscanf(params,"u",idJugador))return SendClientMessage(playerid,-1,"WARNING: /ir [playerid]");
        if(!IsPlayerConnected(idJugador))return SendClientMessage(playerid,-1,"playerid doesn't exists");
        if(IsPlayerInAnyVehicle(playerid))
        {
              new vehiculo = GetPlayerVehicleID(playerid);
              new seat = GetPlayerVehicleSeat(playerid);
              GetPlayerPos(idJugador, xJugador, yJugador, zJugador);
              SetPlayerPos(playerid, xJugador, yJugador+2, zJugador);
              SetVehiclePos(vehiculo, xJugador, yJugador+2, zJugador);
              PutPlayerInVehicle(playerid, vehiculo, seat);
              return 1;
         }
         else
         {
            GetPlayerPos(idJugador, xJugador, yJugador, zJugador);
            SetPlayerPos(playerid, xJugador+1, yJugador+1, zJugador);
            format(StringMen, sizeof StringMen, "[IR] %s fue a %s", pInfo[playerid][Name], pInfo[idJugador][Name]);
            SendClientMessage(playerid, COLOR_RED, StringMen);
            return 1;
         }
    }
    else return SendClientMessage(playerid, COLOR_RED, "No eres staff o no tienes rango suficiente!");
} 


[Flofey]

Quote:

Originally Posted by Mencent That's wrong. d is only for integers and u is for strings and integers. _______________________________________- Try this: PHP Code: CMD:ir(playerid,params[]) {     new idJugador,Float:xJugador,Float:yJugador,Float:zJugador,StringMen[64];     if(pInfo[playerid][Admin] >= 1)     {         if(sscanf(params,"u",idJugador))return SendClientMessage(playerid,-1,"WARNING: /ir [playerid]");         if(!IsPlayerConnected(idJugador))return SendClientMessage(playerid,-1,"playerid doesn't exists");         if(IsPlayerInAnyVehicle(playerid))         {               new vehiculo = GetPlayerVehicleID(playerid);               new seat = GetPlayerVehicleSeat(playerid);               GetPlayerPos(idJugador, xJugador, yJugador, zJugador);               SetPlayerPos(playerid, xJugador, yJugador+2, zJugador);               SetVehiclePos(vehiculo, xJugador, yJugador+2, zJugador);               PutPlayerInVehicle(playerid, vehiculo, seat);               return 1;          }          else          {             GetPlayerPos(idJugador, xJugador, yJugador, zJugador);             SetPlayerPos(playerid, xJugador+1, yJugador+1, zJugador);             format(StringMen, sizeof StringMen, "[IR] %s fue a %s", pInfo[playerid][Name], pInfo[idJugador][Name]);             SendClientMessage(playerid, COLOR_RED, StringMen);             return 1;          }     }     else return SendClientMessage(playerid, COLOR_RED, "No eres staff o no tienes rango suficiente!"); }

Excuse me Mencent, isn't what you did is the same as I did but in a different language :/

[Mencent]

Quote:

Originally Posted by Flofey Excuse me Mencent, isn't what you did is the same as I did but in a different language :/

Sorry, my mistake. Next time I will look better.

[CantBeJohn]

Quote:

Originally Posted by Mencent That's wrong. d is only for integers and u is for strings and integers. [/PHP]

Just to clarify, that's not right either and sounds misleading. "u" is explicitly for Usernames/IDs.

[Joe Staff]

Quote:

Originally Posted by Flofey Excuse me Mencent, isn't what you did is the same as I did but in a different language :/

Not quite, you were trying to use GetPlayerName before ever collecting the ID of the player from the parameter, but then you never actually used the players' names anyway.

[Mencent]

Quote:

Originally Posted by CantBeJohn Just to clarify, that's not right either and sounds misleading. "u" is explicitly for Usernames/IDs.

Well I mean the right but write the wrong. Sorry and thanks for your correction!

[Flofey]

Quote:

Originally Posted by Joe Staff Not quite, you were trying to use GetPlayerName before ever collecting the ID of the player from the parameter, but then you never actually used the players' names anyway.

Code:

format(StringMen, sizeof StringMen, "[GOTO] %s teleported to %s", playername,targetname);

and what do you call these?

[TheToretto]

Use GetPlayerName after the sscanf

[Flofey]

Thanks for correcting me.

PHP Code:

CMD:goto(playerid, params[])
{

    new idPlayer,Float:xPlayer, Float:yPlayer, Float:zPlayer, StringMen[64], playername[MAX_PLAYER_NAME], targetname[MAX_PLAYER_NAME];
    if(IsPlayerAdmin(playerid))
    {
        if(sscanf(params,"u",idPlayer)) return SendClientMessage(playerid,-1, "/goto [ID]");
        if(!IsPlayerConnected(idPlayer)) return SendClientMessage(playerid, -1, "The ID is wrong or the player isn't online!");
        if(IsPlayerInAnyVehicle(playerid))
        {
              new vehicle = GetPlayerVehicleID(playerid);
              new seat = GetPlayerVehicleSeat(playerid);
              GetPlayerPos(idPlayer, xPlayer, yPlayer, zPlayer);
              SetVehiclePos(vehicle, xPlayer, yPlayer+2, zPlayer);
              PutPlayerInVehicle(playerid, vehicle, seat);
              GetPlayerName(idPlayer,targetname,sizeof(targetname));
              GetPlayerName(playerid,playername,sizeof(playername));
              format(StringMen, sizeof StringMen, "[GOTO] %s teleported to %s", playername, targetname);
              SendClientMessage(playerid, -1, StringMen);
         }
         else
         {
            GetPlayerPos(idPlayer, xPlayer, yPlayer, zPlayer);
            SetPlayerPos(playerid, xPlayer+1, yPlayer+1, zPlayer);

            GetPlayerName(idPlayer,targetname,sizeof(targetname));
            GetPlayerName(playerid,playername,sizeof(playername));
            format(StringMen, sizeof StringMen, "[GOTO] %s teleported to %s", playername, targetname);
            SendClientMessage(playerid, -1, StringMen);

         }
    }
    else return SendClientMessage(playerid, -1, "You're not allowed to use this command!");

    return 1;
}