Tank Abziehen jede Sekunde

Sehr geehrte Community,

ich stehe mal wieder auf dem Schlauch ... Ich möchte jetzt, jede Sekunde (Timer) Tank abziehen von dem was in der Datenbank steht. Dies soll nur für Autos sein deren Motor an ist...

Ich habe eine motor Variable die auf 0 bzw. 1 gesetzt wird, sobald man /motor macht. Ich habe die Variable Float:tank[MAX_VEHICLES]; definiert ... Die Tanks werden auch richtig ausgelesen ... nur wie mache ich es jetzt am besten, dass der Tank auch abgezogen wird, wenn keiner im Auto ist aber der Motor an ist...

Ich hatte schon versuche, die allerdings scheiterten... Da war es so, dass entweder gar kein Sprit abgezogen wurde oder ab 2 Spieler bei beiden Autos was abgezogen wurde, sobald nur 1 Motor an war.

Ich hoffe ihr könnt mir helfen...

MfG
XonarZ

Zitat von Jeffry

Folgende Schritte im Callback des Timers:

• Schleife durch MAX_VEHICLES
• Prüfen ob Motor an ist
• Tank abziehen

Den Timer bei OnGameModeInit starten.

Ich habe es jetzt wie folgt:

forward tankLostTimer();


OnGameModeInit: 
//TIMER
SetTimer("tankLostTimer", 1000, true);


public tankLostTimer(){
	for(new i = 0; i < MAX_VEHICLES; i++){
	    if(motor[i] == 0) continue;
		if(tank[i] <= 0){
 			tank[i] = 0;
			motor[i] = 0;
	        SetVehicleParamsEx(i, 0, 0, 0, 0, 0, 0, 0);
   		}else{
	        new vModel = GetVehicleModel(i);
	        for(new j = 0; j < sizeof(mInfo); j++){
	    	    if(mInfo[j][mID] != vModel) continue;
	            for(new player = 0; player < MAX_PLAYERS; player++){
		    	    if(!IsPlayerConnected(player)) continue;
		    	    new Float:lpmnew = cInfo[i][lpm] / 60;
		    	    tank[i] -= lpmnew;
		            if(IsPlayerInVehicle(player, i)){
	              		new string[128];
	               		format(string, sizeof(string), "%0.2f / %0.2f", tank[i], mInfo[vModel][maxTank]);
	               		PlayerTextDrawSetString(player, Tacho_TD[player][15], string);
					}
    			}
			}
		}
	}
	return 1;
}

//EDIT vergessen zu beschreiben, was nicht geht haha ... Der zieht kein Tank ab leider... bzw. Textdraw ändert sich nicht.

MfG
XonarZ

Zitat von Kaliber

Du machst dir das viel zu kompliziert.

Die innere Schleife mit dem mInfo ist komplett unnötig, da du ja durch GetVehicleModel doch schon weißt, um welches Model es sich handelt.

C

public tankLostTimer()
{
    for(new i=GetVehiclePoolSize(); i!=0; i--)
    {
        if(!motor[i]) continue;
        if(tank[i] <= 0)
        {
            tank[i] = motor[i] = 0;
            SetVehicleParamsEx(i, 0, 0, 0, 0, 0, 0, 0);
        }
        else tank[i] -= cInfo[i][lpm] / 60;
    }
    for(new i=GetPlayerPoolSize(),v,m,string[24]; i!=-1; i--)
    {
        if(IsPlayerNPC(i) || !IsPlayerInAnyVehicle(i)) continue;
        v = GetPlayerVehicleID(i), m = GetVehicleModel(v);
        format(string,sizeof(string),"%.02f / %.02f",tank[v],mInfo[m][maxTank]);
        PlayerTextDrawSetString(i, Tacho_TD[i][15], string);
    }
    return 1;
}

Alles anzeigen

Ich glaube so macht das mehr Sinn.

So brauchst du auch keine Schleifen schachteln..weil ja sonst deine Laufzeit extremst wäre..du müsstest ja jedes mal MAX_VEHICLES*MAX_VEHICLES*MAX_PLAYERS durchlaufen...das ist ja Irrsinn.

Alles anzeigen

Danke dir für die Mühe, nur leider passiert nix ...

Zitat von Kaliber

Dann lade mal 1. crashdetect und schaue ob irgendwelche Fehler auftreten
2. Hau in den Timer mal ein print, damit du sicher bist, dass das auch aufgerufen wird

Also ich habe crashdetect schon von anfang an drinne ...

Ich hab jetzt nen Print eingefügt beim Callback... der wird auch jede sekunde ausgeführt.

MfG
XonarZ

Zitat von Kaliber

Pack eventuell unter das PlayerTextDrawSetString mal ein PlayerTextDrawShow.

Sorry das ich mich jetzt erst melde...

Leider hat dies nix gebracht ... Davor hatte ich es mit einer for schleife für cInfo allerdings wäre das dann ja nur für Privatcars...

MfG
XonarZ

Zitat von Jeffry

Versuche es so, das ist näher an deinem ursprünglichen Code dran gehalten:
public tankLostTimer(){ new vModel, vehicleid, string[32], Float:lpmnew; for(new i = 0; i < MAX_VEHICLES; i++) { vModel = GetVehicleModel(i); if(!vModel) continue; if(motor[i] == 0) continue; if(tank[i] <= 0) { tank[i] = 0; motor[i] = 0; SetVehicleParamsEx(i, 0, 0, 0, 0, 0, 0, 0); } else { lpmnew = mInfo[vModel][lpm] / 60; tank[i] -= lpmnew; } } for(new player = 0; player < MAX_PLAYERS; player++) { if(!IsPlayerConnected(player) || !IsPlayerInAnyVehicle(player)) continue; vehicleid = GetPlayerVehicleID(player); vModel = GetVehicleModel(vehicleid); format(string, sizeof(string), "%0.2f / %0.2f", tank[vehicleid], mInfo[vModel][maxTank]); PlayerTextDrawSetString(player, Tacho_TD[player][15], string); } return 1;}

Falls es nicht geht, poste bitte die dein mInfo Array mit Daten aussieht.

mein Enum:

enum modelData{
	mID, //MODELID
	mName[32],
	mBat,
	fuelTyp[16],
	Float:maxTank,
	Float:lpm
}

mInfo:

new mInfo[][modelData] = {
{400, "Landstalker", 12, "Diesel", 10.0, 0.1},
{411, "Infernus", 12, "Benzin", 10.0, 0.1}
};

Noch etwas?

Leider gehts nämlich nicht...

Zitat von Jeffry

Dann bitte so versuchen:
public tankLostTimer(){ new vModel, vehicleid, string[32], Float:lpmnew; for(new i = 0; i < MAX_VEHICLES; i++) { vModel = GetModelIdx(GetVehicleModel(i)); if(vModel == -1) continue; if(motor[i] == 0) continue; if(tank[i] <= 0) { tank[i] = 0; motor[i] = 0; SetVehicleParamsEx(i, 0, 0, 0, 0, 0, 0, 0); } else { lpmnew = mInfo[vModel][lpm] / 60; tank[i] -= lpmnew; } } for(new player = 0; player < MAX_PLAYERS; player++) { if(!IsPlayerConnected(player) || !IsPlayerInAnyVehicle(player)) continue; vehicleid = GetPlayerVehicleID(player); vModel = GetModelIdx(GetVehicleModel(vehicleid)); if(vModel == -1) continue; format(string, sizeof(string), "%0.2f / %0.2f", tank[vehicleid], mInfo[vModel][maxTank]); PlayerTextDrawSetString(player, Tacho_TD[player][15], string); } return 1;}

stock GetModelIdx(modelid){ if(modelid) { for(new i = 0; i < sizeof(mInfo); i++) { if(mInfo[i][mID] == modelid) return i; } } return -1;}

Da du nicht alle Fahrzeuge in mInfo drin hast, muss der Index der Model-ID erst herausgefunden werden.
Das hattest du ja in deinem Code gewissermaßen auch drin, meine Annahme daraus war aber, dass ohnehin alle Fahrzeuge gelistet sind.

Wobei er mit vModel im Index ohnehin auch auf die Bretter gegangen wäre, sehe ich grade.

Alles anzeigen

Okay... Heißt also: Ich hätte theoretisch eine for schleife machen müssen um an den Index der richtigen mID (Modelid) zu kommen um dann den den lpm wert und den maxTank wert auszulesen ?

Und damit ich eben nicht jedes mal eine for schleife machen muss, hast du mir netterweise eine funktion geschrieben die das für mich erledigt ?

hab ich das soweit richtig verstanden ?

MfG
XonarZ