Хороший ли код в этом плагине?

#include <amxmodx>
#include <fun>
#include <cstrike>
#include <hamsandwich>
//#include <engine> // На хер он здесь нужен?

// А где register_plugin(...) ?
#define PLUGIN 	"jVip"
#define VERSION "1.0"
#define AUTHOR 	"tYnE#"	// Мега-скриптер. Сегодня закажу меню оружий.

// Переключатели :D
new iPlayerAction[33][4]

public plugin_init() 
{
	register_clcmd("vipmenu", "VIPMenu")

	// Видимо парень забыл про неё, ведь функция есть, xD
	RegisterHam(Ham_Spawn, "player", "Spawn_player", 1)

	// Отлов события переключения оружия (Фиксим скорость)
	register_event("CurWeapon", "CurWeapon", "be", "1=1")
}

public VIPMenu(id) 
{
	// Не знаю, почему автор не сделал так:
	if(!(get_user_flags(id) & ADMIN_LEVEL_H))
	{
		client_print(id, print_chat, "[Server] У тебя нет прав использовать эту комманду.") 
		return PLUGIN_HANDLED
	}

	new vip_menu = menu_create("\wМеню \yv\d.\yi\d.\yp \w игрока.", "menu_handler")

	// Делаем переключатель для 1 пункта
	if(iPlayerAction[id][1])
	{
   		menu_additem(vip_menu, "Бессмертие\d[ \rВключено \d]", "1", 0)
	}else{
		menu_additem(vip_menu, "Бессмертие\d[ \rВыключено \d]", "1", 0)
	}

	// Делаем переключатель для 2 пункта
	if(iPlayerAction[id][2])
	{
   		menu_additem(vip_menu, "Ходить сквозь стен\d[ \rВключено \d]", "2", 0)
	}else{
   		menu_additem(vip_menu, "Ходить сквозь стен\d[ \rВыключено \d]", "2", 0)
	}

	// Запрещаем юзать мертвому.
	if(is_user_alive(id))
	{
		// Делаем переключатель для 3 пункта
		if(iPlayerAction[id][3])
		{
   			menu_additem(vip_menu, "Скорость\d[ \rВключена \d]", "3", 0)
		}else{
   			menu_additem(vip_menu, "Скорость\d[ \rВыключена \d]", "3", 0)
		}

	}else{
   		menu_additem(vip_menu, "Скорость", "3", ADMIN_ADMIN)
	}

	// Запрещаем юзать мертвому.
	if(is_user_alive(id))
	{
		menu_additem(vip_menu, "Взять жизней \d(\r255\d)", "4", 0)
	}else{
		menu_additem(vip_menu, "Взять жизней \d(\r255\d)", "4", ADMIN_ADMIN)
	}

	// Мега-баг. Запрещаем юзать живому и спеку
	if(!is_user_alive(id) && cs_get_user_team(id) != CS_TEAM_SPECTATOR)
	{
		menu_additem(vip_menu, "Возродиться", "5", 0)
	}else{
		menu_additem(vip_menu, "Возродиться", "5", ADMIN_ADMIN)
	}

	menu_additem(vip_menu, "Взять \y16000$", "6", 0)

	// Запрещаем юзать мертвому.
	if(is_user_alive(id))
	{
		menu_additem(vip_menu, "Взять \yщит", "7", 0)
	}else{
		menu_additem(vip_menu, "Взять \yщит", "7", ADMIN_ADMIN)
	}

	// Запрещаем юзать мертвому.
	if(is_user_alive(id))
	{
		menu_additem( vip_menu, "Взять \yHE\w гранату", "8", 0)
	}else{
		menu_additem( vip_menu, "Взять \yHE\w гранату", "8", ADMIN_ADMIN)
	}

	// Запрещаем юзать мертвому.
	if(is_user_alive(id))
	{
		menu_additem( vip_menu, "Взять пистолет \yAK47", "9", 0)
	}else{
		menu_additem( vip_menu, "Взять пистолет \yAK47", "9", ADMIN_ADMIN)
	}

	// Может логичней убрать навигацию, если всего 9 пунктов?
	// menu_setprop(vip_menu, MPROP_EXIT, MEXIT_ALL)
	menu_setprop(vip_menu, MEXIT_ALL, 0)

	menu_display(id, vip_menu, 0)
	return PLUGIN_HANDLED
}

public menu_handler(id, menu, item) 
{
	if(item == MENU_EXIT) 
	{
		menu_destroy( menu )
		return PLUGIN_HANDLED
	}

	new data[6], iName[64], access, callback
	menu_item_getinfo( menu, item, access, data,5, iName, 63, callback)

	new key = str_to_num( data )

	switch( key ) 
	{
		case 1:
		{
			//Проверяем: если уже есть
			if(iPlayerAction[id][1])
			{
				iPlayerAction[id][1] = false	// Ставим массив на false.
				set_user_godmode(id, 0) 		// Вырубаем
			}else{								// Иначе включаем
				iPlayerAction[id][1] = true		// Ставим массив на true
				set_user_godmode(id,1)			// Включаем
			}

			// Переоткроем меню с понтом дела
			VIPMenu(id) 
		}

		case 2:
		{
			// Проверяем: если уже есть
			if(iPlayerAction[id][2])
			{
				iPlayerAction[id][2] = false	// Ставим массив на false.
				set_user_noclip(id, 0) 			// Вырубаем
			}else{								// Иначе включаем
				iPlayerAction[id][2] = true		// Ставим массив на true
				set_user_noclip(id,1)			// Включаем
			}

			// Переоткроем меню с понтом дела
			VIPMenu(id) 
		}

		case 3:
		{
			// Проверяем: если уже есть
			if(iPlayerAction[id][3])
			{
				iPlayerAction[id][3] = false	// Ставим массив на false.
				set_user_maxspeed(id, 240.0)	// Вырубаем
			}else{								// Иначе включаем
				iPlayerAction[id][3] = true		// Ставим массив на true
				set_user_maxspeed(id, 1000.0)	// Включаем
			}
			
			// Переоткроем меню с понтом дела
			VIPMenu(id) 
		}

		case 4:
		{
			// Если игрок мертв, то нет смысла продолжать
			if(!is_user_alive(id))
				return PLUGIN_HANDLED

			set_user_health(id, 255)
		}

		case 5:
		{
			//o0. Чувак жжет. Получаем унриал баг с экранными худами при таком спавне.
			//Чувак - гуру скриптинга LOL
			//spawn(id)

			//Если игрок жив или спек, то продолжать нельзя =)
			if(is_user_alive(id) || cs_get_user_team(id) == CS_TEAM_SPECTATOR)
				return PLUGIN_HANDLED

			ExecuteHamB(Ham_CS_RoundRespawn, id)
		}

		case 6:
		{
			cs_set_user_money(id,16000)
		}

		case 7:
		{
			if(!is_user_alive(id))
				return PLUGIN_HANDLED

			give_item(id, "weapon_shield")
		}

		case 8:
		{
			if(!is_user_alive(id))
				return PLUGIN_HANDLED

			give_item(id, "weapon_hegrenade")
		}

		case 9:
		{
			if(!is_user_alive(id))
				return PLUGIN_HANDLED

			give_item(id, "weapon_ak47")
			cs_set_user_bpammo(id, CSW_AK47, 90) //Даем патроны
		}
	}

	menu_destroy( menu )
	return PLUGIN_HANDLED
}

public CurWeapon(id)
{
	if(!iPlayerAction[id][3])
		return PLUGIN_CONTINUE

	set_user_maxspeed(id, 1000.0)
	return PLUGIN_CONTINUE
}

//А где, собственно отлов????????
//Но в моем случаи мне функция пригодилась
public Spawn_player(id)
{
	//Так круче :DD
	if(!is_user_alive(id) || !is_user_connected(id))
		return PLUGIN_CONTINUE

	//Присваиваем в зависимости от переключателя
	set_user_godmode(id, iPlayerAction[id][1])
	set_user_noclip(id, iPlayerAction[id][2])

	//Эмулуем переключение на нож, чтобы вызвать кур =)
	engclient_cmd(id, "weapon_knife")

	return PLUGIN_CONTINUE
}