Dialog problem

FamoZ · 1. Februar 2012

Servus,

ich möchte mir einen /wanteds Befehl scripten nur es klappt nicht ganz. Es soll ein Dialog erscheinen in der dann die ganzen Spieler mit Namen und der Wantedanzahl stehen.
Wenn es nur einen Spieler mit Wanteds gibt, soll halt nur der eine Spieler im Dialog drin stehen und wenn es mehrere gibt dann soll eine Zeile darunter der andere Spieler stehen..

Nur wie mach ich das?

Das habe ich bis jetzt und es funktioniert nicht ._.

ocmd:wanteds(playerid, params[])
{
new list[128];
for(new i = 0;i < GetMaxPlayers();i++)
{
if(IsPlayerConnected(i))
{
if(GetPlayerWantedLevel(i) > 0)
{
format(list, sizeof(list), "%s: %i Wanteds", PlayerName(i), GetPlayerWantedLevel(i));
}
else if(GetPlayerWantedLevel(i) == 0)
{
format(list, sizeof(list), "Momentan gibt es keine Wanteds");
}
}
}
ShowPlayerDialog(playerid, DIALOG_WANTEDS, DIALOG_STYLE_MSGBOX, "Wanteds", list, "Okay", "");
return 1;
}

[DoL]Gamer · 1. Februar 2012

Bin mir nicht ganz sicher, habs mal auf die schnelle gemacht

Code

ocmd:wanteds(playerid, params[])
	{
	new string1[250];
 for(new i = 0;i < MAX_PLAYERS;i++)
    {
	    if(IsPlayerConnected(i))
	    {
	     new name[25];
	     GetPlayerName(i,name,sizeof(name));
	     new wanteds = WantedLevel[i];
      format(string1,sizeof(string1),"%s Wanteds:%d\n",name,wanteds);
     }
    ShowPlayerDialog(playerid, DIALOG_WANTEDS, DIALOG_STYLE_MSGBOX, "Wanteds", string1, "Okay", "");
    	return 1;
    }
}

Alles anzeigen

Kann sein, dass es auch nicht geht

FrostWolf · 1. Februar 2012

Versuch mal so:

Von:
if(IsPlayerConnected(i))

zu:
if(!IsPlayerConnected(i)) continue;

FamoZ · 1. Februar 2012

Zitat von [DoL]Gamer

Bin mir nicht ganz sicher, habs mal auf die schnelle gemacht

Code

ocmd:wanteds(playerid, params[])
	{
	new string1[250];
 for(new i = 0;i < MAX_PLAYERS;i++)
    {
	    if(IsPlayerConnected(i))
	    {
	     new name[25];
	     GetPlayerName(i,name,sizeof(name));
	     new wanteds = WantedLevel[i];
      format(string1,sizeof(string1),"%s Wanteds:%d\n",name,wanteds);
     }
    ShowPlayerDialog(playerid, DIALOG_WANTEDS, DIALOG_STYLE_MSGBOX, "Wanteds", string1, "Okay", "");
    	return 1;
    }
}

Alles anzeigen

Kann sein, dass es auch nicht geht

Geht leider nicht.

Zitat von FrostWolf

Versuch mal so:

Von:
if(IsPlayerConnected(i))

zu:
if(IsPlayerConnected(i)) continue;

Und was bringt das? Kannst du das erklären?

FrostWolf · 1. Februar 2012

Schon das Wort "continue" sagt viel!! (fortsetzen)

/e ich würde dir empfehlen, das du von
if(GetPlayerWantedLevel(i) > 0)
zu:
if(GetPlayerWantedLevel(i) >= 1)

machen würdest

FamoZ · 1. Februar 2012

Ja schon aber ich versteh den zusammenhang dazu nicht.

Jedenfalls gehts nicht, danke für den versuch...

//EDIT Und was solls bringen? Ist doch nix anderes?

Noch einer Ideen?

FamoZ · 1. Februar 2012

Bitte unterlasse deine unnötigen Posts.

Ja du schreibst gerade mit mir bei Steam und nein, das stimmt nicht ich habe es getestet!

//EDIT
Brauche immer noch Hilfe.

Austrianer · 1. Februar 2012

Warum funktioniert es denn nicht? Was für Errors gibt es denn, oder gibt es gar keine und es funktioniert einfach so nicht?

FrostWolf · 1. Februar 2012

Das hier sollte 100% funktioniert. Wurde auch getestet!

// Einfach verändern!!
ocmd:wanted(playerid,params[])
{
if(!isPlayerACop(playerid)) return SendClientMessage(playerid,COLOR_ROT,"Du bist kein Beamter");
{
new string[64];
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;
if(GetPlayerWantedLevel(i) >=1)
{
SendClientMessage(playerid,COLOR_ROT,"-----Wanteds-------");
format(string, sizeof(string), "%s(ID:%d) - %d Wanteds\n", SpielerName(i),playerid, GetPlayerWantedLevel(i));
//SendClientMessage(playerid,BLAU,string);
ShowPlayerDialog(playerid,DIALOG_WANTED,DIALOG_STYLE_MSGBOX,"Wanteds",string,"Weiter","Abbrechen");
}
else if(GetPlayerWantedLevel(i) == 0)
{
SendClientMessage(playerid,COLOR_ROT,"Derzeit hat kein Spieler einen Wanted!");
}
}
}
return 1;
}

/ beweis das funk. omg!

Uploaded with ImageShack.us

FamoZ · 1. Februar 2012

Hast du das auch mit mehreren Spielern gleichzeitig getestet?

Es geht halt nicht. Es steht immer nur sein eigener Name im Dialog...

Und hör bitte auf hier sinnlos etwas rein zu posten obwohl ich dir bereits bei Steam 100x gesagt habe das es nicht funktioniert... was bringt dir das?

Goldkiller · 1. Februar 2012

Zitat

Das hier sollte 100% funktioniert. Wurde auch getestet!

Das glaub ich dir gerne,macht es trotzdem nicht richtig. Du hast es sicherlich nur mit einem Spieler getestet, daher wohl fehlerfrei.Mit mehr als einem Spieler funktioniert es nicht. Denn sobald
ein Spieler gefunden wird mit Wanteds, wird für den Spieler der Dialog angezeigt. Nur sein Name taucht in dem Dialog auf. Wird ein weiterer Spieler gefunden, werden seine Daten formatiert und jetzt taucht nur dieser im Dialog auf. Heisst,du beachtest nicht ob zuvor schon Jemand mit Wanteds gefunden wurde.Auch im format ist ein Fehler, playerid anstatt i.
Das else if ist ebenfalls falsch gesetzt.Hat ein Spieler keine Wanteds, kommt für ihn die Nachricht "Derzeit hat kein Spieler einen Wanted!" und das jedes mal wenn ein Spieler keine Wanteds hat.

So ist es richtig, schau es dir mal an... vielleicht lernst du etwas daraus ;).
ocmd:wanted(playerid,params[])
{
if(!isPlayerACop(playerid)) return SendClientMessage(playerid,COLOR_ROT,"Du bist kein Beamter");
{
new
string[ 256 ]; // <- wir brauchen einiges an Speicher. Sollte aber ausreichen so
string[0] = '\0';
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(!IsPlayerConnected(i)) continue;
if(GetPlayerWantedLevel(i) >=1)
{
format(string,sizeof(string),"%s%s(ID:%d) - %d Wanteds\n", string, SpielerName(i),playerid, GetPlayerWantedLevel(i));
//SendClientMessage(playerid,COLOR_ROT,"-----Wanteds-------");
//format(string, sizeof(string), "%s(ID:%d) - %d Wanteds\n", SpielerName(i),playerid, GetPlayerWantedLevel(i));
//SendClientMessage(playerid,BLAU,string);
// ShowPlayerDialog(playerid,DIALOG_WANTED,DIALOG_STYLE_MSGBOX,"Wanteds",string,"Weiter","Abbrechen");
}
/*
else if(GetPlayerWantedLevel(i) == 0)
{
SendClientMessage(playerid,COLOR_ROT,"Derzeit hat kein Spieler einen Wanted!");
}*/

}
if(string[0]) {
ShowPlayerDialog(playerid,DIALOG_WANTED,DIALOG_STYLE_MSGBOX,"Wanteds",string,"Weiter","Abbrechen"); // Sicher DIALOG_STYLE_MSGBOX und nicht vllt LIST ?
}
else { // Wir haben keinen text formatiert, daher wohl auch Niemanden gefunden!
SendClientMessage(playerid,COLOR_ROT,"Derzeit hat kein Spieler einen Wanted!");
}
}
return 1;
}

//Edit: Wird ja wieder grandios eingerückt :\

FrostWolf · 1. Februar 2012

Zitat von FamoZ

Hast du das auch mit mehreren Spielern gleichzeitig getestet?

Und ich wollte es eigentlich im Dialog haben und wenn ich anstatt

SendClientMessage();

einfach

ShowPlayerDialog();

mache geht es nicht...

Und hör bitte auf hier sinnlos etwas rein zu posten wenn ich dir bereits bei Steam 100x gesagt habe das es nicht funktioniert... was bringt dir das?

Alles anzeigen

Zu
Frage 1: Ja, das hab ich mit mehreren Leuten getestet.
Frage 2: Ja, dann ist was andares falsch
@

Goldkiller: Ich habs so gelernt ._.

Goldkiller · 1. Februar 2012

Zitat von FrostWolf

]Zu
Frage 1: Ja, das hab ich mit mehreren Leuten getestet.
Frage 2: Ja, dann ist was andares falsch

Das hast du vllt mit mehreren getestet, zeigt trotzdem immer nur 1 Spieler an.
Ich verweise nochmal auf meinen Post.
http://forum.sa-mp.de/san-andr…ialog-problem/#post857668

Quazii_John · 1. Februar 2012

Könntest du dein immernoch ungelöstes Problem kurz nochmal zusammenfassen?

FrostWolf · 1. Februar 2012

Zitat von Goldkiller

Das hast du vllt mit mehreren getestet, zeigt trotzdem immer nur 1 Spieler an.
Ich verweise nochmal auf meinen Post.
Dialog problem

Also haben alle Threads falsch gemacht? z.B. hier Click ! wtf?
Frage1: Ist die höchste string größe nicht "128"?
Frage2: string[0] = '\0'; // Für was ist das? warum unbedingt 0 und \ ?

FamoZ · 1. Februar 2012

Vielen Dank Goldkiller!

//EDIT Und natürlich an die anderen die es probiert haben.

Funktioniert nun.

//CLOSE PLS

Goldkiller · 1. Februar 2012

Zitat von FrostWolf

Also haben alle Threads falsch gemacht? z.B. hier Click ! wtf?
Frage1: Ist die höchste string größe nicht "128"?
Frage2: string[0] = '\0'; // Für was ist das? warum unbedingt 0 und \ ?

Frage 1. Nein, ist es nicht.Alleine schon ein Array anzulegen mit MAX_PLAYERS ist bei den meisten 500 und somit deutlich mehr als 128 . Es gibt Limits für den Chat, dort war es früher 128 Zeichen. Jetzt ist es aber ~140 wegen den Farbcodes.
Frage 2. \0 ist NULL-Character.Gibt an, dass der Text an dieser Stelle endet.Eigentlich sollte der String beim neu deklarieren sowieso dort einen NULL-Character habe, ich mach es aber aus gewohnheit immer dazu.

Zitat

Also haben alle Threads falsch gemacht? z.B. hier Click ! wtf?

Ich poste nicht in jeden Thread und sehe auch nicht jeden. Dennoch,dort ist es ebenfalls falsch, gut gemerkt ;).

breadfish.de 11. März 2022