Motor aus nach Unfall | OnVehicleDamageUpdate

Guten Abend,

ich möchte, dass wenn ein Fahrzeug nur noch eine gewisse Anzahl an "Leben" hat, der Motor
vom Fahrzeug ausgeht und die Variable "Motorschaden" den Wert "1" bekommt, so dass
der Spieler beim Starten des Motors (mit /motor) sofort mit einer Fehlernachricht returned wird.

Das sieht auch soweit ganz gut aus, jedoch habe ich das Problem, dass sobald das Fahrzeug
unter 400 HP hat, es nicht ausgeht und nur ab und zu die ClientMessage ausgegeben wird!

Soweit das Callback:

public OnVehicleDamageStatusUpdate(vehicleid, playerid)
{
	new Float:health;
	GetVehicleHealth(vehicleid, health);
	new vID = GetPlayerVehicleID(playerid);
    for(new i=0;i<MAX_VEHICLES;i++)
    {
        if(FahrzeugInfo[i][fz_ID] == vID)
        {
			if((floatround(health, floatround_round)/10) <= 40 )
			{
			    if(FahrzeugInfo[i][fz_Motorschaden] == 1) return 1;
				SetVehicleEngineOff(i);
				PlayerTextDrawSetString(playerid,MotorStatus[playerid], "Motor ~r~aus");
				SCM(playerid,COLOR_YELLOW,"[INFO] Der Motor deines Fahrzeugs ist defekt!");
				SCM(playerid,COLOR_YELLOW,"[INFO] Um einen Mechaniker zu kontaktieren /service!");
				FahrzeugInfo[i][fz_Motorschaden] = 1;
			}
		}
	}
	return 1;
}

Zitat von Jeffry

Wie sieht deine SetVehicleEngineOff Funktion aus, kannst du die bitte posten?

Der Stock:

stock SetVehicleEngineOff(vehicleid)
{
	for(new i=0;i<MAX_VEHICLES;i++)
	{
	    if(FahrzeugInfo[i][fz_ID] == vehicleid)
	    {
			new engine, lights, alarm, doors, bonnet, boot, objective;
			GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective);
			SetVehicleParamsEx(vehicleid, 0, lights, alarm, doors, bonnet, boot, objective);
			FahrzeugInfo[i][fz_Motor] = 0;
			return 1;
		}
	}
	return 1;
}

Zitat von hav0K~

Wenn sich meine Scripting-Fähigkeiten nicht komplett verabschiedet haben, müsste das return 1; ganz unten eigentlich weg und dahin gehören, wo das mit dem SendClientMessage, FahrzeugInfo[i][..] = 1; usw steht ans Ende.

Schlagt mich nicht wenns falsch ist.

// Edit

Btw. wofür machst du das überhaupt in einer Schleife? Sehe ich keinen Sinn drin nur unnötige Ressourcen verbrauchen.
vehicleid und playerid sind doch gegeben, also für mich klingt das so danach, sobald ein Spieler Schaden an einem Fahrzeug bzw. an dem, worin er sitzt, anrichtet, wird dieses Callback aufgerufen und die vehicleid vom Spieler und die playerid vom Spieler zurückgegeben.

Oder etwa nicht?

Alles anzeigen

Grundstätzlich kommt das return hinter den Code, da ich aber keine weiteren Codes in diesem Callback habe, wäre das überflüssig.
Und das mit der Schleife (alle Fahrzeuge durchgehen) war ein Test, ob es vielleicht damit läuft

Zitat von Jeffry

C

SetVehicleEngineOff(i);

zu:

C

SetVehicleEngineOff(vID);

Danke schonmal, aber das Ganze ist nur halbfertig.
Ich habe jetzt die Abfrage:

if(FahrzeugInfo[i][fz_Motorschaden] == 1) return 1;

Rausgenommen und jetzt geht das Fahrzeug zumindest schonmal JEDES MAL aus, wenn ich unter diesem abgefragtem Wert liege.
Mein Problem ist jedoch nun, dass die Variable (fz_Motorschaden) für das Fahrzeug (warum auch immer) nicht auf 1 gesetzt wird und ich
daher das Fahrzeug mit dem /motor Befehl wieder ganz normal starten kann...

Das ganze sieht bis jetzt so aus:

Zitat

Zitat von shorttag

WTF wieso immer for? Unnötig...

Code

stock SetVehicleEngineOff(vehicleid)
{
	new engine, lights, alarm, doors, bonnet, boot, objective;
	GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective);
	SetVehicleParamsEx(vehicleid, 0, lights, alarm, doors, bonnet, boot, objective);
	FahrzeugInfo[vehicleid][fz_Motor] = 0;
	return 1;
}

Ganz einfach, da die Fahrzeuge in einem ENUM (Fahrzeuginfo) eine EINDEUTIGE ID habe (fz_ID) !
Daher diese Schleife...

Aber danke

Zitat von Jeffry

Prinzipiell kannst du die Schleife hier schon weg lassen, aber da ich nicht weiß, ob und wie du es noch anders im Code verwendest, habe ich es so gelassen.

Zwecks dem anderen Problem:
So wie es momentan aussieht zeigt es in dem Post nicht an, kannst du das bitte nochmal posten, dass man es sieht?

Ups

Hier nochmal der momentane Stand:

public OnVehicleDamageStatusUpdate(vehicleid, playerid)
{
	new Float:health;
	GetVehicleHealth(vehicleid, health);
	new vID = GetPlayerVehicleID(playerid);
    for(new i=0;i<MAX_VEHICLES;i++)
    {
        if(FahrzeugInfo[i][fz_ID] == vID)
        {
			if((floatround(health, floatround_round)/10) <= 40 )
			{
			    //if(FahrzeugInfo[vID][fz_Motorschaden] == 1) return 1;
				SetVehicleEngineOff(vID);
				PlayerTextDrawSetString(playerid,MotorStatus[playerid], "Motor ~r~aus");
				SCM(playerid,COLOR_YELLOW,"[INFO] Der Motor deines Fahrzeugs ist defekt!");
				SCM(playerid,COLOR_YELLOW,"[INFO] Um einen Mechaniker zu kontaktieren /service!");
				FahrzeugInfo[vID][fz_Motorschaden] = 1;
			}
		}
	}
	return 1;
}

Danke, jetzt läuft das endlich