[SAMMELTHREAD] Kleine Scripting Fragen

XonarZ · 24. September 2018

Zitat von LeonMrBonnie

Du musst für floats auch %f nehmen.

Okay. Nun werden allerdings zu viele nullen angezeigt, kann man das auch irgendwie reduzieren ? Ich würde das ganz gerne dann so auch fürs Tacho übernehmen haha^^

Und noch etwas ... ich hab mich mal eben versucht, dass Fahrzeuge aus der Datenbank geladen und erstellt werden ... Dazu habe ich folgendes gemacht:

Enum:

Code

enum vehData{
	vehID,
	vehMID,
	vehOwner[64],
	vehFuel[64],
	Float:vehTank,
	vehLocked,
	vehColor1,
	vehColor2,
	Float:vehKMStand,
	Float:vehVoltage,
	Float:vehHealth,
	Float:vehLPKM,
	vehLoad,
	Float:vehX,
	Float:vehY,
	Float:vehZ,
	Float:vehA
}
new vInfo[][vehData];

Alles anzeigen

OnGameModeInit:

Code

LoadCars();

und nun der stock:

Code

stock LoadCars(){
	new query[256], rows;
	mysql_format(handle, query, sizeof(query), "SELECT * FROM cars WHERE laden = '1'");
	mysql_pquery(handle, query);
	cache_get_row_count(rows);
	for(new i = 0; i < sizeof(rows); i++)
	{
	    cache_get_value_name_int(i, "vehicleid", vInfo[i][vehMID]);
	    cache_get_value_name_float(i, "posx", vInfo[i][vehX]);
	    cache_get_value_name_float(i, "posx", vInfo[i][vehY]);
	    cache_get_value_name_float(i, "posx", vInfo[i][vehZ]);
	    cache_get_value_name_float(i, "posx", vInfo[i][vehA]);
	    cache_get_value_name_int(i, "color1", vInfo[i][vehColor1]);
	    cache_get_value_name_int(i, "color2", vInfo[i][vehColor1]);
	    CreateVehicle(vInfo[i][vehMID], vInfo[i][vehX], vInfo[i][vehY], vInfo[i][vehZ], vInfo[i][vehA], vInfo[i][vehColor1], vInfo[i][vehColor2], -1);
	}
	return 1;
}

Alles anzeigen

Ich hab das gefühl, dass ich irgendwas falsch verstanden habe haha^^ ... Ich habe mit absicht erstmal nur die paar sachen ausgelesen, damit ich testen kann ob es klappt

Mit freundlichen Grüßen
XonarZ

LeonMrBonnie · 24. September 2018

So wie du das machst ist es nicht ganz falsch.
Aber alles nach dem MySQL_pquery muss in eine Public Funktion die du mit MySQL_pquery aufrufst.
Denn mit mysql_pquery kannst du die Ergebnisse vom query an eine Public Funktion weitergeben und verarbeiten.
Also machst du es zu;_
mysql_pquery(handle,query, "OnLoadCars");

Dann erstellt du die neue Public Funktion:
public OnLoadCars()
{
}
Und kopierst dann den ganzen Code unter dem pquery den du gerade hast einfach in die Funktion rein.

XonarZ · 24. September 2018

Zitat von LeonMrBonnie

So wie du das machst ist es nicht ganz falsch.
Aber alles nach dem MySQL_pquery muss in eine Public Funktion die du mit MySQL_pquery aufrufst.
Denn mit mysql_pquery kannst du die Ergebnisse vom query an eine Public Funktion weitergeben und verarbeiten.
Also machst du es zu;_
mysql_pquery(handle,query, "OnLoadCars");

Dann erstellt du die neue Public Funktion:
public OnLoadCars()
{
}
Und kopierst dann den ganzen Code unter dem pquery den du gerade hast einfach in die Funktion rein.

Alles anzeigen

Also brauche ich sozusagen 2 Callbacks / stocks um die Funktion für die Fahrzeuge zu machen oder kann ich den Query auch unter OnGameModeInit ausführen und dann an LoadCars geben ?

LeonMrBonnie · 24. September 2018

Ja, du kannst den Query auch direkt unter OnGamemodeInit ausführen. Das macht keinen Unterschied.

Jeffry · 24. September 2018

Zitat von XonarZ

Okay. Nun werden allerdings zu viele nullen angezeigt, kann man das auch irgendwie reduzieren ? Ich würde das ganz gerne dann so auch fürs Tacho übernehmen haha^^

%0.2f
Damit kannst du zum Beispiel 2 Nachkommastellen anzeigen.

Zitat von XonarZ

Also brauche ich sozusagen 2 Callbacks / stocks um die Funktion für die Fahrzeuge zu machen oder kann ich den Query auch unter OnGameModeInit ausführen und dann an LoadCars geben ?

Das SELECT Query kannst du bei OnGameModeInit ausführen und darüber das Callback mit den Cache Funktionen aufrufen.

XonarZ · 24. September 2018

Zitat von LeonMrBonnie

Ja, du kannst den Query auch direkt unter OnGamemodeInit ausführen. Das macht keinen Unterschied.

Okay. Ich habs jetzt mal einfach probiert erstmal mit 2 Stocks ... Leider hab ich anscheinend noch einen Fehler ... die Autos werden trotzdem nicht erstellt.

Ich weiß auch leider nicht woran es liegt. Ich kann mir zwar vorstellen, dass die zuweisung mit vInfo[i] ... nicht ganz stimmt... muss ich noch irgendwie beim enum MAX_VEHICLES oder so eintragen ?

PS: wegen dem Float hast du noch nix gesagt, kann man die iwie auf 2 Stellen hinterm Komma begrenzen ?

//EDIT Jeffry hat mir das mit dem Float nun schon gesagt (nicht reloaded gehabt)

Jeffry · 24. September 2018

Zitat von XonarZ

Ich weiß auch leider nicht woran es liegt. Ich kann mir zwar vorstellen, dass die zuweisung mit vInfo ... nicht ganz stimmt... muss ich noch irgendwie beim enum MAX_VEHICLES oder so eintragen ?

Wie sieht dein Code jetzt aus, und wie sieht die Datenbank aus?

XonarZ · 24. September 2018

Zitat von Jeffry

Wie sieht dein Code jetzt aus, und wie sieht die Datenbank aus?

Enum und new vInfo:

Code

enum vehData{
	vehID,
	vehMID,
	vehOwner[64],
	vehFuel[64],
	Float:vehTank,
	vehLocked,
	vehColor1,
	vehColor2,
	Float:vehKMStand,
	Float:vehVoltage,
	Float:vehHealth,
	Float:vehLPKM,
	vehLoad,
	Float:vehX,
	Float:vehY,
	Float:vehZ,
	Float:vehA
}
new vInfo[][vehData];

Alles anzeigen

OnGameModeInit:

Code

//LADEN
	new query[256];
	mysql_format(handle, query, sizeof(query), "SELECT * FROM cars WHERE laden = '1'");
	mysql_pquery(handle, query, "OnLoadCars");

stock OnLoadCars

Code

stock OnLoadCars(){
	new rows;
    cache_get_row_count(rows);
	for(new i = 0; i < sizeof(rows); i++)
	{
	    cache_get_value_name_int(i, "vehicleid", vInfo[i][vehMID]);
	    cache_get_value_name_float(i, "posx", vInfo[i][vehX]);
	    cache_get_value_name_float(i, "posy", vInfo[i][vehY]);
	    cache_get_value_name_float(i, "posz", vInfo[i][vehZ]);
	    cache_get_value_name_float(i, "posa", vInfo[i][vehA]);
	    cache_get_value_name_int(i, "color1", vInfo[i][vehColor1]);
	    cache_get_value_name_int(i, "color2", vInfo[i][vehColor1]);
	    CreateVehicle(vInfo[i][vehMID], vInfo[i][vehX], vInfo[i][vehY], vInfo[i][vehZ], vInfo[i][vehA], vInfo[i][vehColor1], vInfo[i][vehColor2], -1);
	}
	return 1;
}

Alles anzeigen

Datenbank:

SQL

-- Adminer 4.6.3 MySQL dump


SET NAMES utf8;
SET time_zone = '+00:00';
SET foreign_key_checks = 0;
SET sql_mode = 'NO_AUTO_VALUE_ON_ZERO';


DROP TABLE IF EXISTS `cars`;
CREATE TABLE `cars` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `vehicleid` int(11) NOT NULL,
  `owner` varchar(64) NOT NULL,
  `fuel` varchar(64) NOT NULL,
  `tank` float NOT NULL,
  `lock` int(11) NOT NULL,
  `color1` int(11) NOT NULL,
  `color2` int(11) NOT NULL,
  `kmstand` float NOT NULL,
  `voltage` float NOT NULL,
  `health` float NOT NULL,
  `lpkm` float NOT NULL,
  `laden` int(11) NOT NULL,
  `posx` float NOT NULL,
  `posy` float NOT NULL,
  `posz` float NOT NULL,
  `posa` float NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1;


INSERT INTO `cars` (`id`, `vehicleid`, `owner`, `fuel`, `tank`, `lock`, `color1`, `color2`, `kmstand`, `voltage`, `health`, `lpkm`, `laden`, `posx`, `posy`, `posz`, `posa`) VALUES
(1,	401,	'niemand',	'Benzin',	5,	0,	5,	5,	0,	12,	1000,	0.1,	1,	1252.56,	339.167,	19.1865,	246.282),
(2,	411,	'niemand',	'Benzin',	5,	0,	0,	0,	0,	12,	1000,	0.1,	1,	1241.02,	344.355,	19.1334,	70.2362);


-- 2018-09-24 20:31:40

Alles anzeigen

Wusste jetzt nicht ganz wie ich sonst mysql einbinden sollte ... Halt Tabelle "cars" und gleich halt ein paar einträge^^

LeonMrBonnie · 24. September 2018

Nicht Stock OnLoadCars. Du musst dafür ein Public verwenden.
Also Public OnLoadCars

Jeffry · 24. September 2018

Zitat von XonarZ

new vInfo[][vehData];

zu:
new vInfo[50][vehData];

Die Anzahl der Fahrzeuge musst du einmal definieren, sonst weiß der Code nicht, wie viel Speicher er reservieren soll.

Die ID des Fahrzeugs würde ich beim Erstellen noch speichern, die wirst du sicher brauchen:
vInfo[i][vehVID] = CreateVehicle(vInfo[i][vehMID], vInfo[i][vehX], vInfo[i][vehY], vInfo[i][vehZ], vInfo[i][vehA], vInfo[i][vehColor1], vInfo[i][vehColor2], -1);

EDIT:
Plus das, was LeonMrBonnie sagte. Das über mysql_pquery aufgerufene Callback muss ein public sein, sprich auch mit forward Zeile.
stock OnLoadCars(){
zu:
forward OnLoadCars();
public OnLoadCars(){

EDIT 2:
Und die ID ebenfalls auslesen:
cache_get_value_name_int(i, "id", vInfo[i][vehID]);
Die brauchst du später beim Speichern.

XonarZ · 24. September 2018

Also alle änderungen getätigt. Leider keiner Wirkung...

vielleicht eine info von mir ... vInfo[i][vehID] soll die DBID sein, das vehMID soll das Model sein, damit ich besser abfragen kann welches Auto das ist um mit dem maximalen Tank (meinem anderem enum halt) zu vergleichen^^

Jeffry · 24. September 2018

Sollte so eigentlich klappen.
Kannst du den Code nochmal posten?

Gibt dir der MySQL Log etwas aus?

XonarZ · 24. September 2018

Zitat von Jeffry

Sollte so eigentlich klappen.
Kannst du den Code nochmal posten?

Gibt dir der MySQL Log etwas aus?

ich habe zwar mysql_log(); drinne allerdings erstellt der keine mysql log^^

forwards (hab ich ganz oben im script)

Code

//FORWARDS
forward OnUserCheck(playerid);
forward OnUserRegister(playerid);
forward OnUserLogin(playerid);
forward KickTimer(playerid);
forward OnLoadCars();
forward checkVehHealth(vehicleid);

Enum und new vInfo

Code

enum vehData{
	vehID,
	vehMID,
	vehOwner[64],
	vehFuel[64],
	Float:vehTank,
	vehLocked,
	vehColor1,
	vehColor2,
	Float:vehKMStand,
	Float:vehVoltage,
	Float:vehHealth,
	Float:vehLPKM,
	vehLoad,
	Float:vehX,
	Float:vehY,
	Float:vehZ,
	Float:vehA
}
new vInfo[50][vehData];

Alles anzeigen

OnGameModeInit

Code

//LADEN
	new query[256];
	mysql_format(handle, query, sizeof(query), "SELECT * FROM cars WHERE laden = '1'");
	mysql_pquery(handle, query, "OnLoadCars");

public OnloadCars():

Code

public OnLoadCars(){
	new rows;
    cache_get_row_count(rows);
	for(new i = 0; i < sizeof(rows); i++)
	{
	    cache_get_value_name_int(i, "id", vInfo[i][vehID]);
	    cache_get_value_name_int(i, "vehicleid", vInfo[i][vehMID]);
	    cache_get_value_name_float(i, "posx", vInfo[i][vehX]);
	    cache_get_value_name_float(i, "posy", vInfo[i][vehY]);
	    cache_get_value_name_float(i, "posz", vInfo[i][vehZ]);
	    cache_get_value_name_float(i, "posa", vInfo[i][vehA]);
	    cache_get_value_name_int(i, "color1", vInfo[i][vehColor1]);
	    cache_get_value_name_int(i, "color2", vInfo[i][vehColor1]);
	    vInfo[i][vehID] = CreateVehicle(vInfo[i][vehMID], vInfo[i][vehX], vInfo[i][vehY], vInfo[i][vehZ], vInfo[i][vehA], vInfo[i][vehColor1], vInfo[i][vehColor2], -1);
	}
	return 1;
}

Alles anzeigen

Jeffry · 24. September 2018

Zitat von XonarZ

ich habe zwar mysql_log(); drinne allerdings erstellt der keine mysql log^^

Schau mal im Ordner /logs/plugins/mysql.txt

Der Code sieht gut aus.

XonarZ · 24. September 2018

Zitat von Jeffry

Schau mal im Ordner /logs/plugins/mysql.txt
Der Code sieht gut aus.

Okay...

Code

[22:59:37] [ERROR] mysql_format: invalid connection handle '0'
[22:59:37] [ERROR] mysql_pquery: invalid connection handle '0'

das sind die zwei fehler, die nur durch den Start auftreten (ich denke mal halt wenn OnGameModeInit(); aufgerufen wird)

Jeffry · 24. September 2018

Das sieht so aus, als rufst du erst das Query zum Laden der Fahrzeuge auf und stellst danach erst die Verbindung zur MySQL Datenbank her.
Stelle die Verbindung ganz als erstes bei OnGameModeInit her.

XonarZ · 24. September 2018

Zitat von Jeffry

Das sieht so aus, als rufst du erst das Query zum Laden der Fahrzeuge auf und stellst danach erst die Verbindung zur MySQL Datenbank her.
Stelle die Verbindung ganz als erstes bei OnGameModeInit her.

WoW man bin ich dumm ... ja das ist die Lösung... Allerdings muss ich sagen, dass er nur das erste Auto geladen hat, das zweite wurde nicht gespawnt.

Jeffry · 24. September 2018

Wenn du weitere Fahrzeuge hinzufügst, sodass es mindestens 4 Stück in der Datenbank sind, wie viele werden dann geladen? Und welches fehlt?

XonarZ · 24. September 2018

Zitat von Jeffry

Wenn du weitere Fahrzeuge hinzufügst, sodass es mindestens 4 Stück in der Datenbank sind, wie viele werden dann geladen? Und welches fehlt?

Ich habe jetzt insgesamt 7 Autos drinne, davon wird nur das erste (DBID 1) geladen. die restlichen 6 (ID 2 - 7) werden nicht geladen.

Jeffry · 24. September 2018

for(new i = 0; i < sizeof(rows); i++)
zu:
for(new i = 0; i < rows; i++)

Das hast du gut versteckt