Вы находитесь на Pro-Pawn, а не на каком-то "другом портале", поэтому будьте добры, указывайте ник, который у вас на этом форуме.
Либо можете попросить меня или Osetin'а сменить вам ник, здесь это бесплатно.
Теперь по поводу кода.
Если ваш код критикуют, старайтесь вникнуть в суть критики, понять, в чём заключается недостаток, на который вам указывают - возможно он в вашей работе не один.
TheMallard уже правильно подметил, что название "seller" не уникальное - видимо рассчитывал, что вы поймёте, что у других переменных точно такие же неуникальные названия (в чужом моде наверняка могут найтись и какие-нибудь другие "PlayerMenuTD" или "BuyMenu"), но вы послушно сделали только то, о чём вас попросили, и ни грамма больше.
Вот вам небольшая задача. В моде есть 5 переменных:
Определите по названию, какая из них (только одна) относится к системе работы лесоруба.PHP код:
new action[MAX_PLAYERS];
new pickup;
new selected[MAX_PLAYERS char];
new uses[MAX_PLAYERS char];
new object[MAX_PLAYERS];
Надеюсь, сможете сделать из этого какие-нибудь выводы.