Some errors - Printable Version

Some errors - 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: Some errors (/showthread.php?tid=363322)

Some errors - Frosty_LV - 27.07.2012

Hello, may someone help me sort this mess out?

PHP Code:



CMD:buyrifle(playerid, params[])

{

if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))

GivePlayerWeapon(playerid, 33, 1);

GivePlayerMoney(playerid,  -400);

{

else if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))

{

        GetPlayerMoney(playerid) <400);

        SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");

}

error 029: invalid expression, assumed zero
(749) : warning 215: expression has no effect
(749) : error 001: expected token: ";", but found "if"
(751) : warning 215: expression has no effect
(751) : error 001: expected token: ";", but found ")"
(751) : error 029: invalid expression, assumed zero
(751) : warning 215: expression has no effect
(754) : error 030: compound statement not closed at the end of file (started at line 749)

I feel pretty confused

Re: Some errors - Devilxz97 - 27.07.2012

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        GivePlayerWeapon(playerid, 33, 1);
        GivePlayerMoney(playerid,  -400);
    }
    else if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        GetPlayerMoney(playerid) <400);
        SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
    }
    return 1;
}

Re: Some errors - Alexis1999 - 27.07.2012

I fixed & compiled your script

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        GivePlayerWeapon(playerid, 33, 1);
        GivePlayerMoney(playerid,  -400);
    }
    else if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        if( GetPlayerMoney(playerid) < 400 )
            return SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
    }
    return 1;
}

Re: Some errors - Frosty_LV - 27.07.2012

Quote:

Originally Posted by Devilxz97

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        GivePlayerWeapon(playerid, 33, 1);
        GivePlayerMoney(playerid,  -400);
    }
    else if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        GetPlayerMoney(playerid) <400);
        SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
    }
    return 1;
}

(752) : warning 215: expression has no effect
(752) : error 001: expected token: ";", but found ")"
(752) : error 029: invalid expression, assumed zero
(752) : warning 215: expression has no effect

The

PHP Code:



GetPlayerMoney(playerid) <400);

is problem line.

Re: Some errors - Frosty_LV - 27.07.2012

Quote:

Originally Posted by Alexis1999

I fixed & compiled your script

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        GivePlayerWeapon(playerid, 33, 1);
        GivePlayerMoney(playerid,  -400);
    }
    else if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        if( GetPlayerMoney(playerid) < 400 )
            return SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
    }
    return 1;
}

Thanks, no errors! +rep for you.

Re: Some errors - Andi_Evandy - 27.07.2012

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        if(GetPlayerMoney(playerid) < 400) return SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
        GivePlayerWeapon(playerid, 33, 1);
        GivePlayerMoney(playerid,  -400);
    }
    return 1;
}

Edit: Too late XD

Re: Some errors - Devilxz97 - 27.07.2012

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        if(GetPlayerMoney(playerid) < 400);
        return SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
        GivePlayerMoney(playerid,  -400);
        GivePlayerWeapon(playerid, 33, 1);
    }
    return 1;
}

use this , this is simple

Re: Some errors - Alexis1999 - 27.07.2012

Quote:

Originally Posted by Devilxz97

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156))
    {
        if(GetPlayerMoney(playerid) < 400);
        GivePlayerWeapon(playerid, 33, 1);
        GivePlayerMoney(playerid,  -400);
        return SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
    }
    return 1;
}

use this , this is simple

Lol. Your script doesn't work as he needs. The point is to print the message if his money is lower than 400

Re: Some errors - Devilxz97 - 27.07.2012

Quote:

Originally Posted by Alexis1999

Lol. Your script doesn't work as he needs. The point is to print the message if his money is lower than 400

it was fix if you see it clearly -,-

Re: Some errors - M3mPHi$_S3 - 28.07.2012

pawn Code:

CMD:buyrifle(playerid, params[])
{
    if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156));
    {
        GivePlayerWeapon(playerid, 33, 1);
        GivePlayerMoney(playerid,  -400);
    }
    else if(IsPlayerInRangeOfPoint(playerid, 3.0, 295.7310,-38.1204,1001.5156));
    {
        if( GetPlayerMoney(playerid) < 400 );
            return SendClientMessage(playerid, 0xFFFFFFAA, "You don't have enough money!");
    }
    return 1;
}

try with this