Giving me 1 Error
#1

Look , My teams are working and everything , So i tried to add another team { Vagos } , but that happened when i tried :

Codes :

Quote:

#include <a_samp>
#include <Ranks>

#define TEAM_GROVE 1
#define TEAM_BALLA 2
#define TEAM_VAGOS 3
#define TEAM_GROVE_COLOR 0x00FF00AA // Bright Green
#define TEAM_BALLA_COLOR 0xFF00FFAA // Bright Purple
#define TEAM_VAGOS_COLOR 0x00FF00AA // Bright Green

new gTeam[MAX_PLAYERS];

main()
{
print("\n----------------------------------");
print(" Los Santos Gangs War - Made By Tommy");
print("----------------------------------\n");
}

public OnGameModeInit()
{
SetGameModeText("L.S.G.W");
AddPlayerClass(102,2001.1736,-1122.1942,26.6893,179.8825, 0, 0, 0, 0, 0, 0);
AddPlayerClass(107,2521.5156,-1678.9836,15.4372,92.7751, 0, 0, 0, 0, 0, 0);
AddPlayerClass(108,2610.8000,-1097.1997,69.4890,88.0323,0,0,0,0,0,0);
SetPromotionMode(0-5);
SetPromotionMode(2);
AddRank(0, "OutSider", 0, 5000, 50.0, 0, 25, 250);
AddRank(1, "Trusted G", 100, 10000, 75.0, 25.0, 29, 500);
AddRank(2, "Original Gangster", 300, 20000, 100.0, 50.0, 28, 750);
AddRank(3, "Double O.G", 500, 30000, 100.0, 75.0, 24, 1000);
AddRank(4, "Proffesional", 800, 30000, 100.0, 100.0, 30, 1250);
AddRank(5, "Master of War", 1000, 30000, 100.0, 100.0, 31, 1500);
AddRank(6, "Gang Master", 1500, 30000, 100.0, 100.0, 28, 2000);

//--------------------------------------------------------------Cars-----------------------------
AddStaticVehicle(567,2469.7935,-1653.6843,13.2708,90.5193,16,16); // Grove Car ne
AddStaticVehicle(550,2499.4294,-1654.7576,13.2485,254.4190,16,16); // Grove Car two
AddStaticVehicle(445,2506.7881,-1678.3098,13.3342,325.9603,16,16); // Grove Car Three
AddStaticVehicle(580,2474.8115,-1680.0529,13.1468,53.0098,16,16); // Grove Car Four
AddStaticVehicle(554,2473.1538,-1692.3916,13.6217,358.9131,16,16); // Grove car Six
AddStaticVehicle(487,2530.2876,-1677.4856,20.1040,84.7420,16,16); // Grove Chopper
AddStaticVehicle(575,2027.4512,-1141.2002,24.2388,268.0392,174,174); // Ballaz car one
AddStaticVehicle(566,1996.3920,-1130.7765,25.2881,268.9091,174,174); // Ballaz car two
AddStaticVehicle(566,2009.3124,-1131.0549,24.9350,270.7833,174,174); // Ballaz car three
AddStaticVehicle(517,2037.6184,-1131.2960,24.1986,268.1283,174,174); // Ballaz car four
AddStaticVehicle(517,1983.9332,-1140.4438,25.6544,270.9725,174,174); // Ballaz car five
AddStaticVehicle(445,2004.6111,-1141.2178,25.1558,269.5776,174,174); // Ballaz car six
AddStaticVehicle(576,2018.9581,-1140.7462,24.4843,267.8231,174,174); // Ballaz car seven
AddStaticVehicle(576,2051.8625,-1120.1655,24.3129,180.2631,174,174); // Ballaz car Eight
AddStaticVehicle(487,2003.9908,-1109.1211,34.3296,177.8212,174,174); // Ballaz Chopper
AddStaticVehicle(575,2604.3521,-1065.7645,69.1893,5.6632,6,6); // Vagos car 1
AddStaticVehicle(575,2595.9949,-1077.2964,69.1899,359.0371,6,6); // Vagos car 2
AddStaticVehicle(536,2596.3472,-1090.4688,68.9995,0.5688,6,6); // Vagos car 3
AddStaticVehicle(536,2605.9807,-1127.1769,65.6763,26.1111,6,6); // Vagos car 4
AddStaticVehicle(536,2583.3079,-1130.5496,65.1668,15.9907,6,6); // Vagos car 5
AddStaticVehicle(536,2555.7654,-1133.9971,62.7065,267.7499,6,6); // Vagos car 6
AddStaticVehicle(534,2568.5005,-1120.1040,65.0828,289.8283,6,6); // Vagos car 7
AddStaticVehicle(534,2576.3320,-1116.8934,65.7423,291.7586,6,6); // Vagos car 8
AddStaticVehicle(576,2590.9182,-1104.5977,67.4538,327.1710,6,6); // Vagos car 9
AddStaticVehicle(576,2603.7747,-1100.7205,68.6181,356.6893,6,6); // Vagos car 10
AddStaticVehicle(474,2572.7632,-1134.2811,64.8607,272.7364,6,6); // Vagos car 10\1
AddStaticVehicle(474,2572.7632,-1134.2811,64.8607,272.7364,6,6); // Vagos car 11
AddStaticVehicle(474,2546.4392,-1121.2684,62.2027,272.1722,6,6); // Vagos car 12
AddStaticVehicle(487,2589.9819,-1120.9495,66.1942,359.9486,6,6); // Vagos Chopper
return 1;
}

public OnPlayerRequestClass(playerid, classid)
{
SetPlayerPos(playerid, 2531.1123,-899.7438,86.4038,183.2012);
SetPlayerCameraPos(playerid, 2535.2781,-914.8342,86.3623,18.3863);
SetPlayerCameraLookAt(playerid, 2531.1123,-899.7438,86.4038,183.2012);
if(classid == 1)
{
gTeam[playerid] = TEAM_GROVE;
SetPlayerColor(playerid, TEAM_GROVE_COLOR);
}
else
{
gTeam[playerid] = TEAM_BALLA;
SetPlayerColor(playerid, TEAM_BALLA_COLOR);
}
else
{
gTeam[playerid] = TEAM_VAGOS;
SetPlayerColor(playerid, TEAM_VAGOS_COLOR);
}
return 1;
}


public OnPlayerConnect(playerid)
{
GameTextForPlayer(playerid,"~b~Welcome to Los Santos Gangs War",3000,6);
S_OnPlayerConnect(playerid);
return 1;
}
public OnPlayerDisconnect(playerid, reason)
{
S_OnPlayerDisconnect(playerid);
return 1;
}

public OnPlayerSpawn(playerid)
{
S_OnPlayerSpawn(playerid);
return 1;
}
public OnPlayerDeath(playerid, killerid, reason)
{
SendDeathMessage(killerid, playerid, reason); //This is the message that appears on the right of the screen in this form: Killer - Type of gun - The dead player.
{
// Add 1 to this killer's score.
if(killerid != INVALID_PLAYER_ID) SetPlayerScore(killerid, GetPlayerScore(killerid) + 1);
return 1;
}
//We can also add a message that appears in the game GTA San Andreas of "Eliminated".

new string[50];
format(string, sizeof(string), "~R~Wasted");
GameTextForPlayer(playerid, string, 3000, 2);
return 1;
}
public OnVehicleSpawn(vehicleid)
{
return 1;
}

public OnVehicleDeath(vehicleid, killerid)
{
return 1;
}

public OnPlayerText(playerid, text[])
{
return 1;
}

public OnPlayerCommandText(playerid, cmdtext[])
{
if (strcmp("/kill", cmdtext, true, 5) == 0) {
SetPlayerHealth(playerid, 0.0);
return 1;
}
if (strcmp("/Buyhealth", cmdtext, true, 10) == 0) {
if(GetPlayerMoney(playerid) < 10000) {
SendClientMessage(playerid, -1, "You can't afford this , It Cost 10,000$!");
return 1;
}
SetPlayerHealth(playerid, 100.0);
GivePlayerMoney(playerid, -10000); return 1;
}
if (strcmp("/BuyArmor", cmdtext, true, 9) == 0) {
if(GetPlayerMoney(playerid) < 10000) {
SendClientMessage(playerid, -1, "You can't afford this , It Cost 10,000$!");
return 1;
}
SetPlayerArmour(playerid, 100.0);
GivePlayerMoney(playerid, -10000);
return 1;
}
if(strcmp(cmdtext, "/rankinfo", true) == 0) {
GetRankInfo(playerid);
return 1;
}

if(strcmp(cmdtext, "/mystats", true) == 0) {
GetPlayerStats(playerid);
return 1;
}
if(!strcmp(cmdtext, "/BuyWeapons", true))
{
ShowPlayerDialog(playerid, 8, DIALOG_STYLE_LIST, "Weapons List", "Desert Eagle ($20,000)\nM4 ($50,000)\nSpas12 ($100,000)\nSniper Rifle ($80,000)", "Purchase", "Cancel");
return 1;
}
}
public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
return 1;
}

public OnPlayerExitVehicle(playerid, vehicleid)
{
return 1;
}

public OnPlayerStateChange(playerid, newstate, oldstate)
{
return 1;
}

public OnPlayerEnterCheckpoint(playerid)
{
return 1;
}

public OnPlayerLeaveCheckpoint(playerid)
{
return 1;
}

public OnPlayerEnterRaceCheckpoint(playerid)
{
return 1;
}

public OnPlayerLeaveRaceCheckpoint(playerid)
{
return 1;
}

public OnRconCommand(cmd[])
{
return 1;
}

public OnPlayerRequestSpawn(playerid)
{
return 1;
}

public OnObjectMoved(objectid)
{
return 1;
}

public OnPlayerObjectMoved(playerid, objectid)
{
return 1;
}

public OnPlayerPickUpPickup(playerid, pickupid)
{
return 1;
}

public OnVehicleMod(playerid, vehicleid, componentid)
{
return 1;
}

public OnVehiclePaintjob(playerid, vehicleid, paintjobid)
{
return 1;
}

public OnVehicleRespray(playerid, vehicleid, color1, color2)
{
return 1;
}

public OnPlayerSelectedMenuRow(playerid, row)
{
return 1;
}

public OnPlayerExitedMenu(playerid)
{
return 1;
}

public OnPlayerInteriorChange(playerid, newinteriorid, oldinteriorid)
{
return 1;
}

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
return 1;
}

public OnRconLoginAttempt(ip[], password[], success)
{
return 1;
}

public OnPlayerUpdate(playerid)
{
return 1;
}

public OnPlayerStreamIn(playerid, forplayerid)
{
return 1;
}

public OnPlayerStreamOut(playerid, forplayerid)
{
return 1;
}

public OnVehicleStreamIn(vehicleid, forplayerid)
{
return 1;
}

public OnVehicleStreamOut(vehicleid, forplayerid)
{
return 1;
}

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
if(response)// They pressed the first button.
{
switch(dialogid)// If you only have one dialog, then this isn't required, but it's neater for when you implement more dialogs.
{
case 8:// Our dialog!
{
switch(listitem)// Checking which listitem was selected
{
case 0:// The first item listed
{
if(GetPlayerMoney(playerid) < 20000) return SendClientMessage(playerid, 0xFFFFFF, "You don't have enough cash.");
GivePlayerMoney(playerid, -20000);
GivePlayerWeapon(playerid, 24, 250);
}
case 1: // The second item listed
{
if(GetPlayerMoney(playerid) < 50000) return SendClientMessage(playerid, 0xFFFFFF, "You don't have enough cash.");
GivePlayerMoney(playerid, -50000);
GivePlayerWeapon(playerid, 31, 250);
}
case 2: // The third item listed
{
if(GetPlayerMoney(playerid) < 100000) return SendClientMessage(playerid, 0xFFFFFF, "You don't have enough cash.");
GivePlayerMoney(playerid, -100000);
GivePlayerWeapon(playerid, 27, 250);
}
case 3: // The Forth item listed
{
if(GetPlayerMoney(playerid) < 80000) return SendClientMessage(playerid, 0xFFFFFF, "You don't have enough cash.");
GivePlayerMoney(playerid, -80000);
GivePlayerWeapon(playerid, 34, 50);
}
}
}
}
}
return 1;
}
public OnPlayerClickPlayer(playerid, clickedplayerid, source)
{
return 1;
}

Errors :
Quote:

E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(71) : warning 202: number of arguments does not match definition
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(72) : warning 202: number of arguments does not match definition
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(73) : warning 202: number of arguments does not match definition
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(84) : error 029: invalid expression, assumed zero
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(113) : warning 217: loose indentation
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(120) : warning 225: unreachable code
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(120) : warning 217: loose indentation
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(163) : warning 217: loose indentation
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(16 : warning 217: loose indentation
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(177) : warning 209: function "OnPlayerCommandText" should return a value
E:\LOSSAN~1\GAMEMO~1\LSGW2.pwn(341) : warning 217: loose indentation
Pawn compiler 3.2.3664 Copyright © 1997-2006, ITB CompuPhase


1 Error.

Reply
#2

You have two else statements around line 84. I'm not sure exactly what line 84 is, but from what I can see the statement which reads:

else
{
gTeam[playerid] = TEAM_BALLA;
SetPlayerColor(playerid, TEAM_BALLA_COLOR);
}

Needs to be changed too

else if(classid == 2)
{
gTeam[playerid] = TEAM_BALLA;
SetPlayerColor(playerid, TEAM_BALLA_COLOR);
}

Because right now, when you have two else-statements they are both going to take effect under the same circumstances, meaning that your system won't function properly.

All together it should look like:

Code:
public OnPlayerRequestClass(playerid, classid)
{
SetPlayerPos(playerid, 2531.1123,-899.7438,86.4038,183.2012);
SetPlayerCameraPos(playerid, 2535.2781,-914.8342,86.3623,18.3863);
SetPlayerCameraLookAt(playerid, 2531.1123,-899.7438,86.4038,183.2012);
if(classid == 1)
{
gTeam[playerid] = TEAM_GROVE;
SetPlayerColor(playerid, TEAM_GROVE_COLOR);
}
else if(classid == 2)
{
gTeam[playerid] = TEAM_BALLA;
SetPlayerColor(playerid, TEAM_BALLA_COLOR);
}
else
{
gTeam[playerid] = TEAM_VAGOS;
SetPlayerColor(playerid, TEAM_VAGOS_COLOR);
}
return 1;
}
Reply
#3

Well.. yah it Complied , but all i want that when the player choose the 3rd Class his name become Yellow , can anyone help ?
Reply
#4

Looking at your defines it says that Team_Vagos_Color is "BRIGHT GREEN".

Just change it to the define for yellow.
Reply
#5

yah , worked thanks
Reply


Forum Jump:


Users browsing this thread: 2 Guest(s)