Autohaus, Auto soll erstellt werden

Jameso · 30. September 2018

Ich habe einen Dialog erst einmal der zeigt;

Eigene Fahreuge
Andere Fahrzeuge

Respond auf Eigene Fahrzeuge:

Code

if(dialogid == DIALOG_FAHRZEUG1)
	{
	 	if(response)
	 	{
			switch(listitem)
			{
				case 0: //Fahrzeugschlüssel
				{
                 	ShowPlayerDialog(playerid, DIALOG_FAHRZEUG2, DIALOG_STYLE_LIST, "{FFFFFF}Fahrzeug fahrzeugschlüssel", "{FFFFFF}Fahrzeugschlüssel\teigene\nschlüssel\tAndere", "Auswählen", "Zurück");
				}
				case 1: //Nix
		  		{
				}
				case 2: //
				{


				}
				case 3: //
		  		{
				}
			}
		}
	}
	if(dialogid == DIALOG_FAHRZEUG2)
	{
	 	if(response)
	 	{
			switch(listitem)
			{
				case 0: //Eigene Fahrzeugschlüssel
				{
	  				for(new i = 0; i < sizeof(PCar[]); i++)
	  				{
	  				    new name;
						name = PCar[playerid][i][P_MID] - 400;
					    new string[300];
					    format(string,sizeof(string),"{FFFFFF}%d\tSF-1337\tX\nHuntley\tSF-1337\t",carNames[name]);
	                 	ShowPlayerDialog(playerid, DIALOG_FAHRZEUG3, DIALOG_STYLE_LIST, "{FFFFFF}schlüssel\tNummernschild\tAusgewählt", string, "Auswählen","Zurück");
					}
				}
				case 1: //Andere Fahrzeugschlüssel
		  		{


				}
			}
		}
		else
		{
 			if(!response)
		 	{
				switch(listitem)
				{
					case 0: //Wenn er wieder zurück ins Hautpmenü will
					{
					}
					case 1: //Wenn er wieder zurück ins Hautpmenü will
			  		{


					}
				}
			}
		}
	}

Alles anzeigen

Irgendwie habe ich jetzt das Problem, wenn ich auf eigene Fahrzeuge clicke also ich auf DIALOG_FAHRZEUG2 antworte er mir das nächste nicht Anzeigt mit den Fahrzeugen.

Den 4. Punkt habe ich nicht ganz verstanden.

Ich will das es dann auch so aussieht wenn kein Fahrzeug auf dem Slot ist.

Huntley
< Leer >
< Leer

Dazu will ich noch das, sein Kennzeichen angezeigt wird , aber das sollte ich glaube ich hinkriegen.

Bei den anderen Fahrzeugen, das sind dann die von anderen Spielern, steht der Besitzer dann noch in einem Tab ( \t )

Jeffry · 30. September 2018

if(dialogid == DIALOG_FAHRZEUG2)
{
if(response)
{
switch(listitem)
{
case 0: //Eigene Fahrzeugschlüssel
{
new string[300], name;
for(new i = 0; i < sizeof(PCar[]); i++)
{
name = PCar[playerid][i][P_MID] - 400;
if(name < 0 || name >= sizeof(carNames)) format(string,sizeof(string),"%s{FFFFFF}<Leer>\t<Leer>\n", string);
else format(string,sizeof(string),"%s{FFFFFF}%s\tSF-1337\n", string, carNames[name]);
}
ShowPlayerDialog(playerid, DIALOG_FAHRZEUG3, DIALOG_STYLE_LIST, "{FFFFFF}schlüssel\tNummernschild\tAusgewählt", string, "Auswählen","Zurück");
}
case 1: //Andere Fahrzeugschlüssel
{

}
}
}
else
{
//Wenn er wieder zurück ins Hautpmenü will
}
return 1;
}
Versuche es so.

Was verstehst du an Punkt 4 nicht?

Jameso · 30. September 2018

Funktioniert immer noch nicht.
Weiß nicht wie ich das machen soll bei 4.

Jeffry · 30. September 2018

Was funktioniert nicht?

Was weißt du nicht, wie du es machen sollst?

Mehr Informationen, wie jedes Mal. Warum muss ich dir das jedes Mal aufs neue sagen? Mit "Funktioniert nicht" fange ich genau so viel an, wie du, wenn ich dir darauf "Ok" antworte, nämlich gar nichts.

Jameso · 30. September 2018

Du hast mich doch gefragt das ich es so versuchen soll,
ändert nichts soll ich wirklich jetzt unnötig den selben Code rein schicken den du im letzten Post rein geschickt hast?
Hätte ich was anders gemacht, hätte ich schon Bescheid gesagt.

Code

if(dialogid == DIALOG_FAHRZEUG1)
	{
	 	if(response)
	 	{
			switch(listitem)
			{
				case 0: //Fahrzeugschlüssel
				{
                 	ShowPlayerDialog(playerid, DIALOG_FAHRZEUG2, DIALOG_STYLE_LIST, "{FFFFFF}Fahrzeug - Fahrzeugschlüssel", "{FFFFFF}Fahrzeugschlüssel\t-\tEigene\nFahrzeugschlüssel\t-\tAndere", "Auswählen", "Zurück");
				}
				case 1: //Nix
		  		{
				}
				case 2: //
				{


				}
				case 3: //
		  		{
				}
			}
		}
	}
	if(dialogid == DIALOG_FAHRZEUG2)
	{
		if(response)
		{
			switch(listitem)
			{
				case 0: //Eigene Fahrzeugschlüssel
				{
					new string[300], name;
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
						name = PCar[playerid][i][P_MID] - 400;
						if(name < 0 || name >= sizeof(carNames)) format(string,sizeof(string),"%s{FFFFFF}<Leer>\t<Leer>\n", string);
						else format(string,sizeof(string),"%s{FFFFFF}%s\tSF-1337\n", string, carNames[name]);
					}
					ShowPlayerDialog(playerid, DIALOG_FAHRZEUG3, DIALOG_STYLE_LIST, "{FFFFFF}schlüssel\tNummernschild\tAusgewählt", string, "Auswählen","Zurück");
				}
				case 1: //Andere Fahrzeugschlüssel
				{
				}
			}
		}
		else
		{
			//Wenn er wieder zurück ins Hautpmenü will
		}
		return 1;
	}

Alles anzeigen

Selbe Problem wie ebend, er zeigt mir nicht den Dialog DIALOG_FAHRZEUG3

Jeffry · 30. September 2018

Den gleichen Code brauchst du natürlich nicht posten, das macht ja keinen Sinn.

Dass es nicht geht ist mir auch klar, allerdings wäre eine Information hilfreich, wie zum Beispiel, der Code wird überhaupt aufgerufen, es gibt Fehler im Log, etc. Setze doch mal ein paar prints und prüfe ob diese überhaupt aufgerufen werden. Prüfe auch, ob dir crashdetect irgendwelche Fehler ausgibt.

Danach postest du den Code mit den prints und den Server Log, mit den Ausgaben der prints. Oder du siehst den Fehler schon selbst.

Jameso · 30. September 2018

http://prntscr.com/l0j7kv
Das wird mir angezeigt, ist der selbe Code wie im letzten Post mit -400

Jeffry · 30. September 2018

Das kann eigentlich nicht sein, wenn der Code exakt so ist.
Ich habe den Code nach den Posten nochmal direkt danach geändert, hast du diese Änderung im Code? Übernehme den Code bitte nochmal neu in deinen Code.
Dort war bei ersten format noch der Indexzugriff drin, den hatte ich noch entfernt.

Jameso · 30. September 2018

Code

if(dialogid == DIALOG_FAHRZEUG2)
	{
		if(response)
		{
			switch(listitem)
			{
				case 0: //Eigene Fahrzeugschlüssel
				{
					new string[300], name;
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
						name = PCar[playerid][i][P_MID] - 400;
						if(name < 0 || name >= sizeof(carNames)) format(string,sizeof(string),"%s{FFFFFF}<Leer>\t<Leer>\n", string);
						else format(string,sizeof(string),"%s{FFFFFF}%s\tSF-1337\n", string, carNames[name]);
					}
					ShowPlayerDialog(playerid, DIALOG_FAHRZEUG3, DIALOG_STYLE_LIST, "{FFFFFF}schlüssel\tNummernschild\tAusgewählt", string, "Auswählen","Zurück");
				}
				case 1: //Andere Fahrzeugschlüssel
				{
				}
			}
		}
		else
		{
			//Wenn er wieder zurück ins Hautpmenü will
		}
		return 1;
	}

Alles anzeigen

Ich habe vergessen meinen Server neu zu starten, es war mein Fehler tut mir leid.
Im Dialog wird alles vollgespammt mit <Leer>

Jeffry · 30. September 2018

Es kommen so viele Leer Zeilen, wie du an Fahrzeugen pro Spieler erlaubst. Sprich die Anzahl, die bei new PCar im zweiten Array drin steht.

Wie sieht denn das new PCar aus?

Jameso · 30. September 2018

Habs gefunden, danke

Code

if(dialogid == DIALOG_FAHRZEUG2)
	{
		if(response)
		{
			switch(listitem)
			{
				case 0: //Eigene Fahrzeugschlüssel
				{
					new string[300], name;
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
						name = PCar[playerid][i][P_MID] - 400;
						if(name < 0 || name >= sizeof(carNames)) format(string,sizeof(string),"%s{FFFFFF}Fahrzeug\tKennzeichen\tAusgewählt \n\
					 	<Leer>", string);
						else format(string,sizeof(string),"%s{FFFFFF}%s\tSF-1337\n", string, carNames[name]);
					}
					ShowPlayerDialog(playerid, DIALOG_FAHRZEUG3, DIALOG_STYLE_TABLIST_HEADERS, "Fahrzeugschlüssel - Auswählen", string, "Auswählen","Zurück");
				}
				case 1: //Andere Fahrzeugschlüssel
				{
				}
			}
		}
		else
		{
			//Wenn er wieder zurück ins Hautpmenü will
		}
		return 1;
	}

Alles anzeigen

Ich wollte mir das so anzeigen lassen, dass ich dann auch die Unterteilung habe also nach der Überschrift auch noch die Unterüberschrift ist. (Fahrzeug, Kennzeichen usw.)

Wie sollen wir das machen, weil ich ja jetzt in jeder Zeile das ja auch mit rein bekomme.

Habe noch zu TABLIST_HEADERS gewechselt, ist ordentlicher.

Jeffry · 30. September 2018

Die Überschrift musst du vor der Schleife ins format setzen, sonst kommt es für jedes Fahrzeug neu.

Jameso · 30. September 2018

Code

if(response)
		{
			switch(listitem)
			{
				case 0: //Eigene Fahrzeugschlüssel
				{
					new string[300], name;
					format(string,sizeof(string),"%s{FFFFFF}Fahrzeug\tKennzeichen\tAusgewählt \n\
					", string);
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
						name = PCar[playerid][i][P_MID] - 400;
						if(name < 0 || name >= sizeof(carNames)) format(string,sizeof(string),"<Leer>", string);
						else format(string,sizeof(string),"%s{FFFFFF}%s\tSF-1337\n", string, carNames[name]);
					}
					ShowPlayerDialog(playerid, DIALOG_FAHRZEUG3, DIALOG_STYLE_TABLIST_HEADERS, "Fahrzeugschlüssel - Auswählen", string, "Auswählen","Zurück");
				}
				case 1: //Andere Fahrzeugschlüssel
				{
				}
			}
		}

Alles anzeigen

So ist es ja falsch, habe es noch nie gemacht.

Jeffry · 30. September 2018

Vor dem <Leer> fehlt das %s und danach das \n, das hast du entfernt. Das muss beides wieder hin.

Jameso · 1. Oktober 2018

Danke, verstehe das jetzt auch.
Jetzt würde ich gerne das es mit P_Current funktioniert.

Code

if(dialogid == DIALOG_FAHRZEUG3)
	{
		if(response)
		{
			switch(listitem)
			{
				case 0: //Fahrzeug 1
				{
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
                        PCar[playerid][i][P_Current] = 1;
					}
				}
				case 1; //Fahrzeug 2...
				{
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
                        PCar[playerid][i][P_Current] = 1;
					}
				}
			}
		}
		else
		{
			//Wenn er wieder zurück ins Hautpmenü will
		}
		return 1;
	}

Alles anzeigen

So wird ja für jedes Fahrzeug das Current gesetzt, habe es schon gemacht das im Dialog angezeigt wird welches Fahrzeug überhaupt Current ist.
Aber es soll ja nur 1. Mit Current geben

Und muss ich schonmal für das zweite, dritte, vierte... Fahrzeug P_MID2, P_MID3 erstellen?

Jeffry · 1. Oktober 2018

Schaue dir die beiden Punkte nochmal an.

Zitat von Jeffry

Bei Dialog-Klick: Nochmal Schleife durch die Fahrzeuge und counter so lange hochzählen lassen, bis identisch zu listitem (das listitem kann unter Umständen falsch sein, wenn in der Mitte ein Fahrzeug nicht existiert).

Den index aus der Schleife in P_Current setzen.

Davon hast du bis auf die Schleife nichts umgesetzt.
Versuche doch anhand dessen mal, die Punkte exakt so umzusetzen. Nochmal einzeln gesplittet:

Schleife
Zähler hochzählen (bei -1 beginnen)
Zähler mit listitem vergleichen
Wenn Zähler == listitem: Index aus Schleife (i) in P_Current schreiben

P_Current darf übrigens kein enum-Wert von PCar sein, sondern muss zum PlayerInfo (oder wie es bei dir eben heißt) gehören.

Versuche das so im Code umzusetzen. Hake alle 4 Punkte ab. Poste dann den Code nochmal.
Es geht hier um Basics, die musst du verstehen, sonst kann ich dir den Code auch komplett schreiben und du lernst nichts dabei.

Jameso · 1. Oktober 2018

Das verstehe ich ja nicht, also wie mache ich diesen Zähler? (dummer Frage)
Hätte ich das verstanden hätte ich das vorbildlich schon gemacht, aber so verstehe ich das nicht weiß nicht wie.
Hier nochmal angepasst.

Code

if(dialogid == DIALOG_FAHRZEUG3)
	{
		if(response)
		{
			switch(listitem)
			{
				case 0: //Fahrzeug 1
				{
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
                       PInfo[PlayerID][P_Current] = 1;
					}
				}
				case 1; //Fahrzeug 2...
				{
				}
			}
		}
		else
		{
			//Wenn er wieder zurück ins Hautpmenü will
		}
		return 1;
	}

Alles anzeigen

Jeffry · 1. Oktober 2018

Zitat von BrightLeaN

Das verstehe ich ja nicht, also wie mache ich diesen Zähler? (dummer Frage)

Genau so Fragen sollst du ja stellen, endlich! Wenn du etwas nicht verstehst, musst du sagen, was genau du nicht verstehst.
Nur dann kann dir zielgerichtet geholfen werden.

Ein Zähler ist einfach ein Integer:
new counter = -1;
//for(new ...
{
counter++;

Den Counter natürlich nur dann hochzählen lassen, wenn das Fahrzeug auch existiert.
Versuche, ob du damit jetzt weiter kommst. Ansonsten frage gerne zielgerichtet weiter. Wie gesagt, ich möchte, dass du es verstehst, dazu musst du es selbst coden.
Würde ich es mir einfach machen, schreibe ich dir den Code in 2 Minuten hin, das wäre aber nicht Sinn der Sache.

Jameso · 1. Oktober 2018

Zitat von Jeffry

schreibe ich dir den Code in 2 Minuten hin, das wäre aber nicht Sinn der Sache.

Naja, wenn ich dann irgendwas hoch zählen soll und irgendwie den Sinn dahinter nicht verstehe verstehe ich wirklich noch weniger.
Ich muss mir was angucken um zu verstehen ah der zählt also bis er was findet und dann macht er das und das...

So geht es auch, mehr Stress für mich aber wird hoffentlich auch klappen

Verstehe nicht was ich hier counten soll die ModelID? ist der Counter überhaupt richtig?
Mein Compiler stürzt ab irgendwas stimmt nicht, am Counter liegt es nicht.

Code

if(dialogid == DIALOG_FAHRZEUG3)
	{
		if(response)
		{
			switch(listitem)
			{
				case 0: //Fahrzeug 1
				{
				    new counter = -1;
					for(new i = 0; i < sizeof(PCar[]); i++)
					{
 						if(PCar[playerid][d][P_MID] > 0)
 						{
 							for(new counter = 0; counter < sizeof(PCar[]); counter++)
 							{
                       			pInfo[playerid][P_Current] = 1;
                       		}
						}
					}
				}
				case 1; //Fahrzeug 2...
				{
				}
			}
		}
		else
		{
			//Wenn er wieder zurück ins Hautpmenü will
		}
		return 1;
	}

Alles anzeigen

Jeffry · 1. Oktober 2018

Edit:
Mir fällt gerade noch ein, du zeigst ja alle Slots an, auch die leeren. Ist das noch so, oder hast du das geändert?

Wenn ja, dann kannst du die Schleife weg lassen und auch den switch und es einfach so machen:
pInfo[playerid][P_Current] = listitem;