If you think that it can be how its right now, just do what your mind says.
Yea, the code will be so much long, but just use your knowledge with enumerates and arrays, you know so much things with these formats. You can minimize so much lines with this.
Here's my changes:
1. In
ColorChat():
You can cache the value of iPlayers[i] with an interger.
You need not start the
iCount with 1
is_user_connected() is not needed
Here:
PHP Code:
message_begin(MSG_ONE_UNRELIABLE, g_iSayText, _, iPlayers[i])
You can do
.player = Player or your interger's name. Just for skip this '_,'. Just a small thing.
And you're using the g_iSayText just one time, you can cache only if you use a native more than one time. Just add
get_user_msgid("SayText").