Problem mit Dialog (andere Dialog wird ungewollt mitbenutzt)

Vielleicht irgendwo eine gleiche ID verwendet ?

mfg

Du hast es unter die falsche Klammer geschrieben.
Achte darauf, dass du deinen Code richtig einrückst, dann passiert dir sowas nicht.

if(dialogid == DIALOG_HILFE)
{

if(listitem == 0)
{
ShowPlayerDialog(playerid,DIALOG_HILFE1,DIALOG_STYLE_MSGBOX,"Allgemeine Hilfe","/stats /lizenzen (/liz) /kaufen (24/7) /kopfhörer /navi /report","OK","");
}
if(listitem == 1)
{
//ShowPlayerDialog(playerid,DIALOG_HH,DIALOG_STYLE_MSGBOX,"Haushilfe"," -> /kaufehaus - Damit kannst du dir ein Haus kaufen.\n -> /verkaufehaus - Damit kannst du dein Haus verkaufen.\n -> /haustuer - damit kannst du dein Haus auf-/zuschließen.\n -> /hausbetreten - Damtit kannst du dein Haus betreten. -> /hausverlassen - Damit kannst du dein Haus verlassen,\n","Weiter","Schließen");
}
if(listitem == 2)
{
ShowPlayerDialog(playerid,DIALOG_HILFE2,DIALOG_STYLE_MSGBOX,"Handy Hilfe","...","OK","");
}
if(listitem == 3)
{
ShowPlayerDialog(playerid,DIALOG_HILFE3,DIALOG_STYLE_LIST,"Job Hilfe","Drogendealer\nTaxifahrer\nPizzaliferant\nStuntman","OK","");
}

}

if(dialogid == DIALOG_HILFE1)
{
if(response)
{
ShowPlayerDialog(playerid,DIALOG_HILFE,DIALOG_STYLE_LIST,"Hilfe System 2.1","Allgemeine Hilfe\nHaus Hilfe\nHandy Hilfe\nJob Hilfe","OK","");
}
}

if(dialogid == DIALOG_HILFE2)
{
if(response)
{
ShowPlayerDialog(playerid,DIALOG_HILFE,DIALOG_STYLE_LIST,"Hilfe System 2.1","Allgemeine Hilfe\nHaus Hilfe\nHandy Hilfe\nJob Hilfe","OK","");
}
}

// Dialog Jobs
if(dialogid == DIALOG_JOBS)
{
if(response == 0)
{
SendClientMessage(playerid,COLOR_ORANGE,"Jobauswahl abgebrochen!");
return 1;
}
if(listitem == 0)
{
if(Spieler[playerid][Beruf] == 0)
{
Spieler[playerid][Beruf] = 1;
SendClientMessage(playerid,COLOR_YELLOW,"Du hast erfolgreich dein Arbeitslosengeld I beantragt!");
SendClientMessage(playerid,COLOR_YELLOW,"Du erhälst es ab sofort jeden Zahltag!");
return 1;
}
else
{
SendClientMessage(playerid,COLOR_RED,"Du musst arbeitslos sein um Arbeitslosengeld I zu beantragen!");
return 1;
}
}
if(listitem == 1)
{
if(Spieler[playerid][Beruf] > 0)
{
Spieler[playerid][Beruf] = 0;
SendClientMessage(playerid,COLOR_YELLOW,"Du hast deinen Job gekündigt und bist nun Arbeitslos!");
SendClientMessage(playerid,COLOR_YELLOW,"Du kannst Arbeitslosengeld I beantragen um ohne Arbeit Geld zu bekommen!");
EnableStuntBonusForPlayer(playerid,0);
return 1;
}
else
{
SendClientMessage(playerid,COLOR_RED,"Du musst einen Job haben um zu kündigen!");
return 1;
}
}
if(listitem == 2)
{
if(Spieler[playerid][Beruf] == 0)
{
Spieler[playerid][Beruf] = 2;
SendClientMessage(playerid,COLOR_YELLOW,"Du bist nun Drogendealer!");
SendClientMessage(playerid,COLOR_YELLOW,"Unter dem Menü Jobhilfe (/hilfe) findest du alle Befehle!");
return 1;

}
else
{
SendClientMessage(playerid,COLOR_RED,"Du musst arbeitlos sein um einen neuen Job anzunehmen!");
return 1;
}
}
if(listitem == 3)
{
if(Spieler[playerid][Beruf] == 0)
{
Spieler[playerid][Beruf] = 3;
SendClientMessage(playerid,COLOR_YELLOW,"Du bist nun Taxifahrer!");
SendClientMessage(playerid,COLOR_YELLOW,"Unter dem Menü Jobhilfe (/hilfe) findest du alle Befehle!");
return 1;

}
else
{
SendClientMessage(playerid,COLOR_RED,"Du musst arbeitlos sein um einen neuen Job anzunehmen!");
return 1;
}
}
if(listitem == 4)
{
if(Spieler[playerid][Beruf] == 0)
{
Spieler[playerid][Beruf] = 4;
SendClientMessage(playerid,COLOR_YELLOW,"Du bist nun Pizzalieferant!");
SendClientMessage(playerid,COLOR_YELLOW,"Unter dem Menü Jobhilfe (/hilfe) findest du alle Befehle!");
return 1;

}
else
{
SendClientMessage(playerid,COLOR_RED,"Du musst arbeitlos sein um einen neuen Job anzunehmen!");
return 1;
}
}
if(listitem == 5)
{
if(Spieler[playerid][Beruf] == 0)
{
Spieler[playerid][Beruf] = 3;
EnableStuntBonusForPlayer(playerid, 1);
SendClientMessage(playerid,COLOR_YELLOW,"Du bist nun Stuntman!");
SendClientMessage(playerid,COLOR_YELLOW,"Unter dem Menü Jobhilfe (/hilfe) findest du alle Befehle!");
return 1;

}
else
{
SendClientMessage(playerid,COLOR_RED,"Du musst arbeitlos sein um einen neuen Job anzunehmen!");
return 1;
}
}
}

Das return 1; ist an manchen stellen ziemlich überflüssig

mfg

Zitat von CIBERKILLER

Das return 1; ist an manchen stellen ziemlich überflüssig

Im Gegenteil!
Es ist sogar gut, dass er innerhalb der if-Abfragen die returns hat, denn der Ablauf ist "durchfallend", das heißt wenn kein return da ist, dann werden die folgenden if-Abfragen auch gemacht. Ist das return da, nicht. Das schont die CPU.

=> Daher: Wenn von der Skriptlogik her möglich: return verwenden, wenn nichts mehr folgt.

Ja klar, aber return ist für 2 Sachen gedacht!

Zum ersten zum wiedergeben eines Wertes/strings ö.ä!
Zum zweiten für das abbrechen mancher bereiche, wenn die If Be­din­gung nicht stimmt, passiert sowie so nichts.
Klar, aber wenn man weis unten drunter wird sowie nichts mehr aufgerufen, könnte man es schon verwenden.
Dennoch bleibt meine Meinung, in dem Bereich sind manche returns ziemlich überflüssig!

mfg

Zitat von CIBERKILLER

Dennoch bleibt meine Meinung, in dem Bereich sind manche returns ziemlich überflüssig!

Hier zwei Beispiele:
public OnPlayerConnect(playerid)
{
//Nehmen wir an, playerid ist 1:
if(playerid == 0) //nein, gehe zu Zeile 9
{
//...
return 1;
}
if(playerid == 1) //ja
{
//...führe Code aus.
return 1; //Springe aus OnPlayerConnect heraus. Fertig.
}
//Wird nicht aufgerufen.
if(playerid == 2)
{
//...
return 1;
}
if(playerid == 3)
{
//...
return 1;
}
return 1;
}

public OnPlayerConnect(playerid)
{
//Nehmen wir an, playerid ist 1:
if(playerid == 0) //nein, gehe zu Zeile 8
{
//...
}
if(playerid == 1) //ja
{
//...führe Code aus.
}
//hier geht es weiter
if(playerid == 2) //nein
{
//...
}
if(playerid == 3) //nein
{
//...
}
return 1; //Springe aus OnPlayerConnect heraus. Fertig.
}

=> Wir haben im 2. Beispiel 2 if-Abfragen mehr gemacht, das entspricht der doppelten CPU Last. Wenn du jetzt die Zeit misst, für x Durchläufe, dann wird Beispiel 1 doppelt so schnell fertig sein, als Beispiel 2. Jetzt stell dir vor wir machen das mit 1000 Dialogen. Wenn da bereits der zweite aufgerufen wird, dann hast du 998 unnötige if-Abfragen, das entspricht einer fast 500-fachen CPU Last.

Die returns sind also ganz und gar nicht unnötig.

Regelt das Switch denn nicht ?

mfg

Zitat von CIBERKILLER

Regelt das Switch denn nicht ?

Ein switch wäre natürlich noch besser, klar.
Der geht aber nicht immer, sondern nur wenn man konstante Werte abfragt. Bei Dialogen und in meinem Beispiel würde es gehen, aber wenn man zwei Variablen vergleicht, geht es nicht.

Dir wurde doch bereits erklärt dass du es falsch eingegliedert hattest, im Sinne von in den Falschen Klammern, soweit ich das mitgelesen habe.