Erreur Access Violation : API mySQL et Builder 6

Bonjour :D

Je dois, pour un TP, développer un application sous Builder(6) utilisant l'api mySQL.
Dans celui-ci, je dois à un moment lister les tables d'une base de donnée dans une ListBox. Selon le choix de l'utilisateur, le système affichera dans une deuxième ListBox la liste des champs (leurs noms) correspondant à la sélection.
Pour faire simple :

ListBox1 = nom des tables
ListBox2 = nom des champs de la table sélectionnée (par double-clic)

Le soucis viens de la récupération des noms des champs. Lors du double-clic, le programme plante en affichant le message d'erreur suivant :

Citation:

---

Violation d'accès à l'adresse xxxxxx dans le module 'rtl60.bpl'. Lecture de l'adresse yyyyyy

---

Je vous met le morceau de code générant l'erreur 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

AnsiString selection = ListBox1->Items->Strings[ListBox1->ItemIndex]; AnsiString requete = "SELECT * FROM "; requete += selection;   char* requeteChar = requete.c_str();   if(!mysql_query(mySQL, requeteChar)) { myRES = mysql_store_result(mySQL);   if(myRES) { compteur = mysql_num_fields(myRES); myFIELDS = mysql_fetch_fields(myRES);   for(int i=0 ; i<compteur ; i++) { resultat = myFIELDS[i].name; // ligne générant l'erreur ListBox2->Items->Add(resultat); } } }

---

J'ai tout essayé : cast pour voir si ça venait du type, conversion char*->AnsiString... rien n'y fait.
Merci d'avance ;)

Salut,
as mon avis, vu la ligne de code générant l'exception et sont emplacement, tu dois avoir un débordement de tableau, ton i doit être plus grand que la capacité de ton tableau et donc tu accède à une adresse mémoire foireuse et dont tu n'as pas les droits.

Pourtant, la variable "compteur" correspond au nombre de champs dans la table. Du coup, logiquement, je ne devrais pas avoir de débordement...
Je viens d'essayer de parcourir la tableau avec i-1, mais ça revient au même.


Edit : j'ai oublié de préciser que pour myFIELDS[i].type, il n'y a aucune erreur. Ça fonctionne correctement, tant que le retour de myFIELDS[i].xxx n'est pas un char* :?

au lieu de mettre ta variable resultat en AnsiString essaye en la mettant en std::string (ne pas oublier de faire le #include <string>), si l'API mySql à des problème avec l'AnsiString comme tu le dit ca peut venir de la.

C'est tu au bout de combien de passage dans ta boucle cela plante ?

Je vais essayer ça.
Pour le nombre de passage, je ne peux pas te dire exactement. En mode pas-à-pas, ça semble planter dès le premier passage.

Cela me semble logique que ca plante au premier passage si les AnsiString sont en cause ;)

Cool, ça génère une nouvelle erreur ! :D

Toujours une violation d'accès, mais dans le module "CC3260MT.dll" cette fois ci.
J'ai remplacé la boucle for précédente par celle-ci :

Code:

---

1 2 3 4 5 6 7

for(int i=0 ; i<compteur ; i++) { resultatSTDString = myFIELDS[i].name; // Mémorasation dans un variable std::string resultat = AnsiString(resultatSTDString.c_str()); // Passage std::string->AnsiString ListBox2->Items->Add(resultat); // Ne prend que de l'AnsiString }

---

C'est tu quel ligne fait planter et combien valais i au moment du plantage ?

Bonsoir,

Une petite question au passage, juste pour m'éclairer:
"resultat" est de quel type ? Je ne le vois déclaré nulle part dans le code du premier message, il doit être "en dehors" de la partie copiée surement.

@+
;)

C'est un AnsiString ;)

Citation:

---

C'est tu quel ligne fait planter et combien valais i au moment du plantage ?

---

Alors, cette ligne (toujours la même) :

Code:

---

resultatSTDString = myFIELDS[i].name;

---

plante au second passage dans la boucle.
Pour le premier passage, resultat et resultatSTD valent bien le nom du premier champ de la table choisie :roll:
Par contre, dès le deuxième passage : plantage (i = 1).

Désolé pour le temps entre chaque réponse.

Citation:

---

Envoyé par Furanku

Alors, cette ligne (toujours la même) :

Code:

---

resultatSTDString = myFIELDS[i].name;

---

plante au second passage dans la boucle.
Pour le premier passage, resultat et resultatSTD valent bien le nom du premier champ de la table choisie :roll:
Par contre, dès le deuxième passage : plantage (i = 1).

---

On dirai bien que c'est un dépassement de tableau, mais le fait que ça fonctionne avec les propriété non char* prouve que non, c'est très bizarre, peut-être un problème au niveau de l'API mySql qui aurait un bug mais je n'y croit pas trop, il y aurai plus de monde qui aurai ce problème :?

Citation:

---

Envoyé par Furanku

Désolé pour le temps entre chaque réponse.

---

Ce n'est rien, le forum est fait pour résoudre les problème à la vitesse que chacun le souhaite ;)

J'ai oublié de préciser que même si resultat prend bien la valeur au premier passage, cette ligne :

Code:

---

ListBox2->Items->Add(resultat); // Ne prend que de l'AnsiString

---

n'affiche pas pour autant le 1er résultat dans la ListBox... :?
Un dépassement de mémoire, j'y ai pensé aussi, mais je ne vois pas trop où il pourrait être puisque "compteur" correspond au nombre de champs dans ma table.
A la limite, s'il était plus grand, mais là...

Question tu as bien mis en tout premier include et j insiste bien sur premier

#include <windows.h>

??

Chose faite, ça ne change rien ;)

chose toute bete , tu as fait un test de connection ou recuperer des données dans une table ?

Pas de soucis pour l'accès à la base.
J'ai une première ListBox qui me liste convenablement l'ensemble des tables. De même, je peux récupérer y les informations sans problème. Par contre pour le nom des champs... bah voilà quoi :roll:

Salut Furanku
Je ne connais pas mySQL mais sur le site j'ai trouve ce lien

Citation:

---

http://lfe.developpez.com/BCBmySQL/

---

et pour recuperer les noms de champs

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

7. Lister la structure d'une table MYSQL *mySQL; MYSQL_RES *myRES; MYSQL_FIELD *myFields; AnsiString aStr; int CountFields; if (!mysql_query(mySQL, "select * from testbcb")) { myRES = mysql_store_result(mySQL); if (myRES) { CountFields = mysql_num_fields(myRES); myFields = mysql_fetch_fields(myRES); for(unsigned int i = 0; i < CountFields; i++) { aStr.sprintf("Field %s is type %d",myFields[i].name, myFields[i].type); ListBox1->Items->Add(aStr); } } mysql_free_result(myRES); } La fonction mysql_fetch_fields remplit une structure de type MYSQL_FIELDS qui contient alors un tableau contenant toutes les informations des champs renvoyés par la requête.

---

je crois qu'il te manque le type du champ

Citation:

---

myFields[i].name, myFields[i].type);

---

Non, je ne pense pas. myFields[i].type sert bien à récupérer le type du champ, mais je n'en ai pas l'utilité.
J'ai déjà fait ce tuto, et justement cette ligne plantait elle aussi lors de la récupération des noms. Pour le listage des types aucun soucis ;)

Bon, ben ... question conne: rien de spéciale dans le nom des champs ?
Caractéres "spéciaux", taille du nom, etc ...

Je dis ca, mais je sais que si c'était le cas ca aurait du planter dès la création de la table, m'enfin, vaut mieux "checker" éventuellement, au cas ou.

@+
;)

Nop, rien, tout les champs sont écrits en minuscule avec des "_" au lieu des espaces, sans caractères spéciaux.
Je viens de refaire un test, c'est ce foutu myFIELDS[i].name qui pose soucis, c'est certains. Pour i=0, ça fonctionne, et puis dès i=1 le programme plante alors que j'ai pourtant plus d'un champs dans mes tables...

En fait, dès i=1, il me met que myFIELDS[i].name = NULL, ce qui est complètement faux ! J'ai beau chercher, je trouve pas l'erreur... :?

Comme Sunchaser
Une idee conne a tu essaye de demarrer ta boucle avec une valeur superieure a 0, 2..3, pour voir si ce n'ai pas du au contenu du champ

Toujours pareil... Apparemment ça n'est pas le contenu.

Question con :

Quel serveur sql utilises tu ?
As tu des droits suffisemment élevés pour cet utilisateur ?

Donc serveur MySQL (en fait, j'utilise WAMP), et j'accède à la base de donnée en "root" (ce qui ne devrait pas poser de soucis au niveau des droits).

effectivement en root çà devrait pas poser de souci


tu peux pas uploader ton projet ainsi qu un csv contenant ta ou tes bases ?

bon moi je disais çà , c etait juste pour aider , l api sql fonctionne bien chez moi ...

Désolé, j'up mon projet demain.
En fait, en y repensant, il est fort possible que l'erreur vienne du .h de l'api. Mon prof nous a filé une version qu'il a (soit-disant) modifié, et le connaissant (pour ne pas dire que c'est un vrai bon à rien, faut le voir pour comprendre) il a du encore faire n'importe quoi...

Demain, je re-essai tout ça avec l'api fournie avec la dernière version de mysql.

Voilà, j'ai up mon projet ici.
Ne vous inquiétez pas si ce n'est pas clair (manque de commentaires), c'est un TP...

Merci ;)

j ai maté ton projet

Je liste le nom des tables sans souci
Ensuite en double cliquant sur la listbox1 , la boucle coince , j obtiens bien mon 1er nom mais pas les suivants

J ai testé avec le caption d un label si il comptait bien le nombre de champs , pas de souci .

L erreur se fait dans une violation d acces memoire au niveau de la dll cc3260mt.dll

je vais perseverer

Merci !
Je vais continuer à chercher aussi de mon côté, j'ai toute la journée pour ça :lol:

Bon j ai reussi

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

void __fastcall TForm1::ListBox1DblClick(TObject *Sender) { if(ListBox1->Focused()) { //MYSQL_RES *myRES; AnsiString marequete = "SHOW COLUMNS FROM " ; marequete+=ListBox1->Items->Strings[ListBox1->ItemIndex]; if (!mysql_query(mySQL, marequete.c_str() ) )   { myRES = mysql_store_result(mySQL); if (myRES) { Label2->Caption= myRES->row_count; AnsiString aStr;   for(unsigned int i = 0 ; i < myRES->row_count ; i++) { myROW = mysql_fetch_row(myRES); for(unsigned int j = 0; j < 1; j++) { aStr.sprintf("%s", myROW[j]); ListBox2->Items->Add(aStr); } }   }}} }

---

mon cheminement ?
Champ en anglais = field
recherche de field dans mysql.h
je trouve une fonction interessante : mysql_list_fields
je googlelise

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

<?php $result = mysql_query("SHOW COLUMNS FROM sometable"); if (!$result) { echo 'Impossible d\'exécuter la requête : ' . mysql_error(); exit; } if (mysql_num_rows($result) > 0) { while ($row = mysql_fetch_assoc($result)) { print_r($row); } } ?>

---

cette fonction est depreciée , bon pas grave on essaye quand meme
Resultat , toujours la même erreur
Donc on va essayer leur " show collumns from "
Quelques boucles et puis voilà .

Merci beaucoup !
Je vais intégrer ça tout de suite ! :D
Je commençais à rager, il ne me manquait plus que la solution à ce problème et j'avais fini le programme.

Merci encore de votre aide !
Je mettrais le sujet en résolu une fois que j'aurais convenablement intégrer le bout de code ;)

Impeccable !
Je passe donc le sujet en résolu, et encore merci ! ;)