[SAMMELTHREAD] Kleine Scripting Fragen

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.");

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)

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

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.

Code:

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;
}

Laut laden ja siehe:

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

Zitat von Dr. Frauenarzt

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

Was passiert, wenn du da rein schreibst:

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

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

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

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

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

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...

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

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

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?

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

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

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

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);
					}
                }
    		}
		}
	}

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

Schreibt du deinen Code eigentlich absichtlich so verschachtelt, ich habe dir soo oft schon gezeigt wie du das schöner machen kannst.

Und wiiiiiieso startest du Schleifen immer ab 1???

Ich habe dir jetzt schon 1000x erklärt, dass Arrays in Pawn/C/Java/C#/C++/Javascript und allen anderen C ähnlichen Sprachen bei 0 beginnt.

Lässt du das ab 1 starten, überspringst du immer deinen 1. Eintrag..und eine Tankstelle wird nicht funktionieren

//Pack das mal oben in dein Skript:
#define SCME(%0,%1,%2,%3) format(string,sizeof(string),%2,%3),SCM(%0,%1,string)


//Alles andere schreib mal so:
if(Spieler[playerid][pFraktion] != 17) return SCM(playerid, COLOR_WHITE, ERROR_FRAKTION);
new veh = GetPlayerVehicleID(playerid), model = GetVehicleModel(veh), trailer = GetVehicleTrailer(veh);
if(model != 514 && model != 515) return SCM(playerid,0xAFAFAF,"Du befindest dich in einem ungültigen Fahrzeug.");
if(!IsTrailerAttachedToVehicle(veh)) return SCM(playerid, COLOR_WHITE, "{AFAFAF}Du hast dein Produkt Anhänger nicht angeschlossen!");
if(GetVehicleModel(trailer) != 584) return SCM(playerid, COLOR_WHITE, "{AFAFAF}Du hast den falschen Anhänger. Du brauchst den Tank Anhänger!");
new i = GetTankstelle(playerid);
if(i == -1) return SCM(playerid,0xAFAFAF,"Du befindest dich an keiner Tankstelle.");


new gebraucht = floatround(Tanke[i][tMaxLiterBenzin] - Tanke[i][tLiterBenzin]);
new kosten = Tanke[i][tBenzinAnkauf]*gebraucht;


if(Tanke[i][tKasse] < kosten) return SCM(playerid, COLOR_WHITE, "{AFAFAF}Die Tankstelle hat nicht genügend Geld für den Benzin.");
if(!gebraucht) return SCM(playerid, COLOR_WHITE, "{AFAFAF}Dieses Tankstelle benötigt keinen Benzin mehr.");


if(Vehicle[trailer][vehBenzin] > gebraucht)
{
    Vehicle[trailer][vehBenzin] -= gebraucht, Tanke[i][tMaxLiterBenzin] += floatadd(gebraucht, Tanke[i][tLiterBenzin]);
    SCME(playerid,-1,"{21DD00}Es konnten nur %d Liter Benzin abgeliefert werden, die Tankstelle ist voll.", gebraucht);
}
else Tanke[i][tMaxLiterBenzin] += Vehicle[trailer][vehBenzin], Vehicle[trailer][vehBenzin] = 0;


SCME(playerid,-1,"{21DD00}Gehalt: %d$", kosten);
SCME(playerid,-1,"{21DD00}Benzin: %d/2500", Vehicle[trailer][vehBenzin]);
UpdateTankLabel(i, Tanke[i][tOwned]);
Tanke[i][tKasse] -= kosten, GiveMoney(playerid, kosten);


//Und die Funktion haste dann
stock GetTankstelle(playerid)
{
    for(new i; i<MAX_TANKEN; i++)
    {
        if(Tanke[i][tCreated] == 1 && IsPlayerInRangeOfPoint(playerid, 5.0, Tanke[i][tPickupX], Tanke[i][tPickupY], Tanke[i][tPickupZ])) return i;
    }
    return -1;
}

Es ist wichtig Redundanz zu vermeiden.

Desweiteren hattest du da was beim else vertauscht, der Code löst schon mal ein Problem, was dir bei deinem vorherigen Code nicht aufgefallen ist.

Du sagst zwar es werden irgendwo viele Zahlen ausgegeben...aber du das nicht wo...was soll das? Sollen wir jetzt raten bei welcher Ausgabe das passiert?!?!
Gib mal präzise Infos, was genau ausgegeben wird...etc pp
Printe dir eventuell auch mal Werte und gib einen schönen Debug der Variablen an, dann kann man dir auch sofort helfen..

Immer diese halben Sachen und komischen Codes.

Generell könntest du dir soviel arbeit sparen, wenn du deine Systeme mal vernünftig auslagern würdest.
Ich wette bis zu 70% deiner Zeit verschwendest du damit redundanten Code zu schreiben.

new Reward[MAX_PLAYERS];

Reward[playerid] = SetTimerEx("PlayReward", 900000, true, "i", playerid);

forward public PlayReward(playerid);public PlayReward(playerid){ for(new i=0; i<MAX_PLAYERS; i++) { if(PlayerInfo[i][LoggedIn] == false) continue; if(isPlayerAFK(i)) return 1; PlayerInfo[i][Tokens] += 5; SFM(playerid,COLOR_COP,"Reward: {FFFFFF}Du bekommst 5 Tokens!"); SavePlayer(i); } return 1;}

public OnPlayerDisconnect(playerid, reason){ KillTimer(Reward[playerid]);

Das Problem dabei ist dass das System manchmal 5x 5 Tokens gibt. (manchmal auch 2 oder 6, ist immer unterschiedlich.) Hat jemand eine Lösung dafür?

Zitat von Romnus

as Problem dabei ist dass das System manchmal 5x 5 Tokens gibt. Hat jemand eine Lösung dafür?

Du erstellst einen Timer für jeden Spieler und Loopst bei jedem Spieler durch alle Spieler o:

Daaavon ist abzuraten

//Edit:
Ahh wait, du willst, dass für jeden Spieler der Timer individuell gestartet wird.

Mach es so:

//Unter OnPlayerConnect
SetPVarInt(playerid,"reward_timer", SetTimerEx("PlayReward", 900000, true, "i", playerid));


//Unter OnPlayerDisconnect
KillTimer(GetPVarInt(playerid, "reward_timer"));


forward PlayReward(i);
public PlayReward(i)
{
    if(!PlayerInfo[i][LoggedIn] || isPlayerAFK(i)) return 1;
    PlayerInfo[i][Tokens] += 5;
    SFM(i,COLOR_COP,"Reward: {FFFFFF}Du bekommst 5 Tokens!");
    SavePlayer(i); //Are you sure about this?
    return 1;
}

Bist du sicher, dass du da den Spieler direkt speichern magst?

Also kannst du machen, aber reicht es nicht unter OnPlayerDisconnect oder dient das der Server-Crash Vorbeugung?