AMXX.pl: Support AMX Mod X i SourceMod

Na wstępie chciałbym prosić każdego czytelnika o dołożenie swojej wiedzy do tematu i podzielenie się doświadczeniem, żeby wspólnie udoskonalić generator.

Ode mnie takie info:

Usunąć makrodefinicję IsPlayer

#define IsPlayer(%1) (1<=%1<=maxPlayers)

static cell AMX_NATIVE_CALL is_user_connected(AMX *amx, cell *params) /* 1 param */
{
	int index = params[1];
	
	if (index < 1 || index > gpGlobals->maxClients)
		return 0;
	
	CPlayer* pPlayer = GET_PLAYER_POINTER_I(index);
	
	return (pPlayer->ingame ? 1 : 0);
} 

Zależnie od tego, czy chcemy iść z tym wszystkim pod kod easy-to-read czy wydajny:

Zamiana bool'a g_Vip na bity

new bool:g_Vip[33]

new g_Vip;

#define SetBit(%2,%1) (%1 |= (1 << (%2 & MAX_PLAYERS - 1)))
#define RemoveBit(%2,%1) (%1 &= ~(1 <<(%2 & MAX_PLAYERS - 1)))
#define GetBit(%2,%1) (%1 & (1 << (%2 & MAX_PLAYERS - 1)))
#define GetBool(%2,%1) (%1 & (1 << (%2 & MAX_PLAYERS - 1)) == (1 << (%2 & MAX_PLAYERS - 1))) 

#define MAX_PLAYERS 32 

maxPlayers=get_maxplayers(); 

Zmiana syntaxu sprawdzania sumy bitowej flag gracza

if(get_user_flags(id) & 524288 == 524288){ 

if(get_user_flags(id) & ADMIN_LEVEL_H){ 

Zmiana momentu i częstotliwości przypisywania dostępu do vipa

public client_authorized(id){
	if(get_user_flags(id) & 524288 == 524288){
		client_authorized_vip(id);
	}
}
public client_authorized_vip(id){
	g_Vip[id]=true;
} 

public client_authorized(index)
	if(get_user_flags(index) & ADMIN_LEVEL_H)
		SetBit(index, g_Vip); 

public plugin_init()
	register_event("HLTV", "Nowa_Runda", "a", "1=0", "2=0");

public Nowa_Runda()
	for(new i = 1; i <= 32; i++)
		if(is_user_connected(i)	&& get_user_flags(i) & ADMIN_LEVEL_H && !GetBool(i, g_Vip)
			SetBit(i, g_Vip); 

Jednorazowe pobranie nicku gracza, żeby uniknąć powtarzania się tego przy liście vipów, say'u etc.

new userName[MAX_PLAYERS][33];


public client_authorized(index)
{
	if(is_user_hltv(index))
		return;

	get_user_name(index, userName[index], charsmax(userName[]));
} 

Zmiana syntaxu w sayHandle na łatwy do odczytania przy prefixie

public handleSayText(msgId,msgDest,msgEnt){
	new id = get_msg_arg_int(1);
	
	if(is_user_connected(id) && g_Vip[id]){
		new szTmp[256],szTmp2[256];
		get_msg_arg_string(2,szTmp, charsmax(szTmp))
		
		new szPrefix[64] = "^x04[VIP]";
		
		if(!equal(szTmp,"#Cstrike_Chat_All")){
			add(szTmp2,charsmax(szTmp2),szPrefix);
			add(szTmp2,charsmax(szTmp2)," ");
			add(szTmp2,charsmax(szTmp2),szTmp);
		}
		else{
			add(szTmp2,charsmax(szTmp2),szPrefix);
			add(szTmp2,charsmax(szTmp2),"^x03 %s1^x01 :  %s2");
		}
		set_msg_arg_string(2,szTmp2);
	}
	return PLUGIN_CONTINUE;
} 

public sayHandle(msgId, msgDest, msgEnt)
{
	new index = get_msg_arg_int(1);

	if(!is_user_connected(index) || !GetBool(index, g_Vip))
		return PLUGIN_CONTINUE;

	new chatString[2][192];

	get_msg_arg_string(2, chatString[0], charsmax(chatString[]));

	new chatPrefix[64] = "^x04VIP";

	if(equal(chatString[0], "#Cstrike_Chat_All"))
	{
		get_msg_arg_string(4, chatString[0], charsmax(chatString[]));
		set_msg_arg_string(4, "");

		formatex(chatString[1], charsmax(chatString[]), "%s^x03 %s^x01 :  %s", chatPrefix, userName[index], chatString[0]);
	}
	else
		formatex(chatString[1], charsmax(chatString[]), "%s^x01 %s", chatPrefix, chatString[0]);

	set_msg_arg_string(2, chatString[1]);

	return PLUGIN_CONTINUE;
}

Przy okazji tego wszystkiego można by zmienić syntax kodu overall; usunąć zbędne spacje, ogarnąć powtarzający się kod, nazwy zmiennych z np. "g_Vip" na "hasVip" (tak, wiem, że są różne syntaxy pisania kodu i różne firmy wymagają różnych sposobów jego pisania, ale tutaj możemy śmiało powiedzieć, że sami będziemy na tym korzystać. Moim zdaniem łatwiej się czyta "hasVip" niż "g_Vip", ale to już preferencje personalne, ja tylko daję propozycję ), usunąć zbędne przechodzenie do osobnych funkcji, usunąć brackety przy jednoliniowych funkcjach (tak, tutaj też wiem, że to ma swój cel i różni programiści [...]).

W sumie pomysł dobry tyle że z tym nickiem trzeba jeszcze łapać infochanged bo nick można zmienić podczas gry.

W sumie pomysł dobry tyle że z tym nickiem trzeba jeszcze łapać infochanged bo nick można zmienić podczas gry.

public plugin_init()
	register_forward(FM_ClientUserInfoChanged, "clientInfoChanged");

public clientInfoChanged(index)
	get_user_name(index, userName[index], charsmax(userName[]));

Tak tylko, żeby były od razu przykłady. O tym rzeczywiście zapomniałem

Dodam jeszcze ze g_Vip bym zostawił. Nie zmieniał na bitowe smieszki.

Skoro byśmy mieli MAX_PLAYERS (co chyba w tym nowym amx modzie jest juz w którymś z nagłowkow) to nie widze sensu uzywac get_maxplayers() 

Btw, jakbyś dokopał się dlaczego get_maxplayers ma być złe to chętnie się dowiem

Nie możesz użyć zmiennej, do której przypiszesz wartość zwracaną przez get_maxplayers do określenia wielkości tablicy przy jej deklaracji. Makrodefinicja już na to pozwala.

Tyle wiem sam, a o co konkretniej chodziło to nadal się nie dowiedziałem. Tak na chłopski rozum to może ma to coś do czynienia z ukrytym slotem? Dodatkowo trzeba bawić się w odejmowanie HLTV etc. Zwyczajnie w świecie wygodniej jest dla nas (jako skrypterów, nie osobie prowadzącej serwer) użycie makrodefinicji MAX_PLAYERS.

Wersja z bitami to tylko typowe pójście w stronę wydajności.

Nie możesz użyć zmiennej, do której przypiszesz wartość zwracaną przez get_maxplayers do określenia wielkości tablicy przy jej deklaracji.

No ja to wiem. Dlatego napisałem, żeby używać makra

Wersja z bitami to tylko typowe pójście w stronę wydajności.

Czy ja wiem czy taka wydajność. IMO gra nie warta świeczki.

Co do samego generatora to może wystarczy poprawić tylko czytelność. Popularność tego moda podupadła dosyć, nie wiem czy jest sens w ogóle to rozwijać. Ale nie mnie decydować

Dlatego mówiłem, że robimy to pod siebie. Ja, jako skrypter, nie właściciel serwera, wolę mieć to w makrodefinicji, bo łatwiej jest ją zmienić i zrekompilować plugin, a zyskujemy na pamięci. Tego samego nie zrobimy z get_maxplayers. Nie chodzi o maksymalną ilość graczy narzuconą przez silnik (32), tylko o aktualnie maksymalną wartość (ilość slotów serwera), która nie zmienia się raczej nigdy (chyba, że ktoś lubi sobie zmieniać sloty co 2 dni, to wtedy już co innego).

Zbieżność nazw była przypadkowa, nie odnosiłem się do tej stałej z wersji 1.8.3. Tak po prostu zawsze nazywam tę makrodefinicję. Btw. o ile się orientuję to #define może być zmieniony podczas pracy programu, ale kompilator wyrzuci warning, że "warning 201: redefinition of constant/macro (symbol "MAX_PLAYERS")".

To prawda, menu też są jedną wielką katastrofą.

Zamiana bool'a g_Vip na bity

To jest spoko ale zamiast macrodefinicji imho lepiej zrobić poprostu 4 funkcje które zrobią te same operacje bitowe na globalnej zmiennej i będą mogły sobie pobrać ilość slotów przy użyciu get_maxplayers -> zmiana w macro i kompilowanie jest słabe jak chcemy mieć jednego vipa na x serverów. Natomiast czy jest sens to robić? Nie - nie osiągnie się tym większego performacne a tylko skomplikuje się kod dla ludzi którzy nie koniecznie umieją programować

jednocześnie unikając rozdzielania tego na dwie funkcje (które szczerze nie wiem dlaczego w ogóle się tam znalazły?).

Akurat funkcji powinno być jak najwięcej bo to poprawia czytelność kodu oraz łatwiej się tak zamyka logikę zamiast pisać wszystko w jednej funkcji robimy n mniejszych atomowych które potem używamy

usunąć zbędne przechodzenie do osobnych funkcji, usunąć brackety przy jednoliniowych funkcjach

Ludzie, którzy nie znają się na programowaniu i tak nie zaglądają do kodu. Ja osobiście dążę do maksymalnej możliwej wydajności zachowując przy tym umiarkowaną prostotę kodu, chociażby przez (zazwyczaj odgórne, poglądowe, ale jednak) obliczanie średniej długości tekstu przy określaniu wielkości tablicy, utrzymywanie się przy stałej MAX_PLAYERS (i tutaj znowu, zbieżność przypadkowa) i określaniu wielkości tablic posiłkując się wartością makrodefinicji. Możesz mnie nazywać pedantem, ale nie wmówisz mi, że MaciekXkozak2005PL będzie zaglądał w kod, żeby się upewnić, że jest on napisany w taki a nie inny sposób - dla osób średnio zaawansowanych nic się nie zmienia, bo nie muszą znać zasady działania kodu jeśli funkcja nazywa się Get/Set Bit i BoolBit, raczej wszystko jest intuicyjne (tym bardziej, że można je nazwać SetVip, GetVip etc.) .

Sam piszę praktycznie zawsze rozbijając kod na pojedyncze funkcje, zawierając w nich kod zgodny z ich nazwami. W przytoczonym cytacie chodziło o przechodzenie między funkcjami w ten sposób:

public client_authorized(id){
	if(get_user_flags(id) & 524288 == 524288){
		client_authorized_vip(id);
	}
}
public client_authorized_vip(id){
	g_Vip[id]=true;
}
public client_disconnect(id){
	if(g_Vip[id]){
		client_disconnect_vip(id);
	}
}
public client_disconnect_vip(id){
	g_Vip[id]=false;

Czy jest to naprawdę potrzebne, skoro nic innego w tych funkcjach nie jest używane?

Odwołując się od trzeciego cytatu; tutaj zaznaczyłem i wydawało mi się, że jest to dosyć jasne co próbuję przekazać? "usunąć brackety przy jednoliniowych funkcjach (tak, tutaj też wiem, że to ma swój cel i różni programiści [...])."

Czy jest to naprawdę potrzebne, skoro nic innego w tych funkcjach nie jest używane?

Ja osobiście dążę do maksymalnej możliwej wydajności zachowując przy tym umiarkowaną prostotę kodu,

To że to nie jest generator dla ludzi znających się na rzeczy nie znaczy że nie może być czytelny, nie oszukujmy się trudno jest zrobić vipa tak żeby był nieoptymalny i miał jakiś większy wpływ na to jak działa serwer

A ja proponuję poprawić kompatybilność kompilatora z kodem. Przykład: client_disconnect na client_disconnected. Unikniemy wtedy takich tematów.

Dodatkowo można jakoś zmienić wygląd motd, przy zaznaczeniu opcji "Automatycznego generowania MOTD". Zmienić czcionkę, sposób zapisu HTML (bo ten jest trochę 'niemodny'), poprawić literówki (przykład "ba" na "na").

No i zaktualizować kod prefixów na taki, jaki podał _McHappy w jednym z tematów. 

Usunąć makrodefinicję IsPlayer

Zmiana syntaxu sprawdzania sumy bitowej flag gracza

To chyba najprostsze do zrobienia rzeczy, a ludzi piszących o zmiane flagi jest dość sporo, bo sam czasem na PW dostaje jak zmienic flage bo  w kodzie nie ma ADMIN_LEVEL_*

Małymi kroczkami może trzeba zmieniać