Constructeur Destructeur MySQL

Version imprimable

Bonjour à tous,

J'ai développé une classe permettant un accès à MySQL.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 class mysql{ private: sql::Driver *driver; sql::Connection *con; sql::Statement *stmt; sql::ResultSet *res; public: mysql(); ~mysql(); void selectDataBase(string); void useProcedureToInsert(vector<string>, string); };
Dans le constructeur je fais
Code:

1 2 3 4 5 6 7 mysql::mysql(){ driver = get_driver_instance(); con = driver -> connect(DBHOST, USER, PASSWORD); stmt = con->createStatement(); stmt->execute("USE MaDB"); stmt->execute("SET SESSION sql_mode = 'ANSI'"); }
Dans le destructeur je fais
Code:

1 2 3 4 5 mysql::~mysql(){ delete res; delete stmt; delete con; }
Quand le destructeur est appelé, il y a une exception de générée. Et là je ne vois pas pourquoi :cry:
Si quelqu'un a une explication.....

D'avance merci
Cordialement

21/09/2009, 18h17
Alp

Quelle est l'exception ?
21/09/2009, 19h01
zul

Un destructeur avec des delete, et pas de constructeur par copie, ni de surcharge de =, ça sent le "double delete" à plein nez.

Une bonne règle en général, c'est que si ton destructeur est non-trivial, il faut surcharger manuellement le constructeur par copie, et l'opérateur =, sinon elles copient bêtement les pointeurs.
21/09/2009, 20h04
JulienDuSud
Une bonne règle de programmation est de respecter la RAII: celui qui a crée est celui qui doit détruire. Or, dans ta classe mysql, je ne vois aucun new.

Ton destructeur devrait plutôt ressembler à
Code:

1 2 3 4 5 mysql::~mysql(){ stmt->releaseResultSet( res ); // détruit res con->releaseStatement( stmt ); // détruit stmt driver->disconnect( con ); // détruit con }
Ces méthodes pourraient ressembler à
Code:

1 2 3 4 5 6 7 8 9 Connection::releaseStatement( Statement *& stmt ) { if( stmt ) { delete stmt; stmt = 0; } } ...
Tu peux bien sûr améliorer le système en implémentant un système de tracage des objets créer, mais le mieux reste l'utilisation de pointeurs intelligent.

Sinon, comme déjà dit, il faut que tu définisse le constructeur par copie.

Salut à tous :-)

Alp, je n'ai pas réussi à capturer l'exception même en mettant un bloc catch (...) dans le main.

Que signifie RAII ?

Visiblement si une exception a été émise parce que le delete ne fonctionne pas, je suppose que les déclarations ne sont pas des allocations dynamiques.

Y'a un truc que je ne comprend pas. Dans le code ci-dessous

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
int main (){
	sql::Driver *driver;
	sql::Connection *con;
	sql::Statement *stmt;
	sql::ResultSet  *res;
 
 
	try {
 
		driver = get_driver_instance();
		con = driver -> connect(DBHOST, USER, PASSWORD);
		stmt = con->createStatement();
		stmt->execute("USE MaDB");
 
		res = stmt->executeQuery("SELECT ID, NOM, PRENOM FROM MaDB ORDER BY id ASC");
		while (res->next()) {
			cout << res->getInt("ID")	<< "\t" << res->getString("NOM") << "\t" << res->getString("PRENOM") << endl;
		}
 
 
	} catch (sql::SQLException &e) {
		cout << "ERROR: SQLException in " << __FILE__;
		//cout << " (" << __func__<< ") on line " << __LINE__ << endl;
		cout << "ERROR: " << e.what();
		cout << " (MySQL error code: " << e.getErrorCode();
		cout << ", SQLState: " << e.getSQLState() << ")" << endl;
 
		if (e.getErrorCode() == 1047) {
			/*
			Error: 1047 SQLSTATE: 08S01 (ER_UNKNOWN_COM_ERROR)
			Message: Unknown command
			*/
			cout << "\nYour server does not seem to support Prepared Statements at all. ";
			cout << "Perhaps MYSQL < 4.1?" << endl;
		}
 
		return EXIT_FAILURE;
	} catch (std::runtime_error &e) {
 
		cout << "ERROR: runtime_error in " << __FILE__;
		//cout << " (" << __func__ << ") on line " << __LINE__ << endl;
		cout << "ERROR: " << e.what() << endl;
 
		return EXIT_FAILURE;
	}catch(...){
		return EXIT_FAILURE;
	}
 
	delete res;
	delete stmt;
	delete con;
 
	system("PAUSE");
 
 
	return EXIT_SUCCESS;
}

à la fin on fait bien un delete.
Quelle est la différence entre le code de test ci-dessus et celui posté dans le message principal ?

Désolé, c'est peut être simple pour vous, moins pour moi :oops:

Merci

22/09/2009, 11h31
JulienDuSud

RAII = Ressource Acquisition Is Initialisation, et un de ses principes, entre autres, est que c'est celui qui a alloué un objet, ou instancié un objet qui doit le détruire.

Pour les objets statiques, ça marche tout seul grâce au stack, mais il faut l'implémenter si tu utilises des pointeurs. Ou mieux, utiliser des pointeurs intelligents, qui vont se détruire eux même quand ils ne seront plus référencés.

Je t'ai conseillé d'implémenter la déstruction de tes objets du côté de sql::, vu que tu les crées par leur biais.
23/09/2009, 11h09
lephotographe
Hello :-)

Bon en fait je ne suis pas sûr de tout avoir compris.
JulienDuSud, ce que tu me proposes signifie que je devrais faire 4 nouvelles classes héritant respectivement de "Driver, Connection, Statement, ResultSet" pour y implémenter les méthodes que tu proposes.

J'ai donc fais cela
Code:

1 2 3 4 5 mysql::~mysql(){ res->close(); stmt->close(); con->close(); }
Pour ce qui est des attributs, je suppose que leurs destructeurs respectifs sont appelés lorsque mon objet mysql se termine.
23/09/2009, 11h17
JulienDuSud

Attention, si tu dérives d'une classe que tu n'as pas crée, il faut que tu t'assuers qu'elle est complètement virtuelle afin d'assurer la compatibilité.

Si les méthodes que j'ai décrite n'existent pas dans ta librairie, c'est qu'elle est mal conçue de base. En réalité, des méthodes "close()" ne devraient même pas exister dans un code source bien conçu, du moins en C++.

C'est la classe qui crée qui doit tracer les pointeurs qu'il crée afin de pouvoir les nettoyer correctement.
24/09/2009, 10h38
lephotographe

Hello :-)

Je pense que le connecteur C++ pour MySQL a été bien conçu. Enfin je le souhaite.

En fait pour être sûr que les objets soient bien détruits, il faudrait que je trouve un outils qui puisse détecter les fuites mémoires. Je crois avoir vu une fois ce genre d'outil quelque part.

En tout cas merci pour ton aide :ccool:
24/09/2009, 10h40
Alp

Ou sinon, comme dit plus haut, t'appliques le RAII...
24/09/2009, 11h26
3DArchi

Citation:

Envoyé par Alp

Ou sinon, comme dit plus haut, t'appliques le RAII..

Et pour préciser le RAII, quelques lectures ;) :
F.A.Q. : Comment gérer proprement des allocations / désallocations de ressources : le RAII
Tutoriel : Gérer ses ressources de manière robuste en C++, par Aurélien Regat-Barrel
Tutoriel : Pointeurs intelligents, par Loïc Joly