Achslimit Warnings

Jadeclon · 14. Februar 2017

Die Function wird jede Sekunde ausgeführt, bekomme dabei zich ackslimit abfragen, seitdem ich den teil unter else hinzugefügt habe.. Könnte das Problem durch rumprobieren bestimmt auch alleine lösen, aber mich interessieren so performance fragen.
Jemand nh Idee woran´s liegen könnte? Und was sind eigentlich ackslimits?

Code

Function LoseTank(vid)
{
    new owner = GetPlayerIdFromName(GetPlayerCarOwner(vid));
    new vehmodel = GetVehicleModel(vid);
	for(new x = 0; x < MaxVeh; x++)
	{
	    if(PlayerCar[owner][x][CarId] == vid)
	    {
			PlayerCar[owner][x][Fuel] -= CarInfo[vehmodel-400][ci_CarGasConsume];
		}
		else
		{
			new fc;while(fc<sizeof(FactionCars))
			{
				if(FactionCars[fc][fVehid] >= 400 && FactionCars[fc][fVehid] <= 611)
				{
					if(vid == FactionCars[fc][fCarid])
					{
		    			FactionCars[fc][fFuel] -= CarInfo[vehmodel-400][ci_CarGasConsume];
					}
   				}
			}
		}
	}
}

Alles anzeigen

Braun_Strowman · 14. Februar 2017

Wie hoch hast du dein ackslimit denn eingestellt?

Jadeclon · 14. Februar 2017

@Jeff_Hardy dafür müsstest du mir erstmal wie oben schon gefragt, beantworten was ackslimits sind^^

Scheint aber doch nicht an LoseTank zu liegen.

Braun_Strowman · 14. Februar 2017

das ackslimit ist in deiner server.cfg definiert, normalerweise.
Das hat im Normalfall einen Wert von 5000-7000 wenn ich mich nicht täusche, kann auch mal ein wenig mehr werden.
Schau da einfach mal nach, ansonsten füg es hinzu.

ackslimit 7000

Schau dann mal ob du das gleiche Problem hast.

Jadeclon · 14. Februar 2017

Keine chance, das limit schießt über 11k+ hinaus.. Muss daran liegen:

Code

else
					{
					    SendClientMessage(i,F_Gray,"Woran scheiterts 1 ?");
				        new fc;while(fc<sizeof(FactionCars))
						{
						    SendClientMessage(i,F_Gray,"Woran scheiterts 2 ?");
				            if(FactionCars[fc][fVehid] >= 400 && FactionCars[fc][fVehid] <= 611)
							{
							    SendClientMessage(i,F_Gray,"Woran scheiterts 3 ?");
								if(vehid == FactionCars[fc][fCarid])
								{
								    SendClientMessage(i,F_Gray,"Woran scheiterts 4 ?");
									if(FactionCars[fc][fFuel] < 99)
									{
									    SendClientMessage(i,F_Gray,"Woran scheiterts 5 ?");
							       	    if(GetPlayerState(i) == PLAYER_STATE_DRIVER)
							       	    {
							       	        SendClientMessage(i,F_Gray,"Woran scheiterts 6 ?");
											if(vehid != INVALID_VEHICLE_ID)
											{
											    SendClientMessage(i,F_Gray,"Woran scheiterts 7 ?");
											 	SetVehicleParamsEx(vehid,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
											}
							    			SendClientMessage(i,F_Gray,"Woran scheiterts 8 ?");
											FactionCars[fc][fFuel] = 0;
										    format(string, sizeof(string), "~r~Tank leer");
										    GameTextForPlayer(i,string,5000,3);
										    SendClientMessage(i,F_Gray,"Woran scheiterts 9 ?");
										}
									}
								}
							}
						}
					}

Alles anzeigen

Nicht wundern, die "Woran scheiterts?" Nachrichten füg ich immer hinzu um einen Fehler zu suchen und im moment komm ich maximal bis "Woran scheiterts 4".

Jeffry · 14. Februar 2017

Du zählst fc nie hoch.
While Schleifen sind generell nicht zu empfehlen, da genau sowas passieren kann.

new fc;while(fc<sizeof(FactionCars))
zu:
for(new fc; fc<sizeof(FactionCars); fc++)

Jadeclon · 14. Februar 2017

@Jeffry sind jetzt zwar nur noch 4000 ackslimits, aber er kommt nur bis "Woran scheiterts 2"

Alf21 · 14. Februar 2017

Erstmal ka wie das bei PAWN ist, aber normalerweise muss man eine Variable initialisieren, wenn man diese nutzen will.
for(new fc; fc<sizeof(FactionCars); fc++)
for(new fc = 0; fc<sizeof(FactionCars); fc++)

Zudem würde ich dir empfehlen, nicht nur Woran scheiterts {^[0-9]+$} ? auszugeben, sondern ebenso Variablen, um z.B. dein fc zu bekommen oder ähnliche Variablen.

Jadeclon · 15. Februar 2017

@Alf21 Hab das statt den "Woran scheiterts" hinzugefügt:

Code

format(string, sizeof(string), "1 Vehid: %d | Carid: %d | Fuel: %d",FactionCars[fc][fVehid],FactionCars[fc][fCarid],FactionCars[fc][fFuel]);
						    SendClientMessage(i,F_Gray,string);

Aber mir wird bei allen dreien 0 ausgegeben.. Bei dem Befehl zum Fraktionsfahrzeuge erstellen, funktionierts aber...

Code

dcmd_efa(playerid,params[]){
	if(GetPlayerAdmin(playerid) >= NEEDED_ADMINLEVEL)
	{
		new vehid,col1,col2,faction[5];
	    if(!sscanf(params,"dddd",vehid,col1,col2,faction[0]))
		{
			sscanf(params,"dddddddd",vehid,col1,col2,faction[0],faction[1],faction[2],faction[3],faction[4]);
			if(vehid < 400 || vehid > 611) return SendClientMessage(playerid,FACTION_COL_GREY,"Vehicle-ID nur von 400 bis 611.");
			if(col1 < 0 || col1 > 255) return SendClientMessage(playerid,FACTION_COL_GREY,"Falsche Farbe(1)!");
			if(col2 < 0 || col2 > 255) return SendClientMessage(playerid,FACTION_COL_GREY,"Falsche Farbe(2)!");
			if(faction[0] && faction[0] <= MAX_FACTIONS)
			{
			    for(new f;f<5;f++)
				{
			        if(faction[f] <= 0 || faction[f] > MAX_FACTIONS)
					{
			            faction[f] = -255;
					}
				}
				for(new fc;fc<sizeof(FactionCars);fc++)
				{
				    if(FactionCars[fc][fVehid] < 400 || FactionCars[fc][fVehid] > 611)
					{
  						for(new i=0; i<sizeof(AutohausVehs); i++)
						{
					        FactionCars[fc][fVehid] = vehid;
		            		new Float:fX,Float:fY,Float:fZ,Float:fA;
		            		GetPlayerPos(playerid,fX,fY,fZ),GetPlayerFacingAngle(playerid,fA);
		           	 		FactionCars[fc][fPos][0] = fX;
		            		FactionCars[fc][fPos][1] = fY;
		            		FactionCars[fc][fPos][2] = fZ;
		            		FactionCars[fc][fPos][3] = fA;
		            		FactionCars[fc][fCol][0] = col1;
	            			FactionCars[fc][fCol][1] = col2;
		            		for(new f;f<5;f++){
		                		FactionCars[fc][fFaction][f] = faction[f];
		            		}
		            		FactionCars[fc][fFuel] = CarInfo[vehid-400][ci_CarMaxGas]*100;
		            		FactionCars[fc][fPrice] = 0;
		            		for(new mood;mood<14;mood++)
								FactionCars[fc][fTuning][mood] = 0;
	                		FactionCars[fc][fCarid] = CreateVehicle(FactionCars[fc][fVehid],FactionCars[fc][fPos][0],FactionCars[fc][fPos][1],FactionCars[fc][fPos][2],FactionCars[fc][fPos][3],FactionCars[fc][fCol][0],FactionCars[fc][fCol][1],-255);
	                        #if defined FACTION_CARS_LOCKABLE
	                        FactionCars[fc][fLock] = true;
	                        #endif
							SendClientMessage(playerid,FACTION_COL_YELLOW,"Du hast das Fraktionsfahrzeug erfolgreich erstellt.");
							SendClientMessage(playerid,FACTION_COL_YELLOW,"Vergesse es nicht mit '/fpark' umzuparken.");
							PutPlayerInVehicle(playerid,FactionCars[fc][fCarid],0);
							new stringo[128];
							format(stringo, sizeof(stringo), "Vehid: %d | Carid: %d | Fuel: %d",FactionCars[fc][fVehid],FactionCars[fc][fCarid],FactionCars[fc][fFuel]);
					 		SendClientMessage(playerid,F_Gray,stringo);
					        return true;
						}
				    }
				}SendClientMessage(playerid,FACTION_COL_GREY,"Die Maximale Anzahl von Fraktionsfahrzeugen wurde bereits erreicht.");
			}
			else
			{
				SendClientMessage(playerid,FACTION_COL_GREY,"Falsche Fraktion!");
			}
	    }
		else
		{
			SendClientMessage(playerid,FACTION_COL_GREY,"Benutze: '/efa [Model] [Farbe(1)] [Farbe(2)] [Hauptfraktion]'.");
		}
	}
	else
	{
		SendClientMessage(playerid,FACTION_COL_GREY,"Du bist nicht berechtigt diesen Befehl anzuwenden.");
	}
	return true;
}

Alles anzeigen

Jeffry · 15. Februar 2017

Zitat von Alf21

Erstmal ka wie das bei PAWN ist, aber normalerweise muss man eine Variable initialisieren, wenn man diese nutzen will.

Muss man bei SA:MP nicht. Kein Wert bedeutet 0 oder false.

Zitat von Jadeclon

@Jeffry sind jetzt zwar nur noch 4000 ackslimits, aber er kommt nur bis "Woran scheiterts 2"

Alle fVehid sind dann auf 0, oder zumindest nicht im Range von 400-611.

Zitat von Jadeclon

Aber mir wird bei allen dreien 0 ausgegeben.. Bei dem Befehl zum Fraktionsfahrzeuge erstellen, funktionierts aber...

Hast du ein Fahrzeug mit dem Befehl erstellt, während der Server an war und du die prints hast laufen lassen?
Wenn nein, mach das mal, dann müsste es für ein Fahrzeug - das per Befehl erstellte - klappen.
Wenn es klappt, dann fehlt beim Laden der Fahrzeuge aus der Datenbank die Zuweisung von fVehid.

Jadeclon · 15. Februar 2017

@Jeffry funktioniert weder noch...
Hier mal die ganze Funktion: http://pastebin.com/VU4DdPuW
das steht unter Timer2 der jede Sekunde ausgeführt wird.
Seit dem ich in Zeile 90 das else und alles darunter hinzugefügt habe ist das so,
denn die Fraktionsfahrzeuge hatten bis dato keine Fuel Anzeige.
Komischerweise krieg ich bei den Privat Fahrzeugen auch die Nachricht:
"1 Vehid: %d | Carid: %d | Fuel: %d" wenn ich den Motor starte.

Jeffry · 15. Februar 2017

Dann kann es auch sein, dass du die Werte im Array irgendwo überschreibst.
Da in der Variable fVehid 0 drin steht, für alle Indizes, klappt die Abfrage nicht.

Warum dort 0 drin steht kann ich nicht sagen, das kann viele Gründe haben. Da musst du dich durch deinen Code arbeiten und prüfen, wann der korrekt Wert (Model ID) noch drin steht, und wann er plötzlich weg ist.

breadfish.de 12. März 2022