[Relaxing]

            if(warnedTimes[iPlayer] == 0){
                warnedTimes[iPlayer]++;
            } else if(warnedTimes[iPlayer] == 1){
                warnedTimes[iPlayer]++;
            } else if(warnedTimes[iPlayer] == 2){
                warnedTimes[iPlayer]++;

What's the purpose of this?!

[LemoNSK]

Those are cringe lines..

[LemoNSK]

Originally Posted by edon1337 Here's some optimization you could do to the code: 1. PHP Code:             if(warnedTimes[iPlayer] == 0){                 warnedTimes[iPlayer]++;             } else if(warnedTimes[iPlayer] == 1){                 warnedTimes[iPlayer]++;             } else if(warnedTimes[iPlayer] == 2){                 warnedTimes[iPlayer]++;  Instead of if() and else if() you could use switch(). 2. You should specify a HUD time because default is 12 seconds PHP Code: set_hudmessage(0, 255, 0, -1.0, -1.6)  3. Player name can't be longer than 32 characters, so you can use [32]. PHP Code: idName[50]

Thank you for your optimalization.

[edon1337]

Now that I noticed, these lines are completely redundant,

PHP Code:

            if(warnedTimes[iPlayer] == 0){
                warnedTimes[iPlayer]++;
            } else if(warnedTimes[iPlayer] == 1){
                warnedTimes[iPlayer]++;
            } else if(warnedTimes[iPlayer] == 2){
                warnedTimes[iPlayer]++; 


Should be just

PHP Code:

warnedTimes[iPlayer]++ 


Also this part of the code is wrong

PHP Code:

server_cmd("amx_kick #%d ^"Warned too many times!^"", get_user_userid(iPlayer));
                new msg[255], idName[50];
                get_user_name(id, idName, charsmax(idName));
                format(msg, charsmax(msg), "Administrator %s warned player %s ^nReason: %s ^nWarns: 3/3 ^nPlayer has been kicked from the server!", 
idName, szTargetName, szReason, warnedTimes[iPlayer]); 
                warnedTimes[iPlayer] = 0; 


[HamletEagle]

edon gave you some suggestions. Unapproved until you improve your code.