Bekomme Warning nicht weg..

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])

if(dialogid=="hir muss der dialogname rein" !)
{
if(response==1)
{
switch(listitem)
{
case 0 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
case 2 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
case 3 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
case 4 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
}
}
return 1;
}

der dialogname fehlt oder heißt der bei dir 5 ?

und die returns

Du wirst den return nach unten setzen müssen. Bisher returnst du unter der if-Abfrage, aber nicht unter dem public, der benutzt wurde.

Zitat von Fuldara

Ja mein Dialog heißt bei mir 5.

@Cambell: Wie meinst du das?

So wie ich es sage. Bisher returnst du innerhalb der if-Abfrage, was nicht den public returnen wird. Schau dir doch die Fehlermeldung an, es sagt aus, dass der public nicht returnt wurde. D.h.:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
if(dialogid == 5) {
// Switch-Anweisungen ...
}
return 1;
}

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])

if(dialogid==5 !)
{
if(response==1)
{
switch(listitem)
{
case 0 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
case 2 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
case 3 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
case 4 :
{
GivePlayerWeapon(playerid,362,1000);
GivePlayerMoney(playerid,1);
return 1;
}
}
}
return 1;
}
mach es dan so

Radek_Hillers: Es ist kompletter Schwachsinn, was du dort betreibst. In jeder switch-Anweisung zu returnen macht schlicht weg keinen Sinn und du scheinst die Fehlermeldung wohl nicht ganz verstanden zu haben.

Es ist Egal ob bei DialogResponse eine Zahl oder der Dialog Name steht.

#define TESTdialog 5

Heißt man kann

if(dialogid==5 )

Oder

if(dialogid==TESTdialog)

nutzen und es Wird das Gleiche Dialog gezeigt Wichtig ist halt das man Die Richtige Dialog ID eingibt

Nein, bisher ist es so:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
if(dialogid == 5) {
// Switch-Anweisungen ...
return 1;
}
}

Edit: Schau dir nochmal die WaffenIDs an, deine Waffen gibt es garnicht.

Da Du uns die Funktion OnDialogResponse nicht
vollkommen geschickt hast, nehme ich an, dass Du
also die Klammern { und } vergessen hast.

Versuchs so und füge Deine Daten ein:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
if(dialogid == 0000)
{
if(response)
{
switch()
{
case 1:
// Vari 1
// Vari 2

case 2:
// Vari 3
// Vari 4

case 3:
// Vari 5
// Vari 6
}
}
else
{
// ESC
}
}

return 1; // nicht vergessen
}

Du musst auch die WaffenID nehmen und nicht die ObjectID von der Waffe

Waffen IDs: http://wiki.sa-mp.com/wiki/Weapons
Ich habe es dir mal Gemacht

Der Code ist Getestet und Funktioniert Perfekt P.S. : Bei dem Geld GetPlayerMoney machen wenn du Geld abnehmen willst da du für Jede Waffe derzeit 1$ Bekommst^^

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
if(dialogid==5)
{
if(response==1)
{
switch(listitem)
{
case 0:
{
GivePlayerWeapon(playerid,38,1000);
GivePlayerMoney(playerid,1);
}
case 1:
{
GivePlayerWeapon(playerid,22,1000);
GivePlayerMoney(playerid,1);
}
case 2:
{
GivePlayerWeapon(playerid,34,1000);
GivePlayerMoney(playerid,1);
}
case 3:
{
GivePlayerWeapon(playerid,35,1000);
GivePlayerMoney(playerid,1);
}
}
}
}
return 1;
}