[DarknessKnight]

Quote:

Originally Posted by LivingLikeYouDo Dat string is useless. Why are you declaring 128 cells for a 50 cells string? Also, why are you defining the result before the outcomes? Also, for getting name, you use MAX_PLAYER_NAME, which returns a value of 24. This is better: Код: CMD:repair(playerid, params[]) { if(Job[playerid] != 7) return SendClientMessage(playerid, 0xAFAFAFAA, "You not are Mechanic"); if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0xAFAFAFAA, "You are not in a vehicle!"); new name[MAX_PLAYER_NAME + 2], string[50]; GetPlayerName(playerid, name, sizeof(name)); format(string, sizeof(string), "%s repaired his vehicle.", name); SendClientMessageToAll(-1, string); return 1; } @osman2571 : That is the worst example, we are using a temporary new value, and getting name from playername requires opening of file, getting the name, formatting it, and closing the file, which results in slow time.

Thanks alot man
+REP