erledigt

malE · 5. Oktober 2015

@Jeffry das sollte alles sein was man brauch.

OnPlayerConnect - sind auch alle Variablen.

C

PlayerInfo[playerid][p_Kopfgeld] = 0;
PlayerInfo[playerid][p_AuftragID] = -1;
PlayerInfo[playerid][p_KopfgeldID] = -1;
PlayerInfo[playerid][p_AuftraggeberID] = -1;

OnDialogResponse

C

if(dialogid == DIALOG_KOPFGELDER)
	{
		new counter = -1;
		if(!response) return 1;
		for(new i=0; i < MAX_PLAYERS; i++)
		{
			if(PlayerInfo[i][p_Kopfgeld] > 0 && PlayerInfo[i][p_KopfgeldID] == -1)
			{
				counter++;
				if(counter == listitem)
				{
				    if(PlayerInfo[playerid][p_Admin] > 0 && PlayerInfo[playerid][p_Fraktion] != 5)return SendClientMessage(playerid, COLOR_GREY, "Du kannst keine Aufträge annehmen.");
					new str[145];
					format(str, sizeof(str), "** ( Hitmen %s hat %s für $%s angenommen ) **", GetName(playerid), GetName(i), ConvertMoney(PlayerInfo[i][p_Kopfgeld]));
					SendFraktionsMessage(5, 0x8D2705FF, str);
					PlayerInfo[i][p_KopfgeldID] = playerid;
					PlayerInfo[playerid][p_AuftragID] = i;
					PlayerInfo[playerid][p_AuftragTime] = gettime() +60000;
					break;
				}
			}
		}
		return 1;
	}

Alles anzeigen

Die Befehle
Habe bei dem Befehl /kopfgelder die Auftraggeber Variablen etc raus genommen.

C

ocmd:kopfgeld(playerid, params[])
{
	new kopfgeld, pID;
	if(PlayerInfo[playerid][p_Fraktion] == 5)return SendClientMessage(playerid, COLOR_GREY, "  Als Mitglied dieser Fraktion hast du keine Berechtigung für diesen Befehl."); // Zum testen ausklammern
	if(sscanf(params, "ud", pID, kopfgeld))return SendClientMessage(playerid, COLOR_GREY, "Benutze: /kopfgeld [Spieler-ID] [Kopfgeld]");
	if(!IsPlayerConnected(pID) || PlayerInfo[pID][p_LoggedIn] == false)return SendClientMessage(playerid, COLOR_GREY, "Dieser Spieler ist Offline.");
	if(pID == playerid)return SendClientMessage(playerid, COLOR_GREY, "Du kannst dir selbst kein Kopfgeld geben."); // Zum testen ausklammern
	if(kopfgeld < 3500)return SendClientMessage(playerid, COLOR_GREY, "Du kannst dem Spieler kein Kopfgeld unter $3.500 geben.");
	if(kopfgeld > 100000)return SendClientMessage(playerid, COLOR_GREY, "Du kannst dem Spieler kein Kopfgeld über $100.000 geben.");
	if(PlayerInfo[pID][p_Kopfgeld] + kopfgeld > 100000)return SendClientMessage(playerid, COLOR_GREY, "Das Kopfgeld vom Spieler darf nicht über $100.000 liegen.");
	if(PlayerInfo[pID][p_KopfgeldID] != -1) return SendClientMessage(playerid, COLOR_GREY, "Auf diesen Spieler hat bereits jemand einen Auftrag angenommen.");
	PlayerInfo[pID][p_Kopfgeld] = PlayerInfo[pID][p_Kopfgeld] +kopfgeld;
	PlayerInfo[pID][p_AuftraggeberID] = playerid;
	new str[145];
	format(str, sizeof(str), "%s hat ein neuen Auftrag abgegeben.", GetName(playerid));
	SendFraktionsMessage(5, COLOR_BROWN, str);
	SendClientMessage(playerid, COLOR_LIGHTBLUE, "[SERVICE] Dein Auftrag wurde an die Hitmen Agency weitergeleitet.");
	return 1;
}


ocmd:kopfgelder(playerid, params[])
{
	if(PlayerInfo[playerid][p_Fraktion] != 5 || PlayerInfo[playerid][p_Admin] < 5)return 1;
	if(PlayerInfo[playerid][p_AuftragID] != -1) return SendClientMessage(playerid, COLOR_GREY, "Du hast bereits einen Auftrag angenommen.");
	new str[1024];
	for(new i=0; i < MAX_PLAYERS; i++)
	{
		if(!IsPlayerConnected(i) || PlayerInfo[i][p_LoggedIn] != true)continue;
		if(PlayerInfo[i][p_Kopfgeld] > 0 && PlayerInfo[i][p_KopfgeldID] == -1)
		{
			format(str, sizeof(str), "%s\t%s\t%i\t$%s\n", str, GetName(i), i, ConvertMoney(PlayerInfo[i][p_Kopfgeld]));
		}
	}
	format(str, sizeof(str), "Zielperson\tSpieler-ID\tKopfgeld\n%s", str);
	ShowPlayerDialog(playerid, DIALOG_KOPFGELDER, DIALOG_STYLE_TABLIST_HEADERS, "Hitmen Agency: Auftragsliste", str, "Annehmen", "Schließen");
	return 1;
}

Alles anzeigen

Exod_Martinez · 5. Oktober 2015

Und was funktioniert nicht?

malE · 5. Oktober 2015

Zitat von malE

@Jeffry
Taucht unknown command auf :|.
Bei den formats muss das Problem liegen.

C

ocmd:kopfgelder(playerid, params[])
{
if(PlayerInfo[playerid][p_Fraktion] != 5 || PlayerInfo[playerid][p_Admin] < 5)return 1;
if(PlayerInfo[playerid][p_AuftragID] != -1) return SendClientMessage(playerid, COLOR_GREY, "Du hast bereits einen Auftrag angenommen.");
new str[245], str1[245];
for(new i=0; i < MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i) || PlayerInfo[i][p_LoggedIn] != true)continue;
if(PlayerInfo[i][p_Kopfgeld] > 0 && PlayerInfo[i][p_KopfgeldID] == -1)
{
format(str, sizeof(str), "%s%s\t%s\t%s\n", str, GetName(PlayerInfo[i][p_AuftraggeberID]), GetName(i), ConvertMoney(PlayerInfo[i][p_Kopfgeld]));
format(str1, sizeof(str1), "Auftraggeber\tZielperson\tKopfgeld\n%s", str);
ShowPlayerDialog(playerid, DIALOG_KOPFGELDER, DIALOG_STYLE_LIST, "Kopfgelder", str1, "Annehmen", "Schließen");
}
}
return 1;
}

Alles anzeigen

Zitat von Jeffry

Versuche es so:

C

ocmd:kopfgelder(playerid, params[])
{
if(PlayerInfo[playerid][p_Fraktion] != 5 || PlayerInfo[playerid][p_Admin] < 5)return 1;
if(PlayerInfo[playerid][p_AuftragID] != -1) return SendClientMessage(playerid, COLOR_GREY, "Du hast bereits einen Auftrag angenommen.");
new str[1024];
for(new i=0; i < MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i) || PlayerInfo[i][p_LoggedIn] != true)continue;
if(PlayerInfo[i][p_Kopfgeld] > 0 && PlayerInfo[i][p_KopfgeldID] == -1)
{
format(str, sizeof(str), "%s%s\t%s\t%s\n", str, GetName(PlayerInfo[i][p_AuftraggeberID]), GetName(i), ConvertMoney(PlayerInfo[i][p_Kopfgeld]));
}
}
format(str, sizeof(str), "Auftraggeber\tZielperson\tKopfgeld\n%s", str);
ShowPlayerDialog(playerid, DIALOG_KOPFGELDER, DIALOG_STYLE_LIST, "Kopfgelder", str, "Annehmen", "Schließen");


return 1;
}

Alles anzeigen

Zitat von malE
@Jeffry

Abgesehen davon, dass du DIALO_STYLE_LIST statt DIALOG_STYLE_TABLIST_HEADERS nicht ersetzt hast.
Der Code funktioniert einwandfrei, sobald ich GetName(PlayerInfo[i][p_AuftraggeberID]) aus /kopfgelder entferne.

Die Variable hab ich so gesetzt.
OnPlayerConnect
C
PlayerInfo[playerid][p_AuftraggeberID] = -1;
bei meinem /kopfgeld Befehl
C
PlayerInfo[pID][p_AuftraggeberID] = playerid;
Dann wird die Variable über GetName() beim Befehl /kopfgelder ausgelesen.

---
Aber ich denke das wird so sowieso nicht funktionieren.
Weil sich die Variable PlayerInfo[pID][p_AuftraggeberID] spätestens nach dem Reconnect auf -1 setzt. Und dann ein Fehler auftritt, da die Spieler ID nicht gleich die von dem richtigen Auftraggeber ist.
Speichern kann ich die Variable nicht, weil es ja über die playerid läuft. Hast du vielleicht eine nicht all zu komplexe Variante?
Ansonsten muss ich darauf verzichten.
Alles anzeigen

Zitat von Jeffry

Erstelle bitte einen neuen Thread und poste dort den Befehl und die anderen Codes die noch zu dem System gehören.
Wenn ich mich richtig erinnere, dann hatte ich daran schon mal was gemacht, das müsste man dann wahrscheinlich auch etwas ändern.

Jeffry · 5. Oktober 2015

Die Befehle so:

C

ocmd:kopfgeld(playerid, params[])
{
	new kopfgeld, pID;
	if(PlayerInfo[playerid][p_Fraktion] == 5)return SendClientMessage(playerid, COLOR_GREY, "  Als Mitglied dieser Fraktion hast du keine Berechtigung für diesen Befehl."); // Zum testen ausklammern
	if(sscanf(params, "ud", pID, kopfgeld))return SendClientMessage(playerid, COLOR_GREY, "Benutze: /kopfgeld [Spieler-ID] [Kopfgeld]");
	if(!IsPlayerConnected(pID) || PlayerInfo[pID][p_LoggedIn] == false)return SendClientMessage(playerid, COLOR_GREY, "Dieser Spieler ist Offline.");
	if(pID == playerid)return SendClientMessage(playerid, COLOR_GREY, "Du kannst dir selbst kein Kopfgeld geben."); // Zum testen ausklammern
	if(kopfgeld < 3500)return SendClientMessage(playerid, COLOR_GREY, "Du kannst dem Spieler kein Kopfgeld unter $3.500 geben.");
	if(kopfgeld > 100000)return SendClientMessage(playerid, COLOR_GREY, "Du kannst dem Spieler kein Kopfgeld über $100.000 geben.");
	if(PlayerInfo[pID][p_Kopfgeld] + kopfgeld > 100000)return SendClientMessage(playerid, COLOR_GREY, "Das Kopfgeld vom Spieler darf nicht über $100.000 liegen.");
	if(PlayerInfo[pID][p_KopfgeldID] != -1) return SendClientMessage(playerid, COLOR_GREY, "Auf diesen Spieler hat bereits jemand einen Auftrag angenommen.");
	if(strlen(PlayerInfo[pID][p_Auftraggeber])) return SendClientMessage(playerid, COLOR_GREY, "Auf diesen Spieler hat bereits jemand einen Auftrag gegeben.");
	PlayerInfo[pID][p_Kopfgeld] = PlayerInfo[pID][p_Kopfgeld] +kopfgeld;
	format(PlayerInfo[pID][p_Auftraggeber], MAX_PLAYER_NAME, GetName(playerid));
	new str[145];
	format(str, sizeof(str), "%s hat ein neuen Auftrag abgegeben.", GetName(playerid));
	SendFraktionsMessage(5, COLOR_BROWN, str);
	SendClientMessage(playerid, COLOR_LIGHTBLUE, "[SERVICE] Dein Auftrag wurde an die Hitmen Agency weitergeleitet.");
	return 1;
}
ocmd:kopfgelder(playerid, params[])
{
	if(PlayerInfo[playerid][p_Fraktion] != 5 || PlayerInfo[playerid][p_Admin] < 5)return 1;
	if(PlayerInfo[playerid][p_AuftragID] != -1) return SendClientMessage(playerid, COLOR_GREY, "Du hast bereits einen Auftrag angenommen.");
	new str[1024];
	for(new i=0; i < MAX_PLAYERS; i++)
	{
		if(!IsPlayerConnected(i) || PlayerInfo[i][p_LoggedIn] != true)continue;
		if(PlayerInfo[i][p_Kopfgeld] > 0 && PlayerInfo[i][p_KopfgeldID] == -1)
		{
			format(str, sizeof(str), "%s\t%s\t%i\t$%s\n", str, PlayerInfo[i][p_Auftraggeber], i, ConvertMoney(PlayerInfo[i][p_Kopfgeld]));
		}
	}
	format(str, sizeof(str), "Zielperson\tSpieler-ID\tKopfgeld\n%s", str);
	ShowPlayerDialog(playerid, DIALOG_KOPFGELDER, DIALOG_STYLE_TABLIST_HEADERS, "Hitmen Agency: Auftragsliste", str, "Annehmen", "Schließen");
	return 1;
}

Alles anzeigen

Bei OnPlayerConnect und dort wo der Auftrag beendet wird:

C

format(PlayerInfo[playerid][p_Auftraggeber], MAX_PLAYER_NAME, "");

Im Enum dann das p_AuftraggeberID ändern zu:

C

p_Auftraggeber[MAX_PLAYER_NAME],

Dann speichert es den Name unabhängig davon, ob der Auftraggeber online ist oder nicht.

malE · 5. Oktober 2015

Danke dir!
Die Variable dann auch natürlich mit MYSQL abspeichern?

Jeffry · 5. Oktober 2015

Ja, aber eben als String (%s) und VARCHAR in der Datenbank.

malE · 5. Oktober 2015

Das war mir klar.
So lese ich den String korrekt aus, oder?

C

cache_get_field_content(0, "auftraggeber", result);
PlayerInfo[playerid][p_Auftraggeber] = strval(result);

Jeffry · 5. Oktober 2015

Nicht ganz, strval macht aus einem String einen Integer.
Mache die zweite Zeile so:
format(PlayerInfo[playerid][p_Auftraggeber], MAX_PLAYER_NAME, result);

malE · 5. Oktober 2015

Upps .
Danke

breadfish.de 12. März 2022