Verwahrstelle

Hey,
wie wäre es mit einer for schleife?
also

for(new i=0;i<MAX_VEHICLES;i++)//MAX_VEHICLES evt anpassen
{
new string[250];//länge je nach wie viele cars wer sieht
if(abgeschleppt==true)//anpassen
{
format(string,sizeof(string)," Alle angaben die ausgegebenw erden sollen(wichtig nur das \n zum schluss) \n", die angaben zu dem string);
}
ShowPlayerDialog(playerid,Dialog,DIALOG_STYLE_LIST,"Verwahrstelle",string,"Weiter","Abbrechen");
}

das wäre mein ansatz

nur so die frage wie fragst du denn ab obe s in der verwahrstelle ist?

Okay schuldige habe ich wohl schlecht erklärt...

for(new i=0;i<sizeof(cInfo);i++)
{
new string[500];
if(cInfo[i][OWer]==playerid)//anpasssen aufbesitzer also ob der wagen dem gehört
{
if(cInfo[i][c_gesperrt] == 1)
{
format(string,sizeof(string)," %s %d \n", string, cInfo[i][Hier]);
}
}
}
ShowPlayerDialog(playerid,Dialog,DIALOG_STYLE_LIST,"Verwahrstelle",string,"Weiter","Abbrechen");

so Pass auf er geht in der for schleife alles deine wagens die abgeschleppt sind druch und setzt den string
durch das \n wird nach jedem string gleich ne neue Zeile genommen.
heisst bei zwei wagen kommt dann beim string beim nächsten durch gang das neue dazu.

Ja da du hier ein return hast
return SendClientMessage(playerid,COLOR_RED,"Keine gefunden");
das return beendet somit die ganzeschleife und was darutner kommt
somit versuchs mal damit

ocmd:freikaufen(playerid)
{
if(PlayerInfo[playerid][pLoggedIn]==false)return 1;
new string[128],fahrzeugevorhanden=0;
for(new i=0; i<sizeof(cInfo); i++)
{
if(cInfo[i][besitzer] == PlayerInfo[playerid][p_id])
{
if(cInfo[i][c_gespeert] == 1)
{
format(string,sizeof(string),"%s %d \n",string,cInfo[i][model]);
fahrzeugevorhanden=1;
}
}
}
if( fahrzeugevorhanden==1)
{
return ShowPlayerDialog(playerid,DIALOG_CARFREIKAUFEN_AUSWAHL,DIALOG_STYLE_LIST,"Verwahrstelle",string,"Auswählen","Abbruch");
}
else
{
return SendClientMessage(playerid,COLOR_RED,"Keine Fahrzeuge in der Verwahrstelle");
}
return 1;
}

und das

else
{
return SendClientMessage(playerid,COLOR_RED,"Kein Besitzer");
}

lassen wir auch aus der Schleife weil das genau das gleiche wie mit dem c_gesperrt ist
Warum?
also es ist relativ egal ob wir nicht der besitzer vom fahrzeug sind, denn es passiert auch nichts mit dem wagen

Hoffe du verstehst es mfg. Mogly1

evt so?

format(string,sizeof(string),"%s\n %d ",string,cInfo[i][model]);

dann lass mal printen:

ocmd:freikaufen(playerid)
{
if(PlayerInfo[playerid][pLoggedIn]==false)return 1;
new string[128],fahrzeugevorhanden=0;
for(new i=0; i<sizeof(cInfo); i++)
{
if(cInfo[i][besitzer] == PlayerInfo[playerid][p_id])
{
printf("ID:%d | Besitzer:%d | Model:%d | Gesperrt:%d",i,cInfo[i][besitzer],cInfo[i][model],cInfo[i][c_gespeert]);
if(cInfo[i][c_gespeert] == 1)
{
format(string,sizeof(string),"%s %d \n",string,cInfo[i][model]);
fahrzeugevorhanden=1;
}
}
}
if( fahrzeugevorhanden==1)
{
return ShowPlayerDialog(playerid,DIALOG_CARFREIKAUFEN_AUSWAHL,DIALOG_STYLE_LIST,"Verwahrstelle",string,"Auswählen","Abbruch");
}
else
{
return SendClientMessage(playerid,COLOR_RED,"Keine Fahrzeuge in der Verwahrstelle");
}
return 1;
}

Also alles klar funkt eig alles richtig pass auf...

printf("ID:%d | Besitzer:%d | Model:%d | Gesperrt:%d",i,cInfo[i][besitzer],cInfo[i][model],cInfo[i][c_gespeert]);

un die ergebnisse von dir

[19:12:46]
[19:12:46] ID:0 | Besitzer:3 | Model:496 | Gesperrt:0
[19:12:46] ID:1 | Besitzer:3 | Model:496 | Gesperrt:0
[19:12:46] ID:2 | Besitzer:3 | Model:496 | Gesperrt:1

stimmen so apss auf die ID ist nullw eil der print das i vond er schleife wieder gibt...
und dir wird nur ein fahrzeug im dialog gezeigt weil nur 1 Gesperrt ist...
somit Funktioniert doch alles oder wolltest du es anders haben?

na dann zeig mal dein load/save von cars evt. ist da was nicht ganz richtig
und print mal beim Code
cInfo[i][db_id], bzw je nach dem wei du es hast aus

printf("ID:%d | Besitzer:%d | Model:%d | Gesperrt:%d",cInfo[i][db_id],cInfo[i][besitzer],cInfo[i][model],cInfo[i][c_gespeert]);

Edit: zeig mal getfreecarid pls

du müsstest evt. das anpassen bei
getFreeCarID()
da du ja nur die
cInfo[i][id_x]
setzt wenn es nicht gesperrt ist, somit überschreibt sich das glaube anch meiner theorie erstell mal noch nen
"geladen"
cInfo[i][geladen]
das setzt du dann auf 1 bei

OnPlayerCarsLoad
also so dann

forward OnPlayerCarsLoad(playerid);
public OnPlayerCarsLoad(playerid)
{
for(new i=0; i<cache_num_rows(); i++)
{
new id = getFreeCarID();
cInfo[id][geladen]=1;

und bei getfreecarID

getFreeCarID()
{
for(new i=0; i<sizeof(cInfo); i++)
{
if(cInfo[i][geladen]==0)return i;
}
return 0;
}

und wenn das Fahrzeug sich ausloggt alsow eiss ja net wie du es hast musst du geladen wieder 0 setzen dann sollte eigentlich das erstmal wieder richtig gehen