Beim Autos zerstören variable verändern

headman · 30. Dezember 2009

Guten Morgen.
Ich habe mir nen Job gescriptet, also issen Geldtransport minijob. Und man kan den Ausrauben und so.

Nun habe ich ein Problem. man soll ihn nur alle 20 min ausrauben, dies geschieht mit Timer. Nur wenn das Fahrzeug Zerstört wird soll der Timer aktiviert werden, und so kann man es dann nicht mehr rausrauben.

public OnVehicleDeath(vehicleid, killerid)
{
new newcar = GetPlayerVehicleID(newcar);
if(Geldcar(newcar))
{
SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
SendClientMessageToAll(COLOR_RED, "Der Geldtransport scheiterte da das Fahrzeug explodierte und das Geld verbrannte");
SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
Ausraubbar = 0;
nextrobable = 0;
SetTimer("nextrob",1200000,0);
}

return 1;
}

new Geldcar ist als Transporter definiert, bei Public Geldcar

Wenn das Auto zerstört wird, dann kommt dieser Text leider nicht und der timer aktiviert sich nicht, hat jmd ne Lösung?

8D · 30. Dezember 2009

Einfach wäre es so:

public OnVehicleDeath(vehicleid, killerid)
{
if(Geldcar(vehicleid))
{
SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
SendClientMessageToAll(COLOR_RED, "Der Geldtransport scheiterte da das Fahrzeug explodierte und das Geld verbrannte");
SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
Ausraubbar = 0;
nextrobable = 0;
SetTimer("nextrob",1200000,0);
}

return 1;
}

Bei deinem Code war falsch:

public OnVehicleDeath(vehicleid, killerid)
{
new newcar = GetPlayerVehicleID(playerid); // dort muss playerid und nicht newcar hin.
if(Geldcar(newcar))
{
SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
SendClientMessageToAll(COLOR_RED, "Der Geldtransport scheiterte da das Fahrzeug explodierte und das Geld verbrannte");
SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
Ausraubbar = 0;
nextrobable = 0;
SetTimer("nextrob",1200000,0);
}

return 1;
}

headman · 30. Dezember 2009

//edit by 8D. Probiere mal die Lösung von mir.

Schnacke · 30. Dezember 2009

Versuch mal so:

PHP

public OnVehicleDeath(playerid, vehicleid)
{
	new model = GetVehicleModel(vehicleid);
	if(model == deinemodelid)
	{
		SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
		SendClientMessageToAll(COLOR_RED, "Der Geldtransport scheiterte da das Fahrzeug explodierte und das Geld verbrannte");
		SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
		Ausraubbar = 0;
		nextrobable = 0;
		SetTimer("nextrob",1200000,0);
	}
	return 1;
}

Alles anzeigen

Nur halt deine Modelid einfügen.

headman · 30. Dezember 2009

ne @ Schnacke, das wäre mir zu ungenau nur die Model ID zu definieren.

Vielen Dank @ 8D. Es geht jetzt

8D · 30. Dezember 2009

Zitat von Schnacke

Versuch mal so:

PHP

public OnVehicleDeath(playerid, vehicleid)
{
	new model = GetVehicleModel(vehicleid);
	if(model == deinemodelid)
	{
		SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
		SendClientMessageToAll(COLOR_RED, "Der Geldtransport scheiterte da das Fahrzeug explodierte und das Geld verbrannte");
		SendClientMessageToAll(COLOR_GREEN,"----------------------------------------------");
		Ausraubbar = 0;
		nextrobable = 0;
		SetTimer("nextrob",1200000,0);
	}
	return 1;
}

Alles anzeigen

Nur halt deine Modelid einfügen.

Wäre sinnlos, was ist mit anderen Autos der gleichen Modell-ID?

headman np und close.