Auto wird nicht erstellt, geld wird abgezogen

Dachdecker · 3. Januar 2019

Schönen guten Tag ich hab mit einem Tutorial von MrMonat das Autohaus System gescriptet und wenn ich im autohaus bin und im auto bin, dann auf kaufen drücke wird das Geld abgezogen aber das auto nicht erstellt. Also hier nochmal mit PWN code.

Code

if(dialogid == Fahrzeugkaufen)
	{
	    if(response)
	    {
            new idbuy=GetPVarInt(playerid, "buycarid");
			if(!GetPlayerMoney(playerid)>ahCars[idbuy][ahc_preis])
			{
				RemovePlayerFromVehicle(playerid);
				SendClientMessage(playerid, Rot, "Du hast zuwenig Geld");
				return 1;
			}
			// Das wird ausgeführt
			GivePlayerMoney(playerid, -ahCars[idbuy][ahc_preis]);
			SendClientMessage(playerid, Gruen, "Du hast das Auto erfolgreich gekauft");

			//Das aber nicht ---
			SpielerAutoErstellen(playerid, ahCars[idbuy][ahc_model], ahInfo[ahCars[idbuy][ah_id]][ahi_x],ahInfo[ahCars[idbuy][ah_id]][ahi_y],ahInfo[ahCars[idbuy][ah_id]][ahi_z],ahInfo[ahCars[idbuy][ah_id]][ahi_r],1,1);
			printf("Model: %i, X: %f, Y: %f, Z: %f, R: %f",ahCars[idbuy][ahc_model], ahInfo[ahCars[idbuy][ah_id]][ahi_x],ahInfo[ahCars[idbuy][ah_id]][ahi_y],ahInfo[ahCars[idbuy][ah_id]][ahi_z],ahInfo[ahCars[idbuy][ah_id]][ahi_r]);
			RemovePlayerFromVehicle(playerid);
   	    }
	    else
	    {
	        RemovePlayerFromVehicle(playerid);
	        SendClientMessage(playerid, Rot, "Der Vorgang wurde abgebrochen");
	    }
	    return 1;
	}

Alles anzeigen

Kaliber · 3. Januar 2019

Zitat von Dachdecker

SpielerAutoErstellen

Wie sieht denn die Funktion aus?

Dachdecker · 3. Januar 2019

Hey,

Die funktion sieht so aus:

Code

SpielerAutoErstellen(playerid, FahrzeugModel, Float:x, Float:y, Float:z, Float:r, Farbe1, Farbe2)
{
	for(new i=0; i<sizeof(cInfo); i++)
	{
	    if(cInfo[i][id_c] != 0) continue;
		cInfo[i][c_Besitzer] = sInfo[playerid][id];
		cInfo[i][c_x]   =x;
		cInfo[i][c_y]   =y;
		cInfo[i][c_z]   =z;
		cInfo[i][c_r]   =r;
		cInfo[i][c_Farbe1]=Farbe1;
		cInfo[i][c_Farbe2]=Farbe2;
		cInfo[i][c_model] = FahrzeugModel;
		cInfo[i][id_c] = CreateVehicle(FahrzeugModel, x, y, z, r, Farbe1, Farbe2, -1, -1);
		SpielerAutoSpeichernDB(playerid,i);
		return 1;
	}
	return 1;
}

Alles anzeigen

Das funktioniert zu 100%, denn es klappt alles mit dem befehl /veh [Playerid][Modelid][Farbe1][Farbe2] das ist ein test befehl von mir der dann auf diese Funktion zugreift

Kaliber · 3. Januar 2019

Dann lade mal eventuell crashdetect, vielleicht tritt ja ein index Fehler auf

Dachdecker · 3. Januar 2019

Also ich bin jetzt gerade Mal in ein anderes Auto vom Autohaus gestiegen und da ging es, aber es spawnte am falschen Autohaus
Ich habe zwei Autohäuser (provisorisch) und im 1. Autohaus 2 autos (Sultan und Infi) und im 2. Autohaus ein Auto (ModelID 400).

Nur wenn ich in den Infi steige funktioniert alles.

Der Crashdetect gibt das wieder :

[22:31:59] [debug] Run time error 4: "Array index out of bounds"
[22:31:59] [debug] Attempted to read/write array element at index 2 in array of size 2
[22:31:59] [debug] AMX backtrace:
[22:31:59] [debug] #0 0000ae80 in public OnDialogResponse (0, 4, 1, -1, 82208) in MySQLScript.amx

PermanenT · 3. Januar 2019

ahc_model
Wo legst du den Wert hierfür fest?

Dachdecker · 3. Januar 2019

Hier:

Code

enum AutohausCarsEnum
{
	ahc_model,
	Float:ahc_x,
	Float:ahc_y,
	Float:ahc_z,
	Float:ahc_r,
	ahc_preis,
	ahc_id,
	ah_id
}
new ahCars[][AutohausCarsEnum]=
{
	{411,103.6482,-150.6034,2.0219,220.8391, 40000, 0},
	{560,110.9450,-150.6058,1.3968,225.7832, 30000, 0},
	{400,204.1585,-251.1129,1.3129,181.3764, 20000, 1}
};

Alles anzeigen

PermanenT · 3. Januar 2019

ich meine in welcher Funktion sagst du ihm welches Auto es ist?
Sry, meine idbuy
Bzw.
GetPVarInt(playerid, "buycarid");

Und was sagt dein printf in der Console wenn du das erste Auto kaufst?

Dachdecker · 3. Januar 2019

Also hier uberprüfe ich, ob die vID mit einem der Autos von den Autohäusern übereinstimmt und dann setzte ich dieses "i" in die Pvar

Code

if(newstate == PLAYER_STATE_DRIVER)
	{
	    new vID=GetPlayerVehicleID(playerid);
	    for(new i=0; i<sizeof(ahCars); i++)
	    {
	        if(ahCars[i][ah_id] != vID) continue;
	        //Kaufprozess
	        SetPVarInt(playerid, "buycarid", i);
	        new string[200];
	        format(string, sizeof(string), "Möchtest du das Fahrzeug für %i$ kaufen?", ahCars[i][ahc_preis]);
	        ShowPlayerDialog(playerid, Fahrzeugkaufen, DIALOG_STYLE_MSGBOX, "Autokauf",string, "Kaufen", "Abbrechen");
			break;
	    }
	    return 1;
	}

Alles anzeigen

Edgar · 3. Januar 2019

Hallo.

Zeige uns mal bitte ahInfo (enum und dein Array dazu).
Und bitte einmal erwähnen welcher Wert bei welchem Auto bei dieser variable rauskommt: ahCars[idbuy][ah_id]
Du lässt es ja bereits beimn Kaufen printen, einfach mal extra (vor deinem bereits vorhandenen Print) ausgeben.

ich vermute, dass die ID aus der Datenbank gelesen wird?

Grüße.

Dachdecker · 3. Januar 2019

ahInfo und ahCars

Code

enum Autohausenum
{
	Float:ahi_x,
	Float:ahi_y,
	Float:ahi_z,
	Float:ahi_r
}
new ahInfo[][Autohausenum] =
{
	{119.2717,-184.0654,1.2543,269.8809},       //ID=0 == Autohaus1
	{187.9992,-246.2124,1.2286,358.5348}        //ID=1 == Autohaus2
};
enum AutohausCarsEnum
{
	ahc_model,
	Float:ahc_x,
	Float:ahc_y,
	Float:ahc_z,
	Float:ahc_r,
	ahc_preis,
	ahc_id,
	ah_id
}
new ahCars[][AutohausCarsEnum]=
{
	{411,103.6482,-150.6034,2.0219,220.8391, 40000, 0},
	{560,110.9450,-150.6058,1.3968,225.7832, 30000, 0},
	{400,204.1585,-251.1129,1.3129,181.3764, 20000, 1}
};

Alles anzeigen

/* Infernus
[22:43:52] Buycarid: 1
[22:43:52] Model: 411, X: 187.999206, Y: -246.212402, Z: 1.228600, R: 358.534790
*/
/*Sultan
[22:44:01] Buycarid: 2
[22:44:01] [debug] Run time error 4: "Array index out of bounds"
[22:44:01] [debug] Attempted to read/write array element at index 2 in array of size 2
[22:44:01] [debug] AMX backtrace:
[22:44:01] [debug] #0 0000aeac in public OnDialogResponse (0, 4, 1, -1, 82208) in MySQLScript.amx
*/
/* ModelID 400
[22:44:49] Buycarid: 3
[22:44:49] [debug] Run time error 4: "Array index out of bounds"
[22:44:49] [debug] Attempted to read/write array element at index 3 in array of size 2
[22:44:49] [debug] AMX backtrace:
[22:44:49] [debug] #0 0000aeac in public OnDialogResponse (0, 4, 1, -1, 82208) in MySQLScript.amx
*/

Was meinst du? Die Ids werde vom Script zugewiesen laut dem Tut von MrMonat ist alles 1zu1 wie im Tutorial von ihm

Edgar · 3. Januar 2019

Da ist das Problem.

Arrays fangen bei 0 an, deine Autos aber bei 1.

Du musst dem Auto noch einen weiteren Wert (Die ID/Index vom Autohaus) zuweisen.

Ich kenne das Script/Video von ihm nicht, ich halte nicht viel von Tutorials auf Youtube.

// Edit:
Falsch geschaut. Kleinen Moment

Dachdecker · 3. Januar 2019

ich dachte, dass durch das erstellen des Autos bei OnGameModeInit mit einer schleife, die ID gesetzt wird:

Code

// ----- Autohausenum ----- //
	for(new i=0; i<sizeof(ahCars); i++)
	{
	    ahCars[i][ah_id] = AddStaticVehicle(ahCars[i][ahc_model], ahCars[i][ahc_x], ahCars[i][ahc_y], ahCars[i][ahc_z], ahCars[i][ahc_r], 1,1);
	}

Edgar · 3. Januar 2019

Hier: ahInfo[ahCars[idbuy][ah_id]][ahi_x],ahInfo[ahCars[idbuy][ah_id]][ahi_y],ahInfo[ahCars[idbuy][ah_id]][ahi_z],ahInfo[ahCars[idbuy][ah_id]][ahi_r]
verwendest du dauerhaft ah_id statt ahc_id.

Ich bin mir leider grade nicht sicher, aber fangen die Fahrzuge bei ID 1 an?
Wenn dem so ist, musst du idbuy direkt beim auslesen minus 1 rechnen, da wir gesagt ein Array bei Index 0 beginnt, also hat dein Erstes Auto bei ahCars index 0, du verwuchst diesen aber mit der ID vom Fahrzeug (idbuy also 1) aufzurufen.

Dachdecker · 3. Januar 2019

Ja da ah_id durch die schleife bei ongamemode init gesetzt wird oder nicht

Habe es nun auf ahc_id geändert und es funktioniert einwandfrei

Vielen herzlichen dank

Edgar · 3. Januar 2019

Probiere das aber auch bei allen Fahrzeugen aus.

Dachdecker · 6. Januar 2019

Habe ich getan, funktioniert perfekt

Auto wird nicht erstellt, geld wird abgezogen

breadfish.de 12. März 2022

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen

Life Of Economy - Dein neues SAMP-Projekt,