Robiin

Nie mam pojęcia skąd ten kod wytrzasnąłeś i dalczego był napisany w ten sposób, ale był zły.
 

public handleSayText(msgId, msgDest, msgEnt)
{
	new id = get_msg_arg_int(1);

	if(!is_user_connected(id))
	{
		return PLUGIN_CONTINUE;
	}
	
   	new szTmp[192],
   		szTmp2[192],
   		szText[64],
		nazwa_gildi = g_nazwa_gildi(id, szText, charsmax(szText));
		
	get_msg_arg_string(2, szTmp, charsmax(szTmp));

	if(strlen(szText))
	{
		formatex(szText, sizeof(szText), "^x03|%s|", nazwa_gildi);
	}
	
	if(!equal(szTmp, "#Cstrike_Chat_All"))
	{
		add(szTmp2, charsmax(szTmp2), "^x01");
		add(szTmp2, charsmax(szTmp2), szText);
		add(szTmp2, charsmax(szTmp2), " ");
		add(szTmp2, charsmax(szTmp2), szTmp);
	}
	else
	{
		new szPlayerName[64];
		get_user_name(id, szPlayerName, charsmax(szPlayerName));
		
		get_msg_arg_string(4, szTmp, charsmax(szTmp));
		set_msg_arg_string(4, "");
	
		add(szTmp2, charsmax(szTmp2), "^x01");
		add(szTmp2, charsmax(szTmp2), szText);
		add(szTmp2, charsmax(szTmp2), "^x03 ");
		add(szTmp2, charsmax(szTmp2), szPlayerName);
		add(szTmp2, charsmax(szTmp2), "^x01 :  ");
		add(szTmp2, charsmax(szTmp2), szTmp)
	}
	
	set_msg_arg_string(2, szTmp2);

	return PLUGIN_CONTINUE;
} 

Poprzedni tutorial napisany w tym temacie przez R3X był nieco pomieszany i moim zdaniem skupiał sie na niepotrzebnych elementach.
 
Czym są natywy?
 
Kluczowym elementem programowania jest API, którym w AMXXie są natywy.
Natywy to sposób komunikacji pomiędzy dwoma pluginami.
 
Pluginy podzielę na dwa typy:

• Plugin źródłowy - ten, w którym tworzymy natywy i ich działanie
• Plugin docelowy - ten, w którym będziemy używać natywów

Dzięki natywom możemy z pluginu źródłowego pobrać dane w pluginie docelowym lub je dowolnie modyfikować.
Przykładem idealnie odwzorowującym zasadę działania natywów jest exp mod.
Plugin exp moda może mieć w sobie zmienne pokroju

• userExp - zmienna, która przechowuje ilość expa, którą dany gracz posiada
• userLevel - zmienna, która przechowuje poziom gracza

Co jeśli chcemy wiedzieć jaki poziom ma gracz, lub ile ma expa?
Nie ma najmniejszego problemu w pluginie exp moda - w końcu mamy zmienne userExp i userLevel, ale co jeśli chcemy wiedzieć ile gracz ma expa i potrzebujemy tych danych w innym pluginie?
 
Z odpowiedzią przychodzą natywy - jak już wcześniej wspomniałem, pozwalają nam na komunikacje między pluginami.
 
 
Jak wykorzystać natywy?
 
Mimo, że w tutorialu rexa wydawało się to bardzo skomplikowane, natywy są bardzo proste w swojej zasadzie działania.
 
Prosty przykład, jak pobrać jakąś wartość z innego pluginu:
 
Plugin źródłowy:

#include <amxmodx>

#define AUTHOR "aSior - amxx.pl/user/60210-asiorr/"

new const jakasWartosc = 7;

public plugin_init()
{
	register_plugin("x", "v0.1", AUTHOR);
}

// Natywy musza byc rejestrowane w plugin_natives za pomocą register_native.
public plugin_natives()
{
	// Rejestracja natywu o nazwie "pobierzWartosc", który wykona funkcję "publicNatywuPobierzWartosc".
	register_native("pobierzWartosc", "publicNatywuPobierzWartosc");
}

// Co się wykona, kiedy w innym pluginie zostanie użyte pobierzWartosc.
public publicNatywuPobierzWartosc()
{
	return jakasWartosc;
}

 
Plugin docelowy:

#include <amxmodx>

#define AUTHOR "aSior - amxx.pl/user/60210-asiorr/"

native pobierzWartosc();

public plugin_init()
{
	register_plugin("x", "v0.1", AUTHOR);

	new wartosc = pobierzWartosc();

	log_amx("Wartosc = %i", wartosc); // "Wartosc = 7"
}

 
 
Pliki .inc
 
Pliki z roszerzeniem 'inc' to nic innego jak pliki tekstowe, które zawierają zbiór natywów (i nie tylko).
Jak wygląda plik .inc w pluginie dodającym nową walutę:

#if defined _moneySystem_included
	#endinput
#endif

#define _moneySystem_included

native SetMoney(id, value);
native GetMoney(id);
native AddMoney(id, value);

Po krótce: pliki .inc są używane po to, żeby za każdym razem nie pisać 15 linijek z "native SetMoney(id, value);" i całą resztą natywów.

Na co pozwalają natywy

Dzięki nim wiemy jaki gracz ma poziom, exp, ile ma AP, JBPacków czy jaką ma klasę. To tylko powierzchowne przykłady, a natywy mogą zostać wykorzystane na tak wiele sposobów, że kiedy je lepiej poznamy, otwierają nam się drzwi na tak wiele opcji zaprogramowania pluginu, że ogranicza nas już tylko wyobraźnia.

Co dalej?

Jeśli dopiero nauczyłeś się czym są natywy - idź spróbuj sam. Oczywiście po przeczytaniu tego nie masz prawa zrozumieć wszystkiego ot tak, ale z drugiej strony nikt nie będzie Cię trzymał za rączkę.

Podrzucam parę tematów, z których można wyciągnąć informacje o tym, jak natywy zostały użyte w praktyce:

Cod mod

Jailbreak

Zombie

Po przeczytaniu różnych kodów nadejdzie czas na zapoznanie się z takimi rzeczami jak:

• Style natywów
• Typy danych zwracane przez natywy
• Formatowanie napisów / działanie na tablicach w natywach

Do podglądu zostawiam również stronę dokumentacji: register_native.

W amxxie nie możesz zwrócić więcej niż jednej wartości. Dlatego właśnie tworzymy osobną zmienną i robimy get_user_name(indeks, zmienna, charsmax(zmienna)) - przypisujemy nick gracza do 'zmienna' funkcją get_user_name i w ten sposób nie musimy nic zwracać.

Zamiast

return aData[GangName]

daj

set_string(2, aData[GangName], 32);

A w argumentach natywu zamiast

g_nazwa_gildi(id)

daj

g_nazwa_gildi(id, output[], outputSize);

W 2019 roku pukawka i 1s1k to moim zdaniem jedyne sensowne opcje, jakie pozostały. Które wybierzesz zależy od ciebie.

Moje odczucia co do tych dwóch:

• 1shot1kill - stabilniejsze, przejrzyste, bardzo łatwo się odnaleźć jeśli wiesz czego szukasz, support bez języka oficjalnego, lepsza konsola
• pukawka - beginner-friendly, spory wybór ofert, mysql w cenie, ładnie rozłożone zakładki na stronie

Żadne z tych nie jest idealne, ale mimo wszystko ja bym szedł w 1s1k.

Możesz spróbować jeszcze tego, powinno pomóc na /hud i zmniejszyć częstotliwość zanikania hudu.

U mnie w grze wygląda to tak:

Nawet napisałem w pythonie skrypt, żebym czegoś przypadkiem nie przeoczył:

Jak widzisz, działa dobrze. Pokazuje Ci Silver I i nazwę serwera, bo coś jest powalone z indeksami. Dla indeksu == 0 nick będzie nazwą serwera, a statystyki z get_user_stats będą == 0, więc i funkcja getRankIndex zwróci 0, co za tym idzie wyświetli rangę Silver I.

Co do /hud:

set_hudmessage(0, 85, 255, 0.0, 0.17, 0, 6.0, 12.0, 0.1, 0.1);

12.0 odpowiada za czas, jaki wiadomośc będzie się utrzymywała na hudzie.

W związku z przestarzałym tematem i kodem, który ciągle widzę, że się pojawia:

	new players[32], pnum, tempid;
	new szName[32], szTempid[10];
	get_players(players, pnum);
	
	for(new i; i<pnum; i++)
	{
		tempid = players[i];
		
		if(!is_user_connected(tempid)) 
			continue;
	
		get_user_name(tempid, szName, charsmax(szName));
		num_to_str(tempid, szTempid, charsmax(szTempid));
		menu_additem(menu, szName, szTempid);
	}

Dodaję na szybko wam snippet, który wygląda ładniej i generalnie nie używa starych metod:

Przekazywanie indeksu gracza AMXX 1.8.3 i niżej:

public myMenu(index)
{
	new menuIndex = menu_create("Menu:", "myMenu_handler"),
		userName[33],
		userIndex[3];

	for(new i = 1; i <= 32; i++)
	{
		if(!is_user_connected(i))
		{
			continue;
		}

		get_user_name(i, userName, charsmax(userName));

		formatex(userIndex, charsmax(userIndex), "%i", i);

		menu_additem(menuIndex, userName, userIndex);
	}

	menu_display(index, menuIndex);

	return PLUGIN_HANDLED;
}

public myMenu_handler(index, menuIndex, item)
{
	if(item == MENU_EXIT)
	{
		menu_destroy(menuIndex);
	
		return PLUGIN_HANDLED;
	}

	new menuData[2],
		blank,
		target;

	menu_item_getinfo(menuIndex, item, blank, menuData, charsmax(menuData), _, _, blank);

	menu_destroy(menuIndex);

	target = str_to_num(menuData);

	if(!is_user_connected(target))
	{
		return PLUGIN_HANDLED;
	}

	return PLUGIN_HANDLED;
} 

Przekazywanie indeksu gracza AMXX 1.9 i wyżej:

public myMenu(index)
{
	new menuIndex = menu_create("Menu:", "myMenu_handler");

	for(new i = 1; i <= 32; i++)
	{
		if(!is_user_connected(i))
		{
			continue;
		}

		menu_additem(menuIndex, fmt("%n", i), fmt("%i", i));
	}

	menu_display(index, menuIndex);

	return PLUGIN_HANDLED;
}

public myMenu_handler(index, menuIndex, item)
{
	if(item == MENU_EXIT)
	{
		menu_destroy(menuIndex);
	
		return PLUGIN_HANDLED;
	}

	new menuData[2],
		target;

	menu_item_getinfo(menuIndex, item, _, menuData, charsmax(menuData));

	menu_destroy(menuIndex);

	target = str_to_num(menuData);

	if(!is_user_connected(target))
	{
		return PLUGIN_HANDLED;
	}

	return PLUGIN_HANDLED;
}

Opcja jeszcze krótsza:

public myMenu(index)
{
	new menuIndex = menu_create("Menu:", "myMenu_handler"), i[1];


	for(i[0] = 1; i[0] <= 32; i[0]++)
	{
		if(!is_user_connected(i[0]))
		{
			continue;
		}

		menu_additem(menuIndex, fmt("%n", i), i[0]);
	}

	menu_display(index, menuIndex);

	return PLUGIN_HANDLED;
}

public myMenu_handler(index, menuIndex, item)
{
	if(item == MENU_EXIT)
	{
		menu_destroy(menuIndex);
	
		return PLUGIN_HANDLED;
	}

	new menuData[1],
		target;

	menu_item_getinfo(menuIndex, item, _, menuData, charsmax(menuData));

	menu_destroy(menuIndex);

	target = menuData[0];

	if(!is_user_connected(target))
	{
		return PLUGIN_HANDLED;
	}

	return PLUGIN_HANDLED;
} 

Przekazywanie wielu wartości w menu:

public myMenu(index)
{
	new menuIndex = menu_create("Menu:", "myMenu_handler"),
		menuData[4];

	for(new i = 1; i <= 32; i++)
	{
		if(!is_user_connected(i))
		{
			continue;
		}

		menuData[0] = i;
		menuData[1] = cs_get_user_money(i);
		menuData[2] = index;

		menu_additem(menuIndex, fmt("%n", i), menuData);
	}

	menu_display(index, menuIndex);

	return PLUGIN_HANDLED;
}

public myMenu_handler(index, menuIndex, item)
{
	if(item == MENU_EXIT)
	{
		menu_destroy(menuIndex);
	
		return PLUGIN_HANDLED;
	}

	new menuData[6],
		target;

	menu_item_getinfo(menuIndex, item, _, menuData, charsmax(menuData));

	menu_destroy(menuIndex);

	target = menuData[0];
	value = menuData[1];
	otherIndex = menuData[2];

	if(!is_user_connected(target))
	{
		return PLUGIN_HANDLED;
	}

	return PLUGIN_HANDLED;
} 

Przekazywanie dwóch stringów:

public myMenu(index)
{
	new menuIndex = menu_create("Menu:", "myMenu_handler"),
		menuData[64];

	for(new i = 1; i <= 32; i++)
	{
		if(!is_user_connected(i))
		{
			continue;
		}

		get_user_name(i, userName, charsmax(userName));

		formatex(menuData, charsmax(menuData), "%s#%s", userName,  "abcd");	// "aSior abcd"

		menu_additem(menuIndex, fmt("%n", i), menuData);
	}

	menu_display(index, menuIndex);

	return PLUGIN_HANDLED;
}

public myMenu_handler(index, menuIndex, item)
{
	if(item == MENU_EXIT)
	{
		menu_destroy(menuIndex);
	
		return PLUGIN_HANDLED;
	}

	new menuData[64],
		userName[33],
		additionalText[33];

	menu_item_getinfo(menuIndex, item, _, menuData, charsmax(menuData));

	menu_destroy(menuIndex);

	split(menuData, userName, charsmax(userName), additionalText, charsmax(additionalText), "#");

	ColorChat(index, NORMAL, "Wybrales %s %s", userName, additionalText);	// "Wybrales aSior abcd"

	return PLUGIN_HANDLED;
} 

Jeśli znajdziesz kogoś, kto Ci to napisze, to nie ma przecież żadnej filozofii;

• Deklaracja zmiennej userStatTrak[33][33];
• Rejestracja eventu zabicia;
• Pobieranie id broni zabójcy;
• userStatTrak[killer][weaponIndex]++;
• Zapis do MySQL

To wszystko można zrobić w godzinę albo i pół. Na pewno znajdziesz też gotowce.

W ReadData robisz escape string na pustej zmiennej 'name', a używasz nazwa_gracza, która jeszcze nie jest 'przerobiona', bo ReadData wykonuje się od razu po połączeniu. Jeśli to nie to, będziemy szukać dalej.

Zmienna nazwa_gracza jest tablicą jednowymiarową - przechowa najwyżej jeden napis.

Zmień deklaracje z

new nazwa_gracza[64]

na

new nazwa_gracza[33][64]

a przy pobieraniu nazwy rób

get_user_name(id, nazwa_gracza[id], charsmax(nazwa_gracza[]));

public saveDataHandler(failstate, Handle:query, error[], errnum, data[], size)
{
	if(failstate != TQUERY_SUCCESS)
	{
		return PLUGIN_CONTINUE;
	}
	return PLUGIN_CONTINUE;
}

^ empty statement / bezsensowny statement

105 linijka na

		return i - 1;

#include <amxmodx>

#define AUTHOR "aSior - amxx.pl/user/60210-asiorr/"

new const hudColors[][] =
{
	{ 255, 0, 0 }, // Fully red
	{ 0, 255, 0 }, // Fully green,
	{ 0, 0, 255 }, // Fully blue
	{ 128, 0, 0 }, // Half red
	{ 0, 128, 0 }, // Half green
	{ 0, 0, 128 }  // Half blue
};

new currentColors;

public plugin_init()
{
	register_plugin("x", "v0.1", AUTHOR);

	set_task(1.0, "changeColors", .flags = "b");
}

public changeColors()
{
	currentColors = currentColors + 1 >= sizeof(hudColors) ? 0 : currentColors + 1;
}

public client_putinserver(index)
{
	set_task(1.0, "displayHud", index, .flags = "b");
}

public displayHud(taskIndex)
{
	new index = taskIndex;

	if(!is_user_connected(index))
	{
		return;
	}

	set_hudmessage(hudColors[currentColors][0], hudColors[currentColors][1], hudColors[currentColors][2], 0.0, 0.17, 0, 6.0, 12.0);
	show_hudmessage(index, "Your message");
} 

Like that?

Nie pisz mi wiecej na PW, sam zobaczę temat jak będę miał czas.

Nie sprawdzałem, nie testowałem.

Wenti i granie fair-play, wzajemny szacunek i wysoka jakość - przecież to się nie trzyma kupy

I co się stało z waszym zombie? Kiedyś był fajny.

Jeszcze jedno, jak zapisać stringa w sql i go odczytać?

Przecież już zapisujesz - nick. 

// Tworzenie tabeli:
CREATE TABLE IF NOT EXISTS `tableName` (`userName` VARCHAR(33), `someValue` INT(10), `someOtherValue` VARCHAR(33));

// Insertowanie wartości:
INSERT INTO `tableName` (userName, someValue, someOtherValue) VALUES ('abcd', '0', 'efgh');

// Pobieranie wartości:
SELECT * FROM `tableName` WHERE `userName` = 'abcd';

// Aktualizowanie wartości:
UPDATE `tableName` SET `someValue` = '1' WHERE `userName` = 'abcd'; 

Wszystkie zapytania powinny kończyć się średnikiem. Zapytania w kodzie możesz rozbijać na sekcje wieloliniowe poprzez użycie backslasha (\) na końcu linii (niektóre edytory tekstu mogą wywalać błąd, że string nie został zakończony [bo edytor nie znajdzie " na końcu linijki], ale tym się nie przejmuj). Robisz to tak:

format(mysqlRequest, charsmax(mysqlRequest),
						"INSERT INTO `tableName`\
						(userName, someValue, someOtherValue)\
						VALUES ('abcd', '0', 'efgh'");

Możesz odczytywać konkretne wartości zamiast wszystkich:

Odczyt wszystkich rekordów względem wartości 'someValue' jeśli jest >= 0:

SELECT * FROM `tableName` WHERE `someValue` >= '0';

Odczyt konkretej wartości rekordów, których wartość someOtheValue jest równa "efgh":

SELECT `userName` FROM `tableName` WHERE `someOtherValue` = 'efgh';

Ważna jest ta funkcja i to, żebyś wiedział co tak naprawdę robi:

stock mysql_escape_string(output[], len)
{
	static const szReplaceIn[][] = { "\\", "\0", "\n", "\r", "\x1a", "'", "^"" };
	static const szReplaceOut[][] = { "\\\\", "\\0", "\\n", "\\r", "\Z", "\'", "\^"" };
	for(new i; i < sizeof szReplaceIn; i++)
		replace_all(output, len, szReplaceIn[i], szReplaceOut[i]);
}

Jest to zabezpieczenie przed mysql injection.

Usprawnienie tworzenia tabeli:

CREATE TABLE IF NOT EXISTS `tableNane`
(
    `userName` VARCHAR(33) NOT NULL PRIMARY KEY ,
    `someValue` INT(10) NOT NULL UNSIGNED,
    `someOtherValue` VARCHAR(33) NOT NULL
);

NOT NULL - Each row must contain a value for that column, null values are not allowed
DEFAULT value - Set a default value that is added when no other value is passed
UNSIGNED - Used for number types, limits the stored data to positive numbers and zero
AUTO INCREMENT - MySQL automatically increases the value of the field by 1 each time a new record is added
PRIMARY KEY - Used to uniquely identify the rows in a table. The column with PRIMARY KEY setting is often an ID number, and is often used with AUTO_INCREMENT

Można jakoś zabezpieczyć się przed skopiowaniem modelu/mapy przez graczy?

Wiem, że gracze muszą mieć plik po swojej stronie, ale czy można jakoś uniknąć skopiowania

W grę wchodzi możliwość wykonywania poleceń na graczu (bez ograniczeń, chodzi o komendę np. nadpisującą plik), zależy mi tylko na uniknięciu skopiowania pliku lub zablokowaniu możliwości użycia go na innych serwerach.