Erstele Spieler autos in die DB speicher

Onjiary · 8. Januar 2017

Guten Morgen

Ich habe es geschafft das Spielerfahrzeuge erstellt werden aber wie ich die in die Datenbank rein bekommen für die Spieler ist mir noch ein sehr großes rätzel:

Wie ich nun anfangen soll das die Fahrzeuge gespeichert werden in die db noch aus der db geladen werden hoffe ich auf eure hilfe

Mein Code für Spielerfahrzeuge erstellen sieht wie folgt aus:

Fahrzeugenum:
[spoiler

Code

enum fahrzeugEnum
{
	faid,
	besitzer[MAX_PLAYER_NAME],
	model,
	Float:c_x,
	Float:c_y,
	Float:c_z,
	Float:c_r
}
new cInfo[50][fahrzeugEnum];

Alles anzeigen

[/spoiler]

PlayerCar:
[spoiler

Code

PlayerCar(playerid,modelid,Float:x,Float:y,Float:z,Float:r)
{
	for(new i=0; i<sizeof(cInfo); i++)
	{
		if(cInfo[i][faid]!=0)continue;
		GetPlayerName(playerid,cInfo[i][besitzer],MAX_PLAYER_NAME);
		cInfo[i][c_x] =x;
		cInfo[i][c_y] =y;
		cInfo[i][c_z] =z;
		cInfo[i][c_r] =r;
		cInfo[i][model]=modelid;
		cInfo[i][faid] = CreateVehicle(modelid,x,y,z,r,-1,-1,-1);
		new string[128];
		format(string,sizeof(string),"Das Fahrzeug cInfo[%i] wurde erstellt",i);
		SendClientMessageToAll(rot,string);
		return 1;
	}
	return 1;
}

Alles anzeigen

[/spoiler]

createcar:

Spoiler anzeigen

linux_ · 8. Januar 2017

Wie sieht die Tabellenstruktur in Deiner Datenbank zu den Fahrzeugen aus?

Mr.Pawn · 8. Januar 2017

Das sieht mir sehr stark nach dem Fahrzeugsystem aus der MrMonatHD Tutorialreihe aus. Schau mal ein oder 2 Videos weiter, da erklärt er auch die Datenbankerstellung für die Fahrzeuge.

Onjiary · 8. Januar 2017

@Slider hier meine Datenbank Struktur für die Spielerfahrzeuge:

http://www.pic-upload.de/view-32447772/Unbenannt.png.html

linux_ · 8. Januar 2017

Wieso ist 'besitzer' auf float und nicht unsigned int? x, y, z und r zu double machen.

Onjiary · 8. Januar 2017

weil x,y,z,r Flot-Kordinaten sein sollen wegen kommar zeichen

linux_ · 8. Januar 2017

Die muessen aber double sein, da float zu ungenau ist. Hast Du das mit dem Besitzer auf unsigned int geaendert?

Onjiary · 8. Januar 2017

@Slider
was ist denn der unterschied zwischen dem Flot und double?

Ich habe besitzer auf int gestellt mehr auch nicht

TheOneBehindYou1 · 8. Januar 2017

Zitat von Bergustelo

@Slider
was ist denn der unterschied zwischen dem Flot und double?

Ich habe besitzer auf int gestellt mehr auch nicht

Der Unterschied ist, dass doubles mehr komma zahlen anzeigen/speichern
In diesem Fall ist es klüger float zu nehmen, da im Script die Koordinaten in Floats abgespeichert werden und double somit relativ überflüssig ist

seegras · 8. Januar 2017

Zitat von Bergustelo

@Slider
was ist denn der unterschied zwischen dem Flot und double?

Ich habe besitzer auf int gestellt mehr auch nicht

Zitat

MySQL uses four bytes for single-precision values and eight bytes for double-precision value

https://dev.mysql.com/doc/refm…floating-point-types.html

TheOneBehindYou1 · 8. Januar 2017

Zitat von Slider

Wieso ist 'besitzer' auf float und nicht unsigned int?

Zitat von Bergustelo

Ich habe besitzer auf int gestellt mehr auch nicht

Es ist weiters auch sehr unnötig den Besitzer auf int zu setzen da du im Script den Namen und nicht die ID abfragst. Eher auf VARCHAR setzen

linux_ · 8. Januar 2017

Zitat von TheOneBehindYou1

Es ist weiters auch sehr unnötig den Besitzer auf int zu setzen da du im Script den Namen und nicht die ID abfragst. Eher auf VARCHAR setzen

Wenn das Dein Ernst ist, hast Du keine Ahnung von Datenbanken.

@Bergustelo setz es bitte auf unsigned int. Alles andere ist totaler Schwachsinn... Man ordnet ID's zu und keine Namen

Onjiary · 8. Januar 2017

Toll jetzt habt ihr mich alle so verwirt das ich erst recht nicht weiß wie ich das anfangen soll.

linux_ · 8. Januar 2017

Die einzig richtige Loesung: x, y, z, r auf Double setzen fuer die Praezision und den Besitzer auf unsigned int. Und arbeite bitte in Zukunft immer mit User ID's und NICHT mit Namen!

Kinimod · 8. Januar 2017

Zitat von Bergustelo

Toll jetzt habt ihr mich alle so verwirt das ich erst recht nicht weiß wie ich das anfangen soll.

Hör auf das, was Slider sagt. Er trifft nämlich den Punkt.

Das hat mehrere Gründe: Ein wichtiger ist: Wenn der Spielername geändert wird, musst du jede Tabelle, in der der Name dann zugeordnet wird (also bspw. in deiner Fahrzeugtabelle), auch anpassen. Aus diesem Grund ist das, was TheOneIrgendwas geschrieben hat, auch scheiße. Das ist unschöner Stil und ohnehin nur komplizierter. Wenn du von vorneherein mit Spieler ID's arbeitest, hast du den ganzen Firlefanz nicht.

Onjiary · 9. Januar 2017

@Slider @Jeffry

Ich habe jetzt mal versucht es irgendwie hin zu bekommen mit den Spielerfahrzeugen in die Datenbank zu setzen

Spielerfahrzeuge werden auch nicht durch den befehl einfach so in das spiel rein gesetzt:

Der Code sieht so aus:

Ich verwende die Version R41.2 von Mysql

Spielerfahrzeuge erstellen:

Enum

Spoiler anzeigen

PlayerCar

Spoiler anzeigen

Fahrzeuge erstellen und löschen

Spoiler anzeigen

Fahrzeuge Speicher und laden:

forward:

Spoiler anzeigen

spielerfahrzeug

Spoiler anzeigen

carinDB

Spoiler anzeigen

datWeazel · 9. Januar 2017

Zitat von Slider

Die einzig richtige Loesung: x, y, z, r auf Double setzen fuer die Praezision und den Besitzer auf unsigned int. Und arbeite bitte in Zukunft immer mit User ID's und NICHT mit Namen!

Dein Double für die "Praezision" kannst du dir schenken. Sämtliche SAMP Funktionen schlucken nur Float und daher sind deine extra Nachkommastellen der Doubles komplett nutzlos.
SetVehiclePos und CreateVehicle nehmen nur Float für die Koordinaten und GetVehiclePos gibt nur Float zurück.

Und ob du nun nen unsigned int oder ein einfaches int nutzt ist ebenfalls komplett egal.
Ich bezeifle, dass ein samp server die 2.147.483.647 SpielerIDs knackt. Und wenn doch dann entweder Respekt für so viele unique players oder pfui bäh für schlechte Datenbankverwaltung (Denn das würde bedeuten, dass immer wieder Spieler gelöscht und durch Auto-Increment neue IDs benutzt werden.) (Selbes gilt natürlich auch für die Fahrzeug IDs. 2.147.483.647 sind dann doch etwas zu viel für SAMP.)

@TE: Bekommst du denn die Ausgabe "Das Fahrzeug cInfo[xy] wurde erstellt" angezeigt ?

Onjiary · 9. Januar 2017

@datWeazel wenn ich folgenden Befehl ausführe ingame: Sagt er mir Ungültiges Model

Code

ocmd:createcar(playerid,params[])
{
	if(!isAdmin(playerid,6))return SendClientMessage(playerid,rot,"Dein Admingrang ist zu niedrig.");
	new mID,pID;
	if(sscanf(params,"ui",pID,mID))return SendClientMessage(playerid,rot,"INFO: /createcar[playerid][model]");
	if(mID<400||mID>611)return SendClientMessage(playerid,rot,"Ungültiges Model");
	new Float:xc,Float:yc,Float:zc,Float:rc;
	GetPlayerPos(pID,xc,yc,zc);
	GetPlayerFacingAngle(pID,rc);
	PlayerCar(pID,mID,xc,yc,zc,rc);
	return 1;
}

Alles anzeigen

aber leider bekomm ich bei : spielerfahrzeug folgenden Fehler:
spielerfahrzeug

Der Code zu Spielerfahrzeug sieht so aus: weiter kam ich damit nicht

Code

public spielerfahrzeug(carid)
{
	 cInfo[carid][faid]=cache_insert_id(handle);
	 return 1;
}

datWeazel · 9. Januar 2017

Was genau gibst du ein?
Du musst z.b "/createcar 511" eingeben.

Onjiary · 9. Januar 2017

@datWeazel

Normal sollte er Spielerid und die modelID nutzen da ja die Autos an den Spieler gesetzt werden