Verschiedene Tankarten Scripten

Ok, dann ändere mal das:
SetTimer("BenzinVerlust", VERLUST, true);
for (new v=0; v< MAX_VEHICLES; v++)
{
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[v] = false;
Tank[v] = GetVehicleMaxTank(v);
}
ab zu:
SetTimer("BenzinVerlust", VERLUST, true);
SetTimer("SetMaxTank", 3000, false);
und das ganz unten in Gamemode:
forward SetMaxTank();
public SetMaxTank()
{
for (new v=0; v< MAX_VEHICLES; v++)
{
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[v] = false;
Tank[v] = GetVehicleMaxTank(GetVehicleModel(v));
}
return 1;
}
Dann sollte das am Anfang auch klappen. (Ich hoffe, du hast die zwei stocks im Code für GetVehicleMaxTank und das andere).

Und was ist mit der Tankanzeige? Kannst du den Code dazu mal posten?

Ah, das habe ich nicht mehr gesehen, das hast du hinzugefügt während ich gepostet habe.
Schreibe es so:
if(GetPlayerVehicleSeat(playerid) == 0)
{
new v = GetPlayerVehicleID(playerid);
new m = GetVehicleModel(v);
if(GetVehicleModel(v) != 509 && GetVehicleModel(v) != 481 && GetVehicleModel(v) != 510)
{
new name[32];
new string[125];
switch(GetVehicleTankArt(m))
{
case 0: name = "Diesel";
case 1: name = "Benzin";
}
if(Tank[v] < 10)format(string,sizeof string,"Tankart: %s~n~~w~Tank: ~r~%i ~w~liter~n~Km: 0000012~g~%d ~w~",name,Tank[v]);
else if(Tank[v] < 20)format(string,sizeof string,"Tankart: %s~n~~w~Tank: ~y~%i ~w~liter~n~Km: 0000012~g~%d ~w~",name,Tank[v]);
else format(string,sizeof string,"Tankart: %s~n~~w~Tank: ~g~%i ~w~liter~n~Km: 0000012~g~%d ~w~",name,Tank[v]);
PlayerTextDrawSetString(playerid,CarTankart[playerid],string);
}
}
else
{
//TextDrawSetString(Anzeige[playerid], " ");
}

Zitat

Wie meinst du das?

Dass zum Server-Start die Fahrzeuge einen vollen Tank haben.

Ok, super!

Also, die einfachste und für dich sicher am verständlichste Methode ist, dass du zwei Timer erstellst, einen BenzinTimer und einen DieselTimer. Die Aufrufzeit des Timers stellst du dann entsprechend ein. In dem Timer-callback machst du dann eine Schleife durch alle Spieler und dann fragst du ab, ob die TankArt des Fahrzeugs des Spielers Benzin (im BenzinTimer) oder Diesel (im DieselTimer) ist. Wenn ja, dann ziehst du 1 ab.

Zwecks den Preisen:
Da fragst du auch mit der TankArt ab, was getankt wurde, und rechnest das entsprechend auf.

Probiere es für beides erst mal selbst, und wenn du nicht nicht mehr weiter kommst, dann poste bitte den jeweiligen Code und erkläre wo du nicht mehr weiter kommst, kennst du ja schon.

Du brauchst in dieser Abfrage keine Schleife durch alle Spieler.
public BenzinVerlust()
{
for(new i=0; i<MAX_VEHICLES; i++)
{
if(GetVehicleTankArt(p) == 1)
{
if (Motor[i] == true)
{
Tank[i]-=1;
if (Tank[i]<1)
{
Tank[i] = 0;
GetVehicleParamsEx(i,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(i,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[i] = false;
}
}
}
}
return 1;
}
public DieselVerlust()
{
for(new i=0; i<MAX_VEHICLES; i++)
{
if(GetVehicleTankArt(p) == 0)
{
if (Motor[i] == true)
{
Tank[i]-=1;
if (Tank[i]<1)
{
Tank[i] = 0;
GetVehicleParamsEx(i,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(i,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[i] = false;
}
}
}
}
return 1;
}

Wo rufst du den SetMaxTank noch auf? Im Motor Befehl ist der Fehler anscheinend nicht..

Homez

1.)
Entferne bei Motor an/aus das:
Tank[car] = GetVehicleMaxTank(v);

2.)
for (new i=0; i< MAX_PLAYERS; i++)
{
if(GetVehicleTankArt(i) == 1) {
SetTimer("BenzinVerlust", BENZIN, true);
}
else if(GetVehicleTankArt(i) == 0) {
SetTimer("DieselVerlust", DIESEL, true);
}
}
zu:
SetTimer("BenzinVerlust", BENZIN, true);
SetTimer("DieselVerlust", DIESEL, true);

Und die Timer so verwenden, wie du es zuerst hattest:

Nein, das fragst du so ab:
if(GetVehicleTankArt(modelID) == 0)
{
//Diesel
}
else if(GetVehicleTankArt(modelID) == 1)
{
//Benzin
}
Oder eben mit einem switch.

Wann sagt er dir, dass du keinen Tank mehr hast (=leerer Tank, oder?)? Direkt nach dem Server-Start oder nachdem du das Fahrzeug betankt hast?
Geht das tanken?

Ja, stimmt fast.
if(GetVehicleTankArt(i) == 1)
zu:
if(GetVehicleTankArt(GetVehicleModel(i)) == 1)

Ebenso beim Diesel.

Zwecks dem Tank:
Hast du gemacht, was ich in Post #22 geschrieben habe?
http://forum.sa-mp.de/gta-mult…en-scripten-2#post2045451

Ok, dann mache es so, umgehen wir das.
Dort wo du Tank deklariert hast:
new Tank[MAX_VEHICLES];
zu:
new Tank[MAX_VEHICLES] = {-999,...};
(Das ... genau so lassen.)

Und dann dort wo du
Tank[car] = GetVehicleMaxTank(v);
weg gemacht hast, füge das ein:
if(Tank[car] == -999) Tank[car] = GetVehicleMaxTank(v);

Somit heißt -999, dass das Fahrzeug noch nie betreten worden ist.

Dann wird der Timer doch ausgeführt, naja, schmeiß den Timer raus, den brauchen wir nicht mehr.
SetTimer("SetMaxTank", 3000, false);
und
forward SetMaxTank();
public SetMaxTank()
{
for (new v=0; v< MAX_VEHICLES; v++)
{
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[v] = false;
Tank[v] = GetVehicleMaxTank(GetVehicleModel(v));
}
return 0;
}

Kannst du löschen. Dann sollte es aber wirklich klappen.

Zeigt er es dir an, wenn du es so schreibst? Sollte es eigentlich:
if(newkeys & KEY_YES)
{
pHoldingTime[playerid] = gettime();
}
if(RELEASED(KEY_YES))
{
new string[32];
format(string, sizeof string, "%d Sekunden", gettime()-pHoldingTime[playerid]);
SendClientMessage(playerid, -1, string);
}

Einfügen kannst du das dann, indem du das ganze Zeug von deinem Tanken Code Teil den du gepostet hast bei obiger ersten if-Abfrage einfügst (KEY_YES) und bei RELEASED einfach den Timer killst.