[FAHRZEUGE] /carinfo

Dr. Frauenarzt · 8. Juli 2018

Guten Tag,

Ich habe eine frage ich möchte gerne nach den vehicleids abrufen könn wem das Fahrzeug gehört usw aber wen ich /carinfo die vehicleid eingebe kommt unknow command.

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], Float:CarHealth, string[128];
	new vehicleid = GetPlayerNearestCarID(playerid, 5);
	GetVehicleHealth(vehicleid, CarHealth);
	if(!PlayerIsAtAnyCar(playerid, 5))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	if(IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du darfst dich nicht in einem Fahrzeug befinden.");
	for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++)
	{
		for(new i=0;i<GetMaxPlayers();i++)
		{
			if(IsValidVehicle(PrivateVehicle[i][slot][vehID]))
			{
				if(vehicleid == PrivateVehicle[i][slot][vehID])
				{
                    if(GetVehiclePos(PrivateVehicle[i][vehicleid][vehID], Pos[0], Pos[1], Pos[2]) && IsPlayerInRangeOfPoint(i, 5.0, Pos[0], Pos[1], Pos[2]))
		    		{
		                if(sscanf(params, "i", vehicleid))return SendClientMessage(playerid, COLOR_WHITE, "{3592D7}* Benutze {00CC00}/carinfo [vehicleid]");
		                {
		              		format(string, sizeof(string), "Allgemeines:\tBesitzer:\t%s", PrivateVehicle[i][vehicleid][vehBesitzer]);
							ShowPlayerDialog(i, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
		                }
                    }
                }
            }
        }
	}
	return 1;
}

Alles anzeigen

Dies gibt mir der Log aus:

Code

[debug] Run time error 4: "Array index out of bounds"
[debug]  Attempted to read/write array element at index 19 in array of size 4
[debug] AMX backtrace:
[debug] #0 00023f30 in public ocmd_carinfo (0, 6760732) from LRS.amx
[debug] #1 native CallLocalFunction () from samp-server.exe
[debug] #2 000005c8 in public OnPlayerCommandText (0, 6760684) from LRS.amx

//E:

Befehl Aktualisiert.

Das Problem ist es funktioniert immer noch nicht.
Ich arbeite die ganze dran und finde den fehler einfach nicht

@Jeffry

Hast du eventuell eine Lösung?

iTzTrappY · 10. Juli 2018

#Edit

versuche es mal so:

ocmd:carinfo(playerid, params[])
{
new Float:Pos[3], Float:CarHealth, string[128];
new vehicleid = GetPlayerNearestCarID(playerid, 5);
GetVehicleHealth(vehicleid, CarHealth);
if(!PlayerIsAtAnyCar(playerid, 5))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich nicht in der Nähe eines Fahrzeuges.");
if(IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du darfst dich nicht in einem Fahrzeug befinden.");
for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++)
{
for(new i=0;i<GetMaxPlayers();i++)
{
if(IsValidVehicle(PrivateVehicle[i][slot][vehID]))
{
if(vehicleid == PrivateVehicle[i][slot][vehID])
{
GetVehiclePos(PrivateVehicle[i][vehicleid][vehID], Pos[0], Pos[1], Pos[2]);
if(IsPlayerInRangeOfPoint(i, 5.0, Pos[0], Pos[1], Pos[2]))
{
if(sscanf(params, "i", vehicleid))return SendClientMessage(playerid, COLOR_WHITE, "{3592D7}* Benutze {00CC00}/carinfo [vehicleid]");
{
format(string, sizeof(string), "Allgemeines:\tBesitzer:\t%s", PrivateVehicle[i][slot][vehBesitzer]);
ShowPlayerDialog(i, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
}
}
}
}
}
}
return 1;
}

Dr. Frauenarzt · 10. Juli 2018

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], Float:CarHealth, string[128];
	new vehicleid = GetPlayerNearestCarID(playerid, 5);
	GetVehicleHealth(vehicleid, CarHealth);
	if(!PlayerIsAtAnyCar(playerid, 5))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	if(IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du darfst dich nicht in einem Fahrzeug befinden.");
	for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++)
	{
		for(new i=0;i<GetMaxPlayers();i++)
		{
			if(IsValidVehicle(PrivateVehicle[i][slot][vehID]))
			{
				if(vehicleid == PrivateVehicle[i][slot][vehID])
				{
					GetVehiclePos(PrivateVehicle[i][vehicleid][vehID], Pos[0], Pos[1], Pos[2]);
					if(IsPlayerInRangeOfPoint(i, 5.0, Pos[0], Pos[1], Pos[2]))
					{
						if(sscanf(params, "i", vehicleid))return SendClientMessage(playerid, COLOR_WHITE, "{3592D7}* Benutze {00CC00}/carinfo [vehicleid]");
						{
							format(string, sizeof(string), "Allgemeines:\tBesitzer:\t%s", PrivateVehicle[i][slot][vehBesitzer]);
							ShowPlayerDialog(i, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
						}
					}
				}
			}
		}
	}
	return 1;
}

Alles anzeigen

Wen ich jetzt /carinfo 20 mache passiert nichts.
Wen ich hier /carinfo 20 eingebe passiert garnichts.

Weder Unknow command weder irgendwas im log

Wen ich nur /carinfo eingebe kommt unknow command -> Wen ich das nur so eingebe spuckt er mir das aus:

[debug] AMX backtrace:
[debug] #0 00026c50 in public ocmd_carinfo (0, 72407384) from LRS.amx
[debug] #1 native CallLocalFunction () from samp-server.exe
[debug] #2 00000650 in public OnPlayerCommandText (0, 72407348) from LRS.amx

iTzTrappY · 10. Juli 2018

Code

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


ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], Float:CarHealth, string[128];
	new vehicleid = GetPlayerNearestCarID(playerid, 5);
	GetVehicleHealth(vehicleid, CarHealth);
	if(!PlayerIsAtAnyCar(playerid, 5))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	if(IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du darfst dich nicht in einem Fahrzeug befinden.");
	for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++) {
        GetVehiclePos(vehicleid, Pos[0], Pos[1], Pos[2]);
        new VehicleOwner = getVehicleOwner(vehicleid);
        if(IsPlayerInRangeOfPoint(playerid, 5.0, Pos[0], Pos[1], Pos[2])) {
            format(string, sizeof(string), "Allgemeines:\tBesitzer:\t%s", PrivateVehicle[i][slot][vehBesitzer]);
			ShowPlayerDialog(i, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
			return 1;
		}
	}
	return 1;
}

Alles anzeigen

KnoLe_x3 · 10. Juli 2018

Zitat von iTzTrappY

Code

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


ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], Float:CarHealth, string[128];
	new vehicleid = GetPlayerNearestCarID(playerid, 5);
	GetVehicleHealth(vehicleid, CarHealth);
	if(!PlayerIsAtAnyCar(playerid, 5))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	if(IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du darfst dich nicht in einem Fahrzeug befinden.");
	for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++) {
        GetVehiclePos(vehicleid, Pos[0], Pos[1], Pos[2]);
        new VehicleOwner = getVehicleOwner(vehicleid);
        if(IsPlayerInRangeOfPoint(playerid, 5.0, Pos[0], Pos[1], Pos[2])) {
            format(string, sizeof(string), "Allgemeines:\tBesitzer:\t%s", PrivateVehicle[i][slot][vehBesitzer]);
			ShowPlayerDialog(i, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
			return 1;
		}
	}
	return 1;
}

Alles anzeigen

und wenn jetzt bei getVehicleOwner(vehicleid); in der Schleife kein passender Eintrag gefunden wird, dann wird der Index 1 genommen.

iTzTrappY · 10. Juli 2018

Danke, ist noch früh am morgen.

@Dr.Frauenarzt
hab den Befehl nochmal bearbeitet

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], string[128], vehicleid;
	if(sscanf(parama, "i", vehicleid))return SendClientMessage(playerid, -1, "/carinfo [vehicleid]");
	if(!PlayerIsAtAnyCar(playerid, 5))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	if(IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du darfst dich nicht in einem Fahrzeug befinden.");
	for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++) {
        GetVehiclePos(vehicleid, Pos[0], Pos[1], Pos[2]);
        new VehicleOwner = getVehicleOwner(vehicleid);
        if(VehicleOwner == -1)return SendClientMessage(playerid, COLOR_GREY, "Das Fahrzeug hat keinen besitzer.");
        if(IsPlayerInRangeOfPoint(playerid, 5.0, Pos[0], Pos[1], Pos[2])) {
            format(string, sizeof(string), "Allgemeines:\tBesitzer:\t%s", PrivateVehicle[i][slot][vehBesitzer]);
			ShowPlayerDialog(i, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
			return 1;
		}
	}
	return 1;
}

Alles anzeigen

KnoLe_x3 · 10. Juli 2018

Zitat von iTzTrappY

Danke, ist noch früh am morgen.

@Dr.Frauenarzt
hab den Befehl nochmal bearbeitet

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], string[128], vehicleid;
	if(sscanf(parama, "i", vehicleid))return SendClientMessage(playerid, -1, "/carinfo [vehicleid]");
	if(!PlayerIsAtAnyCar(playerid, 5))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	if(IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du darfst dich nicht in einem Fahrzeug befinden.");
	for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++) {
        GetVehiclePos(vehicleid, Pos[0], Pos[1], Pos[2]);
        new VehicleOwner = getVehicleOwner(vehicleid);
        if(VehicleOwner == -1)return SendClientMessage(playerid, COLOR_GREY, "Das Fahrzeug hat keinen besitzer.");
        if(IsPlayerInRangeOfPoint(playerid, 5.0, Pos[0], Pos[1], Pos[2])) {
            format(string, sizeof(string), "Allgemeines:\tBesitzer:\t%s", PrivateVehicle[i][slot][vehBesitzer]);
			ShowPlayerDialog(i, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
			return 1;
		}
	}
	return 1;
}

Alles anzeigen

dann musst du auch den stock anpassen:
stock getVehicleOwner(vehicleid) {
for(new i=0;i<GetMaxPlayers();i++) {
if(vehicleid == PrivateVehicle[i][slot][vehID]) {
return i;
}
}
return -1;
}

iTzTrappY · 10. Juli 2018

Zitat von KnoLe_x3

dann musst du auch den stock anpassen:stock getVehicleOwner(vehicleid) { for(new i=0;i<GetMaxPlayers();i++) { if(vehicleid == PrivateVehicle[i][slot][vehID]) { return i; } } return -1;}

Ja danke habs vergessen, ich bin mit Frauenarzt im Teamspeak

Dr. Frauenarzt · 10. Juli 2018

Danke für eure tatkräftige Unterstützung.

@iTzTrappY
@KnoLe_x3

//E:

Kurze Frage.

Mann kann nur die Infomation von sein eigenen auto sehn?
Möchte das so machen das die Beamten sehen könn wem das Fahrzeug gehört ect.

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], string[256], Float:CarHealth, CarHealthIni, vehicleid;
	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]");
	if(!PlayerIsAtAnyCar(playerid, 30.0))return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	GetVehiclePos(vehicleid, Pos[0], Pos[1], Pos[2]);
	for(new slot = 0;slot<MAX_PLAYER_VEHICLES;slot++) {
        GetVehicleHealth(vehicleid, CarHealth);
		CarHealthIni = floatround((CarHealth - 250.0) / 7.5, floatround_floor);
        new VehicleOwner = getVehicleOwner(vehicleid, slot);
        if(VehicleOwner == -1)return SendClientMessage(playerid, COLOR_GREY, "Das Fahrzeug hat keinen besitzer.");
        if(IsPlayerInRangeOfPoint(playerid, 5.0, Pos[0], Pos[1], Pos[2])) {
            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][slot][vehBesitzer], playerid, PrivateVehicle[playerid][slot][vehFahrgestell], PrivateVehicle[playerid][slot][vehNumberPlate], VehicleName[GetVehicleModel(vehicleid)-400], PrivateVehicle[playerid][slot][vehColor1], PrivateVehicle[playerid][slot][vehColor2], CarHealthIni, Vehicle[vehicleid][vehFuel], Vehicle[vehicleid][vehKMStand]);
			ShowPlayerDialog(playerid, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
			return 1;
        }
    }
	return 1;
}

Alles anzeigen

Douq · 10. Juli 2018

Also ich glaube der Code, der uns zurzeit zur Verfügung steht ist nicht aktuell? Angenommen die Funktion getVehicleOwner() sieht folgend aus:

Code

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

Kann die Schleife in Zeile 8 weg und die Variable slot deklariert werden. Dann nur noch die erste Dimension des Arrays PrivateVehicle von playerid in vehicleOwner umändern und dann sollte das kein Problem mehr darstellen.

Natürlich muss noch ein Check rein, dass der Spieler das Fahrzeug sehen kann, sofern es sein eigenes ist (sprich die Abfrage 4 nicht betrachten, wenn der vehicleOwner der playerID entspricht).

Dr. Frauenarzt · 10. Juli 2018

Der Stock sieht so aus:

Code

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

//E:

Meinst du so?

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], slot, string[256], Float:CarHealth, CarHealthIni, vehicleid;
	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]");
	if(!PlayerIsAtAnyCar(playerid, 30.0))return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Du befindest dich nicht in der Nähe eines Fahrzeuges.");
	GetVehiclePos(vehicleid, Pos[0], Pos[1], Pos[2]);
    GetVehicleHealth(vehicleid, CarHealth);
	CarHealthIni = floatround((CarHealth - 250.0) / 7.5, floatround_floor);
    new VehicleOwner = getVehicleOwner(vehicleid, slot);
    if(VehicleOwner == -1)return SendClientMessage(playerid, COLOR_GREY, "Das Fahrzeug hat keinen besitzer.");
    if(IsPlayerInRangeOfPoint(playerid, 30.0, Pos[0], Pos[1], Pos[2])) {
        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], playerid, PrivateVehicle[VehicleOwner][slot][vehFahrgestell], PrivateVehicle[VehicleOwner][slot][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", "");
		return 1;
    }
	return 1;
}

Alles anzeigen

Douq · 10. Juli 2018

Dann verwende mal die Funktion getVehicleOwner() die ich oben genannt habe - funktioniert's dann?

Die Abfrage in Zeile 4 musst du noch verschieben - sofern dieser Befehl auch für Spieler bei ihrem eigenen Fahrzeug gehen soll. Auch ist die Funktion PlayerIsAtAnyCar nicht sinnvoll, da man bei carinfo scheinbar eine vehicleID angeben muss. Da ergibt es mehr Sinn zu überprüfen, ob der Spieler im Radius des Fahrzeuges ist (was du bereits in Zeile 12 machst, deswegen kann die Abfrage raus).

Dr. Frauenarzt · 10. Juli 2018

Funktioniert danke,

Nein Dies sollte dienen für die Polizei, genauso wie die Ordnungsamlter um nach zu vollziehen wen das Fahrzeug gehört.

@Douq

Ich habe mal eine frage.

Ich wollte das noch hinzufügen das man die Fraktionen auslesen kann das heißt wen man ein Fraktions Fahrzeug "Sieht" oder "Mal eine ID" eingibt das da auch der Dialog kommt.

Er sagt mir immer das das Fahrzeug kein Besitzer hat. "Also die Fraktions Fahrzeuge" die Privaten Fahrzeugen klappen

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], slot, string[256], Float:CarHealth, CarHealthIni, vehicleid;
	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, slot);
    if(VehicleOwner == -1)return SendClientMessage(playerid, COLOR_GREY, "Das Fahrzeug hat keinen besitzer.");
    if(IsPlayerInRangeOfPoint(playerid, 30.0, Pos[0], Pos[1], Pos[2])) {
        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[VehicleOwner][slot][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", "");
		return 1;
    }
    new FVehicleOwner = IsAFraktionsVeh(vehicleid);
    if(IsPlayerInRangeOfPoint(playerid, 30.0, Pos[0], Pos[1], Pos[2])) {
        format(string, sizeof(string), "Allgemeines:\tBesitzer: %s",
        GetFraktionName(Spieler[FVehicleOwner][pFraktion]));
        ShowPlayerDialog(playerid, DIALOG_CARINFO, DIALOG_STYLE_MSGBOX, "Fahrzeug Infomationen", string, "Schließen", "");
        return 1;
    }
	return 1;
}

Alles anzeigen

Code

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

Douq · 11. Juli 2018

Da bieten sich verschiedene Konstruktionen an. Bei der Variation ist es ja derzeit so, dass du den Code ab Zeile 10 nicht mehr weitergehst, da ein Fraktionsfahrzeug nach der Funktion getVehicleOwner == -1 ist. Es bietet sich an, genau an dieser Stelle auf das return zu verzichten, sondern lieber solch eine Konstruktion zu versuchen:

Code

if((vehicleOwner = getVehicleOwner(vehicleid, slot)) != -1) {
// Spielerfahrzeugkram
}
else if((vehicleOwner = IsAFraktionsVeh(vehicleid)) != INVALID_PLAYER_ID) {
// Fraktionskram
}
else {
// Fehlermeldung, weder Spielerfahrzeug noch Fraktionsfahrzeug
}

Eventuell ergibt es in Hinblick der Ressourcennutzung eher Sinn, zuerst das auf Fraktionsfahrzeug und danach aufs Spielerfahrzeug zu überprüfen - aber Optimierungen können sich da sowieso vornehmen ..

Dr. Frauenarzt · 11. Juli 2018

So wie das jetz beschrieben ist macht das ganze auch meiner Meinung auch Sinn.

Nun ja wir schauen mal was sich da machen lässt

//E:

Code

stock GetVehicleFraktionName(vehicleid)
{
    new string[50];
	switch(fVehicle[vehicleid][vehFraktion])
    {
		case 0: string = "Zivilist";
		case 1: string = "LSPD";
		case 2: string = "LVPD";
		case 3: string = "LVPD";
		case 4: string = "LSMD";
		case 5: string = "Russen M.";
		case 6: string = "Yakuza";
		case 7: string = "Regierung";
		case 8: string = "Hitman";
		case 9: string = "San News";
	}
	return string;
}

Alles anzeigen

Kurze frage wie kann ich das am besten machen wen er /carinfo macht das der Name ausgelesen wird?

Komisch ist wen ich das so habe wird mir nur Zivilist angezeigt.

Kaliber · 11. Juli 2018

Zitat von Dr. Frauenarzt

Komisch ist wen ich das so habe wird mir nur Zivilist angezeigt.

Kannst du uns zeigen, wie du die Funktion aufrufst?

Und wie du den Fahrzeugen die Fraktion zuweist wäre auch interessant

Dr. Frauenarzt · 11. Juli 2018

Also es sieht nun so aus also /carinfo:

Code

ocmd:carinfo(playerid, params[])
{
	new Float:Pos[3], slot, string[256], Float:CarHealth, CarHealthIni, vehicleid;
	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, slot);
    if(IsPlayerInRangeOfPoint(playerid, 30.0, Pos[0], Pos[1], Pos[2]))
	{
	    if((VehicleOwner = getVehicleOwner(vehicleid, slot)) != -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[VehicleOwner][slot][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

vehFraktion werden nach nummern eingefragen.

Das heißt wen ich jetzt ein Fraktions Fahrzeug erstelle für die Fraktions 1

gebe ich natürlich 1 an.
Und natürlich 1 ist z.b LSPD

Kaliber · 11. Juli 2018

Zitat von Dr. Frauenarzt

vehFraktion werden nach nummern eingefragen.

Das würde ich gerne mal sehen, da ich vermute, dass dort der Fehler liegt

Zur Überprüfung, kannst du ja auch einfach mal mit printf dir aus Spaß eine Fahrzeug Fraktion ausgeben lassen um zu testen, ob es denn funktioniert, aber ich denke nicht

Dr. Frauenarzt · 11. Juli 2018

Spoiler anzeigen

Code

stock CreateFraktionsVehicle(frakID, vehmodel, Float:posx, Float:posy, Float:posz, Float:posa, preis, neon, color1, color2, siren)
{
	new Query[512];
	for(new fV;fV<MAX_FVEHS;fV++)
	{
		if(fVehicle[fV][modelid] == 0)
		{
			fVehicle[fV][vehFraktion] = frakID;
			fVehicle[fV][modelid] = vehmodel;
			fVehicle[fV][vehPosX] = posx;
			fVehicle[fV][vehPosY] = posy;
			fVehicle[fV][vehPosZ] = posz;
			fVehicle[fV][vehPosA] = posa;
			fVehicle[fV][vehInterior] = 0;
			fVehicle[fV][vehWorld] = 0;
			fVehicle[fV][vehColor1] = color1;
			fVehicle[fV][vehColor2] = color2;
			fVehicle[fV][vehPaintjob] = -1;
			fVehicle[fV][vehSpoiler] = 0;
			fVehicle[fV][vehHood] = 0;
			fVehicle[fV][vehRoof] = 0;
			fVehicle[fV][vehSideskirt] = 0;
			fVehicle[fV][vehLamps] = 0;
			fVehicle[fV][vehNitro] = 0;
			fVehicle[fV][vehExhaust] = 0;
			fVehicle[fV][vehWheels] = 0;
			fVehicle[fV][vehStereo] = 0;
			fVehicle[fV][vehHydraulics] = 0;
			fVehicle[fV][vehFrontBumper] = 0;
			fVehicle[fV][vehRearBumper] = 0;
			fVehicle[fV][vehVentRight] = 0;
			fVehicle[fV][vehVentLeft] = 0;
			fVehicle[fV][vehPreis] = preis;
			fVehicle[fV][vehHealth] = 1000.0;
			fVehicle[fV][vehNeonID] = neon;
			fVehicle[fV][vehTankKM] = 0;
			fVehicle[fV][vehKMStandCount] = 0;
			fVehicle[fV][vehKMStand] = 0;
			fVehicle[fV][vehSiren] = siren;
			fVehicle[fV][vehFahrgestell] = randomex(10000, 99999);
			fVehicle[fV][vehID] = CreateVehicleEx(fVehicle[fV][modelid], fVehicle[fV][vehPosX], fVehicle[fV][vehPosY], fVehicle[fV][vehPosZ], fVehicle[fV][vehPosA], fVehicle[fV][vehColor1], fVehicle[fV][vehColor2], -1, false, fVehicle[fV][vehSiren]);


			format(fVehicle[fV][vehNumberPlate], 32, "%s 1%d", GetFraktionNumberPlate(fVehicle[fV][vehFraktion]), randomex(10000, 99999));
			SetVehicleNumberPlate(fVehicle[fV][vehID], fVehicle[fV][vehNumberPlate]);
			SetVehicleHealth(fVehicle[fV][vehID], fVehicle[fV][vehHealth]);
			mysql_format(MySqlConnection, Query, sizeof(Query), "INSERT INTO `server_fvehicles` (`vehFraktion`, `modelid`, `vehPosX`, `vehPosY`, `vehPosZ`, `vehPosA`, `vehInterior`, `vehWorld`, `vehHealth`, `vehNumberPlate`, `vehPaintjob`, `vehPreis`, `vehColor1`, `vehColor2`, `vehNeonID`, `vehFuel`, `vehSiren`, `vehFahrgestell`) VALUES ('%d', '%d', '%f', '%f', '%f', '%f', '%d', '%d', '%f', '%s', '%d', '%d', '%d', '%d', '%d', '%f', '%d', '%d')",
			fVehicle[fV][vehFraktion], fVehicle[fV][modelid], fVehicle[fV][vehPosX], fVehicle[fV][vehPosY], fVehicle[fV][vehPosZ], fVehicle[fV][vehPosA], fVehicle[fV][vehInterior], fVehicle[fV][vehWorld], fVehicle[fV][vehHealth], fVehicle[fV][vehNumberPlate], fVehicle[fV][vehPaintjob], fVehicle[fV][vehPreis], fVehicle[fV][vehColor1],
			fVehicle[fV][vehColor2], fVehicle[fV][vehNeonID], VehicleData[vehmodel-400][vehMaxTank], fVehicle[fV][vehSiren], fVehicle[fV][vehFahrgestell]);
			mysql_tquery(MySqlConnection, Query, "insertDB", "d", fV);
			SetVehicleToRespawn(fVehicle[fV][vehID]);
			Vehicle[fVehicle[fV][vehID]][vehFuel] = VehicleData[vehmodel-400][vehMaxTank];
			singleSaveFVehicle(fV);
			return 1;
		}
	}
	return 1;
}

Alles anzeigen

[FAHRZEUGE] /carinfo

Beitrag von iTzTrappY (10. Juli 2018)

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen

Fraktion Auto Parken bugt /fpark!

Floatberechnungen drehen durch