1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
Errors, please help
DJTunes
Huge Clucker
****
Posts: 389
Threads: 67
Joined: Dec 2012
Reputation: 0
#1

17.06.2013, 21:29
PAWNO Compiler:
pawn Код:
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\pawno\include\dini.inc(239) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\pawno\include\dutils.inc(28) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\pawno\include\dutils.inc(132) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\pawno\include\dutils.inc(172) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\pawno\include\dutils.inc(280) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\pawno\include\dutils.inc(337) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\pawno\include\dutils.inc(379) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(165) : warning 217: loose indentation
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(170) : error 021: symbol already defined: "format"
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(179) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(185) : error 029: invalid expression, assumed zero
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(190) : warning 217: loose indentation
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(191) : warning 217: loose indentation
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(203) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(218) : warning 217: loose indentation
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(226) : warning 225: unreachable code
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(226) : warning 217: loose indentation
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(230) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(241) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(253) : warning 219: local variable "string" shadows a variable at a preceding level
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(300) : warning 203: symbol is never used: "ct"
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(300 -- 308) : warning 203: symbol is never used: "help"
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(300 -- 308) : warning 203: symbol is never used: "kill"
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(300 -- 308) : warning 203: symbol is never used: "ret_memcpy"
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(300 -- 308) : warning 203: symbol is never used: "rules"
C:\Users\mom2k_000\Documents\samp03x_svr_R1-2_win32\gamemodes\GangWar.pwn(300 -- 308) : warning 203: symbol is never used: "string"
Pawn compiler 3.2.3664          Copyright (c) 1997-2006, ITB CompuPhase


2 Errors.
Can someone please tell me how to fix these warnings, and errors (and maybe possibly paste the fixed script as a reply?) Person that does this will receive reputation.

Here's my script:
pawn Код:
//....._\____________________,,__
//..../ `--¦¦¦¦¦¦¦¦¦----------------------_]
//.../_==o ____________________
//.....),---.(_(__) /
//....// (\) ),------
//...//___//
//../`----' / ...           Los Santos Turf Wars
//./____ / ...            Made by DJTunes/Hollywood
//==============================================================================
// Credits...
// Scripter(s): DJTunes/Hollywood,
// Mapper(s):
//------------------------------------------------
// Information...
// Script Started: 6/16/2013
//==============================================================================
//==============================INCLUDES========================================
#include    < a_samp > // - Includes San Andreas Multiplayer
#include    <OPSP>     // - Includes for Anti-Kill system
#include    <dini>     // - Includes for Login System
#include    <dudb>
//==============================================================================
//==============================DEFINES=========================================
#define                           TEAM_GROVE 0 // - Defines "Grove"
#define                           TEAM_BALLAS 1 // - Defines "Ballas"

#define COLOR_GREEN 0x008000FF
#define COLOR_BLUE 0x0000FFFF
#define COLOR_BRIGHTRED 0xFF000AAA
#define COLOR_AQUA 0x00FFFFAA
#define COLOR_GREY 0xAFAFAFAA
#define COLOR_BEIGE 0xFFF8DCAA
#define COLOR_BLACK 0x000000AA
#define COLOR_LIGHTERBLUE 0x00BFFFAA
#define COLOR_BLUELIGHT 0x1E90FFAA
#define COLOR_BLUEMEDIUM 0x0000CDAA
#define COLOR_BLUEDARK 0x00008BAA
#define COLOR_PINK 0xFF1493AA
#define COLOR_PINKDARK 0xFF00FFAA
#define COLOR_GREENLIGHT 0x00FF00AA
#define COLOR_GREENDARK 0x006400AA
#define COLOR_MAROON 0x800000AA
#define COLOR_OKER 0x808000AA
#define COLOR_ORANGE 0xFF4500AA
#define COLOR_ORANGELIGHT 0xFF8C00AA
#define COLOR_PURPLE 0x800080AA
#define COLOR_VIOLETDARK 0x9400D3AA
#define COLOR_INDIGO 0xAB0082AA
#define COLOR_RED 0xFF0000AA
#define COLOR_SAND 0xFFDEADAA
#define COLOR_SILVER 0xC0C0C0AA
#define COLOR_TEAL 0x008080AA
#define COLOR_WHITE 0xFFFFFFAA
#define COLOR_YELLOW 0xFFFF00AA
#define COLOR_GOLD 0xFFD700AA
#define COLOR_BROWN 0x8B4513AA
#define COLOR_BROWNLIGHT 0xA0522DAA
#define COLOR_GRAY 0xA9A9A9AA
#define COLOR_GRAYDARK 0x696969AA
#define COLOR_INVISIBLE 0xFFFFFF00
#define COLOR_INDIA  0x80000096
#define COLOR_PAKISTAN 0x40808096
#define COLOR_NEPAL 0x8000FF96
#define COLOR_BRAZIL 0xB7B70096
#define COLOR_CHINA 0x80808096
#define COLOR_GREECE 0x00000096
#define COLOR_SINGAPORE 0x00FFFF96
#define COLOR_IRAQ 0x8080C096
#define COLOR_GROVE 0x00FF14FF
#define COLOR_BALLA 0x8200FFFF
//==============================================================================
//==============================VARIABLES=======================================
new gTeam[MAX_PLAYERS]; // - Defines all the families/teams.
new spree[MAX_PLAYERS];
new Text:Spree;
//==============================================================================
//==============================TEAM SETTINGS===================================
forward SetPlayerTeamFromClass(playerid, classid);
   public OnPlayerRequestClass(playerid, classid)
{
    SetPlayerTeamFromClass(playerid, classid);
    return 1;
}
public SetPlayerTeamFromClass(playerid, classid)
{
if(classid == 0)
    {
        gTeam[playerid] = TEAM_BALLAS;
        GameTextForPlayer(playerid,"~w~Ballas",3000,5);
        SetPlayerPos(playerid,1975.2399,-1220.0157,25.0779); 
        SetPlayerCameraPos(playerid,1969.5686,-1224.0016,24.9909);
        SetPlayerCameraLookAt(playerid,1975.2399,-1220.0157,25.0779);
        SetPlayerFacingAngle(playerid,122.4500);
        SetPlayerColor(playerid,0xA000FFFF); 
    }
else if(classid == 1)
    {
        gTeam[playerid] = TEAM_GROVE;
        GameTextForPlayer(playerid,"~w~Grove Street Families",3000,5);
        SetPlayerPos(playerid,2500.6060,-1672.1453,13.3512);
        SetPlayerCameraPos(playerid,2507.0615,-1674.3574,13.3732);
        SetPlayerCameraLookAt(playerid,2500.6060,-1672.1453,13.3512);
        SetPlayerFacingAngle(playerid,252.4717);
        SetPlayerColor(playerid,0x55FF00FF);
    }
}
//==============================================================================
//==============================GAMEMODE INFO===================================
public OnGameModeInit()
{
    UsePlayerPedAnims();
    SetGameModeText("LS Gang Wars");
    AddPlayerClass(104,1979.1917,-1222.0964,25.1824,97.3597,31,100000,27,100000,32, 100000);
    AddPlayerClass(107,2508.5671,-1673.6665,13.3793,86.2585,30,100000,28,100000,31, 100000);
    
    AddStaticVehicleEx(447,2503.60009766,-1667.59997559,13.50000000,90.00000000,-1,-1,15); //Seasparrow
    AddStaticVehicleEx(492,2474.00000000,-1671.00000000,13.19999981,42.00000000,-1,1,15); //Greenwood
    AddStaticVehicleEx(492,2478.19995117,-1656.30004883,13.19999981,91.99523926,-1,1,15); //Greenwood
    AddStaticVehicleEx(492,2487.50000000,-1657.80004883,13.19999981,128.00000000,86,-1,15); //Greenwood
    AddStaticVehicleEx(492,2478.39990234,-1676.69995117,13.19999981,43.99621582,86,-1,15); //Greenwood
    AddStaticVehicleEx(412,2488.89990234,-1680.00000000,13.30000019,20.00000000,86,1,15); //Voodoo
    AddStaticVehicleEx(412,2494.60009766,-1659.90002441,13.30000019,111.99914551,86,1,15); //Voodoo
    AddStaticVehicleEx(425,2494.30004883,-1698.30004883,24.39999962,0.00000000,-1,-1,15); //Hunter
    AddStaticVehicleEx(520,2529.60009766,-1677.50000000,20.79999924,0.00000000,-1,-1,15); //Hydra
    AddStaticVehicleEx(487,2507.39990234,-1695.09997559,18.20000076,0.00000000,-1,-1,15); //Maverick
    
    AddStaticVehicleEx(600,1954.09997559,-1964.59997559,13.50000000,0.00000000,53,53,15); //Picador
    AddStaticVehicleEx(600,1969.09997559,-1964.80004883,13.50000000,0.00000000,53,53,15); //Picador
    AddStaticVehicleEx(487,2041.00000000,-1999.80004883,35.29999924,358.00000000,53,-1,15); //Maverick
    AddStaticVehicleEx(487,2017.30004883,-1999.59997559,35.29999924,357.99499512,53,-1,15); //Maverick
    AddStaticVehicleEx(487,2041.09997559,-2029.90002441,35.29999924,357.99499512,53,-1,15); //Maverick
    AddStaticVehicleEx(487,2017.00000000,-2028.59997559,35.29999924,357.99499512,53,-1,15); //Maverick
    AddStaticVehicleEx(476,2022.30004883,-2049.39990234,36.20000076,0.00000000,53,-1,15); //Rustler
    AddStaticVehicleEx(476,2034.80004883,-2049.69995117,36.20000076,0.00000000,53,-1,15); //Rustler
    AddStaticVehicleEx(425,2006.50000000,-1972.30004883,24.00000000,0.00000000,-1,-1,15); //Hunter
    AddStaticVehicleEx(520,2042.09997559,-1972.19995117,24.00000000,0.00000000,-1,-1,15); //Hydra
    AddStaticVehicleEx(517,1986.80004883,-1969.19995117,16.60000038,36.00000000,53,53,15); //Majestic
    AddStaticVehicleEx(447,1988.00000000,-1971.40002441,23.10000038,0.00000000,53,53,15); //Seasparrow
    AddStaticVehicleEx(517,2008.40002441,-1968.90002441,16.60000038,35.99670410,53,53,15); //Majestic
    AddStaticVehicleEx(517,2014.59997559,-1968.19995117,16.60000038,35.99670410,53,53,15); //Majestic
    AddStaticVehicleEx(517,2024.30004883,-1968.00000000,16.60000038,35.99670410,53,53,15); //Majestic
    AddStaticVehicleEx(517,2032.80004883,-1967.69995117,16.60000038,35.99670410,53,53,15); //Majestic
    AddStaticVehicleEx(517,2042.30004883,-1967.90002441,16.60000038,35.99670410,53,53,15); //Majestic
    AddStaticVehicleEx(517,1954.50000000,-1943.90002441,13.50000000,328.00000000,-1,1,15); //Majestic
    AddStaticVehicleEx(517,1968.50000000,-1944.30004883,13.50000000,41.99487305,-1,1,15); //Majestic
    return 1;
}
//==============================================================================
//==============================ANTI-KILL SYS===================================
public OnPlayerShootPlayer(Shooter,Target,Float:HealthLost,Float:ArmourLost)
{
   if ( Shooter != INVALID_PLAYER_ID )
    {
        if ( GetPlayerTeam( Target ) == GetPlayerTeam( Shooter ) )
        {

            new Float:hp;
            GetPlayerHealth(Target, hp);
            SetPlayerHealth(Target, hp + HealthLost);
            SetPlayerHealth( Shooter, 0 );
            SendClientMessage( Shooter, COLOR_RED, "Team killing is not allowed!" );
            GivePlayerMoney( Shooter, - 5000 );
        }
    }
return 1;
}
//==============================================================================
//==============================LOGIN SYS ENUM==================================
new string[30];
format(string, 30, "%s\r\n", pname); 
fwrite(ftw, string);
//==============================================================================
//==============================PLAYER CONNECT==================================
public OnPlayerConnect(playerid)
{
    SendClientMessage(playerid, COLOR_WHITE,"Welcome to Los Santos Gang War's Server.");
    
    //Score Saver
    new string[128], PlayerName[MAX_PLAYER_NAME]; 
    GetPlayerName(playerid, PlayerName, sizeof(PlayerName)); 
    format(string, sizeof(string), "Scores/%s.ini", PlayerName);
    if(fexist(string))
    {
    SetPlayerScore(playerid, dini_Int(string, "Score"));
    else
    {
    dini_Create(string); 
    dini_IntSet(string, "Score", 0); 
    }
          new pname[24], File:ftw=fopen("names.txt", io_append);
    if(ftw)
    {
        GetPlayerName(playerid, pname, 24);
        fwrite(ftw, pname);
        fclose(ftw);
    }
 }
}
//==============================================================================
//==============================PLAYER DISCONNECT===============================
public OnPlayerDisconnect(playerid, reason)
{
    new string[128], PlayerName[MAX_PLAYER_NAME]; 
    GetPlayerName(playerid, PlayerName, sizeof(PlayerName)); 
    format(string, sizeof(string), "Scores/%s.ini", PlayerName); 
    if(fexist(string))
    dini_IntSet(string, "Score", GetPlayerScore(playerid)); 
    else 
    {
    dini_Create(string); 
    dini_IntSet(string, "Score", GetPlayerScore(playerid)); 
    }
}
//==============================================================================
public OnPlayerDeath(playerid, killerid, reason)
{
    SendDeathMessage(killerid, playerid, reason);
        SetPlayerScore(killerid, GetPlayerScore(killerid) +1);
        GivePlayerMoney(killerid, 5000);
        SendClientMessage(killerid, COLOR_GREEN,"You got 1 score and 5000 cash for killing a player");
        GivePlayerMoney(playerid, -1000);
        SendClientMessage(playerid, COLOR_RED,"You lost $2000 for dieing");
        return 1;
        
        //==============================Spree-System==================================//
    spree[killerid]++;

    if(spree[killerid] == 2)
        {
        new string[128];
        new pName[MAX_PLAYER_NAME];
        GetPlayerName(killerid, pName, sizeof(pName));
        format(string, sizeof(string), "%s is on a killing spree, with 5 kills", pName);
        TextDrawHideForAll(Spree);
        TextDrawSetString(Spree, string);
        TextDrawShowForAll(Spree); SetTimer("TxtReset", 3000, 0);
    }

    if(spree[killerid] == 7)
        {
        new string[128];
        new pName[MAX_PLAYER_NAME];
        GetPlayerName(killerid, pName, sizeof(pName));
        format(string, sizeof(string), "%s is on a killing spree, with 7 kills!", pName);
        TextDrawHideForAll(Spree);
        TextDrawSetString(Spree, string);
        TextDrawShowForAll(Spree); SetTimer("TxtReset", 3000, 0);
        
    }

    if(spree[killerid] == 10)
    {
        new string[128];
        new pName[MAX_PLAYER_NAME];
        GetPlayerName(killerid, pName, sizeof(pName));
        format(string, sizeof(string), "%s is on a killing spre with 10 kills~", pName);
        TextDrawHideForAll(Spree);
        TextDrawSetString(Spree, string);
        TextDrawShowForAll(Spree); SetTimer("TxtReset", 3000, 0);
    }
    return 1;
}
//------------------------------------------------------------------------------------
// Commands
//------------------------------------------------------------------------------------
    public OnPlayerCommandText(playerid, cmdtext[])
{
        return 0;
}
CMD:help(playerid, params[])
{
    SendClientMessage(playerid, COLOR_YELLOW,"=====================Help=========================");
        SendClientMessage(playerid, COLOR_YELLOW,"Welcome to Los Santos Gang Wars Help Menu.");
        SendClientMessage(playerid, COLOR_YELLOW,"Kill enemies, and earn a killing spree.");
        SendClientMessage(playerid, COLOR_YELLOW,"You may donate to the scripter if you wish.");
        SendClientMessage(playerid, COLOR_YELLOW,"Experience is earned on our TDM server...");
        SendClientMessage(playerid, COLOR_YELLOW,"Hope you have fun, and enjoy our script!");
        SendClientMessage(playerid, COLOR_YELLOW,"===================================================");
    return 1;
}
CMD:kill(playerid, params[])
{
    SetPlayerHealth(playerid, 0);
    SendClientMessage(playerid,COLOR_RED,"You have killed yourself.");
    return 1;
}
CMD:rules(playerid, params[])
{
    SendClientMessage(playerid, COLOR_GROVE,"=========================Server-Rules=========================");
    SendClientMessage(playerid, COLOR_YELLOW,"1.Do not car park!");
    SendClientMessage(playerid, COLOR_YELLOW,"2.Do not heli blade kill!");
    SendClientMessage(playerid, COLOR_YELLOW,"3.Do not hack or use mods");
    SendClientMessage(playerid, COLOR_YELLOW,"4.Do not spawn-kill");
    SendClientMessage(playerid, COLOR_YELLOW,"5.Don't shoot your team members");
    SendClientMessage(playerid, COLOR_YELLOW,"6.Do not disrespect/spam/flame anyone");
    SendClientMessage(playerid, COLOR_YELLOW,"7.Do not spam anyones pm or main chat");
    SendClientMessage(playerid, COLOR_GROVE,"==============================================================");
    return 1;
}
CMD:ct(playerid, params[])
{
    ForceClassSelection(playerid);
    SendClientMessage(playerid, COLOR_GREEN,"Returing to team selection after next death ....");
    SendClientMessage(playerid, COLOR_YELLOW,"You can use /kill to go to team selection right now");

    return 1;
}
Find
Reply
JJB562
Big Clucker
***
Posts: 186
Threads: 2
Joined: Aug 2011
Reputation: 0
#2

17.06.2013, 22:07
Since, you have barely started on this script. I would suggest to switch dini to y_ini.
Also, the main problem is this piece of the code.

pawn Код:
new string[30];
format(string, 30, "%s\r\n", pname);
fwrite(ftw, string);
Your user system is terrible. By looking at the script, it seems that you are saving stats based on name. Although this necessarily isn't bad, the system doesn't require a password so anyone could just login as a user, even though it's not their account.

Here's a great tutorial that will help you on your register/login system:
https://sampforum.blast.hk/showthread.php?tid=273088

However, instead of using 'udb_hash' as the hashing system. Use Whirlpool instead since it is far more secure.
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 1 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help