Haus System | Mieter auflisten

Ich habe den Code mal etwas aufgeräumt und performanter gemacht.
if(listitem == 5) //Mieter auflisten
{
new id = INVALID_HOUSE_KEY, string[512];
for(new i = 0; i <MAX_HOUSES; i++)
{
if(Spieler[playerid][pHouseKey] == i)
{
if(strcmp(Haus[i][hBesitzer], GetName(playerid), true) == 0)
{
id = i;
}
break;
}
}
if(id != INVALID_HOUSE_KEY)
{
format(string,sizeof(string),"Liste aller Mieter, die derzeit Online sind:\n");
for(new x;x <MAX_PLAYERS; x++)
{
if(IsPlayerConnected(x) && x != playerid)
{
if(Spieler[x][pHouseKey] == id)
{
format(string,sizeof(string),"%s\n%s (ID: %d),",string, GetName(x), x);
}
}
}
ShowPlayerDialog(playerid, DIALOG_MIETER_LISTE ,DIALOG_STYLE_MSGBOX,"Hausbearbeitung",string,"Weiter","");
}
}

Versuche es so.

Ein Spieler: ShowPlayerDialog -> ID eingeben -> bei OnDialogResponse strval(inputtext) auslesen -> ID rauswerfen
Alle Spieler: ShowPlayerDialog -> OnDialogResponse -> Schleife durch alle Spieler -> Prüfen ob Mieter -> ID rauswerfen

Ich würde das ganze über eine Dialog-Liste machen, über eine Befehl wie /haus, und in der Liste dann anzeigen lassen:

• ...
• Einen Mieter rauswerfen
• Alle Mieter rauswerfen
• ...

So kommt man dann über einen Befehl an alle Funktionen des Hauses ran.

Wieso wird überhaupt eine Schleife verwendet, wenn pHouseKey scheinbar den zugehörigen Index für den Hau-Array enthält?

Zitat von Douq

Wieso wird überhaupt eine Schleife verwendet, wenn pHouseKey scheinbar den zugehörigen Index für den Hau-Array enthält?

Stimmt, darauf habe ich gar nicht geachtet, die Beziehung ist hier wirklich 1:1. Danke für den Hinweis.
Dann kann man es auch so schreiben:
if(listitem == 5) //Mieter auflisten
{
new id = INVALID_HOUSE_KEY, hKey = Spieler[playerid][pHouseKey], string[512];
if(hKey >= 0 && hKey < MAX_HOUSES)
{
if(strcmp(Haus[hKey][hBesitzer], GetName(playerid), true) == 0)
{
id = hKey;
}
}
if(id != INVALID_HOUSE_KEY)
{
format(string,sizeof(string),"Liste aller Mieter, die derzeit Online sind:\n");
for(new x;x <MAX_PLAYERS; x++)
{
if(IsPlayerConnected(x) && x != playerid)
{
if(Spieler[x][pHouseKey] == id)
{
format(string,sizeof(string),"%s\n%s (ID: %d),",string, GetName(x), x);
}
}
}
ShowPlayerDialog(playerid, DIALOG_MIETER_LISTE ,DIALOG_STYLE_MSGBOX,"Hausbearbeitung",string,"Weiter","");
}
}

Das geht dann aber wirklich nur, wenn der pHouseKey exakt immer dem Index des Haus-Arrays entspricht (wie es hier aktuell ist).
Sollte sich das ändern, geht das nicht mehr und es muss wieder mit der Schleife gearbeitet werden. Der Fall wäre dann, wenn das Haus-Array eine Eigenschaft "hID" (z.B.) bekommt, welches dann die ID wiedergibt.
Beide Varianten sind möglich und haben ihre Vor- und Nachteile.

Zitat von Dr. Frauenarzt

Ich weiß nicht wie ich das alles setzen soll.
Heißt den Sein Schlüssel auf 0 das er eben das haus nichtmehr betreten kann.

Spieler[pID][pHouseKey] = INVALID_HOUSE_KEY;

pID ist die eingegebene playerid aus dem inputtext mit strval.
Dazu halt noch die standardmäßigen Abfragen, ob der Spieler online ist, etc.

Die if-Abfrage darf nicht mit strval gemacht werden, sondern mit strlen, sonst kannst du die ID 0 nicht eingeben.

pID sollte auch noch den Wert zugewiesen bekommen, sonst steht dort ja nie was drin.

PS:
Du könntest es auch einfach im Spiel ausprobieren.

Zitat von Jeffry

pID ist die eingegebene playerid aus dem inputtext mit strval.

new pID = strval(inputtext);

Du musst wirklich an dir arbeiten, solche Basics (das ist eigentlich weniger als Basics), solltest du selbst hinbekommen, das ist absolutes Grundverständnis.

Das ist vom Prinzip exakt das gleiche, nur dass dort nicht "params" steht, sondern "inputtext". Das zu übertragen ist das angesprochene Grundverständnis.
Du machst das ja nicht erst seit gestern, von daher solltest du dieses Grundverständnis schnellstens aufbauen. Nimm dir das zu Herzen und arbeite an dir.

strmid(Haus[Spieler[playerid][pHouseKey]], inputtext, 0, strlen(inputtext), 32);
zu:
strmid(Haus[Spieler[playerid][pHouseKey]][hName], inputtext, 0, strlen(inputtext), 32);