ATMs werden nicht Richtig Gespeichert/Gelöscht

BlvckAir · 23. September 2015

Guten Abend,
ich kann keine ATMs entfernen. Glaube sogar das es noch Falsch Gespeichert wird.

Code

enum ATMDaten
{
	ID,
	Float:aX,
	Float:aY,
	Float:aZ,
	Float:aA,
	atmobject,
	bool:erstellt
}
new ATMInfo[MAX_ATMS][ATMDaten];


Entfernen:
	                case 0:
	                {
	                    new id = IsATM(playerid), query[256];
	                    if(id != -1)
	                    {
	                        ATMInfo[id][erstellt] = false;
	                        ATMInfo[id][aX] = 0.0;
	                        ATMInfo[id][aY] = 0.0;
	                        ATMInfo[id][aZ] = 0.0;
	                        DestroyObject(ATMInfo[id][atmobject]);
	                        SendClientMessage(playerid, FARBE_WEISS, "ATM Entfernt!");
	                        format(query, sizeof(query), "SELECT * FROM `ATM` WHERE ID = '%d'", id);
	                        format(query, sizeof(query), "DELETE FROM `ATM` WHERE ID = '%d'", id);
	                        db_query(Datenbank, query);
	                        return 1;
	                    }return SendClientMessage(playerid, FARBE_WEISS, "Kein ATM in der Nähe!!");
	                }


Erstellen:
case 0:
	            	{
	            	    new Float:X, Float:Y, Float:Z, Float:A, query[256];
	            	    GetPlayerPos(playerid, X, Y, Z);
	            	    GetPlayerFacingAngle(playerid, A);
	            	    for(new i = 0; i < MAX_ATMS; i++)
	            	    {
	            	        if(ATMInfo[i][erstellt] == true)continue;
							ATMInfo[i][aX] = X;
							ATMInfo[i][aY] = X;
							ATMInfo[i][aZ] = Z;
							ATMInfo[i][aA] = A;
							CreateDynamicObject(ATMInfo[i][atmobject], X, Y, Z, 0, 0, A);
							format(query, sizeof(query), "INSERT INTO `ATM` (`ID`, `X`, `Y`, `Z`, `A`) VALUES ('%i', '%f', '%f', '%f', '%f')", i, X, Y, Z, A);
							db_query(Datenbank, query);
							SendClientMessage(playerid, FARBE_WEISS, "ATM Erstellt");
							return 1;
						}
					}


stock IsATM(playerid)
{
	new i = 0;
	while(i != MAX_ATMS)
	{
	    if(IsPlayerInRangeOfPoint(playerid, 5.0, ATMInfo[i][aX], ATMInfo[i][aY], ATMInfo[i][aZ]))
	    {
	        return i;
	    }
	    i++;
	}
	return -1;
}


Laden:
stock LoadATM()
{
	new DBResult:Result, string[512], rows;
	for(new i = 0; i < MAX_ATMS; i++)
	{
	    format(string, sizeof(string), "SELECT * FROM `ATM` WHERE `ID` = '%i'", i);
	    Result = db_query(Datenbank, string);
	    rows = db_num_rows(Result);
	    if(rows == 0)continue;
	    db_get_field_assoc(Result, "X", string, sizeof(string));
	    ATMInfo[i][aX] = floatstr(string);
	    db_get_field_assoc(Result, "Y", string, sizeof(string));
	    ATMInfo[i][aY] = floatstr(string);
	    db_get_field_assoc(Result, "Z", string, sizeof(string));
	    ATMInfo[i][aZ] = floatstr(string);
	    db_get_field_assoc(Result, "A", string, sizeof(string));
	    ATMInfo[i][aA] = floatstr(string);
		ATMInfo[i][erstellt] = true;
		ATMInfo[i][atmobject] = CreateDynamicObject(2942,ATMInfo[i][aX],ATMInfo[i][aY],ATMInfo[i][aZ]-0.5,0,0,ATMInfo[i][aA]);
		db_free_result(Result);
	}
	return 1;
}

Alles anzeigen

MfG. Black_Air

@Jeffry Du ne Idee?

Jeffry · 23. September 2015

Nutze anstatt DestroyObject: DestroyDynamicObject

Außerdem beim Erstellen:
ATMInfo[i][erstellt] = true;

Beim entfernen kannst du das hier weg lassen:
format(query, sizeof(query), "SELECT * FROM `ATM` WHERE ID = '%d'", id);

BlvckAir · 23. September 2015

Danke, dies klappt schon einmal.
Erstellt und wird auch Entfernt.
Leider habe ich immer noch ein Fehler,

Code

ocmd:atm(playerid, params[])
{
	if(sInfo[playerid][sKonto] != 1)return SendClientMessage(playerid, FARBE_WEISS, "[{FF0000}FEHLER{FFFFFF}]: Du besitzt kein Bankkonto!");
	if(IsATM(playerid) != -1)
	{
	    ShowPlayerDialog(playerid, DIALOG_PINEINGABE, DIALOG_STYLE_INPUT, "{FFFFFF}ATM - Pineingabe", "{FFFFFF}Gebe bitte deinen vier stelligen Pin ein:", "Bestätigen", "Abbrechen");
	}return SendClientMessage(playerid, FARBE_WEISS, "[{FF0000}FEHLER{FFFFFF}]: Es ist kein Bankautomat in deiner Nähe!");
}

Er sagt mir das kein ATM in der Nähe sei, ich denke mal das die Funktion "IsATM" irgendwas Falsch dran ist oder?

MfG. Black_Air

Jeffry · 23. September 2015

Eigentlich nicht.
Versuche es so:

C

ocmd:atm(playerid, params[])
{
	if(sInfo[playerid][sKonto] != 1)return SendClientMessage(playerid, FARBE_WEISS, "[{FF0000}FEHLER{FFFFFF}]: Du besitzt kein Bankkonto!");
	if(IsATM(playerid) != -1)
	{
		ShowPlayerDialog(playerid, DIALOG_PINEINGABE, DIALOG_STYLE_INPUT, "{FFFFFF}ATM - Pineingabe", "{FFFFFF}Gebe bitte deinen vier stelligen Pin ein:", "Bestätigen", "Abbrechen");
	}
	else SendClientMessage(playerid, FARBE_WEISS, "[{FF0000}FEHLER{FFFFFF}]: Es ist kein Bankautomat in deiner Nähe!");
	return 1;
}

BlvckAir · 23. September 2015

Nunja, nach einem Server Restart kann ich auf die ATMs zugreifen.
Aber selbst dann sagt er mir das kein ATM in der Nähe sei.

Sprich, Dialog kommt aber im Chat kommt auch das kein ATM vorhanden sein.

MfG. Black_Air

Jeffry · 23. September 2015

Ja, das liegt dann daran, dass du da kein else hattest.
Versuche es mit dem Code den ich dir gegeben habe.

BlvckAir · 23. September 2015

Problem liegt vor, sobald ich einen Erstelle kann ich nicht darauf zu greifen.
Nach einem Server Restart kann ich dann aber zu Greifen.

MfG. Black_Air

Jeffry · 23. September 2015

Was meinst du mit zugreifen?

Beim Erstellen fehlt, sehe ich grade, noch die Zuweisung des Objektes
ATMInfo[i][atmobject]=...

BlvckAir · 23. September 2015

Also, ich erstelle einen ATM, ATM Erstellt Object wird auch erstellt, stehe nun davor und tippe /atm dann kommt die Meldung das kein ATM in der Nähe sei, lasse ich den Server Neustarten und gehe zum ATM hin dann kommt der Dialog.

MfG. Black_Air

Ja, aber das Entferne ich gleich noch, da das Object eh beim Laden der ATMs festgelegt ist. Daher ist dies Sinnlos für mich.

Jeffry · 23. September 2015

Den Fehler hast du aber gut versteckt

ATMInfo[i][aY] = X;
Zu
ATMInfo[i][aY] = Y;

BlvckAir · 23. September 2015

Haha
Der wohl dümmste Fehler von mir jemals.

Ich danke dir.

MfG. Black_Air

breadfish.de 12. März 2022