[NeXoR]

PHP код:

stock GiveNexWeapon(playerid, weaponid, ammo)
{
    PlayerPlaySound(playerid, 1052, 0.0, 0.0, 0.0);
    /*
    if(weaponid == 0 || weaponid == 1) PlayerInfo[playerid][pWeapon][0] = weaponid;
    else if(weaponid >= 2 && weaponid <= 9) PlayerInfo[playerid][pWeapon][1] = weaponid;
    else if(weaponid >= 22 && weaponid <= 24) PlayerInfo[playerid][pWeapon][2] = weaponid;
    else if(weaponid >= 25 && weaponid <= 27) PlayerInfo[playerid][pWeapon][3] = weaponid;
    else if(weaponid == 28 || weaponid == 29 || weaponid == 32) PlayerInfo[playerid][pWeapon][4] = weaponid;
    else if(weaponid == 30 || weaponid == 31) PlayerInfo[playerid][pWeapon][5] = weaponid;
    else if(weaponid == 33 || weaponid == 34) PlayerInfo[playerid][pWeapon][6] = weaponid;
    else if(weaponid >= 35 && weaponid <= 38) PlayerInfo[playerid][pWeapon][7] = weaponid;
    else if(weaponid == 16 || weaponid == 17 || weaponid == 18 || weaponid == 39) PlayerInfo[playerid][pWeapon][8] = weaponid;
    else if(weaponid >= 41 && weaponid <= 43) PlayerInfo[playerid][pWeapon][9] = weaponid;
    else if(weaponid >= 10 && weaponid <= 15) PlayerInfo[playerid][pWeapon][10] = weaponid;
    else if(weaponid >= 44 && weaponid <= 46) PlayerInfo[playerid][pWeapon][11] = weaponid;
    else if(weaponid == 40) PlayerInfo[playerid][pWeapon][12] = weaponid;
    */
    switch(weaponid)
    {
        case 0, 1: PlayerInfo[playerid][pWeapon][0] = weaponid; PlayerInfo[playerid][pWeaponAmmo][0] = ammo; // No weapon
        case 2 .. 9: PlayerInfo[playerid][pWeapon][1] = weaponid; PlayerInfo[playerid][pWeaponAmmo][1] = ammo; // Melee
        case 22 .. 24: PlayerInfo[playerid][pWeapon][2] = weaponid; PlayerInfo[playerid][pWeaponAmmo][2] = ammo; // Handguns
        case 25 .. 27: PlayerInfo[playerid][pWeapon][3] = weaponid; PlayerInfo[playerid][pWeaponAmmo][3] = ammo; // Shotguns
        case 28, 29, 32: PlayerInfo[playerid][pWeapon][4] = weaponid; PlayerInfo[playerid][pWeaponAmmo][4] = ammo; // Sub-Machineguns
        case 30, 31: PlayerInfo[playerid][pWeapon][5] = weaponid; PlayerInfo[playerid][pWeaponAmmo][5] = ammo; // Machineguns
        case 33, 34: PlayerInfo[playerid][pWeapon][6] = weaponid; PlayerInfo[playerid][pWeaponAmmo][6] = ammo; // Rifles
        case 35 .. 38: PlayerInfo[playerid][pWeapon][7] = weaponid; PlayerInfo[playerid][pWeaponAmmo][7] = ammo; // Heavy Weapons
        case 16, 17, 18, 39: PlayerInfo[playerid][pWeapon][8] = weaponid; PlayerInfo[playerid][pWeaponAmmo][8] = ammo; // Projectiles
        case 42, 43: PlayerInfo[playerid][pWeapon][9] = weaponid; PlayerInfo[playerid][pWeaponAmmo][9] = ammo; // Special 1
        case 14: PlayerInfo[playerid][pWeapon][10] = weaponid; PlayerInfo[playerid][pWeaponAmmo][10] = ammo; // Gifts
        case 44 .. 46: PlayerInfo[playerid][pWeapon][11] = weaponid; PlayerInfo[playerid][pWeaponAmmo][11] = ammo; // Special 2
        case 40: PlayerInfo[playerid][pWeapon][12] = weaponid; PlayerInfo[playerid][pWeaponAmmo][12] = ammo; // Detonators
    }
    GivePlayerWeapon(playerid, weaponid, ammo);
    return 1;
} 


Код:

D:\MySQL\gamemodes\MVR.pwn(1154) : error 021: symbol already defined: "GivePlayerWeapon"
D:\MySQL\gamemodes\MVR.pwn(1155) : error 010: invalid function or declaration
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


2 Errors.

[Quinncell]

You already defined "GivePlayerWeapon", find on which line the error comes and delete "GivePlayerWeapon" and recompile.

Edit: mistake, put the "GivePlayerWeapon" inside the bracket.

PHP код:

switch(weaponid) 
    { 
        case 0, 1: PlayerInfo[playerid][pWeapon][0] = weaponid; PlayerInfo[playerid][pWeaponAmmo][0] = ammo; // No weapon 
        case 2 .. 9: PlayerInfo[playerid][pWeapon][1] = weaponid; PlayerInfo[playerid][pWeaponAmmo][1] = ammo; // Melee 
        case 22 .. 24: PlayerInfo[playerid][pWeapon][2] = weaponid; PlayerInfo[playerid][pWeaponAmmo][2] = ammo; // Handguns 
        case 25 .. 27: PlayerInfo[playerid][pWeapon][3] = weaponid; PlayerInfo[playerid][pWeaponAmmo][3] = ammo; // Shotguns 
        case 28, 29, 32: PlayerInfo[playerid][pWeapon][4] = weaponid; PlayerInfo[playerid][pWeaponAmmo][4] = ammo; // Sub-Machineguns 
        case 30, 31: PlayerInfo[playerid][pWeapon][5] = weaponid; PlayerInfo[playerid][pWeaponAmmo][5] = ammo; // Machineguns 
        case 33, 34: PlayerInfo[playerid][pWeapon][6] = weaponid; PlayerInfo[playerid][pWeaponAmmo][6] = ammo; // Rifles 
        case 35 .. 38: PlayerInfo[playerid][pWeapon][7] = weaponid; PlayerInfo[playerid][pWeaponAmmo][7] = ammo; // Heavy Weapons 
        case 16, 17, 18, 39: PlayerInfo[playerid][pWeapon][8] = weaponid; PlayerInfo[playerid][pWeaponAmmo][8] = ammo; // Projectiles 
        case 42, 43: PlayerInfo[playerid][pWeapon][9] = weaponid; PlayerInfo[playerid][pWeaponAmmo][9] = ammo; // Special 1 
        case 14: PlayerInfo[playerid][pWeapon][10] = weaponid; PlayerInfo[playerid][pWeaponAmmo][10] = ammo; // Gifts 
        case 44 .. 46: PlayerInfo[playerid][pWeapon][11] = weaponid; PlayerInfo[playerid][pWeaponAmmo][11] = ammo; // Special 2 
        case 40: PlayerInfo[playerid][pWeapon][12] = weaponid; PlayerInfo[playerid][pWeaponAmmo][12] = ammo; // Detonators 
        GivePlayerWeapon(playerid, weaponid, ammo);
    } 


[azzerking]

Quote:

Originally Posted by KeithCooper You already defined "GivePlayerWeapon", find on which line the error comes and delete "GivePlayerWeapon" and recompile. Edit: mistake, put the "GivePlayerWeapon" inside the bracket. PHP код: switch(weaponid)      {          case 0, 1: PlayerInfo[playerid][pWeapon][0] = weaponid; PlayerInfo[playerid][pWeaponAmmo][0] = ammo; // No weapon          case 2 .. 9: PlayerInfo[playerid][pWeapon][1] = weaponid; PlayerInfo[playerid][pWeaponAmmo][1] = ammo; // Melee          case 22 .. 24: PlayerInfo[playerid][pWeapon][2] = weaponid; PlayerInfo[playerid][pWeaponAmmo][2] = ammo; // Handguns          case 25 .. 27: PlayerInfo[playerid][pWeapon][3] = weaponid; PlayerInfo[playerid][pWeaponAmmo][3] = ammo; // Shotguns          case 28, 29, 32: PlayerInfo[playerid][pWeapon][4] = weaponid; PlayerInfo[playerid][pWeaponAmmo][4] = ammo; // Sub-Machineguns          case 30, 31: PlayerInfo[playerid][pWeapon][5] = weaponid; PlayerInfo[playerid][pWeaponAmmo][5] = ammo; // Machineguns          case 33, 34: PlayerInfo[playerid][pWeapon][6] = weaponid; PlayerInfo[playerid][pWeaponAmmo][6] = ammo; // Rifles          case 35 .. 38: PlayerInfo[playerid][pWeapon][7] = weaponid; PlayerInfo[playerid][pWeaponAmmo][7] = ammo; // Heavy Weapons          case 16, 17, 18, 39: PlayerInfo[playerid][pWeapon][8] = weaponid; PlayerInfo[playerid][pWeaponAmmo][8] = ammo; // Projectiles          case 42, 43: PlayerInfo[playerid][pWeapon][9] = weaponid; PlayerInfo[playerid][pWeaponAmmo][9] = ammo; // Special 1          case 14: PlayerInfo[playerid][pWeapon][10] = weaponid; PlayerInfo[playerid][pWeaponAmmo][10] = ammo; // Gifts          case 44 .. 46: PlayerInfo[playerid][pWeapon][11] = weaponid; PlayerInfo[playerid][pWeaponAmmo][11] = ammo; // Special 2          case 40: PlayerInfo[playerid][pWeapon][12] = weaponid; PlayerInfo[playerid][pWeaponAmmo][12] = ammo; // Detonators          GivePlayerWeapon(playerid, weaponid, ammo);     }

You can't put a call to a function inside a switch statement like that, the problem might be that your not continuing the statement and instead your ending it, yet your code is still going to be executed outside the case statement.

Example:

Код:

// Example #1
case 1:
{
    PlayerInfo[playerid][pWeapon][0] = weaponid; 
    PlayerInfo[playerid][pWeaponAmmo][0] = ammo;
}

// Example #2
case 1: PlayerInfo[playerid][pWeapon][0] = weaponid, PlayerInfo[playerid][pWeaponAmmo][0] = ammo;

In example two you will notice instead of using ; after the first statement, I am using a comma. This is to allow for multiple statements without have to use braces.

[NeXoR]

Quote:

Originally Posted by azzerking You can't put a call to a function inside a switch statement like that, the problem might be that your not continuing the statement and instead your ending it, yet your code is still going to be executed outside the case statement. Example: Код: // Example #1 case 1: { PlayerInfo[playerid][pWeapon][0] = weaponid; PlayerInfo[playerid][pWeaponAmmo][0] = ammo; } // Example #2 case 1: PlayerInfo[playerid][pWeapon][0] = weaponid, PlayerInfo[playerid][pWeaponAmmo][0] = ammo; In example two you will notice instead of using ; after the first statement, I am using a comma. This is to allow for multiple statements without have to use braces.

Thanks ..