FAHRZEUG - Fahrzeugschlüssel Infomation

Dr. Frauenarzt · 2. Oktober 2018

Code

ocmd:carkey(playerid)
{
	if(!IsLoggedIn(playerid))return 1;
	new string1[64], string2[64], string3[64], string4[64];
	if(PrivateVehicle[playerid][0][modelid] >= 400 && PrivateVehicle[playerid][0][modelid] <= 611) // Erster Slot
	{
		format(string1, sizeof(string1), "{FFFFFF}%s\n", VehicleName[PrivateVehicle[playerid][0][modelid]-400]);
	}
	else
	{
		format(string1, sizeof(string1), "{FFFFFF}- - -\n");
	}


	if(PrivateVehicle[playerid][1][modelid] >= 400 && PrivateVehicle[playerid][1][modelid] <= 611) // Zweiter Slot
	{
		format(string2, sizeof(string2), "{FFFFFF}%s\n", VehicleName[PrivateVehicle[playerid][1][modelid]-400]);
	}
	else
	{
		format(string2, sizeof(string2), "{FFFFFF}- - -\n");
	}


	if(PrivateVehicle[playerid][2][modelid] >= 400 && PrivateVehicle[playerid][2][modelid] <= 611) // Dritter Slot
	{
		format(string3, sizeof(string3), "{FFFFFF}%s\n", VehicleName[PrivateVehicle[playerid][2][modelid]-400]);
	}
	else
	{
		format(string3, sizeof(string3), "{FFFFFF}- - -\n");
	}


	if(PrivateVehicle[playerid][3][modelid] >= 400 && PrivateVehicle[playerid][3][modelid] <= 611) // Vierter Slot
	{
		format(string4, sizeof(string4), "{FFFFFF}%s\n", VehicleName[PrivateVehicle[playerid][3][modelid]-400]);
	}
	else
	{
		format(string4, sizeof(string4), "{FFFFFF}- - -\n");
	}
	strdel(DialogString, 0, sizeof(DialogString));
	format(DialogString, sizeof(DialogString), "%s%s%s%s", string1, string2, string3, string4);
	ShowPlayerDialog(playerid, DIALOG_CARKEYS, DIALOG_STYLE_LIST, "Deine Autoschlüssel", DialogString, "Auswählen", "Schließen");
	return 1;
}

Alles anzeigen

Hey,

Ich habe kurz eine Frage.
Wie mache ich am besten wen er ein Fahrzeug auswählt das eine Infomation kommt welche Farbe undso in Dialog.

Heißt er wählt den Schlüssel aus.
Es erscheint ein Dialog.

Fahrzeug:
Farbe:
Tank:
usw.

Wie mache ich das am besten?

Kaliber · 2. Oktober 2018

Schau mal hier rein: Schneller & strukturierter skripten

Das könnte dir helfen, dann kannst du dir einiges schenken, wie mehrere Strings, die isLoggedIn Abfrage in jedem Befehl und eine einzelne Auflistung, da solltest du eine Schleife verwenden.

Wie zum teufel, hast du modelid deklariert bzw wie verwendest du das da, so wird das nicht funktionieren.

Und zu deiner Frage, du kannst dann einfach listitem verwenden und dann weißt du, auf welches Fahrzeug er geklickt hat, nämlich:

PrivateVehicle[playerid][listitem][modelid]

Dr. Frauenarzt · 2. Oktober 2018

So habe ich das erstmal gestaltet um zu gucken ob das mit den ID´s klappt ja es klappt

Code

case DIALOG_CARKEYS:
		{
			if(response)
			{
				new id = listitem, string[128];
				if(PrivateVehicle[playerid][id][modelid] >= 400 && PrivateVehicle[playerid][id][modelid] <= 611)
				{
				    format(string, sizeof(string), "Du hast dein Fahrzeugschlüssel Nummer %d gewählt mit den Fahrzeug %s", id, VehicleName[PrivateVehicle[playerid][id][modelid]-400]);
					SendClientMessage(playerid, COLOR_WHITE, string);
					return 1;
				}
				else
				{
					SendClientMessage(playerid, COLOR_WHITE, "Du besitzt auf diesem Schlüssel kein Fahrzeug.");
					return 1;
				}
			}
		}

Alles anzeigen

Habe das nun so:

Code

case DIALOG_CARKEYS:
		{
			if(response)
			{
				new id = listitem, vehicleid, Float:CarHealth, CarHealthIni, string[128];
				if(PrivateVehicle[playerid][id][modelid] >= 400 && PrivateVehicle[playerid][id][modelid] <= 611)
				{
    				GetVehicleHealth(vehicleid, CarHealth);
					CarHealthIni = floatround((CarHealth - 250.0) / 7.5, floatround_floor);
	 				format(string, sizeof(string), "Allgemeines:\tBesitzer: %s (ID: %d)\n\t\tFahrgestellnummer: %d\n\t\tKennzeichen: %s\n\t\tModel: %s\n\t\tFarbe: %d / %d\n\nSonstiges:\tZustand: %d Prozent\n\t\tTank: %.1f L\n\t\tKilometerstand: %.1f km",
					PrivateVehicle[playerid][id][vehBesitzer], playerid, PrivateVehicle[playerid][id][vehFahrgestell], PrivateVehicle[playerid][id][vehNumberPlate], VehicleName[GetVehicleModel(vehicleid)-400], PrivateVehicle[playerid][id][vehColor1], PrivateVehicle[playerid][id][vehColor2], CarHealthIni,
					Vehicle[vehicleid][vehFuel], Vehicle[vehicleid][vehKMStand]);
					ShowPlayerDialog(playerid, DIALOG_FAHRZEUG, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
					return 1;
				}
				else
				{
					SendClientMessage(playerid, COLOR_WHITE, "Du besitzt auf diesem Schlüssel kein Fahrzeug.");
					return 1;
				}
			}
		}

Alles anzeigen

[debug] #0 00064f0c in public OnDialogResponse (playerid=0, dialogid=86, response=1, listitem=0, inputtext[]=@0072d2dc "Sultan") at C:\Users\Patrik\Desktop\Selfmade\gamemodes\German.pwn:6684

Kaliber · 2. Oktober 2018

Interessant wäre die vollständige Debug Meldung...

Da steht ja nicht einmal ein Fehler..

Dr. Frauenarzt · 2. Oktober 2018

Ups mein fehler sry

Code

[debug] Run time error 4: "Array index out of bounds"
[debug]  Attempted to read/write array element at negative index -400
[debug] AMX backtrace:
[debug] #0 00064f0c in public OnDialogResponse (playerid=0, dialogid=86, response=1, listitem=0, inputtext[]=@0072d2dc "Sultan") at C:\Users\Patrik\Desktop\Selfmade\gamemodes\German.pwn:6684

Kaliber · 2. Oktober 2018

Das liegt daran, dass deine Variable vehicleid immer 0 ist. Du setzt da keinen Wert.

Deshalb wenn GetVehicleModel(0) aufgerufen wird und da 0 keine gültige vehicleid ist, wird hier auch immer 0 zurückgegeben und 0-400 = -400 und dann greifst du auf diesen index zu, der natürlich nicht existiert.

Also vehicleid einen Wert zuweisen und sicher sein, dass das Auto auch existiert

Dr. Frauenarzt · 2. Oktober 2018

Das habe ich soweit verstanden

Wie soll ich vehicleid einen wert hinzufügen?

Kaliber · 2. Oktober 2018

Ja...sowas wie: vehicleid = PrivateVehicle[playerid][id][veh_ID] ;

Dr. Frauenarzt · 2. Oktober 2018

habe ich gesehn danke

Habe nun folgendes Problem wen ich auf Auswählen klicke:

Code

case DIALOG_FAHRZEUG:
		{
			if(response)
			{
			    new id = listitem, Float:pos[3], string[128];
			    new vehicleid = PrivateVehicle[playerid][id][vehID]; //6708
				if(PrivateVehicle[playerid][id][modelid] >= 400 && PrivateVehicle[playerid][id][modelid] <= 611)
   				{
					GetVehiclePos(PrivateVehicle[playerid][vehicleid][vehID], pos[0], pos[1], pos[2]);
					SetPlayerCheckpointEx(playerid, pos[0], pos[1], pos[2], 5.0, CP_FINDCAR, INVALID_CHECKPOINT_TYPE);
					format(string, sizeof(string), "{33CCFF}Dein Fahrzeug (%s) wurde auf der Karte rot markiert.", VehicleName[PrivateVehicle[playerid][vehicleid][modelid]-400]);
					SendClientMessage(playerid, COLOR_WHITE, string);
                }
				return 1;
			}
		}

Alles anzeigen

[debug] Run time error 4: "Array index out of bounds"
[debug] Attempted to read/write array element at negative index -1
[debug] AMX backtrace:
[debug] #0 00065344 in public OnDialogResponse (playerid=0, dialogid=87, response=1, listitem=-1, inputtext[]=@0072d3f4 "") at C:\Users\Patrik\Desktop\Selfmade\gamemodes\German.pwn:6708

Kaliber · 2. Oktober 2018

Wie haste es denn geschafft auf ein leeres nicht vorhandenes Feld zu klicken?

Kannst du mal ein Bild vom Dialog machen und zeigen, was du ausgewählt hast?

Dr. Frauenarzt · 2. Oktober 2018

https://picload.org/view/dcgiaorw/inkedsa-mp-044_li.jpg.html

"Auswählen"

Kaliber · 2. Oktober 2018

Das ist doch wieder ein völlig anderer Dialog, das ist doch kein DIALOG_LIST...da kannst du auch kein listitem nutzen.

Spoiler anzeigen

Du musst das listitem, was er zuvor ausgewählt hat z.B. in einer PVar speichern, dann kannst du das da wiederverwenden.

Dr. Frauenarzt · 2. Oktober 2018

Code

case DIALOG_CARKEYS:
		{
			if(response)
			{
				new id = listitem, Float:CarHealth, CarHealthIni, string[256];
				new vehicleid = PrivateVehicle[playerid][id][vehID];
				if(PrivateVehicle[playerid][id][modelid] >= 400 && PrivateVehicle[playerid][id][modelid] <= 611)
				{
    				GetVehicleHealth(vehicleid, CarHealth);
					CarHealthIni = floatround((CarHealth - 250.0) / 7.5, floatround_floor);
	 				format(string, sizeof(string), "Allgemeines:\tBesitzer: %s (ID: %d)\n\t\tFahrgestellnummer: %d\n\t\tKennzeichen: %s\n\t\tModel: %s\n\t\tFarbe: %d / %d\n\nSonstiges:\tZustand: %d Prozent\n\t\tTank: %.1f L\n\t\tKilometerstand: %.1f km",
					PrivateVehicle[playerid][id][vehBesitzer], playerid, PrivateVehicle[playerid][id][vehFahrgestell], PrivateVehicle[playerid][id][vehNumberPlate], VehicleName[GetVehicleModel(vehicleid)-400], PrivateVehicle[playerid][id][vehColor1], PrivateVehicle[playerid][id][vehColor2], CarHealthIni,
					Vehicle[vehicleid][vehFuel], Vehicle[vehicleid][vehKMStand]);
					ShowPlayerDialog(playerid, DIALOG_FAHRZEUG, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Auswählen", "Abbrechen");
					return 1;
				}
				else
				{
					SendClientMessage(playerid, COLOR_WHITE, "Du besitzt auf diesem Schlüssel kein Fahrzeug.");
					return 1;
				}
			}
		}
		case DIALOG_FAHRZEUG:
		{
			if(response)
			{
			    new id = listitem, Float:pos[3], string[128];
			    new vehicleid = PrivateVehicle[playerid][id][vehID];
				if(PrivateVehicle[playerid][id][modelid] >= 400 && PrivateVehicle[playerid][id][modelid] <= 611)
   				{
					GetVehiclePos(PrivateVehicle[playerid][vehicleid][vehID], pos[0], pos[1], pos[2]);
					SetPlayerCheckpointEx(playerid, pos[0], pos[1], pos[2], 5.0, CP_FINDCAR, INVALID_CHECKPOINT_TYPE);
					format(string, sizeof(string), "{33CCFF}Dein Fahrzeug (%s) wurde auf der Karte rot markiert.", VehicleName[PrivateVehicle[playerid][vehicleid][modelid]-400]);
					SendClientMessage(playerid, COLOR_WHITE, string);
                }
				return 1;
			}
		}

Alles anzeigen

Das sind die 2 Codes. Einmal wen er das Fahrzeug auswhält hat wen er dann auf auswählen klickt soll es geortet werden.

Sry bin heute schon neben mir und genervt von den mist

Kaliber · 2. Oktober 2018

Ja...oben im DIALOG_CARKEYS nutzt du z.B.:

SetPVarInt(playerid, "carkey_listitem", listitem);

und unten im DIALOG_FAHRZEUG sagst du:

new id = GetPVarInt(playerid, "carkey_listitem");
DeletePVar(playerid, "carkey_listitem");

Dr. Frauenarzt · 2. Oktober 2018

So das habe ich nun andere sachlage noch dann hab ich den müll endlich hinter mir..

CARINFO:

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], slot, vehicleid, string[256], Float:CarHealth, CarHealthIni;
	if(!(Spieler[playerid][pFraktion] == 1 || Spieler[playerid][pFraktion] == 2 || Spieler[playerid][pFraktion] == 3 || Spieler[playerid][pFraktion] == 13))return SendClientMessage(playerid, COLOR_WHITE, ERROR_FRAKTION);
	if(sscanf(params, "i", vehicleid))return SendClientMessage(playerid, COLOR_WHITE, "{3592D7}* Benutze {00CC00}/carinfo [vehicleid]");
	GetVehiclePos(vehicleid, Pos[0], Pos[1], Pos[2]);
    GetVehicleHealth(vehicleid, CarHealth);
	CarHealthIni = floatround((CarHealth - 250.0) / 7.5, floatround_floor);
    new VehicleOwner = getVehicleOwner(vehicleid);
    if(IsPlayerInRangeOfPoint(playerid, 30.0, Pos[0], Pos[1], Pos[2]))
	{
	    if((VehicleOwner = getVehicleOwner(vehicleid)) != -1)
		{
	 		format(string, sizeof(string), "Allgemeines:\tBesitzer: %s (ID: %d)\n\t\tFahrgestellnummer: %d\n\t\tKennzeichen: %s\n\t\tModel: %s\n\t\tFarbe: %d / %d\n\nSonstiges:\tZustand: %d Prozent\n\t\tTank: %.1f L\n\t\tKilometerstand: %.1f km",
			PrivateVehicle[VehicleOwner][slot][vehBesitzer], VehicleOwner, PrivateVehicle[VehicleOwner][slot][vehFahrgestell], PrivateVehicle[slot][vehicleid][vehNumberPlate], VehicleName[GetVehicleModel(vehicleid)-400], PrivateVehicle[VehicleOwner][slot][vehColor1], PrivateVehicle[VehicleOwner][slot][vehColor2], CarHealthIni,
			Vehicle[vehicleid][vehFuel], Vehicle[vehicleid][vehKMStand]);
			ShowPlayerDialog(playerid, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
		}
		else if((VehicleOwner = IsAFraktionsVeh(vehicleid)) != INVALID_PLAYER_ID)
		{
			format(string, sizeof(string), "Allgemeines:\tBesitzer: %s\n\t\tFahrgestellnummer: %d\n\t\tKennzeichen: %s\n\t\tModel: %s\n\t\tFarbe: %d / %d\n\nSonstiges:\tZustand: %d Prozent\n\t\tTank: %.1f L\n\t\tKilometerstand:  %.1f km",
			GetVehicleFraktionName(vehicleid), fVehicle[VehicleOwner][vehFahrgestell], fVehicle[VehicleOwner][vehNumberPlate], VehicleName[GetVehicleModel(vehicleid)-400], fVehicle[VehicleOwner][vehColor1], fVehicle[VehicleOwner][vehColor2], CarHealthIni, Vehicle[vehicleid][vehFuel], Vehicle[vehicleid][vehKMStand]);
			ShowPlayerDialog(playerid, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
		}
		else
		{
			SendClientMessage(playerid, COLOR_WHITE, "Dieses Fahrzeug hat keinen Besitzer.");
		}
    }
	return 1;
}

Alles anzeigen

Um festzustellen den Besitzer:

Code

stock getVehicleOwner(vehicleid)
{
	for(new i=0;i<GetMaxPlayers();i++)
	{
		for(new slot=0;slot<MAX_PLAYER_VEHICLES;slot++)
		{
			if(vehicleid == PrivateVehicle[i][slot][vehID])
			{
				return 1;
			}
        }
	}
	return -1;
}

Alles anzeigen

Also das heitß:

Wen ich /carinfo und die Fahrzeug ID eingebe kommt der Dialog. -> Das Ist schon soweit richtig.
Nur das Problem ist das es keine Werte ausgibt außer Tank und den Tank. Mehr sagt er mir nicht steht überall 0 oder garnichts

Kaliber · 2. Oktober 2018

Was ein Messi Code...

der stimmt vorne und hinten nicht.

slot wird nirgends zugewiesen.
(VehicleOwner = getVehicleOwner(vehicleid)) != -1 was glaubst du, machst du da? Ich meine es funktioniert, aber ich glaube nicht aus dem Grund, was du ursprünglich wolltest xD im Endeffekt kannst du auch einfach prüfen ob VehicleOwner == 1 ist
Das bringt uns zum nächsten Problem, VehicleOwner ist entweder 1 oder -1. Du glaubst aber, es entspricht der playerid vom Besitzer, was nicht der Fall ist

Dr. Frauenarzt · 2. Oktober 2018

Ja bin gerne eine Messi haha

Ich check nur mehr 0.

Also das mit -1 kacke da habe ich nun == 1 ersetzt

ZITAT:

slot wird nirgends zugewiesen. <- Fraglich ist was ich da zuweisen soll.
Das bringt uns zum nächsten Problem, VehicleOwner ist entweder 1 oder -1. Du glaubst aber, es entspricht der playerid vom Besitzer, was nicht der Fall ist <- Verstehe nicht ganz wo du damit hin willst

bin total heute schon überfordert

Kaliber · 2. Oktober 2018

C

stock IsStaatsFrak(playerid)
{
    switch(Spieler[playerid][pFraktion])
    {
        case 1,2,3,13: return 1;
    }
    return 0;
}
stock GetVehicleHealthIni(vehicleid)
{
    new Float:CarHealth;
    GetVehicleHealth(vehicleid, CarHealth);
    return floatround((CarHealth - 250.0) / 7.5, floatround_floor);
}
stock GetPrivateVehicleOwner(vehicleid)
{
    for(new i=GetPlayerPoolSize(),slot; i!=-1; i--)
    {
        for(slot=0; slot<MAX_PLAYER_VEHICLES; slot++)
        {
            if(vehicleid == PrivateVehicle[i][slot][vehID]) return i;
        }
    }
    return INVALID_PLAYER_ID;
}
stock GetPrivateVehicleOwnerSlot(vehicleid,i)
{
    for(new slot; slot<MAX_PLAYER_VEHICLES; slot++)
    {
        if(vehicleid == PrivateVehicle[i][slot][vehID]) return slot;
    }
    return -1;
}
ocmd:carinfo(playerid, params[])
{
    if(!IsStaatsFrak(playerid)) return SendClientMessage(playerid, COLOR_WHITE, ERROR_FRAKTION);
    new veh;
    if(sscanf(params, "i", veh)) return SendClientMessage(playerid, COLOR_WHITE, "{3592D7}* Benutze {00CC00}/carinfo [vehicleid]");
    if(veh > GetVehiclePoolSize() || veh <= 0) return SendClientMessage(playerid, COLOR_WHITE, "Ungültige Vehicleid!");
    new Float:Pos[3];
    GetVehiclePos(veh, Pos[0], Pos[1], Pos[2]);
    if(!IsPlayerInRangeOfPoint(playerid, 30.0, Pos[0], Pos[1], Pos[2])) return SendClientMessage(playerid, COLOR_WHITE, "Du befindest dich nicht an diesem Auto!");
    new string[256], CarHealthIni = GetVehicleHealthIni(veh),VehicleOwner=GetPrivateVehicleOwner(veh);
    if(VehicleOwner != INVALID_PLAYER_ID)
    {
        new slot = GetVehicleOwnerSlot(veh, VehicleOwner);
        format(string, sizeof(string), "Allgemeines:\tBesitzer: %s (ID: %d)\n\t\tFahrgestellnummer: %d\n\t\tKennzeichen: %s\n\t\tModel: %s\n\t\tFarbe: %d / %d\n\nSonstiges:\tZustand: %d Prozent\n\t\tTank: %.1f L\n\t\tKilometerstand: %.1f km",
        PrivateVehicle[VehicleOwner][slot][vehBesitzer], VehicleOwner, PrivateVehicle[VehicleOwner][slot][vehFahrgestell], PrivateVehicle[slot][veh][vehNumberPlate], VehicleName[GetVehicleModel(veh)-400], PrivateVehicle[VehicleOwner][slot][vehColor1], PrivateVehicle[VehicleOwner][slot][vehColor2], CarHealthIni,
        Vehicle[veh][vehFuel], Vehicle[veh][vehKMStand]);
        ShowPlayerDialog(playerid, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
    }
    else if((VehicleOwner = IsAFraktionsVeh(veh)) != INVALID_PLAYER_ID)
    {
        format(string, sizeof(string), "Allgemeines:\tBesitzer: %s\n\t\tFahrgestellnummer: %d\n\t\tKennzeichen: %s\n\t\tModel: %s\n\t\tFarbe: %d / %d\n\nSonstiges:\tZustand: %d Prozent\n\t\tTank: %.1f L\n\t\tKilometerstand:  %.1f km",
        GetVehicleFraktionName(veh), fVehicle[VehicleOwner][vehFahrgestell], fVehicle[VehicleOwner][vehNumberPlate], VehicleName[GetVehicleModel(veh)-400], fVehicle[VehicleOwner][vehColor1], fVehicle[VehicleOwner][vehColor2], CarHealthIni, Vehicle[veh][vehFuel], Vehicle[veh][vehKMStand]);
        ShowPlayerDialog(playerid, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
    }
    else
    {
        SendClientMessage(playerid, COLOR_WHITE, "Dieses Fahrzeug hat keinen Besitzer.");
    }
    return 1;
}

Alles anzeigen

So sollte das schon eher funktionieren, vorrausgesetzt IsAFraktionsVeh gibt dir eine eindeutige ID zurück und nicht nur 0 oder INVALID_PLAYER_ID.

Dr. Frauenarzt · 2. Oktober 2018

Code

stock IsAFraktionsVeh(vehicleid)
{
	for(new fV;fV<MAX_FVEHS;fV++)
	{
		if(vehicleid == fVehicle[fV][vehID])return fV;
	}
	return INVALID_VEHICLE_ID;
}

Wen ich /carinfo eingebe und die ID´s von den Privaten autos angebe steht das es keine Besitzer gibt von den Fahrzeug obwohl es einen gibt

Kaliber · 2. Oktober 2018

sry natürlich muss da ein != invalid hin.

#Handy