Namen aller Spieler Problem

.Kevin · 23. Dezember 2013

Guten Morgen liebe Community,

ich möchte gerne überprüfen, ob es den Spielernamen den ein Spieler in einem Dialog eingibt auf dem Server gibt.. aber ich bekomme jedes mal
die Meldung das der Spieler nicht online sei ?..

for(new i; i<MAX_PLAYERS;i++) { if(strcmp(inputtext,SpielerName(i),false))return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

Wo liegt hier bitte der Fehler ?

Slash™ · 23. Dezember 2013

Mach mal n ! bei der abfrage.

.Kevin · 23. Dezember 2013

Zitat von Slash™

Mach mal n ! bei der abfrage.

Ändert nichts an der Sache

SylpheedW · 23. Dezember 2013

for(new i; i<MAX_PLAYERS;i++){
if(!strcmp(inputtext,SpielerName(i),false)){
//Bla bla
return 1;
}
}
return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

.Kevin · 23. Dezember 2013

Zitat von Piet_Meier

for(new i; i<MAX_PLAYERS;i++){
if(!strcmp(inputtext,SpielerName(i),false)){
//Bla bla
return 1;
}
}
return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

Alles anzeigen

So kann ich auf jeden ein Kopfgeld setzten, selbst wenn er nicht Online ist

Slash™ · 23. Dezember 2013

Dann änderst du die sachen halt

.Kevin · 23. Dezember 2013

Zitat von Slash™

Dann änderst du die sachen halt

Dann ist das der gleiche Code wie ich oben hatte und funktioniert auch nicht

FACE · 23. Dezember 2013

for(new i; i<MAX_PLAYERS;i++){
if(i==playerid || !IsPlayerConnected(i)) continue;
if(!strcmp(inputtext,SpielerName(i),false)){
//Bla bla
return 1;
}
}
return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

Schonmal so probiert ?

.Kevin · 23. Dezember 2013

Eigentlich ist der Code nicht anders als meiner, ich frage lediglich ab ob der inputtext ein Name von einem Spieler ist ..

FACE · 23. Dezember 2013

Natürlich!
Ich habe lediglich noch eingefügt, dass die Spieler, die nicht online sind (und der spieler selbst), übersprungen werden.

Anbei habe ich noch eine kleine Änderung vorgenommen:

for(new i=0; i<GetMaxPlayres();i++){
if(i==playerid || !IsPlayerConnected(i)) continue;
if(strcmp(inputtext,SpielerName(i),true) == 0){
//Dein Code..
break;
return 1;
}
}
return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

Auf die Gefahr hin, dass dein String (SpielerName) falsch abgefragt wird, hier nochmal direkt deklariert:

new namestr[25];
for(new i=0; i<GetMaxPlayres();i++){
GetPlayerName(i, namestr, sizeof(namestr));
if(i==playerid || !IsPlayerConnected(i)) continue;
if(strcmp(inputtext,namestr,true) == 0){
//Dein Code..
break;
return 1;
}
}
return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

[DT]midomen · 23. Dezember 2013

Schreib die strcmp abfrage vor dem return mal so.
if(strcmp(inputtext,SpielerName(i),false)!=0)

.Kevin · 23. Dezember 2013

Zitat von FACE

Natürlich!
Ich habe lediglich noch eingefügt, dass die Spieler, die nicht online sind (und der spieler selbst), übersprungen werden.

Anbei habe ich noch eine kleine Änderung vorgenommen:

for(new i=0; i<GetMaxPlayres();i++){
if(i==playerid || !IsPlayerConnected(i)) continue;
if(strcmp(inputtext,SpielerName(i),true) == 0){
//Dein Code..
break;
return 1;
}
}
return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

Auf die Gefahr hin, dass dein String (SpielerName) falsch abgefragt wird, hier nochmal direkt deklariert:

new namestr[25];
for(new i=0; i<GetMaxPlayres();i++){
GetPlayerName(i, namestr, sizeof(namestr));
if(i==playerid || !IsPlayerConnected(i)) continue;
if(strcmp(inputtext,namestr,true) == 0){
//Dein Code..
break;
return 1;
}
}
return ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }

Alles anzeigen

Zitat von [DT]midomen

Schreib die strcmp abfrage vor dem return mal so.
if(strcmp(inputtext,SpielerName(i),false)!=0)

Funktioniert leider beides nicht

[DT]midomen · 23. Dezember 2013

Als du den Namen eingegeben hast. Hast du auch die Groß&Kleinschreibung beachtet?

Versuch es mal so:

for(new i;i<MAX_PLAYERS;i++)
{
if(strcmp(inputtext,SpielerName(i),false)==0)
{
//Code
break;
}
else if(i==MAX_PLAYERS-1)
{
ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen");
}
}

.Kevin · 23. Dezember 2013

Zitat von [DT]midomen

Schreib die strcmp abfrage vor dem return mal so.
if(strcmp(inputtext,SpielerName(i),false)!=0)

Zitat von [DT]midomen

Als du den Namen eingegeben hast. Hast du auch die Groß&Kleinschreibung beachtet?

Ja Klar ..

FACE · 23. Dezember 2013

Groß- und Kleinschreibung habe ich mit dem Ignore-Case hinfällig gemacht. Genau um diese Fehlerquelle auszumerzen.

Eine Idee hätte ich noch.
So banal es klingen mag: Wo hast du den Code eingefügt.
Und nicht zu vergessen: Hast du deine DialogIDs richtig definiert (bzw. abgefragt) ? Ist eine häufige Fehlerursache.

.Kevin · 23. Dezember 2013

Zitat von FACE

Groß- und Kleinschreibung habe ich mit dem Ignore-Case hinfällig gemacht. Genau um diese Fehlerquelle auszumerzen.

Eine Idee hätte ich noch.
So banal es klingen mag: Wo hast du den Code eingefügt.
Und nicht zu vergessen: Hast du deine DialogIDs richtig definiert (bzw. abgefragt) ? Ist eine häufige Fehlerursache.

Hier jetzt mal mein code:

if(dialogid==DIALOG_CONTRACT)
{
if(response)
{
new namestr[24];
for(new i=0; i<GetMaxPlayers();i++)
{
GetPlayerName(i, namestr, sizeof(namestr));
if(i==playerid || !IsPlayerConnected(i)) continue;
if(strcmp(inputtext,namestr,true) == 0)
{
format(string,sizeof(string),"Du möchtest auf {FFAA00}%s{FEFEFE} ein Kopfgeld aussetzen.\nBitte gib das Kopfgeld für den Spieler an!",inputtext);
ShowPlayerDialog(playerid,DIALOG_CONTRACT1,DIALOG_STYLE_INPUT,"Contract",string,"absenden","abbrechen");
format(Spieler[playerid][ContractSpieler],24,inputtext);
break;
}
}
return ShowPlayerDialog(playerid,DIALOG_CONTRACT,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen");
//for(new i; i<MAX_PLAYERS; i++) { if(strcmp(inputtext,SpielerName(i),false))return ShowPlayerDialog(playerid,DIALOG_CONTRACT,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Zielperson ein","absenden","abbrechen"); }
}
}
if(dialogid==DIALOG_CONTRACT1)
{
if(response)
{
new a = playerid;
format(string,sizeof(string),"Das Kopfgeld muss höher als 50$ und niedriger als 1000$ sein!\nDu möchtest auf {FFAA00}%s{FEFEFE} ein Kopfgeld aussetzen.\nBitte gib das Kopfgeld für den Spieler an!",inputtext);
if(strval(inputtext) < 50 || strval(inputtext) > 1000)return ShowPlayerDialog(playerid,DIALOG_CONTRACT1,DIALOG_STYLE_INPUT,"Contract",string,"absenden","abbrechen");
format(string,sizeof(string),"INSERT INTO `contracts` (`auftraggeber`,`zielperson`,`kopfgeld`) VALUES ('%s','%s','%d')",SpielerName(playerid),Spieler[playerid][ContractSpieler],strval(inputtext));
mysql_query(string);
format(Auftrag[a][geber],24,SpielerName(playerid));
format(Auftrag[a][ziel],24,Spieler[playerid][ContractSpieler]);
Auftrag[a][kopfgeld] = strval(inputtext);
for(new i; i<MAX_PLAYERS; i++)
{
if(SpielerInfo[i][Fraktion] == 12)
{
SendClientMessage(playerid,Gelb,"Es ist ein neuer Auftrag eingegangen!");
return 1;
}
}
}
}

Kevin1 · 23. Dezember 2013

Zitat von [IRP]K3VIN

for(new i=0; i<GetMaxPlayers();i++)
{
GetPlayerName(i, namestr, sizeof(namestr));
if(i==playerid || !IsPlayerConnected(i)) continue;
if(strcmp(inputtext,namestr,true) == 0)
{
format(string,sizeof(string),"Du möchtest auf {FFAA00}%s{FEFEFE} ein Kopfgeld aussetzen.\nBitte gib das Kopfgeld für den Spieler an!",inputtext);
ShowPlayerDialog(playerid,DIALOG_CONTRACT1,DIALOG_STYLE_INPUT,"Contract",string,"absenden","abbrechen");
format(Spieler[playerid][ContractSpieler],24,inputtext);
break;
}
}
return ShowPlayerDialog(playerid,DIALOG_CONTRACT,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen");

Alles anzeigen

Break beendet nur die Schleife, danach macht er weiter.
Also er zeigt dir wahrscheinlich KURZ den Dialog "Du möchtest auf ..." an, aber DIREKT danach zeigt er dir an, dass der Spieler nicht online ist.
Das "return" da unten wird IMMER nach der Schleife ausgeführt.

.Kevin · 23. Dezember 2013

Zitat von Kevin1

Break beendet nur die Schleife, danach macht er weiter.
Also er zeigt dir wahrscheinlich KURZ den Dialog "Du möchtest auf ..." an, aber DIREKT danach zeigt er dir an, dass der Spieler nicht online ist.
Das "return" da unten wird IMMER nach der Schleife ausgeführt.

Nein es kommt direkt die Meldung das der Spieler nicht online ist.

Kevin1 · 23. Dezember 2013

Zitat von [IRP]K3VIN

Nein es kommt direkt die Meldung das der Spieler nicht online ist.

Ja, weil dein Server schneller ist als dein Auge
Oder weil der Client das 1. noch garnicht richtig verarbeitet hat und sofort zum 2. springt.
Kommentiere einfach mal eben die return-Zeile aus (//)

FACE · 23. Dezember 2013

Bevor das mit dem break noch zu viele Probleme aufwirft, schreibe ich das Problem eben um:

new namestr[25], merk;
merk=-1;
for(new i=0; i<GetMaxPlayres();i++){
GetPlayerName(i, namestr, sizeof(namestr));
if(i==playerid || !IsPlayerConnected(i)) continue;
if(strcmp(inputtext,namestr,true) == 0) { merk=i; break; }
iff(i==GetMaxPlayers() && !IsPlayerConnected(i)) { ShowPlayerDialog(playerid,DIALOG_PERSON,DIALOG_STYLE_INPUT,"Contract","Der Spieler ist nicht Online!\nGib den Namen der Person ein","absenden","abbrechen"); }
}

..
//ShowPlayerDialog..
//if(merk!=-1) //Show (alt.)

das "merk=i;" kann man natürlich direkt mit ShowPlayerDialog ersetzen

Der Rest sollte wohl kein Problem sein

Code

break


break breaks out of a loop, ending it prematurely:
for (new a = 0; a < 10; a++)
{
	if (a == 5) break;
}
This loop will go 6 times but code after the break will only be executed 5 times.

Wir müssen den Dialog aus der Schleife rausziehen, sonst bricht er dort ab.