Vehicle will nicht speichern [mysql]

Cryptex · 26. Oktober 2012

Bin grad dabei kaufbare vehicles zu erstellen,
soweit ein player in einem bestimmten shopfahrzeug sitz kann er den selben wagen an einer bestimmten stelle spawnen

irgendwie will der das aber nicht in die datenbank speichern, kann mal einer drüberschauen, ich finde den fehler nicht

CMD:abuyvehicle(playerid, params[])
{
if(Vehicles[GetPlayerVehicleIDEX(playerid)][fvType] == 10)
{
new Query[500], vModel, vColor1, vColor2;
vModel = GetVehicleModel(GetPlayerVehicleID(playerid));
GetVehicleColor(GetPlayerVehicleID(playerid), vColor1, vColor2);
format(Query, sizeof(Query), "INSERT INTO `Vehicles` (VehicleModel = '%d', VehicleX = '%f', VehicleY = '%f', VehicleZ = '%f', VehicleAngle = '%f', Color1 = '%d', Color2 = '%d', VehicleType = '%d', vBesitzer = '%s', vPreis = '%d')",
vModel, -1630.91, 1289.79, 6.74587, 134.595, vColor1, vColor2, 999, SpielerNameAnzeige(playerid), 100);
mysql_query(Query);
mysql_free_result();

LoadPlayerVehicle(playerid);
SendClientMessage(playerid, green, "{FF0000}[SERVER] {FFFFFF}Du hast dir das Fahzeug gekauft");
}
return 1;
}

IPrototypeI · 26. Oktober 2012

Deine SQL Command ist falsch aufgebaut

dieser sollte so aufgebaut sein
"INSERT INTO `Vehicles` (`VehicleModel`, `VehicleX`, `VehicleY`, `VehicleZ`, `VehicleAngle`, `Color1`, `Color2`, `VehicleType`, `vBesitzer`, `vPreis`) VALUES ('%d', '%f', '%f', '%f', '%f', '%d', '%d', '%d', '%s', '%d')"

Cryptex · 26. Oktober 2012

das macht eig kein untschied, so hatte ich das auch vorher..
damit geht das aber auch nicht habe es eben nochmal getestet

das funktioniert z.b. wenn ich noale autos spawne und speichere
stock AddVehicleToFile(Float:vX, Float:vY, Float:vZ, Float:vA, vModel, vColor1, vColor2)
{
new Query[200];
format(Query, sizeof(Query), "INSERT INTO `Vehicles` (VehicleModel, VehicleX, VehicleY, VehicleZ, VehicleAngle, Color1, Color2) VALUES(%d, %f, %f, %f, %f, %d, %d)",
vModel, vX, vY, vZ, vA, vColor1, vColor2);
mysql_query(Query);
mysql_free_result();
return 1;
}

muss man die werte erst floaten? eig nicht oder, die parameter zeigen ja das es floads sind oder?

//edit 1
float text hinzugefügt

d3adlockz · 26. Oktober 2012

Wieso verwendest du dein stock AddVehicleToFile nicht einfach in deinem Command, wenn dieser Funktioniert? Wäre doch viel einfacher, und wohl auch sinn einer solchen Funktion.

SylpheedW · 26. Oktober 2012

Da er in dem Stock SpielerName bzw. vBesitzer nicht drin hat.
Schreib es doch einfach etwas um.

Und nenne es zbw. AddPlayerVehicleToFile

MFG Piet

d3adlockz · 26. Oktober 2012

Achso, hab ich nicht gesehen. Aber selbst dann hätte ich es für Sinnvoll gehalten diese Funktion in den anderen Code zu kopieren und die 2 Variablen anzuhängen, dann tauchen wohl eher weniger Bugs auf bei solchen minimalen Veränderungen

Cryptex · 27. Oktober 2012

Ich habe das umgeschrieben, habe auch einpar fehler gefunden jedoch geht das immernoch nicht
Da muss ein experte ran Bestimmt ist nur wieder was kleine flasch und ich sehe das wieder nicht...

CMD:abuyvehicle(playerid, params[])
{
new rand = random(sizeof(RandomSpawnVehicleShop)), vModel, vColor1, vColor2, vType, vBesitzer[500], vPreis;
vModel = GetVehicleModel(GetPlayerVehicleID(playerid));
GetVehicleColor(GetPlayerVehicleID(playerid), vColor1, vColor2);
vType = 999;
vBesitzer = SpielerNameAnzeige(playerid);
vPreis = 100;

if(Vehicles[GetPlayerVehicleIDEX(playerid)][fvType] == 10)
{
AddPlayerVehicleToFile(vModel, RandomSpawnVehicleShop[rand][0], RandomSpawnVehicleShop[rand][1], RandomSpawnVehicleShop[rand][2], RandomSpawnVehicleShop[rand][3], vColor1, vColor2, vType, vBesitzer[128], vPreis);
SendClientMessage(playerid, green, "{FF0000}[SERVER] {FFFFFF}Du hast dir das Fahzeug gekauft");
}
return 1;
}

stock AddPlayerVehicleToFile(vModel, Float:vX, Float:vY, Float:vZ, Float:vA, vColor1, vColor2, vType, vBesitzer, vPreis)
{
new Query[200];
format(Query, sizeof(Query), "INSERT INTO `Vehicles` (`VehicleModel`, `VehicleX`, `VehicleY`, `VehicleZ`, `VehicleAngle`, `Color1`, `Color2`, `VehicleType`, `Besitzer`, `Price`) VALUES ('%d', '%f', '%f', '%f', '%f', '%d', '%d', '%d', '%s', '%d')",
vModel, vX, vY, vZ, vA, vColor1, vColor2, vType, vBesitzer, vPreis);
mysql_query(Query);
mysql_free_result();
return 1;
}

Mein SQL Aufbau

SQL

`ID` int(11) NOT NULL AUTO_INCREMENT,
  `VehicleModel` int(3) NOT NULL,
  `VehicleX` float NOT NULL,
  `VehicleY` float NOT NULL,
  `VehicleZ` float NOT NULL,
  `VehicleAngle` float NOT NULL,
  `Color1` float NOT NULL,
  `Color2` float NOT NULL,
  `VehicleType` int(3) NOT NULL DEFAULT '0',
  `Paintjob` int(1) NOT NULL DEFAULT '0',
  `Price` int(11) NOT NULL DEFAULT '0',
  `Locked` int(1) NOT NULL DEFAULT '0',
  `Neon` int(1) NOT NULL DEFAULT '0',
  `Besitzer` varchar(24) NOT NULL DEFAULT ' ',

Alles anzeigen

Die SQLTabelle

d3adlockz · 27. Oktober 2012

Einen Fehler den ich auf Anhieb finde ist hier:
AddPlayerVehicleToFile(vModel, RandomSpawnVehicleShop[rand][0], RandomSpawnVehicleShop[rand][1], RandomSpawnVehicleShop[rand][2], RandomSpawnVehicleShop[rand][3], vColor1, vColor2, vType, vBesitzer[128], vPreis);
Du darfst bei vBesitzer keine Zahl hinten Anhängen. Einfach nur vBesitzer.

Cryptex · 27. Oktober 2012

jo der will für die zeile

vBesitzer = SpielerNameAnzeige(playerid);

ein array, darum habe ich das eig gemacht

Error wenn ich den arry bei new weg mache

Code

error 006: must be assigned to an array

wenn ich es nur unten im quarry weg mache

Code

error 035: argument type mismatch (argument 9)

d3adlockz · 27. Oktober 2012

Das entfernen im Quarry ist richtig. Nur hast du in deinem anderen Code noch einen Fehler.

stock AddPlayerVehicleToFile(vModel, Float:vX, Float:vY, Float:vZ, Float:vA, vColor1, vColor2, vType, vBesitzer, vPreis)

wird zu

stock AddPlayerVehicleToFile(vModel, Float:vX, Float:vY, Float:vZ, Float:vA, vColor1, vColor2, vType, vBesitzer[], vPreis)

Wenns nicht so ist denkt der Code dass vBesitzer ein Integer ist, aber es ist ja ein String.

Cryptex · 27. Oktober 2012

stimmt, ich währe nicht drauf gekommen jedoch macht er immernoch nicht, muss aber nah dran sein

aktuell sieht das so aus

new Float:RandomSpawnVehicleShop[][4] =
{
{-1630.91, 1289.79, 6.74587, 134.595},
{-1630.91, 1289.79, 6.74587, 134.595}
};

CMD:abuyvehicle(playerid, params[])
{
new rand = random(sizeof(RandomSpawnVehicleShop)), vModel, vColor1, vColor2, vType, vBesitzer[500], vPreis;
vModel = GetVehicleModel(GetPlayerVehicleID(playerid));
GetVehicleColor(GetPlayerVehicleID(playerid), vColor1, vColor2);
vType = 999;
vBesitzer = SpielerNameAnzeige(playerid);
vPreis = 100;

if(Vehicles[GetPlayerVehicleIDEX(playerid)][fvType] == 10)
{
AddPlayerVehicleToFile(vModel, RandomSpawnVehicleShop[rand][0], RandomSpawnVehicleShop[rand][1], RandomSpawnVehicleShop[rand][2], RandomSpawnVehicleShop[rand][3], vColor1, vColor2, vType, vBesitzer, vPreis);
SendClientMessage(playerid, green, "{FF0000}[SERVER] {FFFFFF}Du hast dir das Fahzeug gekauft");
}
return 1;
}

stock AddPlayerVehicleToFile(vModel, Float:vX, Float:vY, Float:vZ, Float:vA, vColor1, vColor2, vType, vBesitzer[], vPreis)
{
new Query[200];
format(Query, sizeof(Query), "INSERT INTO `Vehicles` (`VehicleModel`, `VehicleX`, `VehicleY`, `VehicleZ`, `VehicleAngle`, `Color1`, `Color2`, `VehicleType`, `Besitzer`, `Price`) VALUES ('%d', '%f', '%f', '%f', '%f', '%d', '%d', '%d', '%s', '%d')",
vModel, vX, vY, vZ, vA, vColor1, vColor2, vType, vBesitzer, vPreis);
mysql_query(Query);
mysql_free_result();
return 1;
}

d3adlockz · 27. Oktober 2012

Omg, wie konnte ich das nur übersehen
Du kannst einen String nicht mit = ändern. (vBesitzer = SpielerNameAnzeige(playerid);)
Verwende das so:

format(vBesitzer,500,"%s",SpielerNameAnzeige(playerid));
Und die Größe würde ich nebenbei gesagt auch niedriger einstellen, hab noch keine Person gesehen die nen Namen mit 500 Buchstaben hatte

Cryptex · 27. Oktober 2012

Jo, die größe habe ich einfach so eingelegt um zu gucken ob da was flasch ist,
danke für deine hilfe, aber da ist immernoch fas faul...

CMD:abuyvehicle(playerid, params[])
{
new rand = random(sizeof(RandomSpawnVehicleShop)), vModel, vColor1, vColor2, vType, vBesitzer[500], vPreis;
vModel = GetVehicleModel(GetPlayerVehicleID(playerid));
GetVehicleColor(GetPlayerVehicleID(playerid), vColor1, vColor2);
vType = 999;
format(vBesitzer, 500, "%s", SpielerNameAnzeige(playerid));
vPreis = 100;

if(Vehicles[GetPlayerVehicleIDEX(playerid)][fvType] == 10)
{
AddPlayerVehicleToFile(vModel, RandomSpawnVehicleShop[rand][0], RandomSpawnVehicleShop[rand][1], RandomSpawnVehicleShop[rand][2], RandomSpawnVehicleShop[rand][3], vColor1, vColor2, vType, vBesitzer, vPreis);
SendClientMessage(playerid, green, "{FF0000}[SERVER] {FFFFFF}Du hast dir das Fahzeug gekauft");
}
return 1;
}

d3adlockz · 27. Oktober 2012

Was soll denn das hier sein?
if(Vehicles[GetPlayerVehicleIDEX(playerid)][fvType] == 10)

Edit: Oder die bessere Frage, bekommst du die grüne Nachricht?

Blunt · 27. Oktober 2012

SQL

INSERT INTO `Vehicles` (`VehicleModel`, `VehicleX`, `VehicleY`, `VehicleZ`, `VehicleAngle`, `Color1`, `Color2`, `VehicleType`, `Besitzer`, `Price`) VALUES ('%d', '%f', '%f', '%f', '%f', '%d', '%d', '%d', '%s', '%d')

215 Zeichen

new Query[200]

Just sayin

Cryptex · 27. Oktober 2012

und zu guter schluss, es geht
man man man, lag zuletzt an der querry, jop die nachricht bekomme ich, sonst würde ich hier nicht fragen hehe
danke an alle.. und besonders an d3adlockz xD

stock AddPlayerVehicleToFile(vModel, Float:vX, Float:vY, Float:vZ, Float:vA, vColor1, vColor2, vType, vBesitzer[], vPreis)
{
new Query[400];
format(Query, sizeof(Query), "INSERT INTO `Vehicles` (`VehicleModel`, `VehicleX`, `VehicleY`, `VehicleZ`, `VehicleAngle`, `Color1`, `Color2`, `VehicleType`, `Besitzer`, `Price`) VALUES ('%d', '%f', '%f', '%f', '%f', '%d', '%d', '%d', '%s', '%d')",
vModel, vX, vY, vZ, vA, vColor1, vColor2, vType, vBesitzer, vPreis);
mysql_query(Query);
mysql_free_result();
return 1;
}

breadfish.de 11. März 2022