Show IP - Плъгин

[JustyleR]

Информация:
Здравейте, чудех се какъв плъгин да напиша, та реших да напиша един плъгин за Show IP..
Когато играч напише в конзолата : amx_showip name, ще покаже информацията на играча:

Code: Select all

______________

Igrach: Име на играча
Durjava: Държава на играча
IP: IP на играча
UserID: Ид-то на играча
_____________

Ако имате съвети, въпроси, пишете в темата.

Изтегли:

Show IP 1.0.rar

(4.43 KiB) Downloaded 209 times

[/size]

[Dragonspower]

Плъгина е добре измислен и лесно за ориентация, Браво.
Плъгина го сложих в сървърите ни

[ATANASOW]

Плъгина е много полезен,добре са го измислили

[HoBaKaT]

Wut...

Стига бе не ви ли омръзна.
Едно и също след едно и също ...
Дайте нещо ново, нещо свежо.
Нищо ново под слънцето.

+ плъгина ти е пълен copy/paste.
Тук/там нещо неясно.

И в момента какво е това ADMIN_USER ???
Ако е за всички не е ADMIN_USER, a e ADMIN_ALL.
Като не правиш проверка дали има правата ? в функцията.

Отделно какво е това обяснение ?

Code: Select all

register_concmd("amx_showip", "show_players_info", ADMIN_USER, "<nick>")

Ти сега нали се сещаш, че на играча му казваш.
amx_showip <nick>, че така трябва да го напише.
Т.е. им казваш, че от някъде дърпа информацията ?

Оценка:-1/10

[JustyleR]

Wut...

Стига бе не ви ли омръзна.
Едно и също след едно и също ...
Дайте нещо ново, нещо свежо.
Нищо ново под слънцето.

+ плъгина ти е пълен copy/paste.
Тук/там нещо неясно.

И в момента какво е това ADMIN_USER ???
Ако е за всички не е ADMIN_USER, a e ADMIN_ALL.
Като не правиш проверка дали има правата ? в функцията.

Отделно какво е това обяснение ?

Code: Select all

register_concmd("amx_showip", "show_players_info", ADMIN_USER, "<nick>")

Ти сега нали се сещаш, че на играча му казваш.
amx_showip <nick>, че така трябва да го напише.
Т.е. им казваш, че от някъде дърпа информацията ?

Оценка:-1/10

Хм.. Големият се обади.. Както и да е и само да кажа, плъгина не е copy/paste.
Ама, както и да е. Май не си спомняше, като ми рева в скайп да ти оправя системата.. Айде, лека вечер.

[Flickercho]

Премахване на проверката, ненужно е да се задава флаг, при положение, че не е нужна проверка за флаговете.

[JustyleR]

Премахване на проверката, ненужно е да се задава флаг, при положение, че не е нужна проверка за флаговете.

Абе, да ви кажа.. Лесно е да се каже, да имаше уроци по PAWN, да съм направил плъгина на 6.. Ама, като няма уроци..? Както и да е.. Ако имате критики, кажете какви са и как да ги оправя..

[Flickercho]

Добре ще е да смениш флага на ADMIN_ALL, защото така админите, които нямат изрично зададен флаг "z" няма да могат да ползват командата. Може и да греша, но за по-сигурно ще е добре да го обновиш. Също някои неща по стила на кода, които може и някой hater да ги вземе за подозрителни - веднъж си използвал charsmax, а друг път sizeof() - 1; веднъж си използвал console_pritn, а друг път client_pritn(id, print_console... Тези различия развалят идентичността и стила на кода. Също така е по-добре да използваш %d или %i за взимането на IP-то в принт-а, тъй като е цифрова комбинация.

[KiLLeR.]

Също някои неща по стила на кода, които може и някой hater да ги вземе за подозрителни - веднъж си използвал charsmax, а друг път sizeof() - 1; веднъж си използвал console_pritn, а друг път client_pritn(id, print_console... Тези различия развалят идентичността и стила на кода.

Това бих могъл да си го обясня само с това че е имало 'copy/paste'. Макар че и така плъгина пак ще си работи.

[Flickercho]

Именно затова отбелязвам, за стила и идентичността. Защото на практика така изглежда.

[HoBaKaT]

Аз да те светна, че си в грешка:

Code: Select all

ADMIN_ALL		0	//Everyone
ADMIN_IMMUNITY		1	//Flag "a", immunity
ADMIN_RESERVATION	2	//Flag "b", reservation
ADMIN_KICK		4	//Flag "c", kick
ADMIN_BAN		8	//Flag "d", ban
ADMIN_SLAY		16	//Flag "e", slay
ADMIN_MAP		32	//Flag "f", map change
ADMIN_CVAR		64	//Flag "g", cvar change
ADMIN_CFG		128	//Flag "h", config execution
ADMIN_CHAT		256	//Flag "i", chat
ADMIN_VOTE		512	//Flag "j", vote
ADMIN_PASSWORD		1024	//Flag "k", sv_password
ADMIN_RCON		2048	//Flag "l", rcon access
ADMIN_LEVEL_A		4096	//Flag "m", custom
ADMIN_LEVEL_B		8192	//Flag "n", custom
ADMIN_LEVEL_C		16384	//Flag "o", custom
ADMIN_LEVEL_D		32768	//Flag "p", custom
ADMIN_LEVEL_E		65536	//Flag "q", custom
ADMIN_LEVEL_F		131072	//Flag "r", custom
ADMIN_LEVEL_G		262144	//Flag "s", custom
ADMIN_LEVEL_H		524288	//Flag "t", custom
ADMIN_MENU		1048576	//Flag "u", menus
ADMIN_ADMIN		16777216	//Flag "y", default admin
ADMIN_USER		33554432	//Flag "z", default user

Премахване на проверката, ненужно е да се задава флаг, при положение, че не е нужна проверка за флаговете.

Абе, да ви кажа.. Лесно е да се каже, да имаше уроци по PAWN, да съм направил плъгина на 6.. Ама, като няма уроци..? Както и да е.. Ако имате критики, кажете какви са и как да ги оправя..

Още излагания...
Още от самото начало ти казах, прочети малко за амхх ...
В тая документация, тя е написана точно за това да има да се информираш, кое/как/какво и защо, а не да даваш copy/paste кодове.
Документация.

Защо мисля, че ти е copy/paste кода ?

Защото, като правиш нещо за определен флаг администратор то ти трябва да имаш проверка директно в началото на функцията, която да проверява дали дадения потребител id има този флаг или го няма, и след, ако го има ще продължи кода на долу да се изпълнява за него.
Ако ли не, нищо няма да стане.

Като гледам ADMIN_USER ти явно си искал да го направиш за всички и вместо ADMIN_USER можеш да ползваш или ADMIN_ALL или да го оставиш празно.
Но, ако го оставиш празно трябва и при функцията ти да махнеш level,cid понеже няма да са нужни.

Може кода да не ти е copy/paste, но си гледал от други 2/3 кода, за да направиш тази грешка.
Защото на едното място си видял с проверката за правата, а на другото без.

Поне това е моята теория.

Защото ти, ако беше чел до сега от тази документация, щеше да знаеш доста повече от това, защото ако не се лъжа 3/4 месец ти е вече да се занимаваш с тези плъгини.

Колкото до това, аз си казах мнението за кода вече, защото не мисля, че с още 1 show IP амхх става на по-горно ниво.

[StreeT IzroD]

Сложих го, ноооо..

Client with that name or userid not found
Invalid Nick

?

[JustyleR]

Сложих го, ноооо..

Client with that name or userid not found
Invalid Nick

?

Не може да намери играча.. Близко е до акъла.

[h0ligan]

А играчите могат ли дапишат тва в конзолата ?

[OciXCrom]

А играчите могат ли дапишат тва в конзолата ?

Ама не, пише се на лист хартия. Мисли малко преди да задаваш глупави въпроси, за които и сам можеш да намериш отговор.