[ZigGamerx]

Quote:

Originally Posted by Dennis12 Old code .. 1. Why you loop all players? Code: for(new i = 0; i <MAX_PLAYERS; i++) If you don't want to use ITERATORS ( YSI\y_iterate ) to select only from VIP LIST you can make something like this, but with iterators is much better. Code: for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++) This loop will go up to the highest ID on the server, the loop you have with MAX_PLAYERS goes through the 1000 players (define default, even if you have 100 on the server). Why INTEGER only for 0/1 (false / true) ..? Use the boolean variable .. pGOD should be a bool. Code: bool: pGOD ( .. in pInfo ) At least if you used a switch and not if / else at CMD: vgod .. EDIT: OHHH, SORRY MAN, IT'S YOUR FIRST SCRIPT .. Take my words above as some tips, not critics, now I just saw that it is your first script, very good for a first script, you are on a good road.

Next time I'll follow your Tips.