[ahameed4755]

Код:

CMD:agiveweapon(playerid,params[])
{
new targetid;
new TargetID,Weapon,Ammo;
if(!IsPlayerAdmin(playerid)) return SendClientMessage(playerid,0xFFFFFF,"you are not rcon admin");
if(sscanf(params,"u",targetid)) return SendClientMessage(playerid,0xFFFFFF,"usage playerid");
if(!IsPlayerConnected(targetid)) return SendClientMessage(playerid,0xFFFFFF,"player is not connected");
if(sscanf(params,"udd",TargetID,Weapon,Ammo));
SendClientMessage(playerid,0xFFFFFF,"player has recieved the weapon!");
SendClientMessage(targetid,0xFFFFFF,"you have recieved a weapon!");
return 1;
}

\godmode.pwn(121) : error 036: empty statement/

[Aliassassin123456]

PHP код:

CMD:agiveweapon(playerid, params[])
{
    new TargetID, Weapon, Ammo;
    if(!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, 0xFFFFFF, "You are not rcon admin.");
    if(sscanf(params,"udd", TargetID, Weapon, Ammo)) return SendClientMessage(playerid,0xFFFFFF,"Syntax: /agiveweapon <PlayerID/PartOfName> <WeaponID> <Ammo>");
    SendClientMessage(playerid,0xFFFFFF,"Player has recieved the weapon!");
    SendClientMessage(targetid,0xFFFFFF,"You have recieved a weapon!");
    return 1;
} 


But that player will never get their weapon You don't give anything to them!

[Andre02]

if/else if/else statements do not have semicolons ';' at the end!

Wrong:

pawn Код:

if (a == b);
{
    a++;
}

Correct:

pawn Код:

if (a == b)
{
    a++;
}

You should see your code again though..

[saffierr]

ahameed, use GivePlayerWeapon();
And why are you using sscanf twice?
I don't know if that's incorrect or smth, but I've never seen it before.

[Sew_Sumi]

Quote:

Originally Posted by saffierr ahameed, use GivePlayerWeapon(); And why are you using sscanf twice? I don't know if that's incorrect or smth, but I've never seen it before.

PHP код:

if(sscanf(params,"udd",TargetID,Weapon,Ammo)); 


That is where the issue is... That, is an empty statement, as it doesn't have the { }, or anything done in it after the if.

Quote:

Originally Posted by Andre02 if/else if/else statements do not have semicolons ';' at the end!

That is incorrect.

You can make them on one line, and they can have semicolons at the end of them, meaning it's the end of the if portion.

[saffierr]

So, the code you need is this

PHP код:

CMD:agiveweapon(playerid, params[])
{
    new targetid, weapon, ammo;
    if(!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, 0xFFFFFF, "You are not an rcon admin!");
    if(sscanf(params, "udd", targetid, weapon, ammo)) return SendClientMessage(playerid, 0xFFFFFF, "Usage: /agiveweapon [playerid]");
    if(!IsPlayerConnected(targetid)) return SendClientMessage(playerid, 0xFFFFFF, "Player is not connected!");
    if(weapon < 1 || weapon > 46) return SendClientMessage(playerid, COLOR_RED, "Valid weaponids are 1 - 46"); // Because weaponid 88 is not valid...
    
    new string[75], pname[MAX_PLAYER_NAME]; // This is to get the TARGET name  (optional)
    GivePlayerWeapon(targetid, weapon, ammo); // This will give the weapon to the targetid
    GetPlayerName(targetid, pname, MAX_PLAYER_NAME); // This is to get the TARGETS name
    format(string, sizeof string, "%s has received the weapon!", pname); // format is to define a variable,  pname is the TARGETS name
    SendClientMessage(playerid, 0xFFFFFF, string); // The player who committed the cmd, gets the message from the format above
    SendClientMessage(targetid, 0xFFFFFF, "You have received a weapon!"); // Why didn't I use format here either? well, because there's no variable to define
    return 1;
} 


I hope it works, as I didn't test it.

[ahameed4755]

Quote:

Originally Posted by saffierr So, the code you need is this PHP код: CMD:agiveweapon(playerid, params[]) {     new targetid, weapon, ammo;     if(!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, 0xFFFFFF, "You are not an rcon admin!");     if(sscanf(params, "udd", targetid, weapon, ammo)) return SendClientMessage(playerid, 0xFFFFFF, "Usage: /agiveweapon [playerid]");     if(!IsPlayerConnected(targetid)) return SendClientMessage(playerid, 0xFFFFFF, "Player is not connected!");     if(weapon < 1 || weapon > 46) return SendClientMessage(playerid, COLOR_RED, "Valid weaponids are 1 - 46"); // Because weaponid 88 is not valid...          new string[75], pname[MAX_PLAYER_NAME]; // This is to get the TARGET name  (optional)     GivePlayerWeapon(targetid, weapon, ammo); // This will give the weapon to the targetid     GetPlayerName(targetid, pname, MAX_PLAYER_NAME); // This is to get the TARGETS name     format(string, sizeof string, "%s has received the weapon!", pname); // format is to define a variable,  pname is the TARGETS name     SendClientMessage(playerid, 0xFFFFFF, string); // The player who committed the cmd, gets the message from the format above     SendClientMessage(targetid, 0xFFFFFF, "You have received a weapon!"); // Why didn't I use format here either? well, because there's no variable to define     return 1; }  I hope it works, as I didn't test it.

i use the command but it keep saying Usage [PlayerID] (this script dosent work)

[Andre02]

Quote:

Originally Posted by Sew_Sumi That is incorrect. You can make them on one line, and they can have semicolons at the end of them, meaning it's the end of the if portion.

Semicolons are added to 1 line if statements but after the function

pawn Код:

if (a == b) SendClientMessage(playerid, -1, "A has the same value as B");

But not after the if statement.

pawn Код:

if (a == b); SendClientMessage(playerid, -1, "A has the same value as B");

Which means if statements do not have semicolons but the function after it does have it.

[jlalt]

Quote:

Originally Posted by ahameed4755 i use the command but it keep saying Usage [PlayerID] (this script dosent work)

change :

Код:

if(sscanf(params, "udd", targetid, weapon, ammo)) return SendClientMessage(playerid, 0xFFFFFF, "Usage: /agiveweapon [playerid]");

to

PHP код:

if(sscanf(params, "udd", targetid, weapon, ammo)) return SendClientMessage(playerid, 0xFFFFFF, "Usage: /agiveweapon [playerid] [weaponid] [ammo]"); 


then /agiveweapon playerid weaponid ammo

example: /agiveweapon 0 38 5000

[Sew_Sumi]

Quote:

Originally Posted by Andre02 Semicolons are added to 1 line if statements but after the function

Look, fact is, I was meaning about the fact that when you put them on one line, they have a semicolon at the end of the line.

I'd say unless you've been helping in the first place, avoid nitpicking on those who have been.