[Kyance]

Quote:

Originally Posted by Virtual1ty Be very careful with that code, specifically this part: Code: format(string, sizeof(string), "SUBMARINE: Torpedo away! Remaining torpedos: %d/4", Submarine[SubID[playerid]][Torpedos]); CanShoot[playerid] = 0; foreach(Player, i) { if(IsPlayerInRangeOfPoint(i, 20, X, Y, Z)) { SendClientMessage(i, -1, string), string = "\0"; } } The way I see it, you send the first player found in range a message, and then you "NULL terminate" the string, perhaps you wanted to "end it" after the foreach loop? Also I suggest detecting stuff like that under OnPlayerUpdate with static variables, simply because it gets called more frequently.

Too many things, doing a loop each second ( or even less! ) would cause the server to crash imo.
It's the same as running a 1 second ( or 500 ms ) timer which repeats itself..

Still need help though.