[SAMMELTHREAD] Kleine Scripting Fragen

Kaliber · 23. Januar 2019

Zitat von Kaliber

in der Frak[1][frakEisen] das war leer.

Anscheinend hast du da einen Fehler beim Laden

Liest du eig was ich schreibe? xD

Und noch was...

der Log oben, stimmt doch überhaupt nicht mit dem Code...ein auf der vorherigen Seite...

Also printe dir das und zeig nochmal deinen aktuellen Befehl

Dr. Frauenarzt · 23. Januar 2019

Frak[1][frakEisen] <- Negativ ich kann mich ausrüsten also sind Materialien vorhanden.
habe dies schon getestet wen keine vorhanden sind kann man sich nicht ausrüsten.

Verstehe nicht was genau du gerade willst das ist der Log wen ich das Geladen habe:

Spoiler anzeigen

Der Log wen ich die Materialien geladen habe:

Materialien geladen: 15000

So das ist der Command für das Abladen:

Spoiler anzeigen

C

ocmd:unload(playerid, params[])
{
    new string[128];
    new vehicleid = GetPlayerVehicleID(playerid);
    if(IsPlayerInRangeOfPoint(playerid, 5.0, 1563.4137, -1694.5535, 5.8906))
    {
		if(GetVehicleModel(vehicleid) == 428)
	 	{
			if(!(Spieler[playerid][pFraktion] == 17))return SendClientMessage(playerid, COLOR_WHITE, ERROR_FRAKTION);
			if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Du befindest dich in keinem Lieferwagen.");
			if(Vehicle[GetPlayerVehicleID(playerid)][vehMats] <= 0)return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Du hast keine Materialien geladen.");
			new gebraucht = Frak[1][frakMaxEisen]-Frak[1][frakEisen];
			new money = Vehicle[vehicleid][vehMats]*6;
	        if(Vehicle[vehicleid][vehMats] > gebraucht)
			{
			    Vehicle[vehicleid][vehMats] -= gebraucht;
				Frak[1][frakEisen] += gebraucht;
				format(string, sizeof(string), "{6E7271}Verbrecher mit /arrest einsperren\n\n{7EF201}Waffenkammer (LSPD)\n%d/%d\nMaterialien", Frak[1][frakEisen], Frak[1][frakMaxEisen]);
				Update3DTextLabelText(lspd3d, COLOR_WHITE, string);
				printf("Fraktion 1 - 1: Gebraucht: %d", gebraucht);
				format(string, sizeof(string), "{21DD00}Gehalt: %d$", money);
				SendClientMessage(playerid, COLOR_WHITE, string);
				format(string, sizeof(string), "{21DD00}Materialien: %d/50000", Vehicle[vehicleid][vehMats]);
				SendClientMessage(playerid, COLOR_WHITE, string);
				Spieler[playerid][pBank] += money;
				format(string, sizeof(string), "{33AA33}* Staatskasse hat dir %d$ überwiesen. Betreff: Materialien abgeliefert (%d)", money, gebraucht);
				SendClientMessage(playerid, COLOR_WHITE, string);
				printf("Fraktion 1 - 2: Gebraucht: %d", gebraucht);
			}
			else
			{
			    Vehicle[vehicleid][vehMats] -= gebraucht;
   				Frak[1][frakEisen] += gebraucht;
				format(string, sizeof(string), "{6E7271}Verbrecher mit /arrest einsperren\n\n{7EF201}Waffenkammer (LSPD)\n%d/%d\nMaterialien", Frak[1][frakEisen], Frak[1][frakMaxEisen]);
				Update3DTextLabelText(lspd3d, COLOR_WHITE, string);
				printf("+ Fraktion 1 - 2: Gebraucht: %d", gebraucht);
				format(string, sizeof(string), "{21DD00}Gehalt: %d$", money);
				SendClientMessage(playerid, COLOR_WHITE, string);
				format(string, sizeof(string), "{21DD00}Materialien: %d/50000", Vehicle[vehicleid][vehMats]);
				SendClientMessage(playerid, COLOR_WHITE, string);
				Spieler[playerid][pBank] += money;
				format(string, sizeof(string), "{33AA33}* Staatskasse hat dir %d$ überwiesen. Betreff: Materialien abgeliefert (%d)", money, gebraucht);
				SendClientMessage(playerid, COLOR_WHITE, string);
				printf("+ Fraktion 1 - 1: Gebraucht: %d", gebraucht);
				Vehicle[vehicleid][vehMats] -= gebraucht;
			}
        }
    }
    return 1;
}

Alles anzeigen

So das sind beide Command.
Wie bereits ich geschreieben habe habe ich den fehler schon beim laden gesucht da liegt kein fehler da die anzeige genauso daran liegt (3DTextLabel)
Da steht es genauso in der DB: 245.000 und maximale Materialien 250.0000
So verstehe jetzt nicht ganz wo wie was? printe doch schon und er gibt mir immer 0 aus siehe da:

Fraktion 1 - 1: Gebraucht: 0
Fraktion 1 - 2: Gebraucht: 0

Kaliber · 23. Januar 2019

Zitat von Dr. Frauenarzt

So das sind beide Command.

Jaaa und die sind falsch.

Wie ist es möglich...dass du es verkackt hast 4 Zeilen zu kopieren?!?!?!?!

Zitat von Dr. Frauenarzt

else
{
Vehicle[vehicleid][vehMats] -= gebraucht;
Frak[1][frakEisen] += gebraucht;

Das hast du da stehen.

Zitat von Kaliber

else
{
Vehicle[vehicleid][vehMats] = 0;
Frak[1][frakEisen] += Vehicle[vehicleid][vehMats];

Das hab ich da stehen...fällt dir irgendwie..irgendwas auf?!?!?!

Zitat von Dr. Frauenarzt

Negativ ich kann mich ausrüsten also sind Materialien vorhanden.
habe dies schon getestet wen keine vorhanden sind kann man sich nicht ausrüsten.

Habe ich dich gefragt ob du dich ausrüsten kannst oder ob ich wissen will OB da was drinnen ist?
Nein ich habe dich gefragt WIE VIELE GENAU geladen werden und dafür musst du das printen.

Zitat von Dr. Frauenarzt

Wie bereits ich geschreieben habe habe ich den fehler schon beim laden gesucht da liegt kein fehler da die anzeige genauso daran liegt (3DTextLabel)
Da steht es genauso in der DB: 245.000 und maximale Materialien 250.0000

Das hattest du nicht geschrieben, aber gut zu wissen.

Ja du solltest natürlich noch prüfen ob gebraucht 0 ist, dann ist ja klar, dass er nichts mehr benötigt und du kannst eine Fehlermeldung ausgeben.

Dr. Frauenarzt · 23. Januar 2019

Vehicle[vehicleid][vehMats] = 0;
Frak[1][frakEisen] += Vehicle[vehicleid][vehMats];

Habe ich nun eingetragen.

Nun habe ich beides ausgegeben einmal was ich eingegeben habe und einmal das was im Fahrzeug ist!

Materialien geladen: 25000
printf("Materialien geladen: %d", int);

Materialien geladen [Fahrzeug]: 25000
printf("Materialien geladen [Fahrzeug]: %d", Vehicle[vehicleid][vehMats]);

Fehlermeldung habe ich so hinzugefügt:
if(Frak[1][frakEisen] + gebraucht > Frak[1][frakMaxEisen])return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Soviele Materialien kannst du hier nicht abladen.");

Kaliber · 23. Januar 2019

Wieso schickst du mir das von dem load Befehl?!?!

Wir reden hier nur über den unload Befehl.

Und bei der Fehlermeldung schreib doch einfach:

if(gebraucht == 0)

Dr. Frauenarzt · 23. Januar 2019

Vehicle[vehicleid][vehMats] -= gebraucht;
printf("Materialien [Fahrzeug]: %d", Vehicle[vehicleid][vehMats]);

Frak[1][frakEisen] += gebraucht;
printf("Materialien Fraktion: %d", gebraucht);

format(string, sizeof(string), "{6E7271}Verbrecher mit /arrest einsperren\n\n{7EF201}Waffenkammer (LSPD)\n%d/%d\nMaterialien", Frak[1][frakEisen], Frak[1][frakMaxEisen]);
Update3DTextLabelText(lspd3d, COLOR_WHITE, string);
printf("%d/%d\nMaterialien", Frak[1][frakEisen], Frak[1][frakMaxEisen]);

format(string, sizeof(string), "{21DD00}Gehalt: %d$", money);
SendClientMessage(playerid, COLOR_WHITE, string);
printf("Gehalt: %d", money);

format(string, sizeof(string), "{21DD00}Materialien: %d/50000", Vehicle[vehicleid][vehMats]);
SendClientMessage(playerid, COLOR_WHITE, string);
printf("Materialien: %d", Vehicle[vehicleid][vehMats]);

Spieler[playerid][pBank] += money;

format(string, sizeof(string), "{33AA33}* Staatskasse hat dir %d$ überwiesen. Betreff: Materialien abgeliefert (%d)", money, gebraucht);
SendClientMessage(playerid, COLOR_WHITE, string);
printf("Staatskasse Überwiesen: %d, Materialien abgeladen: %d", money, gebraucht);

Das ist vom unload befehl habe mir mal alles ausgegeben:

Materialien [Fahrzeug]: 25000
Materialien Fraktion: 0
0/0
Materialien
Gehalt: 150000
Materialien: 25000
Staatskasse Überwiesen: 150000, Materialien abgeladen: 0

Kaliber · 23. Januar 2019

Zitat von Dr. Frauenarzt

0/0

Siehst du das?

Das sagt aus, dass beide: Frak[1][frakEisen], Frak[1][frakMaxEisen])

0 sind.

Und genau das habe ich gemeint gehabt mit..printe ob das geladen wird.

Übrigens poste bitte immer ganze Befehle.

Dr. Frauenarzt · 23. Januar 2019

Code:

Spoiler anzeigen

C

public LoadFraktion()
{
	new rows, fields;
	cache_get_data(rows, fields);
	for(new i=0;i<rows;i++)
	{
		new fraktion = cache_get_field_content_int(i, "frakID");
		if(fraktion < MAX_FRACS)
		{
			Frak[fraktion][frakIDx] = cache_get_field_content_int(i, "frakID");
			printf("Fraktion ID: %d", Frak[fraktion][frakIDx]);
			Frak[fraktion][frakKasse] = cache_get_field_content_int(i, "frakKasse");
			Frak[fraktion][frakKasseOn] = cache_get_field_content_int(i, "frakKasseOn");
			Frak[fraktion][frakEisen] = cache_get_field_content_int(i, "frakEisen");
			printf("Fraktion Materialien: %d", Frak[fraktion][frakEisen]);
			Frak[fraktion][frakMaxEisen] = cache_get_field_content_int(i, "frakMaxEisen");
			printf("Fraktion Max Materialien: %d", Frak[fraktion][frakMaxEisen]);
			Frak[fraktion][frakEisenOn] = cache_get_field_content_int(i, "frakEisenOn");
			Frak[fraktion][frakDrugs] = cache_get_field_content_int(i, "frakDrugs");
			Frak[fraktion][frakDrugsOn] = cache_get_field_content_int(i, "frakDrugsOn");
			Frak[fraktion][frakType] = cache_get_field_content_int(i, "frakType");
			cache_get_field_content(i, "frakName", Frak[fraktion][frakName], MySqlConnection, 32);
			Frak[fraktion][frakOpened] = cache_get_field_content_int(i, "frakOpened");
			Frak[fraktion][spawnX] = cache_get_field_content_float(i, "spawnX");
			Frak[fraktion][spawnY] = cache_get_field_content_float(i, "spawnY");
			Frak[fraktion][spawnZ] = cache_get_field_content_float(i, "spawnZ");
			Frak[fraktion][spawnA] = cache_get_field_content_float(i, "spawnA");
			Frak[fraktion][spawnInterior] = cache_get_field_content_int(i, "spawnInterior");
			Frak[fraktion][spawnVirtualWorld] = cache_get_field_content_int(i, "spawnVirtualWorld");
			Frak[fraktion][frakEisenPreis] = cache_get_field_content_int(i, "frakEisenPreis");
			Frak[fraktion][frakDrugPreis] = cache_get_field_content_int(i, "frakDrugPreis");


			if(Frak[fraktion][frakIDx] == 2)
			{
				new string[128];
				format(string, sizeof(string), "{6E7271}Verbrecher mit /arrest einsperren.\n\n{7EF201}Waffenkammer (LSPD)\n%d/%d\nMaterialien", Frak[fraktion][frakEisen], Frak[fraktion][frakMaxEisen]);
				Update3DTextLabelText(lspd3d, COLOR_WHITE, string);
			}
			if(Frak[fraktion][frakIDx] == 3)
			{
				new string[128];
				format(string, sizeof(string), "{6E7271}Verbrecher mit /arrest einsperren.\n\n{7EF201}Waffenkammer (FBI)\n%d/%d\nMaterialien", Frak[fraktion][frakEisen], Frak[fraktion][frakMaxEisen]);
				Update3DTextLabelText(fbi3d, COLOR_WHITE, string);
			}
			if(Frak[fraktion][frakIDx] == 4)
			{
				new string[128];
				format(string, sizeof(string), "{6E7271}Verbrecher mit /arrest einsperren.\n\n{7EF201}Waffenkammer (LVPD)\n%d/%d\nMaterialien", Frak[fraktion][frakEisen], Frak[fraktion][frakMaxEisen]);
				Update3DTextLabelText(lvpd3d, COLOR_WHITE, string);
			}
			if(Frak[fraktion][frakIDx] == 16)
			{
				new string[128];
				format(string, sizeof(string), "{9B2736}San News ¦ Lagerbestand\n{FFFFFF}%d/%d\nZeitungspapier", Frak[fraktion][frakEisen], Frak[fraktion][frakMaxEisen]);
				Update3DTextLabelText(news3d, COLOR_WHITE, string);
			}
		}
	}
	return 1;
}

Alles anzeigen

Laut laden ja siehe:

Fraktion ID: 2 = LSPD
Fraktion Materialien: 245000
Fraktion Max Materialien: 250000

Kaliber · 23. Januar 2019

Zitat von Dr. Frauenarzt

printf("Fraktion ID: %d", Frak[fraktion][frakIDx]);

Was passiert, wenn du da rein schreibst:

printf("Fraktion ID: %d", fraktion);

Dr. Frauenarzt · 23. Januar 2019

Fraktion ID: 0
Fraktion ID: 1
Fraktion ID: 2
Fraktion ID: 3
Fraktion ID: 4
Fraktion ID: 5
Fraktion ID: 6
Fraktion ID: 7
Fraktion ID: 8
Fraktion ID: 9
Fraktion ID: 10
Fraktion ID: 11
Fraktion ID: 12
Fraktion ID: 13
Fraktion ID: 14
Fraktion ID: 15
Fraktion ID: 16
Fraktion ID: 17

Kaliber · 23. Januar 2019

............

Manchmal hab ich echt das Gefühl, dass du mich verarschen willst

Was glaubst du bringt mir das?!

Du scriptest jetzt doch schon auch seit mindestens 3 Jahren...

Du musst mir natürlich den GANZEN Log schicken...weil die nachfolgenden prints doch entscheidend sind

Dr. Frauenarzt · 23. Januar 2019

Weiß nicht was sich das bringen soll ist das meiner meinung das selbe oder etwa nicht?

Spoiler anzeigen

Kaliber · 23. Januar 2019

Zitat von Dr. Frauenarzt

Fraktion ID: 1
Fraktion Materialien: 0
Fraktion Max Materialien: 0

Nein siehst du das.

Für Fraktion 1 werden keine Materialien geladen...

Dr. Frauenarzt · 23. Januar 2019

Ich sehe das das ist auch der Präsident.
Wen ich jetzt welche rein machen würde werden die geladen o.O?

//E:

Siehe hier die Tabellen für die Fraktion:

https://imgur.com/a/n8JIEhO

//E:

Habe es gerade eben gesehn oh man bin ich blöd

Sub Royal · 23. Januar 2019

Hat jemand gerade die verbesserte strtok Funktion parat? Finde die gerade nicht im Web.

Dr. Frauenarzt · 23. Januar 2019

Zitat von Sub Royal

Hat jemand gerade die verbesserte strtok Funktion parat? Finde die gerade nicht im Web.

https://forum.sa-mp.com/showthread.php?t=69992 meinst du das?

Sub Royal · 23. Januar 2019

Das ist die alte, gibt aber irgendwie eine bessere von jemanden hier aus dem Forum.

Kaliber · 23. Januar 2019

Zitat von Sub Royal

Das ist die alte, gibt aber irgendwie eine bessere von jemanden hier aus dem Forum.

Jeffry hatte mal eine geschrieben: Nützliche Codeschnipsel

Meine Finde ich leider nicht mehr xD Kp wo ich die mal hingepackt hatte xD

Aber ich rate dir eh zu sscanf ganz ehrlich

Brauchst eigentlich nirgens strotk, dass kann man alles eleganter lösen

Sub Royal · 23. Januar 2019

Da hast du Recht, ich mag lieber auch sscanf.
Ich teste nur gerade mal den Urgodfather, und da ist das drinnen, habe nicht soviel Lust das ueberall zu wechseln

Dr. Frauenarzt · 24. Januar 2019

Spoiler anzeigen

C

for(new i=1;i<MAX_TANKEN;i++)
	{
		if(Tanke[i][tCreated] == 1)
		{
			if(IsPlayerInRangeOfPoint(playerid, 5.0, Tanke[i][tPickupX], Tanke[i][tPickupY], Tanke[i][tPickupZ]))
			{
			    if(GetVehicleModel(vehicleid) == 514 || GetVehicleModel(vehicleid) == 515)
       			{
    				new gebraucht = floatround(Tanke[i][tMaxLiterBenzin])-floatround(Tanke[i][tLiterBenzin]);
				    new TankeMoney = Tanke[i][tKasse];
					new kosten = (Tanke[i][tBenzinAnkauf]*gebraucht);
					new Float:intfloat = float(gebraucht);
					new Float:add = floatadd(intfloat, Tanke[i][tLiterBenzin]);
			        if(!(Spieler[playerid][pFraktion] == 17))return SendClientMessage(playerid, COLOR_WHITE, ERROR_FRAKTION);
			        if(!IsTrailerAttachedToVehicle(GetPlayerVehicleID(playerid))) return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Du hast dein Produkt Anhänger nicht angeschlossen!");
			        if(GetVehicleModel(GetVehicleTrailer(GetPlayerVehicleID(playerid))) != 584) return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Du hast den falschen Anhänger. Du brauchst den Tank Anhänger!");
			        if(TankeMoney < kosten)return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Die Tankstelle hat nicht genügend Geld für den Benzin.");
			        if(gebraucht == 0)return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Dieses Tankstelle benötigt keinen Benzin mehr.");
			        if(Vehicle[GetVehicleTrailer(vehicleid)][vehBenzin] > gebraucht)
					{
					    Vehicle[GetVehicleTrailer(vehicleid)][vehBenzin] -= gebraucht;
						Tanke[i][tMaxLiterBenzin] += add;
                        UpdateTankLabel(i, Tanke[i][tOwned]);
                        format(string, sizeof(string), "{21DD00}Es konnten nur %d Liter Benzin abgeliefert werden, die Tankstelle ist voll.", gebraucht);
                        SendClientMessage(playerid, COLOR_WHITE, string);
  						format(string, sizeof(string), "{21DD00}Gehalt: %d$", kosten);
						SendClientMessage(playerid, COLOR_WHITE, string);
						format(string, sizeof(string), "{21DD00}Benzin: %d/2500", Vehicle[GetVehicleTrailer(vehicleid)][vehBenzin]);
						SendClientMessage(playerid, COLOR_WHITE, string);
						Tanke[i][tKasse] -= kosten;
						GiveMoney(playerid, kosten);
					}
					else
					{
		    			Vehicle[GetVehicleTrailer(vehicleid)][vehBenzin] = 0;
						Tanke[i][tMaxLiterBenzin] += Vehicle[GetVehicleTrailer(vehicleid)][vehBenzin];
                        UpdateTankLabel(i, Tanke[i][tOwned]);
  						format(string, sizeof(string), "{21DD00}Gehalt: %d$", kosten);
						SendClientMessage(playerid, COLOR_WHITE, string);
						format(string, sizeof(string), "{21DD00}Benzin: %d/2500", Vehicle[GetVehicleTrailer(vehicleid)][vehBenzin]);
						SendClientMessage(playerid, COLOR_WHITE, string);
						Tanke[i][tKasse] -= kosten;
						GiveMoney(playerid, kosten);
					}
                }
    		}
		}
	}

Alles anzeigen

Ich habe noch ein Problem..

iwie funktioniert das mit den float nicht ganz...
Es sind Millionen Zahlen...

So ist das im Enum enthalten:

Float:tMaxLiterBenzin