Timer Fehler

RickFace · 24. August 2018

Hallo liebe com,

ich bräuchte noch einmal eure Hilfe..

Ich habe mir einen Timer erstellt der alle 2 Sekunden die Tank Anzeige Aktualisieren soll allerdings habe ich jetzt das Phänomen dass sobalt ich in ein Beliebiges Auto einsteige der Gesammte Server hängen bleibt.. ich denke ich hab da einen Fetten Fehler gemacht bin finde ihn gerade nur nicht...

Hier mal der Timer:

Code

public TachoTimer()
{
    for(new i=0; i<MAX_PLAYERS; i++)
	{
	    if(!IsPlayerConnected(i))continue;
	    if(IsPlayerInAnyVehicle(i))
	    {
		    new string[164],tankfuellung;
		    for(new ix=0; ix<sizeof playerCar; ix++)
		    {
		        if(playerCar[ix][p_car_id_x]!=GetPlayerVehicleID(i))continue;
				tankfuellung = floatround(playerCar[ix][p_car_fuel],floatround_round);
			    format(string,sizeof string,"Tank: %i L",tankfuellung);
			    PlayerTextDrawSetString(i,Tacho_Tank[i],string);
                SendClientMessage(i,COLOR_GREW,"-> UPDATE");
			}
		    for(new ix=0; ix<sizeof fcarInfo; i++)
		    {
		        if(fcarInfo[ix][fcar_id_x]!=GetPlayerVehicleID(i))continue;
		        tankfuellung = floatround(fcarInfo[ix][fcar_fuel],floatround_round);
		        format(string,sizeof string,"Tank: %i L",tankfuellung);
		        PlayerTextDrawSetString(i,Tacho_Tank[i],string);
		        SendClientMessage(i,COLOR_GREW,"-> UPDATE");
		    }


		    for(new ix=0; ix<sizeof ahCarInfo; ix++)
		    {
		        if(ahCarInfo[ix][ah_car_id_x]!=GetPlayerVehicleID(i))continue;
				tankfuellung = floatround(ahCarInfo[ix][ah_car_fuel],floatround_round);
			    format(string,sizeof string,"Tank: %i L",tankfuellung);
			    PlayerTextDrawSetString(i,Tacho_Tank[i],string);
                SendClientMessage(i,COLOR_GREW,"-> UPDATE");
		    }
            SendClientMessage(i,COLOR_GREW,"-> UPDATEEE");
		}


	}
	return 1;
}

Alles anzeigen

Und natürlich wieder schonmal vielen dank im Vorraus

Kaliber · 24. August 2018

Du solltest mal sowas wie break benutzen...

Ansonsten rattert der dir da ja eine Schleife mit 500*MAX_CARS*MAX_CARS*MAX_CARS durch...und das halt eben alle paar Millisekunden, je nach timer-intervall.

Das ist ja jetzt nicht so das, was du eigentlich willst..

Es ist an sich nicht sehr schlau den Tank für jedes Fahrzeug in nem anderen Array zu speichern

Vielleicht sollte man das vereinheitlichen.

Zudem deklariert man keine Variablen in Schleifen...so muss der nämlich immer und immer wieder alles neu allozieren.

Man kann das auch alles kürzen und eleganter schreiben, das werd ich dir machen, wenn du mir sagst wie groß jeweils:

playerCar, fcarInfo und ahCarInfo sind.

Also wie hast du die deklariert?

RickFace · 24. August 2018

Code

new ahCarInfo[MAX_VEHICLES][AutoHausCarInfos];
new fcarInfo[MAX_VEHICLES][FraktionsAutoDaten];
new playerCar[MAX_VEHICLES][SpielerAutoDaten];

So hatte ich die..

//Edit

falls hilfreich noch hier die enums:

Code

enum AutoHausCarInfos{
	ah_car_id,
	ah_car_id_x,
	Float:ah_car_x,
	Float:ah_car_y,
	Float:ah_car_z,
	Float:ah_car_r,
	ah_car_autohaus_id,
	ah_car_vehicleid,
	ah_car_kosten,
	Float:ah_car_fuel,
	Text3D:ah_car_id_y
}

Alles anzeigen

Code

enum FraktionsAutoDaten{
	fcar_id,
	fcar_id_x,
	fcar_fraktion,
	fcar_vehicleid,
	Float:fcar_spawn_x,
	Float:fcar_spawn_y,
	Float:fcar_spawn_z,
	Float:fcar_spawn_r,
	fcar_color1,
	fcar_color2,
	fcar_paintjob,
	fcar_paintjob_state,
	Float:fcar_fuel
}

Alles anzeigen

Code

enum SpielerAutoDaten{
	p_car_id,
	p_car_id_x,
	p_car_vehicleid,
	p_car_besitzer,
	Float:p_car_fuel,
	Float:p_car_x,
	Float:p_car_y,
	Float:p_car_z,
	Float:p_car_r,
	p_car_kosten,
	p_car_color_1,
	p_car_color_2
}

Alles anzeigen

//edit2

Habe das Problem gerade selber gelöst

Habe statt einen dauerhaften Timer einfach bei OnPlayerStateChange SetTimerEx und KillTimer genutzt außerdem hab ich bei den for-schleifen jeweils sobalt es geupdated wurde ein return eingebaut dadurch verbraucht er nicht soviel kapazität..

Vielen dank trozdem für die Hilfestellung

Kaliber · 24. August 2018

Zitat von RickFace

return eingebaut

Damit zerstört du aber die Funktion der Schleife, deshalb sagte ich break. (Also für andere Spieler geht dann die Schleife evtl nicht mehr.)

C

public TachoTimer()
{
    for(new i=GetPlayerPoolSize(),string[32]; i!=-1; i--)
    {
        if(!IsPlayerConnected(i) || !IsPlayerInAnyVehicle(i)) continue;
        format(string,sizeof string,"Tank: %i L",GetVehicleTank(GetPlayerVehicleID(i)));
        PlayerTextDrawSetString(i,Tacho_Tank[i],string);
    }
    return 1;
}
stock GetVehicleTank(veh)
{
    for(new v; v<MAX_VEHCILES; v++)
    {
        if(playerCar[v][p_car_id_x] == veh) return floatround(playerCar[v][p_car_fuel],floatround_round);
        if(fcarInfo[v][fcar_id_x] == veh) return floatround(fcarInfo[v][fcar_fuel],floatround_round);
        if(ahCarInfo[v][ah_car_id_x] == veh) return floatround(ahCarInfo[v][ah_car_fuel],floatround_round);
    }
    return -1;
}

Alles anzeigen

Auslagern hilft auch hier und hält den Code strukturierter.

Hier mal ein Tutorial, was ich geschrieben hatte: Schneller & strukturierter skripten

Timer Fehler

breadfish.de 12. März 2022

Ähnliche Themen

Bankraub System funktioniert nicht

Server stürzt nach gmx ab

Scheine an Fahrzeugen zuordnen und abfragen

Jetpack

Timer Fehler