MySQLQuery++ - a C++ MySQL Query Class

[nico_w]

EN:
Hey,

as I'm currently developing FoxBot, I was in need for a small class which is able to create prepared MySQL statements and execute them, preferably using C++ data types like std::string. As I was unable to get the MySQL C++ Connector running, I instead coded my own MySQL query class which is able to create a connection to a MySQL server and execute queries just like I wanted it to do.
It still needs the MySQL C headers and libs though which you can find in MySQL server installations and on the official MySQL website.

The most recent version can be found on my GitHub account in this repository:
Documentation (pdf): (although it should be self-explaining )

Best regards,
Nico

DE:
Hey,

da ich momentan am FoxBot arbeite brauchte ich die Möglichkeit, MySQL-Queries effizient und mit C++-Datentypen wie std::string ausführen zu können. Den offiziellen MySQL C++-Connector konnte ich leider mit MingW nicht zum Laufen bekommen, daher hab' ich mir kurzerhand eine eigene kleine Klasse geschrieben, die Prepared MySQL-Statements erstellen und ausführen kann.
Ihr benötigt dafür trotzalledem die MySQL Includes und Libraries, die sind für gewöhnlich bei einer MySQL-Server-Installation vorhanden oder man kann sie sich auf der MySQL-Webseite runterladen.

Die aktuellste Version findet ihr auf meinem GitHub-Account in diesem Repository:
Dokumentation (pdf, nur Englisch): (sollte aber eigentlich recht selbsterklärend sein )

Gruß,
Nico

[Requi]

Darf ich dich küssen?

Das kommt gerade passend für mein PServer Projekt.

:***

[Delinquenz]

Kann dir da nur empfehlen, eine meines Erachtens nach sehr ordentliche Bibliothek, die außerdem nicht auf MySQL beschränkt ist, sondern auch PostgreSQL, SQLite3 etc. unterstützt.

Wenn man allerdings nicht zu viel Overload möchte, würde ich deine Bibliothek empfehlen, da sie auch ziemlich ordentlich aussieht.

[nico_w]

Dass es da noch nen paar Libraries gibt ist klar, nur wollte ich etwas kleines, auf MySQL beschränktes - da kommt mir meine Klasse hier nun recht gelegen, denn sie unterstützt eigentlich alles was ich brauche, ist nicht überladen und übersichtlich gehalten.

[Schlüsselbein]

3 Dinge sind mir beim groben drüberschauen aufgefallen:
1. Keine const correctness.
2. Warum die ganzen std::string Objekte nicht als const ref übergeben?
3. Deine escapeStr-Funktion allokiert dynamischen Speicher ohne dieses freizugeben. Das ist schlechter Stil. Das Objekt, welches Speicher anfordert, sollte dafür verantwortlich sein, diesen wieder freizugeben. Außerdem gibt die Methode einen char* statt std::string zurück.

[nico_w]

Originally Posted by Schlüsselbein

3 Dinge sind mir beim groben drüberschauen aufgefallen:
1. Keine const correctness.
2. Warum die ganzen std::string Objekte nicht als const ref übergeben?
3. Deine escapeStr-Funktion allokiert dynamischen Speicher ohne dieses freizugeben. Das ist schlechter Stil. Das Objekt, welches Speicher anfordert, sollte dafür verantwortlich sein, diesen wieder freizugeben. Außerdem gibt die Methode einen char* statt std::string zurück.

Danke, ich werd' mich drum kümmern, das zu berichtigen. Hatte das alles nach Feierabend schnell hingeschrieben und werde den Code nochmal dementsprechend verbessern.

[Ende!]

Originally Posted by Schlüsselbein

3 Dinge sind mir beim groben drüberschauen aufgefallen:
1. Keine const correctness.
2. Warum die ganzen std::string Objekte nicht als const ref übergeben?
3. Deine escapeStr-Funktion allokiert dynamischen Speicher ohne dieses freizugeben. Das ist schlechter Stil. Das Objekt, welches Speicher anfordert, sollte dafür verantwortlich sein, diesen wieder freizugeben. Außerdem gibt die Methode einen char* statt std::string zurück.

^dies, plus:

• Die Initialisierung der std::strings in Konstruktor macht keinen Sinn, da der Default-Constructor sie bereits so initialisiert. Weiterhin würde ich da wohl 'ne Initialisierungsliste bevorzugen.
• MySQLConnection::EscapeString: Bösartiger Heap-Overflow, Output von mysql_real_escape_string kann bis zu doppelt so lang sein, wie der Input Buffer
• Schrecklich viel redundanter Code bei den gettern .. fauler Hund, kapseln! :P
• Keine einheitliche Konvention für Benamung von Methoden (oder erkenne ich sie nur nicht?)

Sind noch 'n paar kleinere Dinge drin .. schreib mich dafür einfach mal aufm Jabber an, wenn du Bock hast.

[Schlüsselbein]

Es gibt n Jabber?

[Ende!]

Originally Posted by Schlüsselbein

Es gibt n Jabber?

Für Lv2+ :P Überlegen momentan noch, den auch für andere Leute zu öffnen, mal sehen.

[.Marcel']

Sehr schön, danke.

[nico_w]

So, ist alles behoben :3

[Schlüsselbein]

Paar Methoden sollten noch als const gekennzeichnet werden und primitive Datentypen übergibt man besser by value. Als Referenz bringt dir keinen Vorteil (sollte sogar noch langsamer sein als by value, auch wenns Nanooptimization ist und demnach gar keine Rolle spielt) - es sieht einfach unnatürlich aus

Sieht sonst ganz okay aus.

[th0rex]

Hey kannst du mir sagen warum, wenn ich <mysql.h> include, sehr viele Fehler von wegen Neudefinition bekommen ? (123 IntelliSense: Mindestens eine Instanz der überladenen Funktion ""WSAStartup"" weist eine "C"-Verknüpfung auf. Ungefähr 110 Stück davon)
Habe schon gegoogelt aber nicht wirklich was brauchbares gefunden ...

Edit : Habs gelöst ...