While (level < 0 || level > 4)
{
return SendClientMessage(playerid,0x3740B3,"Admin level must be between 0 and 4");
}
What's the point of using while when it's going to be called only once? It could also cause problem if someone forgot to use return and that would cause an infinite loop. Plus it's while and not While.