Problem mit OnPlayerSelectedMenuRow

DerFamer · 28. Mai 2016

/*public OnPlayerSelectedMenuRow(playerid, row)
{
if(GetPlayerMenu(playerid) == Stadthalle)
{
switch(row)
{
case 0: { ShowPlayerDialog(playerid, DIALOG_JOBS, DIALOG_STYLE_LIST, "Los Santos Arbeitsamt", "Detektiv\nMüllmann\nPilot\nAngler\nBodyguard\nZeitungsjunge\nBoxer\nBusfahrer\nPizzalieferant\nProdlieferant (FreizeitJob)\nMechaniker\nAnwalt", "Auswählen", "Abbrechen"); }
case 1: { if(PlayerInfo[playerid][pOwnPerso] == 1) }
}
}
return 1;
}*/

Hallo habe das Problem wen ich kompeliere bekomm ich 26 errors aber nicht von diesen Code. wen ich das Ausklammer habe ich 0 Errors an was liegt das?

Knolle · 28. Mai 2016

C

public OnPlayerSelectedMenuRow(playerid, row) {
	if(GetPlayerMenu(playerid) == Stadthalle) {
		switch(row) {
			case 0: { ShowPlayerDialog(playerid, DIALOG_JOBS, DIALOG_STYLE_LIST, "Los Santos Arbeitsamt", "Detektiv\nMüllmann\nPilot\nAngler\nBodyguard\nZeitungsjunge\nBoxer\nBusfahrer\nPizzalieferant\nProdlieferant (FreizeitJob)\nMechaniker\nAnwalt", "Auswählen", "Abbrechen"); }
			case 1: { if(PlayerInfo[playerid][pOwnPerso] == 1){} }
		}
	}
	return 1;
}

Dein Problem lag darin, das der Compiler sehr primtiv ist und du eine offene If abfrage hattest ohne eine Anweisung dahinter.
Aber wie gesagt, primitiv, daher wurde quasi eine geschweite Klammer mit dem if in verbindung gebracht.
Nun habe ich einfach geschweifte Klammern hinzugefügt und dies sollte nicht mehr der Fall sein.

DerFamer · 28. Mai 2016

C:\Windows\Vss\German v1.1\German[Scripts]\German-Next-Era-Life---v2.5\German Next Era Life - v2.5\gamemodes\GNE.pwn(6006) : error 029: invalid expression, assumed zero
C:\Windows\Vss\German v1.1\German[Scripts]\German-Next-Era-Life---v2.5\German Next Era Life - v2.5\gamemodes\GNE.pwn(6006) : error 001: expected token: ";", but found "{"
C:\Windows\Vss\German v1.1\German[Scripts]\German-Next-Era-Life---v2.5\German Next Era Life - v2.5\gamemodes\GNE.pwn(6016) : warning 225: unreachable code

Hm

Knolle · 28. Mai 2016

Was ist genau Zeile 6006?
Ich kann die Zeilen nicht gerade konkret unserem bzw. deinem Schnipsel zuordnen

DerFamer · 28. Mai 2016

public OnPlayerSelectedMenuRow(playerid, row) { //Hier
if(GetPlayerMenu(playerid) == Stadthalle) {
switch(row) {
case 0: { ShowPlayerDialog(playerid, DIALOG_JOBS, DIALOG_STYLE_LIST, "Los Santos Arbeitsamt", "Detektiv\nMüllmann\nPilot\nAngler\nBodyguard\nZeitungsjunge\nBoxer\nBusfahrer\nPizzalieferant\nProdlieferant (FreizeitJob)\nMechaniker\nAnwalt", "Auswählen", "Abbrechen"); }
case 1: { if(PlayerInfo[playerid][pOwnPerso] == 1){} }
}
}
return 1;
}

Knolle · 29. Mai 2016

Der Callback "OnPlayerSelectedMenuRow" ist nicht ganz zufällig auch das letzte im Script? Wenn doch, dann schau mal, ob der Funktion/Callback zuvor etwas fehlt bzw. platziere diese mal um, schauen ob der Fehler nach wie vor der gleiche ist.
So können wir ohne große weiteres die Fehlerhafte Funktion ausfindig machen.
Ich schlage dies vor, da der Callback selbst richtig definiert ist und auch keine geschweifte Klammer hier fehlt.

DerFamer · 10. Juni 2016

Die Funktion OnPlayerSelectedMenuRow gibt es schon wie kann ichs beheben?

Knolle · 11. Juni 2016

Also hast du die nun ein zweites mal im Script?
Oder verstehe ich dich da falsch?
Falls zutreffend:
- kopiere den Inhalt was in der public Funktion passieren soll in die jeweils andere hinein.
- Alternativ: hook dein zweites Callback (nur geeignet, wenn man gute Kenntnisse besitzt im Scripten bzw. man verwendety_hook)
Falls nicht zutreffend:
lies zweite Zeile: verstehe dich falsch.

DerFamer · 11. Juni 2016

Function OnPlayerSelectedMenuRow(playerid, row)
{
new Menu:CurrentMenu = GetPlayerMenu(playerid);
if(CurrentMenu == Navigation)
{
switch(row)
{
case 0://Stadthalle
{
SetPlayerCheckpoint(playerid,1481.1517,-1763.9393,18.7958,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Stadthalle] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 1://Krankenhaus
{
SetPlayerCheckpoint(playerid,1184.7716,-1321.6627,13.5732,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Krankenhaus] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 2://Polizei
{
SetPlayerCheckpoint(playerid,1513.7441,-1667.4794,14.0469,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Polizei Department] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 3://Schwarzmarkt
{
SetPlayerCheckpoint(playerid,2620.4971,-1499.0149,16.2748,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Zivispawn] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 4://Arbeitsamt
{
SetPlayerCheckpoint(playerid,2228.3035,-1157.6097,25.7999,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Arbeitsamt] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 5://Bank
{
SetPlayerCheckpoint(playerid,1462.6842,-1014.6383,26.4997,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Bank] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 6://Handyladen
{
SetPlayerCheckpoint(playerid,1104.1304,-1449.0028,15.7969,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Handyladen] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 7://uhrenladen
{
SetPlayerCheckpoint(playerid,1699.9642,-1170.1368,23.8281,5.0);
SendClientMessage(playerid, COLOR_GREEN, "Ziel [Uhrenladen] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
}
case 8://Exit Menu
{
HideMenuForPlayer(Navigation, playerid);
}
}
}
/*public OnPlayerSelectedMenuRow(playerid, row) {
if(GetPlayerMenu(playerid) == Stadthalle) {
switch(row) {
case 0: { ShowPlayerDialog(playerid, DIALOG_JOBS, DIALOG_STYLE_LIST, "Los Santos Arbeitsamt", "Detektiv\nMüllmann\nPilot\nAngler\nBodyguard\nZeitungsjunge\nBoxer\nBusfahrer\nPizzalieferant\nProdlieferant (FreizeitJob)\nMechaniker\nAnwalt", "Auswählen", "Abbrechen"); }
case 1: { if(PlayerInfo[playerid][pOwnPerso] == 1){} }
}
}
return 1;
}*/

Knolle · 11. Juni 2016

Du hattest einen kleinen Fehler.
Du hattest eine Geschweifte Klammer vergessen. Aber nun habe ich deine Stadthalle in die dafür vorgesehene Funktion eingepflegt.

Code

Function OnPlayerSelectedMenuRow(playerid, row) {
	new Menu:CurrentMenu = GetPlayerMenu(playerid);
	if(GetPlayerMenu(playerid) == Stadthalle) {
	switch(row) {
		case 0: { 
			ShowPlayerDialog(playerid, 
				DIALOG_JOBS, DIALOG_STYLE_LIST, 
				"Los Santos Arbeitsamt", 
				"Detektiv\n\
				Müllmann\n\
				Pilot\n\
				Angler\n\
				Bodyguard\n\
				Zeitungsjunge\n\
				Boxer\n\
				Busfahrer\n\
				Pizzalieferant\n\
				Prodlieferant (FreizeitJob)\n\
				Mechaniker\n\
				Anwalt", 
				"Auswählen", "Abbrechen"); }
		case 1: { 
			if(PlayerInfo[playerid][pOwnPerso] == 1){
				//code goes here
			} 
		}
	}
	if(CurrentMenu == Navigation){
		switch(row) {
			case 0: {//Stadthalle
				SetPlayerCheckpoint(playerid,1481.1517,-1763.9393,18.7958,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Stadthalle] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 1: {//Krankenhaus
				SetPlayerCheckpoint(playerid,1184.7716,-1321.6627,13.5732,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Krankenhaus] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 2: {//Polizei
				SetPlayerCheckpoint(playerid,1513.7441,-1667.4794,14.0469,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Polizei Department] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 3: {//Schwarzmarkt
				SetPlayerCheckpoint(playerid,2620.4971,-1499.0149,16.2748,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Zivispawn] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 4: {//Arbeitsamt
				SetPlayerCheckpoint(playerid,2228.3035,-1157.6097,25.7999,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Arbeitsamt] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 5: {//Bank
				SetPlayerCheckpoint(playerid,1462.6842,-1014.6383,26.4997,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Bank] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 6: {//Handyladen
				SetPlayerCheckpoint(playerid,1104.1304,-1449.0028,15.7969,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Handyladen] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 7: {//uhrenladen
				SetPlayerCheckpoint(playerid,1699.9642,-1170.1368,23.8281,5.0);
				SendClientMessage(playerid, COLOR_GREEN, "Ziel [Uhrenladen] auf der MiniMap. (Benutze /chilfe um den Marker zu löschen)");
			}
			case 8: {//Exit Menu
				HideMenuForPlayer(Navigation, playerid);
			}
		}
	}
}

Alles anzeigen

Btw: Bei deiner Navigation kannst du ganz gut auch Arrays verwenden.

Code

enum E_NavigationData{
	Float:CPPosX,
	Float:CPPosY,
	Float:CPPosZ,
	Float:CPRadius,
	Name[32],
	//...
};
new NavigationData[][E_NavigationData] = {
	{1481.1517,-1763.9393,18.7958,5.0,"Stadthalle"},
	{1184.7716,-1321.6627,13.5732,5.0,"Krankenhaus"},
	//...
};

Alles anzeigen

Und dann kannst du direkt die erhaltene Row nutzen als Index Zeiger.
Geht natürlich ganz gut, solange du nichts sonderbar großes haben möchtest. Wie in deinem Fall, ist es ja nur in den eckigen Klammern den Namen einfügen (format()) und dann die Nachricht Senden.

Problem mit OnPlayerSelectedMenuRow

breadfish.de 12. März 2022

Ähnliche Themen

Hilfe beim Autohaus

MYSQL-Script

Repairsystem mit Enum

PROBLEM MIT PICKUP. Filterscript!

[Erledigt] [MySQL-Problem] MySQL R39-3 // Accountspeicherung

problem mit autohaus