[Andre02]

Quote:

Originally Posted by Lucases This one is better, and you won't need to script the loop inside every time command. PHP Code: #define SetTimeForAll(0%, %1)     for(new i; i<MAX_PLAYERS; i++) { SetPlayerTime(i, 0%, %1) }  PHP Code: CMD:day(playerid, params[]) {     if(PlayerInfo[playerid][pAdmin] <= 2) return SendClientMessage(playerid, COLOR_RED, "ERROR: Insufficient Permissions!");         SetTimeForAll(12,0);     SendClientMessageToAll(COLOR_RED, "Server's time has been set to day time!");     return 1; }

For sure, specially since he doesn't need a loop at all for that since he has SetWorldTime(12) and SendClientMessageToAll()?

I don't recommend using a loop for that.
The default value of MAX_PLAYERS is 500 meaning that, the loop you're doing will run 500 times which is totally useless, unless you have 500 slots/players in your server.
Not only it's useless IMO, but if you have other loops running at the same time as this one, it may cause lag which is something your players wouldn't enjoy that much.

I recommend you to do this:

Quote:

Originally Posted by TomRedlake Why not replacing it like this (much better than doing loop): Code: CMD:day(playerid, params[]) { if(PlayerInfo[playerid][pAdmin] <= 2) return SendClientMessage(playerid, COLOR_RED, "ERROR: Insufficient Permissions!"); SetWorldTime(12); SendClientMessageToAll(COLOR_RED, "Server's time has been set to day time!"); return 1; } Check it more here: SetWorldTime