Possible for working?with 6 warnings?

Possible for working?with 6 warnings? - 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: Possible for working?with 6 warnings? (/showthread.php?tid=370546)

Possible for working?with 6 warnings? - Sanady - 20.08.2012

Hello I made flame bullet shop and effect is taking healths but I made this

pawn Код:

public OnPlayerGiveDamage(playerid, damagedid, Float: amount, weaponid)
{
    if(flame[playerid] == 1)
    {
        new Float:X,Float:Y,Float:Z;
        GetPlayerPos(damagedid,X,Y,Z);
        CreateExplosion(X,Y,Z,12,0.0);
        if(health[playerid] == 1)
        {
            SetPlayerHealth(damagedid,GetPlayerHealth(damagedid) -20);
        }
        if(health1[playerid] == 1)
        {
            SetPlayerHealth(damagedid,GetPlayerHealth(damagedid) -50);
        }
        if(health2[playerid] == 1)
        {
            SetPlayerHealth(damagedid,GetPlayerHealth(damagedid) -70);
        }
    }
    if(flame[playerid] == 0)
    {
    
    }
    return 1;
}

WARNINGS

Код:

D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(623) : warning 202: number of arguments does not match definition
D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(627) : warning 202: number of arguments does not match definition
D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(631) : warning 202: number of arguments does not match definition
D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(655) : warning 215: expression has no effect
D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(666) : warning 215: expression has no effect
D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(677) : warning 215: expression has no effect
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


6 Warnings.

Re: Possible for working?with 6 warnings? - Cjgogo - 20.08.2012

pawn Код:

public OnPlayerGiveDamage(playerid, damagedid, Float: amount, weaponid)
{
    if(flame[playerid] == 1)
    {
        new Float:X,Float:Y,Float:Z;
        new Float:health;
        health=GetPlayerHealth(damagedid,health);
        GetPlayerPos(damagedid,X,Y,Z);
        CreateExplosion(X,Y,Z,12,0.0);
        if(health[playerid] == 1)
        {
            SetPlayerHealth(damagedid,health -20);
        }
        if(health1[playerid] == 1)
        {
            SetPlayerHealth(damagedid,health -50);
        }
        if(health2[playerid] == 1)
        {
            SetPlayerHealth(damagedid,health -70);
        }
    }
    return 1;
}

It won't work,try my code instead,should return no errors/warnings

Re: Possible for working?with 6 warnings? - Sanady - 20.08.2012

Quote:

Originally Posted by Cjgogo

pawn Код:

public OnPlayerGiveDamage(playerid, damagedid, Float: amount, weaponid)
{
    if(flame[playerid] == 1)
    {
        new Float:X,Float:Y,Float:Z;
        new Float:health;
        health=GetPlayerHealth(damagedid,health);
        GetPlayerPos(damagedid,X,Y,Z);
        CreateExplosion(X,Y,Z,12,0.0);
        if(health[playerid] == 1)
        {
            SetPlayerHealth(damagedid,health -20);
        }
        if(health1[playerid] == 1)
        {
            SetPlayerHealth(damagedid,health -50);
        }
        if(health2[playerid] == 1)
        {
            SetPlayerHealth(damagedid,health -70);
        }
    }
    return 1;
}

It won't work,try my code instead,should return no errors/warnings

It`s working fine now those warnings

Код:

D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(660) : warning 215: expression has no effect
D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(671) : warning 215: expression has no effect
D:\SWAT\samp03dsvr_win32(1)\gamemodes\Reg.pwn(682) : warning 215: expression has no effect
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


3 Warnings.

Code

pawn Код:

if(dialogid == 1221)
    {
        if(!response) return 1;
        switch(listitem)
        {
            case 0:
            {
                if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":20%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health[playerid] == 1;//Those lines
                }
            }
            case 1:
            {
                if(GetPlayerMoney(playerid) < 700) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":50%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health2[playerid] == 1;//Those lines
                }
            }
            case 2:
            {
                if(GetPlayerMoney(playerid) < 2000) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":70%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health3[playerid] == 1;//Those lines
                }
            }
        }
    }

Re: Possible for working?with 6 warnings? - Cjgogo - 20.08.2012

pawn Код:

if(dialogid == 1221)
    {
        if(!response) return 1;
        switch(listitem)
        {
            case 0:
            {
                if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":20%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health[playerid] = 1;//Those lines
                }
            }
            case 1:
            {
                if(GetPlayerMoney(playerid) < 700) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":50%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health2[playerid] = 1;//Those lines
                }
            }
            case 2:
            {
                if(GetPlayerMoney(playerid) < 2000) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":70%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health3[playerid] = 1;//Those lines
                }
            }
        }
    }

Fixed,the error was that the assigning(a value to a variable) is not done by "==",but by "=","==" is used for checking a variable's value.

Re: Possible for working?with 6 warnings? - Sanady - 20.08.2012

Quote:

Originally Posted by Cjgogo

pawn Код:

if(dialogid == 1221)
    {
        if(!response) return 1;
        switch(listitem)
        {
            case 0:
            {
                if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":20%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health[playerid] = 1;//Those lines
                }
            }
            case 1:
            {
                if(GetPlayerMoney(playerid) < 700) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":50%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health2[playerid] = 1;//Those lines
                }
            }
            case 2:
            {
                if(GetPlayerMoney(playerid) < 2000) return SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You don`t have enough money to buy flame bullets!");
                {
                    cont[playerid] = 1;
                    SendClientMessage(playerid,0xFFFFFFFF,""embed_orange"[FLAME BULLET SHOP]"embed_white"You bought "embed_red"1 "embed_white"flame bullet.Press Y to active it!"embed_red"DAMAGE"embed_white":70%!");
                    GivePlayerMoney(playerid,GetPlayerMoney(playerid)-100);
                    key[playerid] = 1;
                    health3[playerid] = 1;//Those lines
                }
            }
        }
    }

Fixed,the error was that the assigning(a value to a variable) is not done by "==",but by "=","==" is used for checking a variable's value.

Ohh Yea...