/lock befehl

Ist car_besitzer ein Name? Wenn ja musst du %s verwenden

Kannst du mir mal verraten, wieso du den "Besitzer" nach Integer speichern lässt. ?
Ist ziemlich schwachsinnig, wenn sich da jemand was kauft, wird 1-unendlich gesetzt ? Woher möchtest du dann wissen, wenn einer ein Problem mit seinem Auto hat, welches das ist ? Kannst ja dann nicht alle löschen ?

MFG.

Wenn du den Car Besitzer speichern willst musst du es als String angeben

Ach Leute! Er fragt die Datenbank ID ab, und die ist nunmal ein Integer. Er muss nichts als String speichern.
if(cInfo[vID][besitzer] == sInfo[playerid][db_id])

Zeige doch mal den Teil des Codes bei dem du sInfo[playerid][db_id] einen Wert zuweist.

Was gibt er denn aus, wenn du unter:
cInfo[id][besitzer]=cache_get_field_content_int(i,"besitzer",dbhandle);

das hier einfügst:
printf("Vehicle %d: Besitzer %d", id, cInfo[id][besitzer]);

Prüfe auch mal, ob
new id=getFreeCarID()
und
vID = GetClosestVehicle(playerid, 20);
beim selben Auto die selben Werte haben, das müssen sie nämlich, sonst macht alles keinen Sinn.

stock GetClosestVehicle(playerid, Float:range)
{
new Float:p_X;
new Float:p_Y;
new Float:p_Z;

new Float:Distance;
new Float:PretendentDistance = range +1;
new Pretendent;

GetPlayerPos(playerid, p_X, p_Y, p_Z);

for(new vehicleid=1; vehicleid < MAX_VEHICLES; vehicleid++)
{
Distance = GetVehicleDistanceFromPoint(vehicleid, p_X, p_Y, p_Z);
printf("Vehicleid: %d -> %0.2f", vehicleid, Distance);
if(Distance <= range && Distance <= PretendentDistance)
{
printf("%0.2f <= %0.2f => vehicleid = %d", Distance, PretendentDistance, vehicleid);
Pretendent = vehicleid;
PretendentDistance = Distance;
}
}

return Pretendent;
}

Dann poste was passiert, wenn du nahe dem Auto bist (ID 52). Die 52 ist ja richtig, die muss auch bei der Funktion ausgegeben werden. Solange sie das nicht tut, brauchen wir gar nicht weiter zu machen.

Aber Hallo! So eine Ausgabe ist Gold wert.

Ändere
if(Distance <= range && Distance <= PretendentDistance)

Zu
if(Distance <= range && Distance <= PretendentDistance && Distance != 0.0)

Dann dürfte beides mal die 52 zurückgeben werden.

Schildere dann, falls bestehend, das nächste Problem.

Hast du inzwischen eine Lösung? Weil ich hab das selbe Problem.

stock GetCarID(vID)
{
printf("vID-GetCarID:%i",vID);
for(new i=0; i<MAX_PRIV_CARS; i++) if(cInfo[i][id_x] == vID) return i;
return -1;
}

MAX_PRIV_CARS=> Das was du bei "new cInfo[HIER][...]" hast.

Und dann beim /lock:
printf("vID:%i",vID);
printf("car_besitzer:%i", cInfo[vID][besitzer]);
printf("besitzer_dbid:%i", sInfo[playerid][db_id]);

zu:
vID = GetCarID(vID);
if(vID == -1) return SendClientMessage(playerid, 0xFF000FF", "ERROR: Auto ist kein mit MySQL erstelltes Auto.");
printf("vID:%i",vID);
printf("car_besitzer:%i", cInfo[vID][besitzer]);
printf("besitzer_dbid:%i", sInfo[playerid][db_id]);

Was wird geprintet?

Zitat

MAX_PRIV_CARS=> Das was du bei "new cInfo[HIER][...]" hast.

Das was bei "HIER" steht nimmst du und schreibst du da hin, wo ich MAX_PRIV_CARS geschrieben habe.

Nein.

Du sollst in diesem Code:
stock GetCarID(vID)
{
printf("vID-GetCarID:%i",vID);
for(new i=0; i<MAX_PRIV_CARS; i++) if(cInfo[i][id_x] == vID) return i;
return -1;
}
MAX_PRIV_CARS ersetzen, und zwar mit dem was bei dir im Code dort steht, wo ich "HIER" geschrieben habe.
Wenn du es immer noch nicht verstehst, poste bitte die Zeile
new cInfo....