C++ et SQL : soucis de requête

Version imprimable

Bonjour tout le monde,

Je suis nouveau sur le forum et je pense que ça va être l'ultime solution pour résoudre mon problème. Enfin je l'espère.

Voila je suis en train de réaliser une mediathèque en C++, développé sous Code::blocks.

Je plante sur une fonction de recherche par titre de disque dans laquelle je dois récupérer le titre à rechercher d'un cin.

Voici la fonction :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 void ConnectionDisque::rechercher(const Disque &d){ string titre; cout << "entrez le titre" << endl; cin >> titre; int err = mysql_query(&connection, "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'"); if(err != 0){ throw SQLErrorException(mysql_error(&connection)); // On recupere le resultat result = mysql_use_result(&connection); } }
Et voici mon erreur générée :

Citation:

error: cannot convert `std::basic_string<char, std::char_traits<char>, std::allocator<char> >' to `const char*' for argument `2' to `int mysql_query(MYSQL*, const char*)'|

Ceci est pour mon projet de fin d'année de master que je dois rendre dans 2 semaines...

J'espère que vous pourrez faire quelque chose pour moi :)

Merci d'avance

02/07/2009, 09h45
3DArchi
Bonjour et bienvenu,

Code:

"SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'"

a comme résultat un std::string et il n'y a pas de conversion implicite vers const char*.
Il faut que tu passes par une variable intermédiaire puis utilises c_str() :
Code:

1 2 std::string requete = "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'"; int err = mysql_query(&connection, requete.c_str());
02/07/2009, 09h53
Winners21

Je te remercie je vais essayer, cependant pourrais-tu me dire ce que je fais de la ligne :

Code:

int err = mysql_query(&connection, "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'");

Je la remplace par :

Code:

std::string requete = "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'";

?

Merci d'avance
02/07/2009, 10h20
Winners21

Bon tout se compile tres bien mais maintenant quand j'éxécute il me balance une erreur de mémoire :

L'instruction à "0x100052a6" emploie l'adresse memoire "0x0000000c". La mémoire ne peut pas être "read"

Une idée ?

Apres avoir lancé le debugger, j'ai trouvé d'ou venait le bug.

Voici ce que le debugger me dit :

Citation:

100052A6 mysql_fetch_row()
004023EB ConncetionDisque::get_next_disque(this=0x22fba0)
0040429d main()

Voici ma fonction get_next_disque() :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
Disque ConnectionDisque::get_next_disque(){
    MYSQL_ROW row;
    if(row = mysql_fetch_row(result)){
        return
        (Disque(atoi(row[0]), /* Id */
        row[1], /* Titre */
        atoi(row[2]),/* Année */
        Artiste (atoi(row[4]), /* Id Artiste (Table Artiste) */
        row[5], /* Nom */
        row[6]))); /* Prénom */
    }
    else{
        throw NoMoreRowException();
    }
}

Voici son appel dans le main() :

Code:

1
2
3
4
5
6
7
8
9
10
11
try{
                        Disque d;
                        //conn.effectuer_requete();
                        conn.rechercher(d);
                        //while(true){
                        d = conn.get_next_disque();
                        d.afficher();
                    //}
                }
                catch(NoMoreRowException &e){
            }

02/07/2009, 13h38
Médinoc

Au passage, ce code est la porte ouverte vers de l'injection SQL (si on ignore le fait que cin >> titre s'arrêtera au premier espace venu, car il paraît intentionnel).

Utiliser des paramètres SQL ou bien mysql_escape_string().
02/07/2009, 13h46
Winners21

Pour l'instant c'est une phase de test, les titres sont sans espaces
02/07/2009, 13h47
3DArchi
Citation:

Envoyé par Winners21

Je te remercie je vais essayer, cependant pourrais-tu me dire ce que je fais de la ligne :

Code:

int err = mysql_query(&connection, "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'");

Je la remplace par :

Code:

std::string requete = "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'";

?

Merci d'avance

Comme indiqué dans mon message précédent, tu remplaces ta ligne :

Code:

int err = mysql_query(&connection, "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'");

par les 2 lignes :
Code:

1 2 std::string requete = "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'"; int err = mysql_query(&connection, requete.c_str());
Citation:

Envoyé par Winners21

Voici ce que le debugger me dit :

Mets un point d'arrêt, regarde l'état de tes différentes variables et avance pas à pas pour voir ce qui pose problème, en n'hésitant pas à entrer dans les fonction (F11).
02/07/2009, 14h06
Winners21

Je te remercie en faisant des pauses j'ai pu constater que ma requete prenait bien en compte la variable du 'cin'.

De ce point de vue la tout va bien mais au niveau de ma fonction get_next_disque c'est toujours pareil, on me donne une erreur dans libmysql.dll avec mysql_fetch_row...
02/07/2009, 14h21
3DArchi

D'où vient result dans l'appel if(row = mysql_fetch_row(result)), quel est son type et quelle est sa valeur ?

Le result vient de la précédente fonction qui est lancée.

Je te mets une copie de la déclaration des deux fonctions ainsi que de leur appel.

Fonctions :

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
void ConnectionDisque::rechercher(const Disque &d){
    string titre;
    cout << "entrez le titre" << endl;
    cin >> titre;
    std::string requete = "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id AND disque.titre = '"+titre+"'";
    int err =  mysql_query(&connection, requete.c_str());
    //cout << requete.c_str() << endl;
    if(err != 0){
         //throw SQLErrorException(mysql_error(&connection));
 
    // On recupere le resultat
    result = mysql_use_result(&connection);
    }
 
}
//Fin de fonction
 
//---------------------------------------------------------------------------------
//	/!\	oublie pas de rajouter les lignes : throw(NoMoreRowException) au début !!
//		Par contre pas terrible le coup de lever une exception quand on arrive à la
//		fin. Tu ferais mieux de renvoyer un Disque "vierge" et de le vérifier...
//---------------------------------------------------------------------------------
Disque ConnectionDisque::get_next_disque() throw(NoMoreRowException){
	//	on récupère le résultat suivant
	cout << "hello" << endl;
    MYSQL_ROW row;
	row = mysql_fetch_row(result);
    system("PAUSE");
	//	S'il n'y en a plus, on lève une exception
	if ( row == NULL)
		throw NoMoreRowException();
	//	On renvoit le disque récupéré
	return
        (Disque(atoi(row[0]), /* Id */
        row[1], /* Titre */
        atoi(row[2]),/* Année */
        Artiste (atoi(row[4]), /* Id Artiste (Table Artiste) */
        row[5], /* Nom */
        row[6]))); /* Prénom */

Appel dans le main :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
 try{
                        Disque d;
                        //conn.effectuer_requete();
                        conn.rechercher(d);
                        while(true){
                        d = conn.get_next_disque();
                        d.afficher();
                    }
                }
                catch(NoMoreRowException &e){
            }

J'ai mis des pause comme tu me l'a dis. Et le programme bloque arrivé a la ligne ou il y a ecrit 'mysql_fetch_rox()'.

02/07/2009, 14h47
3DArchi

Question bête : result ne vaudrait pas NULL par hasard ?

En fait, si tu as bien recopié le code que tu exécutes, alors, je pense que result est NULL :

Code:

1
2
3
4
5
6
if(err != 0){
   //throw SQLErrorException(mysql_error(&connection));
 
   // On recupere le resultat
   result = mysql_use_result(&connection);
}

Alors que tu voulais probablement écrire :

Code:

1
2
3
4
5
6
7
if(err != 0){
   //throw SQLErrorException(mysql_error(&connection));
}
else{
   // On recupere le resultat
   result = mysql_use_result(&connection);
}

Non, il n'y a aucune déclaration mettant result a Null...

Quand j'execute une fonction nommé "effectuer_requete", tout se passe bien et un affichage se fait tandis que pour la fonction rechercher ce n'est pas le cas alors que les 2 ne sont pas si différentes...
Code:

1 2 3 4 5 6 7 8 void ConnectionDisque::effectuer_requete(){ int err = mysql_query(&connection, "SELECT * FROM disque, artiste WHERE id_artiste = artiste.id"); if(err != 0) throw SQLErrorException(mysql_error(&connection)); // On recupere le resultat result = mysql_use_result(&connection); }

02/07/2009, 15h19
Winners21

Donc voila je ne comprends pas tout...

Au cas ou je peux toujours envoyer le projet complet pour bien pouvoir voir l'erreur générée...

Merci d'avance
02/07/2009, 15h57
Winners21

Encore et toujours moi, voici l'erreur clair donnée par le debugger :

Citation:

Program received signal SIGSEGV, Segmentation fault.
In mysql_fetch_row () (E:\Projet au propre\projet\projet\libmysql.dll)

Quelqu'un aurait une idée de la cause ?
02/07/2009, 16h01
Winners21
Citation:
Envoyé par 3DArchi

En fait, si tu as bien recopié le code que tu exécutes, alors, je pense que result est NULL :

Code:

1 2 3 4 5 6 if(err != 0){ //throw SQLErrorException(mysql_error(&connection)); // On recupere le resultat result = mysql_use_result(&connection); }

Alors que tu voulais probablement écrire :

Code:

1 2 3 4 5 6 7 if(err != 0){ //throw SQLErrorException(mysql_error(&connection)); } else{ // On recupere le resultat result = mysql_use_result(&connection); }
Je n'avais pas vu ton message !
La faute était tellement évidente que je ne l'ai meme pas vue !
Merci a toi c'était ça qui était a l'origine de ce P***** de bug !

Merci encore de ton aide ;)
03/07/2009, 01h14
nicroman

Ceci dit, ton code a un problême pour les titres contenant un '
(et ça va arriver)

La librairie native de mysql ne comportant pas de notion de 'Statement' (et donc de 'statement parameters', oui l'API C de mysql est très sale), il te faut faire à la main un mysql_real_escape_string()

D'autre part la manière dont tu parses le resultat est à proscrire absolument... Pour une simple raison, si tu rajoutes un field à la table 'disque', tous les fields de la table 'artiste' seront décalés dans les rows...

Il y a deux façons de faire ces requêtes...

La 'sale' (que personne ne fait en réalité)...
Tu fais le 'select *' comme maintenant,
Tu fais un mysql_fetch_fields() pour avoir la définition de tous les fields.
Tu "trouves" manuellement l'indice des fields qui t'interessent
Pour chaque row (mysql_fetch_row()) tu utilises les indices trouvés au lieu d'indices constants.

La 'propre' (que tout le monde fait).
Tu fais un 'select disque.field1, disque.field2, artiste.field3 from ...'
Et tu es sur d'avoir les fields au bon endroit. Et en prime, ne rappatrier que les informations nécessaires (imagines que la table 'disque' contienne en BLOB les données binaires du disque lui-même !!!)

Et puis... la question que tout le monde se pose... pourquoi ne pas utiliser libMySql++ ?
Tu aurais le bénéfice des statements (objet 'Query') et donc des paramètres automatiquement 'escapés'. Et aussi des Row::value_list()
03/07/2009, 09h00
Winners21

Pour répondre à toutes tes questions je débute seulement en C++, je ne peux pas tout connaitre. C'est bête et méchant mais quand on débute on essaye de bricoler avec les bout de code que nous donne le prof...
Quand on a un bon prof ça va mais quand on en a un comme le notre ça ne va pas !!
Tout ce que tu peux voir comme code, je n'ai rien produit par moi même, c'est le code donné par le prof qui est ré utilisé...
03/07/2009, 09h19
3DArchi

Citation:

Envoyé par Winners21

Pour répondre à toutes tes questions je débute seulement en C++, je ne peux pas tout connaitre. C'est bête et méchant mais quand on débute on essaye de bricoler avec les bout de code que nous donne le prof...
Quand on a un bon prof ça va mais quand on en a un comme le notre ça ne va pas !!
Tout ce que tu peux voir comme code, je n'ai rien produit par moi même, c'est le code donné par le prof qui est ré utilisé...

Si ton cours ne te donne pas entièrement satisfaction, n'hésites pas à consulter les ressources du site : Tutoriels (notamment des cours pour débuter) et FAQ.
03/07/2009, 11h52
white_tentacle

Citation:

La librairie native de mysql ne comportant pas de notion de 'Statement' (et donc de 'statement parameters', oui l'API C de mysql est très sale), il te faut faire à la main un mysql_real_escape_string()

:nono: Ça a changé, désormais il y a des statements dans l'api C.
04/07/2009, 01h01
nicroman

Haaa vi....

mysql_stmt_init()
mysql_stmt_prepare()
mysql_stmt_bind_param().

Tiens... je découvre.... ca fait tellement longtemps que j'ai plus utilisé mysql (nativement en tout cas)....
Mercie tentacle ! :)