Waffenshop mit mehreren Dialogen aber wie? :o

if(dialogid == DIALOG_WAFFENSHOPLS)
{
if(response == 1)
{
if(listitem == 0)
{
ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLS,DIALOG_STYLE_LIST,"Nahkampfwaffen","Schlagring: PREIS\nMesser: PREIS\nKatana: PREIS\n","Kaufen","Abbruch");
return 1;
}

}
else
{

SendClientMessage(playerid,ROT,"Vielleicht überlegst du es dir.");
return 1;

}
}

Zitat von Showtek

if(dialogid == DIALOG_WAFFENSHOPLS)
{
if(response == 1)
{
if(listitem == 0)
{
ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLS,DIALOG_STYLE_LIST,"Nahkampfwaffen","Schlagring: PREIS\nMesser: PREIS\nKatana: PREIS\n","Kaufen","Abbruch");
return 1;
}

}
else
{

SendClientMessage(playerid,ROT,"Vielleicht überlegst du es dir.");
return 1;

}
}

Alles anzeigen

Warum diese returns?

Towelie123 - Ziehe das Geld ab und gib ihm die Waffe, die entsprechenden Funktionen findest du hier: http://wiki.sa-mp.com/wiki/Function:GivePlayerWeapon, http://wiki.sa-mp.com/wiki/Function:GivePlayerMoney

Du fragst einfach mit listitem ab, welche Waffe er angeklickt hat. Dann gibst du ihm in der Abfrage die Waffe + Munition und ziehst ihm Geld ab

Nein, du kannst ja nicht beide Dialog exakt gleich überprüfen. Du musst sie schon voneinander trennen.

Zitat von Towelie123

und wie mache ich das :D?

Indem man die beiden Dialoge voneinander trennt. Warum gibt es denn eine dialogid, wieso überprüft man in OnDialogResponse() denn überhaupt dialogid, was tue ich eigentlich generell, wenn ich Dialoge erstelle? Mache dir das klar, schaue dir nochmals ShowPlayerDialog() an und trenne dann einen Dialog vom anderen.

Vielleicht hilft dir das.
ZITAT AUS SAMP WIKI


#define DIALOG_WEAPONS 3
// In some command
ShowPlayerDialog(playerid, DIALOG_WEAPONS, DIALOG_STYLE_LIST, "Weapons", "Desert Eagle\nAK-47\nCombat Shotgun", "Select", "Close");

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
if(dialogid == DIALOG_WEAPONS)
{
if(response) // If they clicked 'Select' or double-clicked a weapon
{
// Give them the weapon
if(listitem == 0) // They selected the first item - Desert Eagle
{
GivePlayerWeapon(playerid, WEAPON_DEAGLE, 14); // Give them a desert eagle
}
if(listitem == 1) // They selected the second item - AK-47
{
GivePlayerWeapon(playerid, WEAPON_AK47, 120); // Give them an AK-47
}
if(listitem == 2) // They selected the third item - Desert Eagle
{
GivePlayerWeapon(playerid, WEAPON_SHOTGSPA, 28); // Give them a Combat Shotgun
}
}
return 1; // We handled a dialog, so return 1. Just like OnPlayerCommandText.
}

return 0; // You MUST return 0 here! Just like OnPlayerCommandText.
}

Campbell ©

Weil ich seine Returns drinnen gelassen habe.
Sind zwar unnötig aber was solls.

Arbeite lieber mit case, anstatt mit listitems... Viel Ressourcen Schonender.

Zitat von Code.[21]

Arbeite lieber mit case, anstatt mit listitems... Viel Ressourcen Schonender.

In einem switch verwendet man auch listitem - deine Formulierung ist falsch.

Du musst die Dialoge verschieden benennen.
Außerdem musst du den Dialog auch abfragen, um Aktionen bei "response" zu setzen.
Btw: Benutz zum Einrücken lieber Tab anstatt Leertaste.

-----

Ganz oben unter die Includes musst du noch DIALOG_WAFFENSHOPLSPISTOLEN und DIALOG_WAFFENSHOPLSNAHMAPF definen.

Und

ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLS,DIALOG_STYLE_LIST,"Waffen Menü","Nahkampfwaffen\nPistolen","Weiter","Abbruch");

solltest du je nach dem wie man bei dir Waffen kauft z.B. unter OnPlayerCommandText oder OnPlayerKeyStateChange setzen.
Und wenn er dabei noch irgendwo in der Nähe sein muss, IsPlayerInRangeOfPoint benutzen.

if(dialogid == DIALOG_WAFFENSHOPLS)
{
if(response == 1)
{


if(listitem == 0)//Nahkampfwaffen
{
ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLSNAHKAMPF,DIALOG_STYLE_LIST,"Nahkampfwaffen","Schlagring: PREIS\nMesser: PREIS\nKatana: PREIS\n","Kaufen","Zurück");

}
if(listitem == 1)//Pistolen
{
ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLSPISTOLEN,DIALOG_STYLE_LIST,"Pistolen","9MM: PREIS\nSilenced 9MM: PREIS\nDesert Eagle: PREIS\n","Kaufen","Zurück");
}

}
else
{

SendClientMessage(playerid,ROT,"Vielleicht überlegst du es dir.");
return 1;

}
}
if(dialogid==DIALOG_WAFFENSHOPLSNAHKAMPF)//Nahkampfwaffen
{
if(response)
{
if(listitem == 0)
{
GivePlayerWeapon(playerid,1,1);
SendClientMessage(playerid,BLAU,"Du hast dir einen Schlagring gekauft!");
}
}
else
{
ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLS,DIALOG_STYLE_LIST,"Waffen Menü","Nahkampfwaffen\nPistolen","Weiter","Abbruch");
}

}
if(dialogid==DIALOG_WAFFENSHOPLSPISTOLEN)//Pistolen
{
if(response)
{
if(listitem == 0)
{
//Hier das Geld abziehen und die Waffen vergeben

}
}
else
{
ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLS,DIALOG_STYLE_LIST,"Waffen Menü","Nahkampfwaffen\nPistolen","Weiter","Abbruch");
}

}

Weil du schon wieder mehrere Dialoge mit einer DialogID verwendest...

So sollte es gehen:

if(pickupid==ammulsbuy)
{
ShowPlayerDialog(playerid,DIALOG_WAFFENSHOPLS,DIALOG_STYLE_LIST,"Waffen Menü","Nahkampfwaffen\nPistolen","Weiter","Abbruch");
}
return 1;