Keystate (ENTER/F) hat keine Auswirkung mit der for-Schleife bei RangeOfPoint

EllBundy · 22. Mai 2019

Ich habe mein dynamisches Fahrradverleihsystem soweit fertig, habe aber ein Problem, dass sich einfach beheben lässt.
Und zwar wird der Fahrradverleih aus der Datenbank ausgelesen, erstellt und ist vollkommen sichtbar.
Ein Harken bleibt dennoch, sobald ich auf das erstellte Icon gehe und die Taste ENTER/F Drücke erscheint einfach der Dialog nicht wie geplant.
Die Console gibt wiederrum beide erstellten Verleihs als druchlauf an.

Ich hoffe Ihr könnt mir helfen

Code

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
    new str[256];
	switch(newkeys)
	{
	    case 16: //Enter
		{
		    for(new i; i!=FVerleihCount; i++)
		    {
		        if(IsPlayerInRangeOfPoint(playerid, 1.0, FVerleih[i][fCoords][0], FVerleih[i][fCoords][1], FVerleih[i][fCoords][2]))
		        {
		            SetPVarInt(playerid, "FVerleih", i);
		            format(str, sizeof(str),"{FFFFFF}Herzlich Willkommen "#SERVERFARBE"%s{FFFFFF},\nMöchtest du dir ein Fahrrad für 30 Minuten leihen ?", SpielerInfo[playerid][pName]);
        			ShowPlayerDialog(playerid, DIALOG_FAHRRADLEIH, DIALOG_STYLE_MSGBOX, ""#SERVERFARBE"Fahrradverleih",str, "Ja","Nein");
        			return 1;
				}
				printf("Fahrradverleih ENTER NR. %d", i);
		    }
//...

Alles anzeigen

im nachfolgenden ist der Ausschnitt, wo ich den Verleih lade und auch erfolgreich erstellt wird

Code

public LoadDataInfo(database)
{
	switch(database)
	{
	    case 0: // fahrradverleih_list
	    {
			if(cache_num_rows() != 0)
			{
			    for(new i; i!=cache_num_rows(); i++)
			    {
			        cache_get_value_name_int(i,"pID",FVerleih[i][fID]);   
			        cache_get_value_name_int(i,"PickupID",FVerleih[i][fIconID]);   
			        cache_get_value_name(i,"Name",FVerleih[i][fName]);    	
			        cache_get_value_name_float(i,"coordsX",FVerleih[i][fCoords][0]);
			        cache_get_value_name_float(i,"coordsY",FVerleih[i][fCoords][1]);
			        cache_get_value_name_float(i,"coordsZ",FVerleih[i][fCoords][2]);
			        cache_get_value_name_float(i,"coordsX2",FVerleih[i][fCoords][3]);
			        cache_get_value_name_float(i,"coordsY2",FVerleih[i][fCoords][4]);
			        cache_get_value_name_float(i,"coordsZ2",FVerleih[i][fCoords][5]);
			        cache_get_value_name_float(i,"coordsFA",FVerleih[i][fCoords][6]);
			        cache_get_value_name_int(i,"BikePrice",FVerleih[i][fBikePrice]);   
			        cache_get_value_name_int(i,"BMXPrice",FVerleih[i][fBMXPrice]);   
			        cache_get_value_name_int(i,"MBikePrice",FVerleih[i][fMBikePrice]);   
			        cache_get_value_name_int(i,"SchlossPrice",FVerleih[i][fSchlossPrice]);
			        cache_get_value_name_int(i,"GPSPrice",FVerleih[i][fGPSPrice]); 
			        cache_get_value_name_int(i,"Konto",FVerleih[i][fKonto]);
			        FVerleih[i][fLogged] = true;
			        new str[128];
			        FVerleih[i][fPickupID] = CreatePickup(FVerleih[i][fIconID], 1, FVerleih[i][fCoords][0],FVerleih[i][fCoords][1],FVerleih[i][fCoords][2]);
			        format(str, sizeof(str), ""#SERVERFARBE"%s\n{FFFFFF}Drücke ENTER/F",FVerleih[i][fName]);
			        FVerleih[i][fTXT3D] = Create3DTextLabel(str, 0xFFFFFFFF, FVerleih[i][fCoords][0],FVerleih[i][fCoords][1],FVerleih[i][fCoords][2], 5, 0);
       			}
			}
else
			{
			    return printf("[DATABASE] '%s' wurde kein Eintrag gefunden.", TableName[database]);
			}
			FVerleihCount = cache_num_rows();
			return printf("[DATABASE] '%s' wurden %d Einträge geladen.", TableName[database], cache_num_rows());
	    }

Alles anzeigen

Akino Kiritani · 22. Mai 2019

Das Problem bei Keystates ist, dass sich diese nicht einfach prüfen lassen, wie normale Zahlen, da diese als so genannte "Bit Mask" in PAWN landen.
Das hat damit zu tun, das man mehrere Tasten gleichzeitig drücken kann.

Hier findest du eigentlich eine gute Erklärung dazu wie man Keys richtig prüft. [wiki]OnPlayerKeyStateChange#How_NOT_to_check_for_a_key[/wiki][wiki]OnPlayerKeyStateChange#God_mode_while_holding_use[/wiki]

EllBundy · 22. Mai 2019

Ja warum funktioniert es dann bei den vordefinierten Koordinaten ?

Hier erstelle im Scriptkopf die Standartgebäude:

Code

stock const ServerStates[MAX_SERVERSTATES][sCFG] = {
	{ PICKUP_PFEIL,3,0,""#SERVERFARBE"Stadthalle\n{FFFFFF}Drücke ENTER/F",1480.8480,-1771.5313,18.7958,390.7539,173.8786,1008.3828,true},
	{ PICKUP_PFEIL,0,0,""#SERVERFARBE"Citybank\n{FFFFFF}Drücke ENTER/F",1469.4771,-1011.4132,26.8438,2305.5535,-16.3067,26.7496,true}
};

hier habe ich sie auch bei KeyState unter der taste ENTER/F

Code

case 16: //Enter
		{
		    for(new i; i!=FVerleihCount; i++)
		    {
		        if(IsPlayerInRangeOfPoint(playerid, 1.0, FVerleih[i][fCoords][0], FVerleih[i][fCoords][1], FVerleih[i][fCoords][2]))
		        {
		            SetPVarInt(playerid, "FVerleih", i);
		            format(str, sizeof(str),"{FFFFFF}Herzlich Willkommen "#SERVERFARBE"%s{FFFFFF},\nMöchtest du dir ein Fahrrad für 30 Minuten leihen ?", SpielerInfo[playerid][pName]);
        			ShowPlayerDialog(playerid, DIALOG_FAHRRADLEIH, DIALOG_STYLE_MSGBOX, ""#SERVERFARBE"Fahrradverleih",str, "Ja","Nein");
        			return 1;
				}
				printf("Fahrradverleih ENTER NR. %d", i);
		    }
		    for(new i; i!=MAX_SERVERSTATES; i++)
		    {
			    if(IsPlayerInRangeOfPoint(playerid, 1.0, ServerStates[i][sX],ServerStates[i][sY],ServerStates[i][sZ]))
	     		{
	     		    SetPlayerPos(playerid, ServerStates[i][sX2],ServerStates[i][sY2],ServerStates[i][sZ2]);
	     		    SetPlayerVirtualWorld(playerid, ServerStates[i][sVirtualWorld]);
	     		    SetPlayerInterior(playerid, ServerStates[i][sInterior]);
	     		    format(str, sizeof(str), "~g~%s", ServerStates[i][sTXT]);
	     		    GameTextForPlayer(playerid, str, 600, 3);
	     		    return 1;
				}
				else if(IsPlayerInRangeOfPoint(playerid, 1.0, ServerStates[i][sX2],ServerStates[i][sY2],ServerStates[i][sZ2]) && GetPlayerInterior(playerid) == 3)
	     		{
	     		    SetPlayerPos(playerid, ServerStates[i][sX],ServerStates[i][sY],ServerStates[i][sZ]);
	     		    SetPlayerVirtualWorld(playerid, 0);
	     		    SetPlayerInterior(playerid, 0);
	     		    GameTextForPlayer(playerid, "~r~Auf Wiedersehen", 600, 3);
	     		    return 1;
				}
			}

Alles anzeigen

Hier funktioniert das ganze einwandfrei
Also denke ich nicht das es an der Keyprüfung liegt

Akino Kiritani · 22. Mai 2019

Dann musst du anfangen zu debugen.

Lass dir via printf mal die ID des Verleihs ausgeben und die Koordinaten, wenn du die Enter Taste drückst.

EllBundy · 22. Mai 2019

Code: server_log.log

[22:49:34] Fahrradverleih 0:
[22:49:34] 0.000000 0.000000 0.000000
[22:49:34] Fahrradverleih 1:
[22:49:34] 1182.340332 -1332.666992 13.583100

Habe es mal bei dem zweiten Verleih probiert und da funktioniert ist scheinbar.
Beim Ersten Verleih (ID 0) nicht, da ja die Coords 0 betragen, woran kann das den liegen ?

EDIT:
Habe mal ein 3. Verleih hinzugefügt, aber beim Start des Server, bekomme ich keinen Debug von public LoadDataInfo, das die Tabelle ausgelesen und geladen wurde. Sobald ich den 3. Eintrag lösche ist alles wieder beim alten

EllBundy · 23. Mai 2019

Ich bin der Sache nun näher auf de Grund gegangen und hab wie du gesagt hast gedebugt.
Mir kam das seltsam vor, dass er erstellt wird, aber danach nicht exestiert. Somit habe ich die for-schleife bei LoadData unter die Lupe genommen und mit hilfe des Debugs festegestellt,
das es beim ersten durchlauf gelesen und erstellt wird. Sobald der 2. durchlauf startet, wird der der 1. resettet.
Ich habe mir den Code näher angeschaut und kann mir einfach nicht erklären, woran das liegen

Ich hoffe ihr könnt mir bei dem Problem helfen, da ich selbst schon am verzweifeln bin, aber einfach nicht aufgeben will und alles Manuell eintragen will, sowie einzeln auslesen...

Danke im Vorraus

Code

forward LoadDataInfo(database);
public LoadDataInfo(database)
{
	switch(database)
	{
	    case 0: // fahrradverleih_list
	    {
			if(cache_num_rows() != 0)
			{
			    for(new i; i!=cache_num_rows(); i++)
			    {
			        cache_get_value_name_int(i,"pID",FVerleih[i][fID]);   
			        cache_get_value_name_int(i,"PickupID",FVerleih[i][fIconID]);   
			        cache_get_value_name(i,"Name",FVerleih[i][fName]);    	
			        cache_get_value_name_float(i,"coordsX",FVerleih[i][fCoords][0]);
			        cache_get_value_name_float(i,"coordsY",FVerleih[i][fCoords][1]);
			        cache_get_value_name_float(i,"coordsZ",FVerleih[i][fCoords][2]);
			        cache_get_value_name_float(i,"coordsX2",FVerleih[i][fCoords][3]);
			        cache_get_value_name_float(i,"coordsY2",FVerleih[i][fCoords][4]);
			        cache_get_value_name_float(i,"coordsZ2",FVerleih[i][fCoords][5]);
			        cache_get_value_name_float(i,"coordsFA",FVerleih[i][fCoords][6]);
			        cache_get_value_name_int(i,"BikePrice",FVerleih[i][fBikePrice]);   
			        cache_get_value_name_int(i,"BMXPrice",FVerleih[i][fBMXPrice]);   
			        cache_get_value_name_int(i,"MBikePrice",FVerleih[i][fMBikePrice]);   
			        cache_get_value_name_int(i,"SchlossPrice",FVerleih[i][fSchlossPrice]);
			        cache_get_value_name_int(i,"GPSPrice",FVerleih[i][fGPSPrice]); 
			        cache_get_value_name_int(i,"Konto",FVerleih[i][fKonto]);
			        FVerleih[i][fLogged] = true;
			        FVerleih[i][fPickupID] = CreatePickup(FVerleih[i][fIconID], 1, FVerleih[i][fCoords][0],FVerleih[i][fCoords][1],FVerleih[i][fCoords][2]);
			        FVerleih[i][fTXT3D] = Create3DTextLabel(""#SERVERFARBE"Fahrradverleih\n{FFFFFF}Drücke ENTER/F", 0xFFFFFFFF, FVerleih[i][fCoords][0],FVerleih[i][fCoords][1],FVerleih[i][fCoords][2], 5, 0);
			        printf("[LOADDATA]Fahrradverleih %d:", i); // Ob der Durchlauf erfolgreich ausgelesen wurde
					printf("%f %f %f",FVerleih[i][fCoords][0], FVerleih[i][fCoords][1], FVerleih[i][fCoords][2]);
       			}
			}
			else
			{
			    return printf("[DATABASE] '%s' wurde kein Eintrag gefunden.", TableName[database]);
			}
			FVerleihCount = cache_num_rows();
			ServerStartChecks[1] = true;
			for(new i; i!=FVerleihCount; i++) // Der Check ob alle durchläufe geklappt haben
			{
			    printf("[LOADDATA:CHECK]Fahrradverleih %d:", i);
				printf("%f %f %f",FVerleih[i][fCoords][0], FVerleih[i][fCoords][1], FVerleih[i][fCoords][2]);
			}
			return printf("[DATABASE] '%s' wurden %d Einträge geladen.", TableName[database], cache_num_rows());
	    }
	    case 1: // serverinfo_price
	    {
	        cache_get_value_name_int(0,"Perso",ServerPrice[Perso]);
	        cache_get_value_name_int(0,"CKonto",ServerPrice[CKonto]);
	        ServerStartChecks[2] = true;
	        return printf("[DATABASE] '%s' wurde geladen.", TableName[database]);
	    }
	}
	return 1;
}

Alles anzeigen

hier die Ausgabe der Checks, an die server_log.log

Code: server_log.log

[15:54:12] [LOADDATA]Fahrradverleih 0:
[15:54:12] 1753.994384 -1887.939941 13.557000
[15:54:12] [LOADDATA]Fahrradverleih 1:
[15:54:12] 1182.340332 -1332.666992 13.583100


[15:54:12] [LOADDATA:CHECK]Fahrradverleih 0:
[15:54:12] 0.000000 0.000000 0.000000
[15:54:12] [LOADDATA:CHECK]Fahrradverleih 1:
[15:54:12] 1182.340332 -1332.666992 13.583100

Akino Kiritani · 23. Mai 2019

Ich hab es nun 1:1 nachgebaut bzw. deinen vorhanden Code genutzt, mehrere Datenbanken ausprobiert und kann es einfach nicht reproduzieren.

Ich weiß nicht ob es daran liegt, das ich einen anderen Compiler benutze.
Ich werde das mal probieren.

// Edit: Auch nach mehrfachen versuchen (mit dem Standard Compiler) bleibt es dabei, dass ich es nicht reproduzieren kann.

EllBundy · 23. Mai 2019

reproduzieren heißt genau was ?

Studs · 23. Mai 2019

Versuch mal die return 1;s überall bis auf das Ende des Publics OnPlayerKeyStateChange rauszunehmen

Akino Kiritani · 23. Mai 2019

Zitat von EllBundy

reproduzieren heißt genau was ?

Das ich versucht habe den Fehler/das Problem irgendwie nach zu stellen.

EllBundy · 23. Mai 2019

Ich habe eben nochmal gegrübelt und dachte mir wenn der erste durchlauf durch den 2. resettet wird.
Sollte was mit der Variable nicht stimmen wo ich die Infos abspeichere.
Daraufhin habe ich mir die Variable näher betrachtet:
enum FVerleihData{
//...
}
new FVerleih[][FVerleihData];

und habe einfach eine vordefinierten array eingefügt, in dem Fall die 2, da ich 2 Verleihs habe.
Also so
enum FVerleihData{
//...
}
new FVerleih[2][FVerleihData];

Nach dem Test in der Console und auf dem Server funktioniert nun alles.
Dennoch bleibt die Frage wie mache ich das am besten, da ich das ja dyn. haben will.
Das ich jederzeit InGame einen erstellen kann, damit er auch geladen wird und abgespeichert...

Würde das so funktionieren ?
#define MAX_FVERLEIH 10 // Beispielanzahl
enum FVerleihData{
//...
}
new FVerleih[MAX_FVERLEIH][FVerleihData];

und bei der for-Schleife dann nur FVerleihCount nehme, da die Variable ja wiedergibt wieviele Einträge in der Datenbank sind.
Dann muss ich zwar wenns mal über 10 gehen sollte das im Script erhöhen...

Akino Kiritani · 24. Mai 2019

Zitat von EllBundy

Daraufhin habe ich mir die Variable näher betrachtet:

Ach herrje, daran das du evtl. eine leere Variablenngröße angegeben hast hab ich gar nicht gedacht.
PAWN funktioniert leider so nicht. Man kann es so machen, aber dann füllt PAWN halt immer nur einen Wert.

Das ist halt das Problem mit statischen Sprachen - sie brauchen idR eine feste Größe mit der sie arbeiten können.

Zitat von EllBundy

Würde das so funktionieren ?

Ja, dass wird so funktionieren und ist auch Erstrebenswert so zu arbeiten.
Wie gesagt, PAWN braucht feste Größen.
Es gibt Möglichkeiten Variablengrößen richtig dynamisch zu gestalten, ist aber für einen Anfänger definitiv etwas viel.

Zitat von EllBundy

und bei der for-Schleife dann nur FVerleihCount nehme, da die Variable ja wiedergibt wieviele Einträge in der Datenbank sind.

Ja und nein.
Im Grunde eine gute Idee. Du wirst nur ein Problem bekommen, wenn du mal einen Eintrag während der Runtime löschen solltest.

Wir nehmen mal dein Beispiel mit 2 Verleihs.
Sofern auch 2 vorhanden sind ist alles gut.

Nun bist du aber der Meinung - eigentlich reicht auch einer und löscht den ersten Verleih.
Entsprechend sinkt nun auch der Wert FVerleihCount um eins, da nun ein Verleih wegfällt.

Wir erinnern uns, dass du 2 geladen hast, und die Arraynummern 0 und 1 wurden gefüllt.
Löscht du jetzt den ersten also 0 und versucht dann mit deinem Wert FVerleihCount zu arbeiten, wirst du feststellen. dass deine Schleife nur den ersten Array (0) versucht auszulesen.
Dieser ist durch die Löschung nun nur leider leer und du kommst auf direktem Wege nicht mehr an den 2. Verleih.

Entsprechend ist es sinnvoller, die Schleife so zu gestalten, dass sie theoretisch jeden Wert erreichen kann.
Nur musst du hierbei darauf achten, einen Check zu erstellen, ob die Werte auch geladen wurden.

Diesen Check kannst du gestallten wie du möchtest. Du kannst dir eine Variable in der Enum Struktur erstellen und es darüber machen oder du prüfst einfach ob ein Wert nicht 0 bzw. leer ist.
Hierfür könntest du bspw. pID oder sogar einfach die IconID nehmen. Diese sollten idR nicht 0 sein, wenn Werte vorhanden sind.

breadfish.de 12. März 2022