Fehler bei format-Befehl

RickFace · 23. August 2018

Hallo liebe com,

habe ein Problem und zwar hab ich mir nen Befehl erstellt um die Mieter des Hauses in einer Liste anzeigen zu lassen:

Code

ocmd:showtenands(playerid,params[])
{
    new statee=0,ix;
	for(new i=0; i<sizeof hInfo; i++)
	{
	    if(statee==1)continue;
	    if(IsPlayerInRangeOfPoint(playerid,2.0,hInfo[i][h_x],hInfo[i][h_y],hInfo[i][h_z]))
	    {
			statee=1;
			ix=i;
		}
	}
	if(statee==0)return SendClientMessage(playerid,COLOR_RED,"Fehler: Du bist nicht nahe genug an einem Haus");
	statee=0;
    if(hInfo[ix][h_besitzer]==sInfo[playerid][p_id])
    {
        statee=1;
    }
	if(statee==0)return SendClientMessage(playerid,COLOR_RED,"Fehler: Du bist nicht der Besitzer dieses Hauses");
	new query[164];
	if(hInfo[ix][h_mieter_a]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_a]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,1);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"Mieter A: Frei");
	}
	if(hInfo[ix][h_mieter_b]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_b]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,2);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter B: Frei",ShowTenands[playerid]);
	}
	if(hInfo[ix][h_mieter_c]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_c]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,3);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter C: Frei",ShowTenands[playerid]);
	}
	if(hInfo[ix][h_mieter_d]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_d]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,4);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter D: Frei",ShowTenands[playerid]);
	}
	if(hInfo[ix][h_mieter_e]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_e]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,5);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter E: Frei",ShowTenands[playerid]);
	}
	if(hInfo[ix][h_mieter_f]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_f]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,6);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter F: Frei",ShowTenands[playerid]);
	}
	if(hInfo[ix][h_mieter_g]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_g]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,7);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter G: Frei",ShowTenands[playerid]);
	}
	if(hInfo[ix][h_mieter_h]!=INVALID_OWNER)
	{
		format(query,sizeof query,"SELECT * FROM user WHERE id='%i'",hInfo[ix][h_mieter_h]);
		mysql_function_query(dbhandle,query,true,"OnUserLoadTenands","ii",playerid,8);
	}
	else
	{
	    format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter H: Frei",ShowTenands[playerid]);
	}
	ShowPlayerDialog(playerid,DIALOG_SHOW_TENANDS,DIALOG_STYLE_MSGBOX,"Mieter deines Hauses:",ShowTenands[playerid],"Okay","");
	return 1;
}

Alles anzeigen

und das Public zu der mysql_function_query:

Code

new ShowTenands[MAX_PLAYERS][300];

Code

public OnUserLoadTenands(playerid,slot)
{
	new name[MAX_PLAYER_NAME];
	cache_get_field_content(0,"username",name,dbhandle,sizeof name);
	if(slot==1)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"Mieter A: %s",name);
	}
	else
	if(slot==2)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter B: %s",ShowTenands[playerid],name);
	}
	else
	if(slot==3)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter C: %s",ShowTenands[playerid],name);
	}
	else
	if(slot==4)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter D: %s",ShowTenands[playerid],name);
	}
	else
	if(slot==5)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter E: %s",ShowTenands[playerid],name);
	}
	else
	if(slot==6)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter F: %s",ShowTenands[playerid],name);
	}
	else
	if(slot==7)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter G: %s",ShowTenands[playerid],name);
	}
	else
	if(slot==8)
	{
		format(ShowTenands[playerid],sizeof ShowTenands[playerid],"%s\nMieter H: %s",ShowTenands[playerid],name);
	}
	return 1;
}

Alles anzeigen

angezeigt bekomme ich folgenden fehler:

Code

C:\Users\Riagine\Desktop\GTA\GTA Server\gamemodes\invisible.pwn(324) : error 001: expected token: "]", but found "-identifier-"
C:\Users\Riagine\Desktop\GTA\GTA Server\gamemodes\invisible.pwn(324) : warning 215: expression has no effect
C:\Users\Riagine\Desktop\GTA\GTA Server\gamemodes\invisible.pwn(324) : error 001: expected token: ";", but found "]"
C:\Users\Riagine\Desktop\GTA\GTA Server\gamemodes\invisible.pwn(324) : error 029: invalid expression, assumed zero
C:\Users\Riagine\Desktop\GTA\GTA Server\gamemodes\invisible.pwn(324) : fatal error 107: too many error messages on one line

die Zeile 324 ist in dem fall bei dem Public die format funktion in Zeile 7. allerdings komme ich nicht drauf wo der Fehler liegen soll.

Vielen Dank schonmal für eure Hilfe

Kaliber · 23. August 2018

Zitat von RickFace

sizeof ShowTenands[playerid]

Das hier ist dein Fehler

Das müsste: sizeof(ShowTenans[]) heißen.

Allerdings würde ich dir gerne auch ein paar Optimierungstipps geben, schau mal hier rein: Schneller & strukturierter skripten

Gibt viele Möglichkeiten den unteren Teil extremst zu kürzen, hier mal eine trickige:

C

public OnUserLoadTenands(playerid,slot)
{
	new name[MAX_PLAYER_NAME];
	cache_get_field_content(0,"username",name,dbhandle,sizeof name);
	format(ShowTenands[playerid],sizeof ShowTenands[],"%s\nMieter %c: %s",ShowTenands[playerid],64+slot,name);
	return 1;
}

%c macht aus einer Zahl, einen Character, nach der ASCII Tabelle: https://www.torsten-horn.de/techdocs/ascii.htm

65 = A
Dementsprechend sind B..C..D
66,67,68, also immer 64+slot

//Edit: Gerade gesehn, kann ja nur einen Slot geben

RickFace · 23. August 2018

Vielen Dank für dein Hilfe Kailber,

das mit dem ASCII Code ist auch eine gute idee, werde mir dass jedenfalls merken.

Kaliber · 23. August 2018

Gerade erst gemerkt, dass du mit slot 1 beginnst, also musst du natürlich 64+slot rechnen, damit du bei A anfängst

Und du hast noch einen kleinen Denkfehler gemacht, wenn ich mir deine Querys so ansehe.

Zitat von RickFace

SELECT * FROM user WHERE id='%i'

Da suchst du nur nach dem Slot.

Das bedeutet ja aber, dass es nur ein Haus geben kann

Du solltest dir da evtl nochmal ein paar Tutorials anschauen, wie so Häuser Systeme realisiert werden, denn du brauchst hier halt auch noch eine HausID, damit du von diesem Haus die Mieter abfragen kannst

RickFace · 23. August 2018

Ne da hast du was falsch verstanden..
mit dieser query frage ich nicht die Häuser ab sondern lasse damit die row vom mieter laden um den Usernamen anzeigen zu lassen.
eine Haus ID gibt es ja natürlich schon bin ja nicht blöd ^^.

Code

enum HausDaten{
	h_id,
	h_besitzer,
	h_interior,
	h_kauf_level,
	h_kosten,
	Float:h_x,
	Float:h_y,
	Float:h_z,
	Float:h_i_x,
	Float:h_i_y,
	Float:h_i_z,
	h_pickup,
	Text3D:h_text[200],
	h_interior_pickup,
	h_status,
	bool:h_state,
	h_mieter_a,
	h_mieter_b,
	h_mieter_c,
	h_mieter_d,
	h_mieter_e,
	h_mieter_f,
	h_mieter_g,
	h_mieter_h,
	h_miet_kosten,
	h_payday_sek,
	h_payday_min,
	h_payday_std
}

Alles anzeigen

Code

public OnHousesLoad()
{
	new rows,fields;
	cache_get_data(rows,fields,dbhandle);
	if(rows==0)return print("Keine Einträge in der DB houses");
	for(new ix=0; ix<rows; ix++)
	{
		new i = GetFreeHausID();
		hInfo[i][h_id]=cache_get_field_content_int(ix,"id",dbhandle);
		hInfo[i][h_besitzer]=cache_get_field_content_int(ix,"besitzer",dbhandle);
		hInfo[i][h_interior]=cache_get_field_content_int(ix,"interior",dbhandle);
		hInfo[i][h_kosten]=cache_get_field_content_int(ix,"kosten",dbhandle);
		hInfo[i][h_kauf_level]=cache_get_field_content_int(ix,"kauf_level",dbhandle);
		hInfo[i][h_mieter_a]=cache_get_field_content_int(ix,"mietera",dbhandle);
		hInfo[i][h_mieter_b]=cache_get_field_content_int(ix,"mieterb",dbhandle);
		hInfo[i][h_mieter_c]=cache_get_field_content_int(ix,"mieterc",dbhandle);
		hInfo[i][h_mieter_d]=cache_get_field_content_int(ix,"mieterd",dbhandle);
		hInfo[i][h_mieter_e]=cache_get_field_content_int(ix,"mietere",dbhandle);
		hInfo[i][h_mieter_f]=cache_get_field_content_int(ix,"mieterf",dbhandle);
		hInfo[i][h_mieter_g]=cache_get_field_content_int(ix,"mieterg",dbhandle);
		hInfo[i][h_mieter_h]=cache_get_field_content_int(ix,"mieterh",dbhandle);
		hInfo[i][h_miet_kosten]=cache_get_field_content_int(ix,"miet_kosten",dbhandle);
		hInfo[i][h_x] = cache_get_field_content_float(ix,"ax",dbhandle);
		hInfo[i][h_y] = cache_get_field_content_float(ix,"ay",dbhandle);
		hInfo[i][h_z] = cache_get_field_content_float(ix,"az",dbhandle);
		hInfo[i][h_i_x] = cache_get_field_content_float(ix,"aix",dbhandle);
		hInfo[i][h_i_y] = cache_get_field_content_float(ix,"aiy",dbhandle);
		hInfo[i][h_i_z] = cache_get_field_content_float(ix,"aiz",dbhandle);
		hInfo[i][h_payday_sek] = cache_get_field_content_int(ix,"payday_sek",dbhandle);
		hInfo[i][h_payday_min] = cache_get_field_content_int(ix,"payday_min",dbhandle);
		hInfo[i][h_payday_std] = cache_get_field_content_int(ix,"payday_std",dbhandle);
		hInfo[i][h_state]=false;
		if(hInfo[i][h_besitzer]==INVALID_OWNER)
		{
		    new string[164];
		    hInfo[i][h_pickup]=CreatePickup(PICKUP_HOUSE_GREEN,1,hInfo[i][h_x],hInfo[i][h_y],hInfo[i][h_z],0);
			format(string,sizeof string,"Haus ID: %i\nAb Level %i\nFür %i$ zu Kaufen",hInfo[i][h_id],hInfo[i][h_kauf_level],hInfo[i][h_kosten]);
		    hInfo[i][h_text]=Create3DTextLabel(string,COLOR_GOLD,hInfo[i][h_x],hInfo[i][h_y],hInfo[i][h_z],15.0,0);
		}
		else
		{
            new string[164];
            format(string,sizeof string,"SELECT * FROM user WHERE id='%i'",hInfo[i][h_besitzer]);
            mysql_function_query(dbhandle,string,true,"OnHouseOwnerLoad","i",i);
            hInfo[i][h_pickup]=CreatePickup(PICKUP_HOUSE_BLUE,1,hInfo[i][h_x],hInfo[i][h_y],hInfo[i][h_z],0);
		}
		hInfo[i][h_interior_pickup] = CreatePickup(PICKUP_INFO,1,hInfo[i][h_i_x],hInfo[i][h_i_y],hInfo[i][h_i_z],0);
		printf("Haus ID: %i(%i) wurde geladen",hInfo[i][h_id],i);
	}
	return 1;
}

Alles anzeigen

Also mit deiner ersten Verbesserung wo das ShowTenands[playerid] zu ShowTenands[] wird hat wieder alles Funktioniert.

Habe auch schon das ganze house System schon fast fertig das sind jetzt nur noch extra Funktionen.

buyhouse,sellhouse,openhouse,closehouse,leaverent,kickrent,renthouse,setrent,showtenands,copyhouseinterior,unloadhouses,loadhouses,reloadhouses,changehousespawn,edithousecosts,edithousebuylevel,edithouseinterior,edithouseinteriorpos,sethouseowner,deletehouse,createhouse... das hab ich schon alles eingebaut und ist auch Vollständig Funktionsfähig.

breadfish.de 12. März 2022