Befehl funktioniert nur bei der ersten Pflanze

malE · 21. November 2015

Hallo
habe mir aus Testzwecken ein /pushpflanze Befehl erstellt.
Problem ist aber das der Befehl nur bei der ersten Pflanze funktioniert.

C

ocmd:pushpflanze(playerid, params[])
{
	if(PlayerInfo[playerid][p_Admin] < 6)return 1;
	for(new i=0; i < sizeof(DrugInfo); i++)
	{
	    if(IsPlayerInRangeOfPoint(playerid, 1.5, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ]))
	    {
			DrugInfo[i][d_Prozent] = 100;
			new str[145];
			format(str, sizeof(str), " Du hast die Pflanze %i auf 100 Prozent gesetzt.", i);
			SendClientMessage(playerid, 0xFF5559FF, str);
			UpdateDrugLabel(i);
		}
		else return SendClientMessage(playerid, COLOR_GREY, "Es ist keine Drogenpflanze in deiner Nähe.");
	}
	return 1;
}

Alles anzeigen

Jeffry · 21. November 2015

Du musst das return aus der Schleife nehmen:
ocmd:pushpflanze(playerid, params[])
{
if(PlayerInfo[playerid][p_Admin] < 6)return 1;
for(new i=0; i < sizeof(DrugInfo); i++)
{
if(IsPlayerInRangeOfPoint(playerid, 1.5, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ]))
{
DrugInfo[i][d_Prozent] = 100;
new str[145];
format(str, sizeof(str), " Du hast die Pflanze %i auf 100 Prozent gesetzt.", i);
SendClientMessage(playerid, 0xFF5559FF, str);
UpdateDrugLabel(i);
return 1;
}
}
return SendClientMessage(playerid, COLOR_GREY, "Es ist keine Drogenpflanze in deiner Nähe.");
}

malE · 21. November 2015

Ich hatte es auch schon ohne return. Deswegen bin ich verwundert.
Es funktioniert einfach nicht.

SendClientMessage Output:

Code

Du hast die Pflanze 3 auf 100 Prozent gesetzt.

Jeffry · 21. November 2015

Was ist daran falsch? Soll es nicht die dir am nächsten stehende Pflanze auf 100 setzen (in dem Fall die 3)?

malE · 21. November 2015

Keine schlechte Frage .
Aber es ist egal an welche Pflanze ich stehe, es wird nur die Pflanze ID 3 gepusht.

Jeffry · 21. November 2015

Oh. Wie lädst du die Pflanzen?

malE · 21. November 2015

Code

forward OnPlayerDrugsLoad();
public OnPlayerDrugsLoad()
{
	new num_fields, num_rows, count = 0;
	cache_get_data(num_rows, num_fields, handle);
	if(!num_rows)return 1;
	for(new i=0; i < num_rows; i++)
	{
		DrugInfo[i][d_Besitzer] = cache_get_field_content_int(i, "besitzer", handle);
		DrugInfo[i][d_Zustand] = cache_get_field_content_int(i, "zustand", handle);
		DrugInfo[i][d_Wasser] = cache_get_field_content_int(i, "wasser", handle);
		DrugInfo[i][d_Prozent] = cache_get_field_content_int(i, "prozent", handle);
		DrugInfo[i][d_PosX] = cache_get_field_content_float(i, "x", handle);
		DrugInfo[i][d_PosY] = cache_get_field_content_float(i, "y", handle);
		DrugInfo[i][d_PosZ] = cache_get_field_content_float(i, "z", handle);
		DrugInfo[i][d_Objekt] = CreateObject(DROGENOBJEKT, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ], 0.0, 0.0, 0.0);
		DrugInfo[i][d_Erstellt] = true;
  		new str[145];
	    format(str, sizeof(str), "[ Zustand: %d ]\n[ Wasser: %d ]\n[ Prozent: %d ]", DrugInfo[i][d_Zustand], DrugInfo[i][d_Wasser], DrugInfo[i][d_Prozent]);
	    Create3DTextLabel(str, COLOR_WHITE, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ], 10.0, 0, 0);
		count ++;
	}
	printf("[INFO] %i/%i Drogenpflanze geladen.", count, sizeof(DrugInfo));
	return 1;
}

Alles anzeigen

Außerdem wird die Pflanze ID 3 anscheint nicht gespeichert.
OnGameModeExit:

Code

stock SavePlayerDrugs()
{
    new query[256];
	for(new i=0; i < sizeof(DrugInfo); i++)
	{
	    mysql_format(handle, query, sizeof(query), "UPDATE drugs SET prozent='%d', wasser='%d', zustand='%d' WHERE id='%d'", DrugInfo[i][d_Prozent], DrugInfo[i][d_Wasser], DrugInfo[i][d_Zustand], DrugInfo[i][d_ID]);
	    mysql_pquery(handle, query);
	    print(query);
	}
}

Es wird nichts geprintet.

Jeffry · 21. November 2015

Debugge den Befehl bitte mal so:
ocmd:pushpflanze(playerid, params[])
{
if(PlayerInfo[playerid][p_Admin] < 6)return 1;
new Float:x, Float:y, Float:z;
GetPlayerPos(playerid, x, y, z);
printf("%f / %f / %f", x, y, z);
for(new i=0; i < sizeof(DrugInfo); i++)
{
printf("i:%d | %f / %f / %f | %d", i, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ], IsPlayerInRangeOfPoint(playerid, 1.5, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ]));
if(IsPlayerInRangeOfPoint(playerid, 1.5, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ]))
{
printf("Setze zu 100");
DrugInfo[i][d_Prozent] = 100;
new str[145];
format(str, sizeof(str), " Du hast die Pflanze %i auf 100 Prozent gesetzt.", i);
SendClientMessage(playerid, 0xFF5559FF, str);
UpdateDrugLabel(i);
return 1;
}
}
printf("Nicht in der Nähe.");
return SendClientMessage(playerid, COLOR_GREY, "Es ist keine Drogenpflanze in deiner Nähe.");
}

Zwecks der Speicherung:
Wo rufst du die Funktion auf?

malE · 22. November 2015

Vorab es hat an jeder Pflanze funktioniert, aber nur die Pflanze ID 0 wurde auf 100 gesetzt laut 3DTextLabel.

Konsole

Wie gesagt es wird im Callback OnGameModeExit() aufgerufen.
Falls die Frage auftaucht, Ja ich lasse den stock vor mysql_close() aufrufen.

Jeffry · 22. November 2015

Poste bitte dein UpdateDrugLabel.

Zwecks der Speicherung:
Das schauen wir uns dann danach an, jetzt erst mal ein Problem lösen.

malE · 22. November 2015

C

stock UpdateDrugLabel(id)
{
	for(new i=0; i < sizeof(DrugInfo); i++)
	{
		new label[145];
	    format(label, sizeof(label), "[ Zustand: %d ]\n[ Wasser: %d ]\n[ Prozent: %d ]", DrugInfo[id][d_Zustand], DrugInfo[id][d_Wasser], DrugInfo[id][d_Prozent]);
	    Update3DTextLabelText(DrugInfo[id][d_Label], 0xFFFFFFFF, label);
	}
}

Jeffry · 22. November 2015

Schreibe das so:
stock UpdateDrugLabel(id)
{
new label[145];
format(label, sizeof(label), "[ Zustand: %d ]\n[ Wasser: %d ]\n[ Prozent: %d ]", DrugInfo[id][d_Zustand], DrugInfo[id][d_Wasser], DrugInfo[id][d_Prozent]);
Update3DTextLabelText(DrugInfo[id][d_Label], 0xFFFFFFFF, label);
return 1;
}

Und ändere beim Erstellen das
Create3DTextLabel(str, COLOR_WHITE, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ], 10.0, 0, 0);
zu:
DrugInfo[i][d_Label] = Create3DTextLabel(str, COLOR_WHITE, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ], 10.0, 0, 0);

malE · 22. November 2015

Okey. Soweit so gut. Es scheint jetzt zufunktionieren.

Bei dem Befehl wird die Pflanze nicht aus der Datenbank entfernt.

C

ocmd:abernten(playerid, params[])
{
	for(new i=0; i < sizeof(DrugInfo); i++)
	{
	    if(IsPlayerInRangeOfPoint(playerid, 1.5, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ]))
		{
		    if(DrugInfo[i][d_Besitzer] != PlayerInfo[playerid][p_ID])return 1;
		    if(DrugInfo[i][d_Prozent] < 90)return SendClientMessage(playerid, COLOR_GREY, "Die Pflanze ist noch nicht ausgereift.");
	    	DrugInfo[i][d_PosX] = 0;
			DrugInfo[i][d_PosY] = 0;
			DrugInfo[i][d_PosZ] = 0;
			DrugInfo[i][d_Wasser] = 0;
			DrugInfo[i][d_Zustand] = 0;
			DrugInfo[i][d_Prozent] = 0;
			DestroyObject(DrugInfo[i][d_Objekt]);
			Delete3DTextLabel(DrugInfo[i][d_Label]);
			new query[256];
			format(query, sizeof(query), "DELETE FROM drugs WHERE id='%d'", DrugInfo[i][d_ID]);
			mysql_function_query(handle, query, false, "", "");
		}
	}
	return 1;
}

Alles anzeigen

Außerdem funktioniert die Speicherung leider immer noch nicht.

-
Darf ich Fragen wie du den PAWN-Quellcode benutzt?
Bei mir wird das immer so abgesendet: Bild.

Akino Kiritani · 22. November 2015

Zitat von malE

if(DrugInfo[i][d_Besitzer] != PlayerInfo[playerid][p_ID])return 1;

Du müsstet hier das return 1; in continue; ändern

Jeffry · 22. November 2015

Das dürfte am gleichen Fehler liegen, und zwar habe ich gesehen, dass du die id gar nicht lädst.
Füge das hier beim Laden hinzu:
DrugInfo[i][d_ID] = cache_get_field_content_int(i, "id", handle);

Der Befehl kann außerdem so aussehen, damit nicht einfach nichts passiert, wenn dir die Pflanze nicht gehört:
ocmd:abernten(playerid, params[])
{
for(new i=0; i < sizeof(DrugInfo); i++)
{
if(IsPlayerInRangeOfPoint(playerid, 1.5, DrugInfo[i][d_PosX], DrugInfo[i][d_PosY], DrugInfo[i][d_PosZ]))
{
if(DrugInfo[i][d_Besitzer] != PlayerInfo[playerid][p_ID]) return SendClientMessage(playerid, COLOR_GREY, "Die Pflanze gehört dir nicht.");
if(DrugInfo[i][d_Prozent] < 90) return SendClientMessage(playerid, COLOR_GREY, "Die Pflanze ist noch nicht ausgereift.");
DrugInfo[i][d_PosX] = 0;
DrugInfo[i][d_PosY] = 0;
DrugInfo[i][d_PosZ] = 0;
DrugInfo[i][d_Wasser] = 0;
DrugInfo[i][d_Zustand] = 0;
DrugInfo[i][d_Prozent] = 0;
DestroyObject(DrugInfo[i][d_Objekt]);
Delete3DTextLabel(DrugInfo[i][d_Label]);
new query[256];
format(query, sizeof(query), "DELETE FROM drugs WHERE id='%d'", DrugInfo[i][d_ID]);
mysql_function_query(handle, query, false, "", "");
return 1;
}
}
return SendClientMessage(playerid, COLOR_GREY, "Du bist nicht in der Nähe einer Pflanze.");
}

Zitat von malE

Darf ich Fragen wie du den PAWN-Quellcode benutzt?

Ich nutze den BBCode Modus (erster Button in der Post-Forum).

malE · 22. November 2015

Gut danke. Die Pflanze wird jetzt auch in der Datenbank entfernt.
Aber das Problem, welches die Pflanzen nicht speichert besteht immer noch.

Jeffry · 22. November 2015

Was gibt dir denn der print in der Speicher-Funktion aus?

malE · 22. November 2015

Gar nichts.
Also wenn ich die server.exe schließen und im server-log.txt reinschaue steht kein Print.

Jeffry · 22. November 2015

Das liegt daran, dass OnGameModeExit nicht aufgerufen wird, wenn du die Konsole schließt.

malE · 22. November 2015

Wie soll ich sonst den Server beenden?