Beiträge von Kaliber

Warum zur Hölle so kompliziert?

Machs einfach nur so:

stock const Float:H_W[][3] = {
{244.4221, 305.0592, 999.1484},
{266.7091, 305.0197, 999.1484},
{2324.4587, -1148.8004, 1050.7101},
{235.3885, 1187.0400, 1080.2578},
{225.7580, 1239.9197, 1082.1406},
{223.1530, 1287.4996, 1082.1406},
{225.7527, 1021.9715, 1084.0162},
{295.0053, 1472.9153, 1080.2578},
{328.0984, 1478.0658, 1084.4375},
{386.7174, 1471.7593, 1080.1875},
{2269.7668, -1210.4520, 1047.5625},
{2365.2903, -1135.0590, 1050.8750}
};

ocmd:exit(playerid)
{
for(new i; i<sizeof(H_W); i++)
{
if(!IsPlayerInRangeOfPoint(playerid, 7.0, H_W[i][0], H_W[i][1], H_W[i][2])) continue;
new v=GetPlayerVirtualWorld(playerid);
printf("Welt: %d | %.02f | %.02f | %.2f",v,HausInfo[v][hX],HausInfo[v][hY],HausInfo[v][hZ]);
SetPlayerInterior(playerid, 0),SetPlayerVirtualWorld(playerid, 0);
return SetPlayerPos(playerid, HausInfo[v][hX],HausInfo[v][hY],HausInfo[v][hZ]),1;
}
return SendClientMessage(playerid,-1,!"Sie befinden sich an keinem Haus!");
}

Schau mal was dir geprintet wird...wenn da 0|0|0 in den Koordinaten rauskommt, stimmt die VirtuelleWelt nicht mit dem index von HausInfo überein...

Du solltest dir vielleicht überlegen, dass alles in ein Array zu schreiben

mfg.

Zitat von Apokrate

Werde aber trotzdem gekickt, wenn ich den Namen habe.

Machs so:

stock const Betatester[][] = { //oben ins Skript
"Admin","Test"
};

new b_name[MAX_PLAYER_NAME];
GetPlayerName(playerid, b_name, MAX_PLAYER_NAME);
for(new j; j < sizeof(Betatester); j++)
{
if(strcmp(b_name, Betatester[j]) != 0)
{
SendClientMessage(playerid, COLOR_ERROR, "Der Server befindet sich in der Betaphase. Du musst eingetragen sein, um zu spielen.");
return Kick(playerid);
}
}

Zitat von Jeffry

In einem Timer:

Wofür brauchst du bitte einen Timer?

Man muss die Daten doch nur 1x beim Ausloggen speichern...und da ist es immer wieder die selbe Struktur.

Und bei Registrieren muss man halt 1x einen Insert Query ausführen und der hat natürlich auch die selbe Struktur.

Für triviale kleine Querys, brauchst du das natürlich nicht verwenden...

Mein Gott

Zitat von Jeffry

Du hast das Array mit den Spalten, das macht es im Endeffekt mehr Code. Da jedes Query unterschiedlich ist, muss ich für jedes Query die Spalten neu dort angeben, das macht es eher mehr Code, als weniger.

Dir sollte bewusst sein, wenn du meinen Text gelesen hast, dass es nur um das Speichern z.B. von Spielerdaten o.ä. geht.

Und da reicht es 1x die Tabellen Struktur zu deklarieren, somit es es wesentlich weniger Code!

Zitat von Jeffry

Auf den ersten Blick sehe ich hier nicht, welches davon richtig ist, und welches falsch ist. Sowas kann schnell passieren und man merkt es nicht. Das bezieht sich auch auf die Wartbarkeit. Ich muss immer hin und her scrollen, zwischen dem rows Array und der Zeile hier, um herauszufinden, dann entweder alter oder money richtig ist. Bei einem Query mit 100 Attributen wird das lustig.

Dir ist klar, dass es bei dem normalen format nicht anders ist du scherzkeks.

Zudem kann man sich die Struktur auch dahin als Kommentar schreiben...

Zitat von Jeffry

Da sehe ich ganz genau, was zu was gehört.

Das ist aber langsamer...
und wie gesagt...die Struktur kann man angeben, das nicht so kompiziert als kommentar...

Zitat von Jeffry

Nur weiß das kaum einer.

Ändert nichts an der Tatsache...

Zitat von Jeffry

Queries wird das ziemlich unübersichtlich

Nun, wenn man es 1x am Anfang richtig gemacht hat...ist es sehr einfach weitere dazuzutun.
Man muss das halt schritt für schritt machen, aber dafür ist es wesentlich einfacher, neue Sachen leicht hinzuzufügen.

und ein normales Query in diesem Maße ist noch viel unübersichtlicher...

Aber wie gesagt..ist jedem selbst überlassen wie er es macht...

Ein paar Leute wollten so eine Include, also habe ich sie Ihnen zur Verfügung gestellt, ich verstehe dein Problem damit nicht.

Das format so schreiben:

format(string,sizeof(string),"%sName: %s (ID: %i) | Ort: %s | Situation: %s\n",string,SpielerName(i),i,ZoneName(i),lengt);

mfg.

Du darfst hier kein Prepared Statement verwenden:

Zitat

However, they are not allowed for identifiers (such as table or column names), in the select list that names the columns to be returned by a SELECT statement, or to specify both operands of a binary operator such as the = equal sign.

Du musst hier ein normales Query senden...

PS: Deine Nutzung ist übrigens falsch, lies dir dazu nochmal eine Dokumentation durch...

@Jack_Bauer:

Machs mal so:

if(preg_match("#^[a-zA-Z]+$#", $this->currentPage))

mfg.

Zitat von Wellendoxairfix

oder wie ihr es gelöst habt.

Da gibt es diesen Callback:

public OnActorStreamIn(actorid, forplayerid)
{
SetActorPos(actorid,acInfo[actorid][ac_x],acInfo[actorid][ac_y],acInfo[actorid][ac_z]);
SetActorFacingAngle(actorid,acInfo[actorid][ac_a]);
return 1;
}

Dir sollte aufgefallen sein, dass das 3DTextLabel immer richtig sitzt, aber der Actor nicht.

Das liegt daran, dass die Positionen zwar richtig geladen werden, der Actor an sich aber aSynchron ist und du ihn somit immer richtig setzen musst

mfg.

Zitat von Ronny17

Weil ich will das ganze nämlich mit dem Wetter System zb bauen das es auch mal Wetter gibt wo Plötzlich die Stadt Überschwemmt wird Habe sowas bei Samp bisher noch nicht gesehen

Nun in SA:MP ist das mit den bisherigen nativen Funktionen auch nicht so gut machbar.

Wenn du sowas gut umsetzen willst und neue Features brauchst, schau dir das hier mal an: SAMPFox - Über 90 neue Natives! Jetzt für SA:MP 0.3z!

mfg.

Schau dir diesen Callback mal an, damit ist das sehr gut möglich: https://wiki.sa-mp.com/wiki/OnPlayerWeaponShot

mfg.

Dann haste schlecht gesucht

Hier: mysql_static.so

Suche mal die mysql_static.so und lade die nur

Zitat von Jeffry

Mehr Code

Also eine simple Zeile ist länger, als 6 Zeilen wo man mit jedem Komma und jedem Sonderzeichen zu kämpfen hat...ok

Zitat von Jeffry

Langsamer

Bei 100 Aufrufen...1ms

Zitat von Jeffry

Fehleranfälliger

Eigentlich ja eher nicht.
Wieso auch....wenn man es so macht wie ich es gezeigt habe, werden eben keine Fehler auftreten, dafür ist es ja da...man muss sich mit den Kommas etc nicht mehr rumplagen und es wird ja geprüft ob Spalten & Anzahl der Attribute übereinstimmt

PS: Wenn String > 1024 kann man ihn erhöhen, das steht dann in der Konsole, genauso die anderen Fehlermeldungen...

Zitat von Jeffry

Unübersichtlicher als ein ganz normales Query

Du findest eine Zeile wo du klar siehst welche Variable zu was gehört unübersichtlicher als 6 oder mehr Zeilen, die voll geschrieben sind mit Sonderzeichen und Feldern...alles klar

Zitat von Jeffry

Schlechte Wartbarkeit

Wieso? Wenn was nicht funktioniert...muss es ja an einem der 2 Kriterien liegen...eigentlich ist das eine sehr gute Wartbarkeit

Zitat von Jeffry

Allgemeinverständlichkeit (niemand weiß, dass Name|Alter für Name und Alter steht, da hätte ich wenigstens & genommen)

Es stehen beide Zeichen indirekt dafür:

new x,y,z
if(x | y | z) //Hier geht der Compiler auch durch alle Variablen, anstatt bei der 1. falschen abzubrechen ^^

Zitat von Jeffry

Zudem kommt, dass wenn man mehrere Queries hat, die unterschiedliche Spalten in derselben Tabelle ändern, dann muss man die "rows" Variable für dieses Query jedes mal, bzw. oft, neu anlegen, das ist Speicherverschwendung, wenn man in Betracht zieht, dass kein wirklicher Vorteil dabei entsteht.

Ich hoffe dir ist bewusst, dass man nicht gezwungen ist diese Funktionen zu verwenden, aber in den Meisten Fällen kann dies sehr praktisch sein.

Zitat von IPrototypeI

Um noch etwas anzumerken das MySQL Query von BlueG/ maddinator beinhaltet das ORM - System, damit spart man sich genug Zeit.

Joa das ist ja jedem selbst überlassen

Probier mal mein System aus, das sollte einfacher und übersichtlicher sein

Es ist zwar ressourcenfressender logischerweise, allerdings macht es bei 10.000 Aufrufen nur ca 50ms Unterschied

//Edit: Bei 100 aufrufen, liegt der unterschied nur bei 1ms

Schau mal hier rein: MySQL Builder Library

mfg.

Hallo, viele von euch kennen das Problem mit den langen MySQL Querys und das es sehr unübersichtlich ist.

Aus diesem Grund habe ich mal eine kleine Library geschrieben womit man einfach diese Querys erzeugen lassen kann.

Funktionen:

native MySQL_Insert(handle,const rows[][],const target_Table[],const form[],{Float, _}:...);
native MySQL_Update(handle,const rows[][],const target_Table[],const where[], const form[],{Float, _}:...);

Beispiel:

//Unter OnGameModeInit
handle = mysql_connect(...);

//Oben ins Skript:
stock const rows[][] = {
"`Geld`","`Alter`","`X`","`Name`" //Hier sind die Spalten Namen, wichtig sind hier die `
};
//Dann da wo Ihr den User registriert (INSERT INTO) benötigt:
new money = 5000,alter=16,Float:x=3.141,name[] = "Test123"; //Das sind Beispiel Daten zum speichern

MySQL_Insert(handle,rows,"Test","ddfs",money,alter,x,name);
/*
"Test" = Der Tabellen-Name
"ddfs" = Die Form (die Reihenfolge der Datentypen) d=Integer,f=Float,s=String
*/

//Angenommen wir wollen jetzt den Spieler nur noch Updaten:
new money = 9500,alter=16,Float:x=3.141,name[] = "Test123";

MySQL_Update(handle,rows,"Test","Name","ddfws",money,alter,x,name);
/*
"Test" = Wieder der Tabellen Name
"Name" = Das Where Statement.
"w" = Das w in dem Format gibt an, dass das nachfolgende das WHERE Statement ist (und s dahinter der Typ halt von dem).
*/

Mehrere Where Statements:

new money = 9500,alter=16,Float:x=3.1412,name[] = "Test123";

MySQL_Update(handle,table,"Test","Name|Alter","dfwswd",money,x,name,alter);

Download:
Source-Code: Pastebin
Direkter-Download: MediaFire

mfg.

Hier eine nützliche Funktion:

stock ReplaceString(const search[],const replace[], source[], size=sizeof source) //(c) by Kaliber
{
new i=strfind(source,search),tmp=strlen(search);
if(i==-1) return 0;
for( ; i!=-1; i=strfind(source,search,false,i+1))
{
strdel(source,i,i+tmp),strins(source,replace,i,size);
}
return 1;
}

//Beispiel:
new string[64] = "Ich liebe Gemüse und Gemüse!";
ReplaceString("Gemüse", "Schokolade", string);
print(string); //-> "Ich liebe Schokolade und Schokolade!"

Gibt es zwar schon, aber diese Funktion ist um einiges schneller, als das was ich sonst so gesehen habe

Es kommt auch auf das Betriebssystem an.

Da hat Pawno irgendwie eine Macke, wenn du eine Version über Windows 7 hast...also (8/8.1/10) stell mal den Kompatibilitätsmodus auf Windows 7

mfg.

Zeig uns mal was im Skript dazu steht...ggf auch die Variable größe erhöhen

Naja, das komma am ende des Querys ist falsch...und ich schätze mal der String ist zu klein oder nicht leer...