SA-MP Forums Archive
/w - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: /w (/showthread.php?tid=650199)


/w - InSain - 23.02.2018

I'm trying to make it so that the player id is optional and that it defaults to yourself with sscanf. When I run the command without the player id, it doesn't give me the "Usage" output, indicating that sscanf recognizes that it's optional, but it doesn't give me the playerid.

Код:
CMD:w(playerid, params[])
{
	new weapid, recieverid, ammo;
	recieverid = playerid;
	if (!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, COLOR_GRAY, "You are not authorized to use that command!");
	if (sscanf(params, "diU", weapid, ammo, recieverid)) return SendClientMessage(playerid, COLOR_GRAY, "Usage: /w [weapon id] [ammo] [player id]");
	if (weapid < 0 || weapid > 46) return SendClientMessage(playerid, COLOR_GRAY, "Invalid weapon ID!");
	else
		{
		GivePlayerWeapon(recieverid, weapid, ammo);
		}
	return 1;
}


Re: /w - BulletRaja - 23.02.2018

EDIT: removed
2: you cant give weapon without playerid to anyone
3: by removing playerid only who press /w [weap id] [ammo] will get the gun.

Re: /w - RogueDrifter - 23.02.2018

Quote:
Originally Posted by BulletRaja
Посмотреть сообщение
PHP код:
CMD:w(playerid, params[])
{
    new weapid, ammo;
    if (!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, COLOR_GRAY, "You are not authorized to use that command!");
    if (sscanf(params, "di", weapid, ammo)) return SendClientMessage(playerid, COLOR_GRAY, "Usage: /w [weapon id] [ammo]");
    if (weapid < 0 || weapid > 46) return SendClientMessage(playerid, COLOR_GRAY, "Invalid weapon ID!");
    else
        {
        GivePlayerWeapon(playerid, weapid, ammo);
        }
    return 1;
} 
1- Don't just send code, explain it.
2- That code will never work.

Re: /w - BulletRaja - 23.02.2018

Quote:
Originally Posted by RogueDrifter
Посмотреть сообщение
1- Don't just send code, explain it.
2- That code will never work.
ok edited

Re: /w - rfr - 23.02.2018

Quote:
Originally Posted by RogueDrifter
Посмотреть сообщение
1- Don't just send code, explain it.
2- That code will never work.
that code that bulletraja sent works fine for me, what you on about

Re: /w - RogueDrifter - 23.02.2018

Quote:
Originally Posted by rfr
Посмотреть сообщение
that code that bulletraja sent works fine for me, what you on about
The thread:
Quote:
Originally Posted by InSain
I'm trying to make it so that the player id is optional
What he posted:
Quote:

if (sscanf(params, "di", weapid, ammo)) return SendClientMessage(playerid, COLOR_GRAY, "Usage: /w [weapon id] [ammo]");
How's that going to give weapons to other players?

Re: /w - rfr - 23.02.2018

Quote:
Originally Posted by RogueDrifter
Посмотреть сообщение
1- Don't just send code, explain it.
2- That code will never work.
Quote:
Originally Posted by RogueDrifter
Посмотреть сообщение
The thread:


What he posted:


How's that going to give weapons to other players?
why don't you run the code and see for yourself



Re: /w - Kane - 23.02.2018

PHP код:
CMD:w( playerid, params[] ) {
    new weapid, ammo, 
        userid;
    if( !IsPlayerAdmin( playerid ) )
        return SendClientMessage(playerid, COLOR_GRAY, "You are not authorized to use that command!"); 
    if( sscanf( params, "iiU(65535)", weapid, ammo, userid ) ) 
        return SendClientMessage( playerid, COLOR_GRAY, "Usage: /w [weapon id] [ammo] [optional playerid]" );
    if( weapid < 0 || weapid > 46 )
        return SendClientMessage(playerid, COLOR_GRAY, "Invalid weapon ID!"); 
    if( ammo <= 0 )
        return SendClientMessage(playerid, COLOR_GRAY, "Invalid ammo!"); 
    if( userid == INVALID_PLAYER_ID ) {
        userid = playerid;
    } else {
        if( !IsPlayerConnected( userid ) )
            return SendClientMessage( playerid, COLOR_GRAY, "Player isn't connected." );
    }
    GivePlayerWeapon( userid, weapid, ammo );
    SendClientMessage( playerid, COLOR_GRAY, "Gun spawned!" );
    return 1;
} 
Read up on sscanf specifiers. A capital U means optional.
https://sampforum.blast.hk/showthread.php?tid=570927

Re: /w - Stev - 23.02.2018

Staying on topic of this thread, you had the right idea with the code itself, just didn't set it up correctly.

Using sscanf you can set a default value if the param is not used.

EDIT: Use @Arthur Kane, he beat me too it

Re: /w - kingmk - 23.02.2018

I hope i helped u, i didn't test the code.

Код:
CMD:w(playerid, params[])
{
	new weapid, recieverid, ammo;
	if (!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, COLOR_GRAY, "You are not authorized to use that command!");
        if(sscanf(params, "diu", weapid, ammo, recieverid))
        {
               if (weapid < 0 || weapid > 46) return SendClientMessage(playerid, COLOR_GRAY, "Invalid weapon ID!");
               GivePlayerWeapon(playerid, weapid, ammo);
        }
        else if(!sscanf(params, "diu", weapid, ammo, recieverid))
        {
               if(IsPlayerConnected(recieverid))
               {
                          if(weapid < 0 || weapid > 46) return SendClientMessage(playerid, COLOR_GRAY, "Invalid weapon ID!");
                          GivePlayerWeapon(recieverid, weapid, ammo);
               }
        }
        else return SendClientMessage(playerid, COLOR_WHITE,"Usage: /w [WeapID] [Ammo] [PlayerID(Optional)]");
        return 1;
}