Originally Posted by EnzoMetlc
Just some suggstions:
1.
pawn Code:
CMD:exit(playerid, params[])
{
SpawnPlayer(playerid);
ActualizarJuegos();
EsHumano[playerid] = 0;
EsZombie[playerid] = 0;
Carnes[playerid] = 0;
MediKits[playerid] = 0;
Game[playerid][Zhomviez] = 0;
Game[playerid][CSMap1] = 0;
Game[playerid][CSMap2] = 0;
Game[playerid][CSMap3] = 0;
Game[playerid][CSMap4] = 0;
Game[playerid][CSMap5] = 0;
Game[playerid][CSMap6] = 0;
Game[playerid][CSMap7] = 0;
Game[playerid][CSMap8] = 0;
Game[playerid][CSMap9] = 0;
Game[playerid][CSMap10] = 0;
Game[playerid][CSMap11] = 0;
Game[playerid][MinigunDM] = 0;
Game[playerid][Minigun2DM] = 0;
Game[playerid][RocketDM] = 0;
Game[playerid][RocketDM2] = 0;
Game[playerid][ZonaWW] = 0;
Game[playerid][ZonaRW] = 0;
Game[playerid][Counter] = 0;
return 1;
}
//...
CALLBACK: ActualizarJuegos()
{
SetTimer("UsersCS", 1000, true);
SetTimer("UsersCS11", 1000, true);
SetTimer("UsersCS10", 1000, true);
SetTimer("UsersCS9", 1000, true);
SetTimer("UsersCS8", 1000, true);
SetTimer("UsersCS7", 1000, true);
SetTimer("UsersCS6", 1000, true);
SetTimer("UsersCS5", 1000, true);
SetTimer("UsersCS4", 1000, true);
SetTimer("UsersCS3", 1000, true);
SetTimer("UsersCS2", 1000, true);
SetTimer("UsersCS1", 1000, true);
SetTimer("UsersMinigun", 1000, true);
SetTimer("UsersMinigun2", 1000, true);
SetTimer("UsersRocket", 1000, true);
SetTimer("UsersRocket2", 1000, true);
SetTimer("UsersMDM", 1000, true);
SetTimer("UsersZonaWW", 1000, true);
SetTimer("UsersZonaRW", 1000, true);
SetTimer("UsersZombieAttack", 1000, true);
return 1;
}
That may over-load your server because it will be called often by players and will update these mini-games. You only need one timer, and use it for update all mini-games.
2. What Sreyas said.
3. What Sreyas said x2, or this:
pawn Code:
stock UsersCS6()
{
new Gaim12;
for(new i, g = GetMaxPlayers(); i < g; i++)
{
//if(IsPlayerConnected(i)) You doesn't check if it's connected, cause "CSMap6" can verify that in some way.
if(Game[i][CSMap6] == 1) Gaim12++;
}
return Gaim12;
}
3. Don't use "CALLBACK:" for callbacks that are pre-defined by SA-MP includes. Use "public" instead.
4. You have many arrays size problems.
But don't get mad! You will improve this in a future 
It's a nice system, so keep it up  |
