Fahrzeugsystem - Tank & Kilometerstand

malE · 17. Januar 2016

Hallo,

habe jeweils ein Problem mit meiner Tank & Kilometerstand Funktion.
An meiner Tank Funktion wird die Tank-Menge der Privatfahrzeuge zwar in die Variable ausgelesen, aber im TextDraw ( Tacho ) wird mir ein leerer Tank gezeigt.
Sowie bei meinem /motor Befehl, man kann logischer Weise nämlich nur den Motor starten mit einem gefüllten Tank, jedoch steht auch dort, dass mein Tank leer wäre.
In der MySQL Spalte des Fahrzeugs steht jedoch 40, also 40 Liter im Tank.
Habe mir es auch schon printen lassen, dort steht auch in der Variable 40.
Jedoch tritt dieses Problem bei den Polizeifahrzeugen nicht auf, die separat geladen werden.

Problem: Tank

Code

forward OnPlayerVehiclesLoad(playerid);
public OnPlayerVehiclesLoad(playerid)
{
	new  engine, lights, alarm, doors, bonnet, boot, objective;
	new rows;
	rows = cache_num_rows();
	for(new i=0; i < rows; i++)
	{
		VehicleInfo[i][v_Police] = bool:cache_get_field_content_int(i, "police", handle);
		if(VehicleInfo[i][v_Police] == true)continue;
		VehicleInfo[i][v_ID] = cache_get_field_content_int(i, "id", handle);
		VehicleInfo[i][v_Besitzer] = cache_get_field_content_int(i, "besitzer", handle);
		VehicleInfo[i][v_X] = cache_get_field_content_float(i, "x", handle), VehicleInfo[i][v_X] = VehicleInfo[i][v_KMX];
		VehicleInfo[i][v_Y] = cache_get_field_content_float(i, "y", handle), VehicleInfo[i][v_Y] = VehicleInfo[i][v_KMY];
		VehicleInfo[i][v_Z] = cache_get_field_content_float(i, "z", handle), VehicleInfo[i][v_Z] = VehicleInfo[i][v_KMZ];
		VehicleInfo[i][v_R] = cache_get_field_content_float(i, "r", handle);
		VehicleInfo[i][v_Health] = cache_get_field_content_float(i, "health", handle);
		VehicleInfo[i][v_Tank] = cache_get_field_content_int(i, "liter", handle);
		VehicleInfo[i][v_Sirene] = bool:cache_get_field_content_int(i, "siren", handle);
		VehicleInfo[i][v_Model] = cache_get_field_content_int(i, "model", handle);
		VehicleInfo[i][v_Farbe] = cache_get_field_content_int(i, "farbe", handle);
		VehicleInfo[i][v_Farbe2] = cache_get_field_content_int(i, "farbe2", handle);
		VehicleInfo[i][v_Kilometer] = cache_get_field_content_int(i, "kmstand", handle);
		VehicleInfo[i][v_Bombe] = bool:cache_get_field_content_int(i, "bombe", handle);
		VehicleInfo[i][v_WeelsKugelsicher] = bool:cache_get_field_content_int(i, "weelskugelsicher", handle);
		VehicleInfo[i][v_Panzerglas] = bool:cache_get_field_content_int(i, "panzerglas", handle);
		//printf("Health: %0.2f | Tank: %d | Sirene: %d | Farbe: %d | Farbe: %d | Kilometer: %d | Bombe: %d | Kugelsicher: %d | Panzerglas: %d",
		//VehicleInfo[i][v_Health], VehicleInfo[i][v_Tank], VehicleInfo[i][v_Sirene], VehicleInfo[i][v_Farbe], VehicleInfo[i][v_Farbe2], VehicleInfo[i][v_Kilometer], VehicleInfo[i][v_Bombe], VehicleInfo[i][v_WeelsKugelsicher], VehicleInfo[i][v_Panzerglas]);
		if(VehicleInfo[i][v_Model] >= 400 && VehicleInfo[i][v_Model] <= 611)
		{
			VehicleInfo[i][v_Objekt] = CreateVehicle(VehicleInfo[i][v_Model], VehicleInfo[i][v_X], VehicleInfo [i][v_Y], VehicleInfo[i][v_Z], VehicleInfo[i][v_R], VehicleInfo[i][v_Farbe], VehicleInfo[i][v_Farbe2], -1, VehicleInfo[i][v_Sirene]);
			if(VehicleInfo[i][v_Health] > 250.99) SetVehicleHealth(VehicleInfo[i][v_Objekt], VehicleInfo[i][v_Health]);
			else SetVehicleHealth(VehicleInfo[i][v_Objekt], 1000);
			GetVehicleParamsEx(VehicleInfo[i][v_Objekt], engine, lights, alarm, doors, bonnet, boot, objective);
			SetVehicleParamsEx(VehicleInfo[i][v_Objekt], engine, lights, alarm, 1, bonnet, boot, objective);
		}
		else printf("[FEHLER] Fahrzeug-Model %d existiert nicht (DB-ID: %d).", VehicleInfo[i][v_Model], i);
	}
	return 1;
}

Alles anzeigen

Code

forward TachoTimer();
public TachoTimer()
{
	for(new i=0; i < MAX_PLAYERS; i++)
	{
	    new vID = GetPlayerVehicleID(i);
	    if(!IsPlayerConnected(i) || PlayerInfo[i][p_LoggedIn] != true)continue;
	    if(!IsPlayerInAnyVehicle(i))continue;
	    new tacho[256], Float:hp;
	    GetVehicleHealth(vID, hp);
	    VehicleInfo[vID][v_Kilometer] += e_GetDistance(vID);
		format(tacho, sizeof(tacho), "%s~n~Geschwindigkeit: %dkm/h~n~Kilometerstand: %d.%dkm~n~Treibstoff: %dl~n~Schaden: %0.2f", VehicleNames[GetVehicleModel(vID)-400], GetPlayerSpeed(i), VehicleInfo[vID][v_Kilometer]/1000, VehicleInfo[vID][v_Kilometer]%1000, VehicleInfo[vID][v_Tank], hp);
	    PlayerTextDrawSetString(i, Tacho[i][0], tacho);
		if(VehicleEngineOn(vID))PlayerTextDrawSetString(i, Tacho[i][2], "~g~Motor");
		else PlayerTextDrawSetString(i, Tacho[i][2], "~w~Motor");
		if(VehicleLightsOn(vID))PlayerTextDrawSetString(i, Tacho[i][3], "~g~Licht");
		else PlayerTextDrawSetString(i, Tacho[i][3], "~w~Licht");
		if(VehicleDoorLocked(vID))PlayerTextDrawSetString(i, Tacho[i][4], "~g~Tueren");
		else PlayerTextDrawSetString(i, Tacho[i][4], "~w~Tueren");
		GetVehiclePos(vID, VehicleInfo[vID][v_KMX], VehicleInfo[vID][v_KMY], VehicleInfo[vID][v_KMZ]);
	}
	return 1;
}

Alles anzeigen

Dann kommen wir mal zum zweiten Problem.
Hierbei wird erst gar nicht die Kilometer-Strecke gespeichert.
Aber im Tacho steht eine Strecke, und zwar die vom 0 Punkt bis zur Position des Fahrzeuges.
Wenn ihr versteht was ich meine.

Problem: Kilometerstand

Tabelle

Das sollte alles notwendige sein.
Ich hoffe ihr könnt mir helfen.

Jeffry · 18. Januar 2016

Das Problem ist, dass du auf den Index von VehicleInfo mit der vehicleid drauf zu greifst, das geht aber nicht, da die vehicleid nicht der Index-ID entspricht.
Erstelle dazu eine folgende Funktion, um aus der vehicleid den Index zu bekommen:
stock getVehIndex(vehicleid)
{
for(new i = 0; i < sizeof(VehicleInfo); i++)
{
if(VehicleInfo[i][v_Objekt] == vehicleid)
{
return i;
}
}
return -1;
}

Dann kannst du die TachoTimer Funktion so schreiben:
forward TachoTimer();
public TachoTimer()
{
for(new i=0; i < MAX_PLAYERS; i++)
{
new vID = GetPlayerVehicleID(i);
if(!IsPlayerConnected(i) || PlayerInfo[i][p_LoggedIn] != true)continue;
if(!IsPlayerInAnyVehicle(i))continue;
new idx = getVehIndex(vID);
if(idx == -1) continue;
new tacho[256], Float:hp;
GetVehicleHealth(vID, hp);
VehicleInfo[idx][v_Kilometer] += e_GetDistance(vID);
format(tacho, sizeof(tacho), "%s~n~Geschwindigkeit: %dkm/h~n~Kilometerstand: %d.%dkm~n~Treibstoff: %dl~n~Schaden: %0.2f", VehicleNames[GetVehicleModel(vID)-400], GetPlayerSpeed(i), VehicleInfo[idx][v_Kilometer]/1000, VehicleInfo[idx][v_Kilometer]%1000, VehicleInfo[idx][v_Tank], hp);
PlayerTextDrawSetString(i, Tacho[i][0], tacho);
if(VehicleEngineOn(vID))PlayerTextDrawSetString(i, Tacho[i][2], "~g~Motor");
else PlayerTextDrawSetString(i, Tacho[i][2], "~w~Motor");
if(VehicleLightsOn(vID))PlayerTextDrawSetString(i, Tacho[i][3], "~g~Licht");
else PlayerTextDrawSetString(i, Tacho[i][3], "~w~Licht");
if(VehicleDoorLocked(vID))PlayerTextDrawSetString(i, Tacho[i][4], "~g~Tueren");
else PlayerTextDrawSetString(i, Tacho[i][4], "~w~Tueren");
GetVehiclePos(vID, VehicleInfo[idx][v_KMX], VehicleInfo[idx][v_KMY], VehicleInfo[idx][v_KMZ]);
}
return 1;
}

Zu dem anderen Problem:
Da musst du die e_GetDistance Funktion ebenfalls abändern. Und zwar zu:
stock e_GetDistance(vehicleid)
{
new Float:x2, Float:y2, Float:z2, Float:value, idx = getVehIndex(vehicleid);
if(idx == -1) return 0;
GetVehiclePos(vehicleid, x2, y2, z2);
value = floatsqroot(floatpower(floatabs(floatsub(x2, VehicleInfo[idx][v_KMX])), 2)+floatpower(floatabs(floatsub(y2, VehicleInfo[idx][v_KMY])), 2)+floatpower(floatabs(floatsub(z2, VehicleInfo[idx][v_KMZ])), 2));
return floatround(value);
}
Zudem musst du dann beim Laden der Fahrzeuge den Wert von diesen Variablen
VehicleInfo[idx][v_KMX], VehicleInfo[idx][v_KMY], VehicleInfo[idx][v_KMZ]
auf den Wert von
VehicleInfo[idx][v_X], VehicleInfo [idx][v_Y], VehicleInfo[idx][v_Z]
setzen, damit nicht der Null-Punkt als Berechnungsstart genommen wird.

Dann musst du außerdem, wenn die Distanz dazu gerechnet wurde, also in der Funktion zur Positionsberechnung, oder dort wo es der Variable hinzugefügt wird (je nach dem wie der Rest deines Systems aussieht) die Koordinaten aktualisieren, sprich:
GetVehiclePos(vehicleid, VehicleInfo[idx][v_KMX], VehicleInfo[idx][v_KMY], VehicleInfo[idx][v_KMZ]);
Damit diese neue Position als neuer Berechnungsstart genommen wird.

malE · 18. Januar 2016

Vielen Dank erstmal!

Zitat von Jeffry

Zudem musst du dann beim Laden der Fahrzeuge den Wert von diesen Variablen
VehicleInfo[idx][v_KMX], VehicleInfo[idx][v_KMY], VehicleInfo[idx][v_KMZ]
auf den Wert von
VehicleInfo[idx][v_X], VehicleInfo [idx][v_Y], VehicleInfo[idx][v_Z]
setzen, damit nicht der Null-Punkt als Berechnungsstart genommen wird.

Soll ich idx durch i ersetzen oder den stock benutzen den du mir erstellt hast?

Zitat von Jeffry

Dann musst du außerdem, wenn die Distanz dazu gerechnet wurde, also in der Funktion zur Positionsberechnung, oder dort wo es der Variable hinzugefügt wird (je nach dem wie der Rest deines Systems aussieht) die Koordinaten aktualisieren, sprich:

GetVehiclePos(vehicleid, VehicleInfo[idx][v_X], VehicleInfo [idx][v_Y], VehicleInfo[idx][v_Z]);
Damit diese neue Position als neuer Berechnungsstart genommen wird.

Das hier soll ich dann durch GetVehiclePos(vID, VehicleInfo[idx][v_X], VehicleInfo [idx][v_Y], VehicleInfo[idx][v_Z]); im public TachoTimer ersetzen?
Fehler: Sobald ich in ein Fahrzeug einsteige wird der Kilometerstand automatisch hochgezählt.

Logischer Weise tritt der Fehler mit dem Tank der Privatfahrzeuge auch im Motor-Befehl auf.
Soll ich im Befehl den Index von vID zu idx ersetzen?

Code

ocmd:motor(playerid,params[])
{
	new vID = GetPlayerVehicleID(playerid);
	new engine, lights, alarm, doors, bonnet, boot, objective;
	if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)return 1;
	GetVehicleParamsEx(vID, engine, lights, alarm, doors, bonnet, boot, objective);
	new str[145];
	if(engine == 1){
	    engine = 0;
	    VehicleInfo[vID][v_Motor] = false;
	    ShowInfoBox(playerid, "Motor ~r~aus", 2000);
	}else{
	    if(VehicleInfo[vID][v_Tank] == 0)return ShowInfoBox(playerid, "Treibstoff ist leer.", 3500);
	    engine = 1;
	    VehicleInfo[vID][v_Motor] = true;
	    ShowInfoBox(playerid, "Motor ~g~an", 2000);
	    if(VehicleInfo[GetPlayerVehicleID(playerid)][v_Bombe] == true)
	    {
	        for(new i=0; i < MAX_PLAYERS; i++)
	        {
	            if(!IsPlayerConnected(i) || IsPlayerNPC(i))continue;
	            if(GetPlayerVehicleID(i) != GetPlayerVehicleID(playerid))continue;
    	        format(str, sizeof(str), " * %s starb durch eine Fahrzeugbombe.", PlayerInfo[i][p_Name]);
	    		SendLocalMessage(i, 15.0, COLOR_PURPLE, str);
	    		VehicleInfo[GetPlayerVehicleID(playerid)][v_Bombe] = false;
	    		SetPlayerHealth(i, 0);
	        }
	        new Float:X, Float:Y, Float:Z;
	        GetPlayerPos(playerid, X, Y, Z);
	        CreateExplosion(X, Y, Z, 0, 25);
	    }
	}
	SetVehicleParamsEx(vID, engine, lights, alarm, doors, bonnet, boot, objective);
	return 1;
}

Alles anzeigen

Jeffry · 18. Januar 2016

Zitat von malE

Soll ich idx durch i ersetzen oder den stock benutzen den du mir erstellt hast?

Ersetze es durch "i".

Zitat von malE

Das hier soll ich dann durch GetVehiclePos(vID, VehicleInfo[idx][v_X], VehicleInfo [idx][v_Y], VehicleInfo[idx][v_Z]); im public TachoTimer ersetzen?
Fehler: Sobald ich in ein Fahrzeug einsteige wird der Kilometerstand automatisch hochgezählt.

Da bin ich mit den Variablen durcheinander gekommen.
Nutze
GetVehiclePos(vehicleid, VehicleInfo[idx][v_KMX], VehicleInfo[idx][v_KMY], VehicleInfo[idx][v_KMZ]);
Diese Werte müssen aktualisiert werden.
Mit idx, ja.

Zitat von malE

Soll ich im Befehl den Index von vID zu idx ersetzen?

Ja, für die Indexe ([vID]) jeweils.

malE · 18. Januar 2016

Scheint zu funktionieren, Kilometerstand wird auch gespeichert sowie der Tank.
Jedoch hakt es nun am Motor Befehl.
Der Motor geht nicht an, ich kann so oft wie ich möchte /motor eingeben, aber der Motor geht nicht an.
Jedoch wird mir die InfoBox angezeigt, dass der Motor an wäre.

Code

ocmd:motor(playerid, params[])
{
    new vID = GetPlayerVehicleID(playerid);
	new idx = getVehIndex(vID);
	new engine, lights, alarm, doors, bonnet, boot, objective;
	if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)return 1;
	GetVehicleParamsEx(idx, engine, lights, alarm, doors, bonnet, boot, objective);
	new str[145];
	if(engine == 1){
	    engine = 0;
	    VehicleInfo[idx][v_Motor] = false;
	    ShowInfoBox(playerid, "Motor ~r~aus", 2000);
	}else{
	    if(VehicleInfo[idx][v_Tank] == 0)return ShowInfoBox(playerid, "Treibstoff ist leer.", 3500);
	    engine = 1;
	    VehicleInfo[idx][v_Motor] = true;
	    ShowInfoBox(playerid, "Motor ~g~an", 2000);
	    if(VehicleInfo[GetPlayerVehicleID(playerid)][v_Bombe] == true)
	    {
	        for(new i=0; i < MAX_PLAYERS; i++)
	        {
	            if(!IsPlayerConnected(i) || IsPlayerNPC(i))continue;
	            if(GetPlayerVehicleID(i) != GetPlayerVehicleID(playerid))continue;
    	        format(str, sizeof(str), " * %s starb durch eine Fahrzeugbombe.", PlayerInfo[i][p_Name]);
	    		SendLocalMessage(i, 15.0, COLOR_PURPLE, str);
	    		VehicleInfo[GetPlayerVehicleID(playerid)][v_Bombe] = false;
	    		SetPlayerHealth(i, 0);
	        }
	        new Float:X, Float:Y, Float:Z;
	        GetPlayerPos(playerid, X, Y, Z);
	        CreateExplosion(X, Y, Z, 0, 25);
	    }
	}
	SetVehicleParamsEx(idx, engine, lights, alarm, doors, bonnet, boot, objective);
	return 1;
}

Alles anzeigen

malE · 18. Januar 2016

Wie oben schon erwähnt, wird der Motor nicht gestartet, weil der Tank leer ist.
Obwohl der nicht leer ist. Deswegen kann ich vID schon mal nicht benutzen.

// Edit:
Mir ist gerade aufgefallen, dass nur bei den Polizeifahrzeugen der Tacho funktioniert, außerdem wird mir beim Kilometerstand die Entfernung vom Nullpunkt bis zur
Position des Fahrzeugs angezeigt.

YaBroo · 18. Januar 2016

Schonmal im Befehl die ID's und den Tank printen lassen?

Dann siehst du ja ob die ID's auch richtig überprüft werden und ob der Wert der Vari korrekt ist.

Jeffry · 18. Januar 2016

Du darfst den Index (idx) auch nur im Index (das was zwischen [ ] steht) von VehicleInfo nutzen, nicht als vehicleid.
vehicleid != Index

GetVehicleParamsEx(idx, engine, lights, alarm, doors, bonnet, boot, objective);
zu:
GetVehicleParamsEx(vID, engine, lights, alarm, doors, bonnet, boot, objective);
Gleiches beim Set... .

malE · 18. Januar 2016

Gut zu wissen , getVehIndex muss ich nun immer benutzen wenn ich VehicleInfo[... benutze?
Jetzt werden die leider die Privatfahrzeuge immer am Nullpunkt gespawnt.
Am speichern hängt es nicht, habe es mir schon printen lassen.
Ich weiß jedoch nicht wo hier der Fehler liegt.

Code

forward OnPlayerVehiclesLoad(playerid);
public OnPlayerVehiclesLoad(playerid)
{
	new  engine, lights, alarm, doors, bonnet, boot, objective;
	new rows;
	rows = cache_num_rows();
	for(new i=0; i < rows; i++)
	{
		VehicleInfo[i][v_Police] = bool:cache_get_field_content_int(i, "police", handle);
		if(VehicleInfo[i][v_Police] == true)continue;
		VehicleInfo[i][v_ID] = cache_get_field_content_int(i, "id", handle);
		VehicleInfo[i][v_Besitzer] = cache_get_field_content_int(i, "besitzer", handle);
		VehicleInfo[i][v_X] = cache_get_field_content_float(i, "x", handle), VehicleInfo[i][v_X] = VehicleInfo[i][v_KMX];
		VehicleInfo[i][v_Y] = cache_get_field_content_float(i, "y", handle), VehicleInfo[i][v_Y] = VehicleInfo[i][v_KMY];
		VehicleInfo[i][v_Z] = cache_get_field_content_float(i, "z", handle), VehicleInfo[i][v_Z] = VehicleInfo[i][v_KMZ];
		VehicleInfo[i][v_R] = cache_get_field_content_float(i, "r", handle);
		VehicleInfo[i][v_Health] = cache_get_field_content_float(i, "health", handle);
		VehicleInfo[i][v_Tank] = cache_get_field_content_int(i, "liter", handle);
		VehicleInfo[i][v_Sirene] = bool:cache_get_field_content_int(i, "siren", handle);
		VehicleInfo[i][v_Model] = cache_get_field_content_int(i, "model", handle);
		VehicleInfo[i][v_Farbe] = cache_get_field_content_int(i, "farbe", handle);
		VehicleInfo[i][v_Farbe2] = cache_get_field_content_int(i, "farbe2", handle);
		VehicleInfo[i][v_Kilometer] = cache_get_field_content_int(i, "kmstand", handle);
		VehicleInfo[i][v_Bombe] = bool:cache_get_field_content_int(i, "bombe", handle);
		VehicleInfo[i][v_WeelsKugelsicher] = bool:cache_get_field_content_int(i, "weelskugelsicher", handle);
		VehicleInfo[i][v_Panzerglas] = bool:cache_get_field_content_int(i, "panzerglas", handle);
		if(VehicleInfo[i][v_Model] >= 400 && VehicleInfo[i][v_Model] <= 611)
		{
			VehicleInfo[i][v_Objekt] = CreateVehicle(VehicleInfo[i][v_Model], VehicleInfo[i][v_X], VehicleInfo [i][v_Y], VehicleInfo[i][v_Z], VehicleInfo[i][v_R], VehicleInfo[i][v_Farbe], VehicleInfo[i][v_Farbe2], -1, VehicleInfo[i][v_Sirene]);
			if(VehicleInfo[i][v_Health] > 250.99) SetVehicleHealth(VehicleInfo[i][v_Objekt], VehicleInfo[i][v_Health]);
			else SetVehicleHealth(VehicleInfo[i][v_Objekt], 1000);
			GetVehicleParamsEx(VehicleInfo[i][v_Objekt], engine, lights, alarm, doors, bonnet, boot, objective);
			SetVehicleParamsEx(VehicleInfo[i][v_Objekt], engine, lights, alarm, 1, bonnet, boot, objective);
		}
		else printf("[FEHLER] Fahrzeug-Model %d existiert nicht (DB-ID: %d).", VehicleInfo[i][v_Model], i);
		if(GetVehicleModel(MAX_VEHICLES-1) != 0) return print("Es können keine weiteren Fahrzeuge geladen werden, Limit (MAX_VEHICLES) erreicht.");
	}
	return 1;
}

Alles anzeigen

YaBroo · 18. Januar 2016

Code

VehicleInfo[i][v_X] = cache_get_field_content_float(i, "x", handle), VehicleInfo[i][v_X] = VehicleInfo[i][v_KMX];

Warum speicherst'n du Floats in einer Variable die du direkt durch einen anderen Wert anschließend neudefinierst?

malE · 18. Januar 2016

Da sonst im Kilometerstand die Entfernung vom Nullpunkt bis zum Fahrzeug dazu addiert wird, ohne das sich das Fahrzeug bewegt hat.

YaBroo · 18. Januar 2016

Hä, das einzige was du da machst sind deine geladenen Koordinaten mit neuen Daten überschreiben, bevor du das Fahrzeug überhaupt erstellst.
Hast sie leicht vertauscht; ich glaube du wolltest es so handhaben.

VehicleInfo[i][v_KMX] = VehicleInfo[i][v_X];
VehicleInfo[i][v_KMY] = VehicleInfo[i][v_Y];
VehicleInfo[i][v_KMZ] = VehicleInfo[i][v_Z];

Vari1 = Vari2;
Vari2 wird in Vari1 gespeichert, nicht andersrum.

lg,
YaBroo

malE · 18. Januar 2016

Vielen Dank an euch beiden!
Nun funktionieren die beiden Systeme.

Fahrzeugsystem - Tank & Kilometerstand

Beitrag von YaBroo (18. Januar 2016)

breadfish.de 12. März 2022