SA-MP Forums Archive
<<<<<<<<please help >>>>>>>>> - 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)
+---- Forum: Help Archive (https://sampforum.blast.hk/forumdisplay.php?fid=89)
+---- Thread: <<<<<<<<please help >>>>>>>>> (/showthread.php?tid=267343)


<<<<<<<<please help >>>>>>>>> - MA_proking - 08.07.2011

pawn Код:
if(AccInfo[playerid][pVip] = 1)
    {
    SendClientMessage(playerid,red, "VIP Silver");
    }
    if(AccInfo[playerid][pVip] = 2)
    {
    SendClientMessage(playerid,red, "VIP Gold");
    }
    if(AccInfo[playerid][pVip] = 3)
    {
    SendClientMessage(playerid,red, "VIP Premium");
    }
    if(AccInfo[playerid][Level] = 1)
    {
    SendClientMessage(playerid,red, "Trail moderator");
    }
     if(AccInfo[playerid][Level] = 2)
    {
    SendClientMessage(playerid,red, "Moderator");
    }
     if(AccInfo[playerid][Level] = 3)
    {
    SendClientMessage(playerid,red, "Administrator");
    }
     if(AccInfo[playerid][Level] = 4)
    {
    SendClientMessage(playerid,red, "Master Administrator");
    }
     if(AccInfo[playerid][Level] = 5)
    {
    SendClientMessage(playerid,red, "Manager");
    }
     if(AccInfo[playerid][Level] > 5)
    {
    SendClientMessage(playerid,red, "Owner");
    }
It is in LUX admin
But it show these warnings
Код:
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 225: unreachable code
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1515) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1515) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1519) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1523) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1527) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1527) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1531) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1535) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1539) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1549) : warning 209: function "dcmd_stats" should return a value
Please help

Re: <<<<<<<<please help >>>>>>>>> - [HiC]TheKiller - 08.07.2011

pawn Код:
if(AccInfo[playerid][pVip] = 1)
Should be

pawn Код:
if(AccInfo[playerid][pVip] == 1)
etc..

A single equals is setting a value, a double equals is checking a value.

Re: <<<<<<<<please help >>>>>>>>> - MA_proking - 08.07.2011

Still showing these warnings
Код:
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 225: unreachable code
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1515) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1527) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1549) : warning 209: function "dcmd_stats" should return a value


Re: <<<<<<<<please help >>>>>>>>> - PotH3Ad - 08.07.2011

-_-

pawn Код:
if(AccInfo[playerid][pVip] == 1)
    {
    SendClientMessage(playerid,red, "VIP Silver");
    }
    if(AccInfo[playerid][pVip] == 2)
    {
    SendClientMessage(playerid,red, "VIP Gold");
    }
    if(AccInfo[playerid][pVip] == 3)
    {
    SendClientMessage(playerid,red, "VIP Premium");
    }
    if(AccInfo[playerid][Level] == 1)
    {
    SendClientMessage(playerid,red, "Trail moderator");
    }
     if(AccInfo[playerid][Level] == 2)
    {
    SendClientMessage(playerid,red, "Moderator");
    }
     if(AccInfo[playerid][Level] == 3)
    {
    SendClientMessage(playerid,red, "Administrator");
    }
     if(AccInfo[playerid][Level] == 4)
    {
    SendClientMessage(playerid,red, "Master Administrator");
    }
     if(AccInfo[playerid][Level] == 5)
    {
    SendClientMessage(playerid,red, "Manager");
    }
     if(AccInfo[playerid][Level] > 5)
    {
    SendClientMessage(playerid,red, "Owner");
    }
It was as simple as pressing "Ctrl + H" Find "=" and replace with "==".

Re: <<<<<<<<please help >>>>>>>>> - MA_proking - 08.07.2011

replacement done but still warnings are there

Re: <<<<<<<<please help >>>>>>>>> - emon - 08.07.2011

the same code written should be same down line life

if(strcmp, "/cmd"
return 1;


this is right way to write code
---------------

(strcmp, "/cmd
return 1;

wrong way this should give u warning

Re: <<<<<<<<please help >>>>>>>>> - PotH3Ad - 08.07.2011

@emon He is using DCMD, what are you talking about.

@MA_proking Post lines 1511-1549, there is more code causing warnings then what you posted.

Re: <<<<<<<<please help >>>>>>>>> - MA_proking - 08.07.2011

Here is the whole code
pawn Код:
dcmd_stats(playerid,params[])
{
    new string[128];
    new pDeaths;
    new player1, h, m, s;

    if(!strlen(params)) player1 = playerid;
    else player1 = strval(params);

    if(IsPlayerConnected(player1))
    {
    TotalGameTime(player1, h, m, s);
    if(AccInfo[player1][Deaths] == 0) pDeaths = 1;
    else pDeaths = AccInfo[player1][Deaths];
    format(string, sizeof(string), "|- %s's Statistics -|",PlayerName2(player1));
    SendClientMessage(playerid, green, string);
    format(string, sizeof(string), "Kills: [%d] | Deaths: [%d] | Ratio: [%0.2f] | Money: [$%d] | Time: [%d] hrs [%d] mins [%d] secs |", AccInfo[player1][Kills], AccInfo[player1][Deaths], Float:AccInfo[player1][Kills]/Float:pDeaths,GetPlayerMoney(player1), h, m, s);
    return SendClientMessage(playerid, green, string);
if(AccInfo[playerid][pVip] = 1)
    {
    SendClientMessage(playerid,red, "VIP Silver");
    }
    if(AccInfo[playerid][pVip] == 2)
    {
    SendClientMessage(playerid,red, "VIP Gold");
    }
    if(AccInfo[playerid][pVip] == 3)
    {
    SendClientMessage(playerid,red, "VIP Premium");
    }
    if(AccInfo[playerid][Level] ==1)
    {
    SendClientMessage(playerid,red, "Trail moderator");
    }
     if(AccInfo[playerid][Level] == 2)
    {
    SendClientMessage(playerid,red, "Moderator");
    }
     if(AccInfo[playerid][Level] == 3)
    {
    SendClientMessage(playerid,red, "Administrator");
    }
     if(AccInfo[playerid][Level] == 4)
    {
    SendClientMessage(playerid,red, "Master Administrator");
    }
     if(AccInfo[playerid][Level] == 5)
    {
    SendClientMessage(playerid,red, "Manager");
    }
     if(AccInfo[playerid][Level] > 5)
    {
    SendClientMessage(playerid,red, "Owner");
    }
    } else
    return SendClientMessage(playerid, red, "ERROR: Player Not Connected!");
}
#endif
Still showing these warnings
Код:
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 225: unreachable code
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1511) : warning 211: possibly unintended assignment
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1515) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1527) : warning 217: loose indentation
C:\DOCUME~1\Prabhat\Desktop\LuxAdmin.pwn(1549) : warning 209: function "dcmd_stats" should return a value


Re: <<<<<<<<please help >>>>>>>>> - PotH3Ad - 08.07.2011

pawn Код:
dcmd_stats(playerid, params[])
{
    new string[128];
    new pDeaths;
    new player1, h, m, s;

    if(!strlen(params)) player1 = playerid;
    else player1 = strval(params);

    if(!IsPlayerConnected(player1)) return SendClientMessage(playerid, red, "ERROR: Player Not Connected!");

    TotalGameTime(player1, h, m, s);
    if(AccInfo[player1][Deaths] == 0) pDeaths = 1;
    else pDeaths = AccInfo[player1][Deaths];
    format(string, sizeof(string), "|- %s's Statistics -|", PlayerName2(player1));
    SendClientMessage(playerid, green, string);
    format(string, sizeof(string), "Kills: [%d] | Deaths: [%d] | Ratio: [%0.2f] | Money: [$%d] | Time: [%d] hrs [%d] mins [%d] secs |", AccInfo[player1][Kills], AccInfo[player1][Deaths], Float:AccInfo[player1][Kills]/Float:pDeaths,GetPlayerMoney(player1), h, m, s);
    SendClientMessage(playerid, green, string);

    switch(AccInfo[playerid][pVip])
    {
        case 1: SendClientMessage(playerid,red, "VIP Silver");
        case 2: SendClientMessage(playerid,red, "VIP Gold");
        case 3: SendClientMessage(playerid,red, "VIP Premium");
    }
    switch(AccInfo[playerid][Level])
    {
        case 1: SendClientMessage(playerid,red, "Trail moderator");
        case 2: SendClientMessage(playerid,red, "Moderator");
        case 3: SendClientMessage(playerid,red, "Administrator");
        case 4: SendClientMessage(playerid,red, "Master Administrator");
        case 5: SendClientMessage(playerid,red, "Manager");
        default:
        {
            if(AccInfo[playerid][Level] > 5) SendClientMessage(playerid,red, "Owner");
        }
    }
    return 1;
}
#endif


Re: <<<<<<<<please help >>>>>>>>> - MA_proking - 08.07.2011

it works thanks