[SAMMELTHREAD] Kleine Scripting Fragen

LeonMrBonnie · 22. Juni 2019

Zitat von BrightLeaN

Oh, ich bin dumm dachte Array wäre, wenn man die Stirng-Länge angibt :X

Das stimmt auch quasi, da in PAWN ein String einfach nur ein Array ist in dem die Buchstaben einzelnd gespeichert werden.

Die Query kann so garnicht funktionieren ~~und sollte eigentlich einen Fehler ausgeben~~, da du vor dem WHERE ID = %d ein Komma zu viel hast. Nach dem letzten Wert der eingetragen werden soll, darf kein Komma folgen.
Merke beim schreiben gerade, dass du die Query auch nirgendwo ausführst.

Und bist du sicher, dass du es nur einmal bei OnPlayerDisconnect ausrufst und sonst nirgends? Zeig uns mal den Codeteil.

Jameso · 22. Juni 2019

In Zeile 55. ist kein Komma. Das Komma kommt nach "

Wie gesagt, beim Disconnect:
SpielerSpeichern(playerid);

Oh, ja die Query wird jetzt auch ausgeführt.
Jetzt updatet der Wert auch.

Warum es da doppelt kommt weiß ich allerdings nicht

LeonMrBonnie · 22. Juni 2019

STRG + F und mal gucken ob du sicher auch nirgendwo anders SpielerSpeichern ausführst?

Jameso · 23. Juni 2019

STRG + F gedrückt und geschaut, wird sonst nicht ausgeführt, nur beim Disconnect erscheint beides sofort.
Ist auch egal, Fehler wurde aufgeschrieben, bis ich vorerst keine negativen Auswirkungen festelle, verschiebe ich das mal...

Bin gerade überfordert, wie frage ich in einer Schleife ab, ob ein Spieler in der Nähe ist mit einer Nachricht wenn nicht.
So hört er ja komplett auf, wenn ich das return und wenn ich in der schleife den else part mache ist ja komplett doof

Was wäre effektiv? (Zeile 4.)

Also so gehts nicht

Code

{
	for(new i;i<sizeof(gInfo);i++)
	{
		if(!IsPlayerInRangeOfPoint(playerid,2.5,gInfo[i][gX],gInfo[i][gY],gInfo[i][gZ])) return SCM(playerid,-1,"Nicht in näheee");
		{
			if(gInfo[i][gLock] > 0) return 1;
			if(gInfo[i][gBesitzer_dbID] == pInfo[playerid][pGarage])
			{
	            SCM(playerid,-1,"TEST_1");
	            return 1;
			}
			else
			{
			    if(IsPlayerInAnyVehicle(playerid)) return SCM(playerid,-1,"Test_0");
			}
		}
	}
	return 1;
}

Alles anzeigen

SkeeZy · 23. Juni 2019

Zitat von BrightLeaN
STRG + F gedrückt und geschaut, wird sonst nicht ausgeführt, nur beim Disconnect erscheint beides sofort.
Ist auch egal, Fehler wurde aufgeschrieben, bis ich vorerst keine negativen Auswirkungen festelle, verschiebe ich das mal...

Bin gerade überfordert, wie frage ich in einer Schleife ab, ob ein Spieler in der Nähe ist mit einer Nachricht wenn nicht.
So hört er ja komplett auf, wenn ich das return und wenn ich in der schleife den else part mache ist ja komplett doof

Was wäre effektiv? (Zeile 4.)

Also so gehts nicht
Code
{
	for(new i;i<sizeof(gInfo);i++)
	{
		if(!IsPlayerInRangeOfPoint(playerid,2.5,gInfo[i][gX],gInfo[i][gY],gInfo[i][gZ])) return SCM(playerid,-1,"Nicht in näheee");
		{
			if(gInfo[i][gLock] > 0) return 1;
			if(gInfo[i][gBesitzer_dbID] == pInfo[playerid][pGarage])
			{
	            SCM(playerid,-1,"TEST_1");
	            return 1;
			}
			else
			{
			    if(IsPlayerInAnyVehicle(playerid)) return SCM(playerid,-1,"Test_0");
			}
		}
	}
	return 1;
}
Alles anzeigen
Alles anzeigen

Code

{


	for(new i;i<sizeof(gInfo);i++)


	{


		if(!IsPlayerInRangeOfPoint(playerid,2.5,gInfo[i][gX],gInfo[i][gY],gInfo[i][gZ])) return SCM(playerid,-1,"Nicht in näheee");


		if(gInfo[i][gLock] > 0) return 1;


		if(gInfo[i][gBesitzer_dbID] == pInfo[playerid][pGarage])


		{


	            SCM(playerid,-1,"TEST_1");


	            return 1;


		}


		else


		{


	            if(IsPlayerInAnyVehicle(playerid)) return SCM(playerid,-1,"Test_0");


		}
	}


	return 1;


}

Alles anzeigen

versuchs mal damit.

Wenn du diese Zeile nutzt, solltest du dahinter keine Klammer öffnen. Es sind beide Fälle durchs return geregelt:

if(!IsPlayerInRangeOfPoint(playerid,2.5,gInfo[i][gX],gInfo[i][gY],gInfo[i][gZ])) return SCM(playerid,-1,"Nicht in näheee");

ich hoffe mal ich habe das ganze richtig verstanden

Jameso · 23. Juni 2019

Ja das letzte ist mir bewusst, habe es eben geändert, um zu zeigen was mein Ziel ist

Aber es ist doch so, dass er ja bei der Schleife mit der 0 beginnt, wenn ich dann returne gibts keinen Callback mehr zumindest geht der nicht weiter.
Heißt die Schleife macht theoretisch dann nicht mehr weiter oder?

Ist der Spieler nicht in der Nähe von i returnt er ja, was ist mit dem Rest?

SkeeZy · 23. Juni 2019

Code

}
	for(new i;i<sizeof(gInfo);i++)
	{
		if(IsPlayerInRangeOfPoint(playerid,2.5,gInfo[i][gX],gInfo[i][gY],gInfo[i][gZ]))
		{
			if(gInfo[i][gLock] > 0)
			{
			    //was von hier
				if(gInfo[i][gBesitzer_dbID] == pInfo[playerid][pGarage])
				{
	            	SCM(playerid,-1,"TEST_1");
	            	return 1;
				}
				else
				{
   					if(IsPlayerInAnyVehicle(playerid))
					{
                        SCM(playerid,-1,"Test_0");
					}
				}//bis hier passieren soll ist mir ein wenig unschlüssig.
			}
		} else SCM(playerid, -1, "Nicht in der Nähe!");
	}
	return 1;
}

Alles anzeigen

Einfach alle Returns rausnehmen und anders bauen. So sollte es klappen, allerdings ist mir die Funktion des Bereichs zwischen meinen Kommentaren ein wenig fragwürdig.

Jameso · 23. Juni 2019

Ich habe in einer Datenbank einmal die WorldIDs http://prntscr.com/o5jzt9
Und einmal ein InteriorType http://prntscr.com/o5k001

Wie kriege ich den höchsten Wert raus wo InteriorType 1 ist?
Damit ich für den nächsten Eintrag +1 rechnen kann. Jeder Typ kann ja 255 Virtual Worlds haben

Douq · 23. Juni 2019

Zitat von BrightLeaN

Ich habe in einer Datenbank einmal die WorldIDs http://prntscr.com/o5jzt9
Und einmal ein InteriorType http://prntscr.com/o5k001

Wie kriege ich den höchsten Wert raus wo InteriorType 1 ist?
Damit ich für den nächsten Eintrag +1 rechnen kann. Jeder Typ kann ja 255 Virtual Worlds haben

Prinzipiell kannst du Merkmalsabfragen per WHERE spezifizieren => WHERE InteriorType = 1. Falls du den letzten Eintrag haben möchtest, kannst du per ORDER BY id DESC LIMIT 1 dir deine Dateneinträge abwärts nach ID sortieren und dir einen Eintrag (LIMIT 1) holen. Damit hättest du dann den höchsten Wert innerhalb der Tabelle -> dein Vorhaben klingt aber so, dass das auch ein AUTO_INCREMENT übernehmen könnte?

Jameso · 23. Juni 2019

Ja aber, wo das InteriorType = 2 ist soll ja auch neu gezählt werden. So das jeder Typ die maximale Anzahl an VirtuellenIDS erstellen kann, dann habe ich am Ende mehr davon.
AUTO_INCREMENT hilft ja dann, hätte ich dieses Types nicht würde ich sagen.

Wie erstelle ich ein DynamicPickup in allen Virtuellen Welten? Das selbe auch für 3DTextLabels,
soll ich wirklich eine Schleife durchlaufen lassen? Oder gibts da was anderes?

SkeeZy · 23. Juni 2019

Zitat von BrightLeaN

Wie erstelle ich ein DynamicPickup in allen Virtuellen Welten? Das selbe auch für 3DTextLabels,
soll ich wirklich eine Schleife durchlaufen lassen? Oder gibts da was anderes?

Normalerweise solltest du beim Pickups-Erstellen eine virtuelle welt eintragen können. Wenn du dort -1 einträgst, ist dieses in allen VW's sichtbar.

Jameso · 23. Juni 2019

Bin am verzweifeln, was ist denn hier falsch? er zeigt mir sogar den Wert an, heißt das es doch richtig sein muss.

C

mysql_format(handle, query, sizeof(query), "INSERT INTO garage (X, Y, Z, ID, Kaufpreis, InteriorType, existiert, Lock) VALUES");
			mysql_format(handle, query, sizeof(query), "%s ('%f','%f','%f','%d','%d','%d','%d','%d')"//query,........

http://prntscr.com/o5lawi

Code

[16:06:41] [DEBUG] processing internal specifier 'c'
[16:06:41] [DEBUG] pushed value '1' onto AMX stack
[16:06:41] [DEBUG] processing internal specifier 's'
[16:06:41] [DEBUG] pushed value 'INSERT INTO garage (X, Y, Z, ID, Kaufpreis, InteriorType, existiert, Lock) VALUES ('775.807861','-1379.737549','13.651802','3','25825','1','1','1')' onto AMX stack
[16:06:41] [DEBUG] processing internal specifier 's'
[16:06:41] [DEBUG] pushed value '' onto AMX stack
[16:06:41] [DEBUG] processing internal specifier 's'
[16:06:41] [DEBUG] pushed value 'You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'Lock) VALUES ('775.807861','-1379.737549','13.651802','3','25825','1','1','1')' at line 1' onto AMX stack
[16:06:41] [DEBUG] processing internal specifier 'c'

LeonMrBonnie · 23. Juni 2019

Warum setzt du die alle als Strings? Das sind alles Zahlen, du musst also die ' weglassen.

Jameso · 23. Juni 2019

Das wusste ich gar nicht, habe es überall so und es geht, heißt quasi nur bei %e die ' benutzen ?

Allerdings ändert das nichts, Fehler tritt immer noch auf

SkeeZy · 23. Juni 2019

Huhu,
ich habe vor wenn der Server startet alle Häuser aus der MySQl-Tabelle in die Enumvariable zu setzen.
Es gibt max. 700 Häuser(MAX_HOUSES). Und da ich aktuell per Schleife 700 Datenbankanfragen habe, suche ich nach einer besseren alternative, die häuser anhand ihrer hID in die richtige Variable zu setzen.
Ein Haus, dass einmal die hID 15 hatte, soll diese nach dem Speichern und laden auch weiterhin behalten.
Soweit klappt es nur stört mich diese Schleife im....

...OnGameModeInit:

Code

for(new i=0, j=MAX_HOUSES; i<=j; i++)
	{
		loadHouse(i);//Hier lade ich das Haus anhand seiner ID
	}


	new housesstring[64];
	format(housesstring, sizeof(housesstring), "Geladene Häuser: %d", loadedHouses);
	print(housesstring);

stock loadHouse(houseid):

Code

stock loadHouse(houseid)
{
	new query[256];
	mysql_format(handle, query, sizeof(query), "SELECT * FROM houses WHERE h_id = '%d'", houseid);
	mysql_pquery(handle, query, "onHouseLoaded", "d", houseid);
	return 1;
}

onHouseLoaded:

Code

forward onHouseLoaded(houseid);
public onHouseLoaded(houseid)
{
	new rows;
	cache_get_row_count(rows);
	if(rows != 0)
	{
		cache_get_value_name_int(0, "h_db", hInfo[houseid][hDB]);
		cache_get_value_name_float(0, "h_x", hInfo[houseid][hX]);
		cache_get_value_name_float(0, "h_y", hInfo[houseid][hY]);
		cache_get_value_name_float(0, "h_z", hInfo[houseid][hZ]);
		cache_get_value_name_int(0, "h_owner", hInfo[houseid][hOwner]);
		cache_get_value_name_int(0, "h_int", hInfo[houseid][hInt]);
		cache_get_value_name_int(0, "h_price", hInfo[houseid][hPrice]);
		cache_get_value_name_int(0, "h_rent", hInfo[houseid][hRent]);
		cache_get_value_name_int(0, "h_rentable", hInfo[houseid][hRentable]);
		cache_get_value_name_int(0, "h_level", hInfo[houseid][hLevel]);
		cache_get_value_name(0, "h_db", hInfo[houseid][hDesc]);
		updateHouse(houseid);
		loadedHouses++;
	}
	return 1;
}

Alles anzeigen

Ich bedanke mich schonmal im Voraus und hoffe auf eine passendere Lösung.

LeonMrBonnie · 23. Juni 2019

Zitat von BrightLeaN

Das wusste ich gar nicht, habe es überall so und es geht, heißt quasi nur bei %e die ' benutzen ?

Allerdings ändert das nichts, Fehler tritt immer noch auf

Du brauchst auch bei %e die ' nicht, die werden durch das Escapen automatisch eingefügt.
Aber ja, die ' brauchst du nur bei Strings.

Zitat von SkeeZy
Huhu,
ich habe vor wenn der Server startet alle Häuser aus der MySQl-Tabelle in die Enumvariable zu setzen.
Es gibt max. 700 Häuser(MAX_HOUSES). Und da ich aktuell per Schleife 700 Datenbankanfragen habe, suche ich nach einer besseren alternative, die häuser anhand ihrer hID in die richtige Variable zu setzen.
Ein Haus, dass einmal die hID 15 hatte, soll diese nach dem Speichern und laden auch weiterhin behalten.
Soweit klappt es nur stört mich diese Schleife im....

...OnGameModeInit:
Code
for(new i=0, j=MAX_HOUSES; i<=j; i++)
	{
		loadHouse(i);//Hier lade ich das Haus anhand seiner ID
	}


	new housesstring[64];
	format(housesstring, sizeof(housesstring), "Geladene Häuser: %d", loadedHouses);
	print(housesstring);
stock loadHouse(houseid):
Code
stock loadHouse(houseid)
{
	new query[256];
	mysql_format(handle, query, sizeof(query), "SELECT * FROM houses WHERE h_id = '%d'", houseid);
	mysql_pquery(handle, query, "onHouseLoaded", "d", houseid);
	return 1;
}
onHouseLoaded:
Code
forward onHouseLoaded(houseid);
public onHouseLoaded(houseid)
{
	new rows;
	cache_get_row_count(rows);
	if(rows != 0)
	{
		cache_get_value_name_int(0, "h_db", hInfo[houseid][hDB]);
		cache_get_value_name_float(0, "h_x", hInfo[houseid][hX]);
		cache_get_value_name_float(0, "h_y", hInfo[houseid][hY]);
		cache_get_value_name_float(0, "h_z", hInfo[houseid][hZ]);
		cache_get_value_name_int(0, "h_owner", hInfo[houseid][hOwner]);
		cache_get_value_name_int(0, "h_int", hInfo[houseid][hInt]);
		cache_get_value_name_int(0, "h_price", hInfo[houseid][hPrice]);
		cache_get_value_name_int(0, "h_rent", hInfo[houseid][hRent]);
		cache_get_value_name_int(0, "h_rentable", hInfo[houseid][hRentable]);
		cache_get_value_name_int(0, "h_level", hInfo[houseid][hLevel]);
		cache_get_value_name(0, "h_db", hInfo[houseid][hDesc]);
		updateHouse(houseid);
		loadedHouses++;
	}
	return 1;
}
Alles anzeigen
Ich bedanke mich schonmal im Voraus und hoffe auf eine passendere Lösung.
Alles anzeigen

Warum lädst du nicht einfach alle Häuser aus der Datenbank, indem du das WHERE Statement weglässt und loopst dann durch alle Zeilen durch? Dann lädst du doch alle vorhandenen Häuser.

SkeeZy · 23. Juni 2019

Zitat von LeonMrBonnie

Warum lädst du nicht einfach alle Häuser aus der Datenbank, indem du das WHERE Statement weglässt und loopst dann durch alle Zeilen durch? Dann lädst du doch alle vorhandenen Häuser.

Hey, danke für den Tipp. Könntest du mir ggf. auch sagen, wie ich durch die Zeilen loope?
Und prüfe welche hID die entsprechende Zeile hat bzw. diese dann zuweise?
Stelle mir das wie folgt vor, bin mir aber ein wenig unsicher.

Code

stock loadHouses()
{
	new query[256];
	mysql_format(handle, query, sizeof(query), "SELECT * FROM houses");
	mysql_pquery(handle, query, "onHouseLoaded");
	return 1;
}

Code

forward onHouseLoaded(houseid);
public onHouseLoaded(houseid)
{
	new rows;
	cache_get_row_count(rows);
	if(rows != 0)
	{
		for(new i=0, j=MAX_HOUSES; i<=j; i++) //gehört hier j=MAX_HOUSES(700) oder j=rows hin ?
		{
			//ka wie ich jetzt prüfe, in welche hID in dieser row gespeichert ist
		}
	}
	return 1;
}

Alles anzeigen

LeonMrBonnie · 23. Juni 2019

Code

forward onHouseLoaded();
public onHouseLoaded()
{
	new rows;
	cache_get_row_count(rows);
	if(rows != 0)
	{
		for(new i=0, j=rows; i<j; i++)
		{
		cache_get_value_name_int(i, "h_db", hInfo[i][hDB]);
		cache_get_value_name_float(i, "h_x", hInfo[i][hX]);
		cache_get_value_name_float(i, "h_y", hInfo[i][hY]);
		cache_get_value_name_float(i, "h_z", hInfo[i][hZ]);
		cache_get_value_name_int(i, "h_owner", hInfo[i][hOwner]);
		cache_get_value_name_int(i, "h_int", hInfo[i][hInt]);
		cache_get_value_name_int(i, "h_price", hInfo[i][hPrice]);
		cache_get_value_name_int(i, "h_rent", hInfo[i][hRent]);
		cache_get_value_name_int(i, "h_rentable", hInfo[i][hRentable]);
		cache_get_value_name_int(i, "h_level", hInfo[i][hLevel]);
		cache_get_value_name(i, "h_db", hInfo[i][hDesc]);
		updateHouse(i);
		loadedHouses++;
		}
	}
	return 1;
}

Alles anzeigen

SkeeZy · 23. Juni 2019

Ah ok, im Endeffekt bleibt es ja fast gleich. #BrainDamagedHorse

Eine Sache noch:
-Die h_db ist mein Primary Key in der Enumvariable sollte i allerdings hID(Wert der die houseid speichert) sein.
Kann ich dann einfach folgendes machen?

C

new houseid; //dies
for(new i=0, j=rows; i<j; i++)


		{
                cache_get_value_name_int(i, "h_id", houseid);//und dies


		cache_get_value_name_int(i, "h_db", hInfo[houseid][hDB]);
		cache_get_value_name_float(i, "h_x", hInfo[houseid][hX]);
		cache_get_value_name_float(i, "h_y", hInfo[houseid][hY]);
		cache_get_value_name_float(i, "h_z", hInfo[houseid][hZ]);
		cache_get_value_name_int(i, "h_owner", hInfo[houseid][hOwner]);
		cache_get_value_name_int(i, "h_int", hInfo[houseid][hInt]);
		cache_get_value_name_int(i, "h_price", hInfo[houseid][hPrice]);
		cache_get_value_name_int(i, "h_rent", hInfo[houseid][hRent]);
		cache_get_value_name_int(i, "h_rentable", hInfo[houseid][hRentable]);
		cache_get_value_name_int(i, "h_level", hInfo[houseid][hLevel]);
		cache_get_value_name(i, "h_db", hInfo[houseid][hDesc]);
		updateHouse(houseid);
		loadedHouses++;


		}

Alles anzeigen

Beitrag von Jameso (23. Juni 2019)