Discussion :

[KindPlayer]

Bonjour

J'aimerais avoir quelques précisions sur la classe std::string de C++. En fait j'utilise une macro qui retourne une string. Pour récupérer la string dans une variable j'ecris std::string *str = new string(MACRO(parametres)) en utilisant le constructeur par copie. Quand j'ecris simplement std::string = MACRO(parametres) j'ai une violation mémoire (fichiers dbgheap.c et dbgdel.c pour ceux qui ont deja vu ca). Malheureusement je sais pas comment est ecrite la fonction MACRO. Si vous avez une idée de pourquoi la deuxieme solution marche pas?
Autre chose j'aimerais pouvoir reattribuer ma variable str. J'ai tenté
str = &(MACRO(nouveaux parametres)) mais ca plante aussi (mais pas tout le temps :s). Je suis assez nul en C++ comme vous vous en doutez en lisant ces lignes alors ne me moquez pas trop vite, merci

[loicounet]

un petit mot avant d'aller en pause
pose des bouts de code avec les balises...c'est plus facile a lire car ton post est assez condensé et ca donne pas envie de lire

[nikko34]

MACRO te renvoie une std::string ou un char*?

[KindPlayer]

Bon je reformate un peu mon message. Donc je souhaite récupérer une instance de std::string via une marco qui retourne une string (et pas un char* pour repondre au message précédent). J'écris:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 std::string *str = new string(MACRO(parametres))

en utilisant le constructeur par copie. Ca ca marche. En revanche, si j'écris

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::string = MACRO(parametres)

ca plante, une alloc qui a pas été faite je présume mais en principe l'alloc a lieu dans la fonction MACRO. Peut-etre dois je déclarer ma variable comme étant une référence sur le retour de la fonction?
J'aimerais également pouvoir reattribuer ma variable str. J'ai testé

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

str = &(MACRO(nouveaux parametres))

mais ca plante aussi (mais pas tout le temps :s). Voila si vous avez qq idées et précisions à me fournir sur le fonctionnement précis de la classe String (je préfère la classe String de java ).

[nikko34]

ta MACRO elle renvoie un std::string, pas un std::string* ?

alors:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
std::string str = MACRO( param );

ça ça ne marche pas?

les string C++ ne sont pas compliqués. Ce qui est compliqué c'est que tu utilises une MACRO qui a l'air bien pourrie et que tu te mélanges les pinceaux avec les pointeurs ( alors qu'en C++ ils ne sont pas fait pour être utilisés tout le temps ).

la classe string s'utilise très bien comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
std::string str = "toto";
std::string str2 = str;
std::string str3 = str + str2;
 
std::cout << str3 << std::endl;

[KindPlayer]

quand j'ai ecrit

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::string = MACRO(parametres)

il fallait bien sur lire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::string str = MACRO(parametres)

c'est ce que j'avais d'abord écris dans mon code mais comme ca plantait je me suis résolu a faire une allocation dynamique qui elle a fonctionné. Ouai la macro doit etre bien pourrie et elle retourne bien un string pas un string* (d'ou l'emploi du constructeur par copie pour l'alloc dynamique). Mais jcomprends toujours pas pourquoi la première déclaration sans passer par un pointeur marche pas. Et pour réattribuer ma string je suis obligé d'écrire à chaque fois

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

str = new string(MACRO(nouveaux parametres))

ce qui est bien laid je trouve. Je précise que tt ça s'exécute au sein d'une JNI ca joue peut-être un rôle.. Enfin pour l'instant je comprends pas bien.

[nikko34]

t'as aucun moyen d'avoir le code de ta MACRO?

[KindPlayer]

elle est dans une dll. Et c'est pas le premier truc chelou que j'ai avec cette dll d'ailleurs. J'my connais pas très bien en programmation mais plein de trucs ont l'air bizarrement écrit dedans. Bon ben jcrois que jvais rester avec ma solution pourrie à base de pointeurs. Comme quoi du code sale engendre du code sale...

[KindPlayer]

pour tes remarques

[Médinoc]

Une macro ne peut pas être dans une DLL, puisqu'elle est résolue par le préprocesseur et non le compilateur ni l'éditeur de liens.
Tu devrais donc trouver la macro en question dans le fichier d'en-tête associé à la DLL...

[nikko34]

je pense que tu dois avoir moyen de retrouver ce code, ton compilateur y arrive.
( demande lui? )

[r0d]

Difficile à deviner ce qu'il se passe exactement sans avoir le code de la macro.
Mais je suppute l'existence de structure conditionnelle (if) dans cette macro, ce qui peut entrainer des comportements indéfinis.

Un exemple de ce qui peut arriver:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
struct Param{
Param(/*parametres constructeur*/){/* constructeur */};
bool constructStrings(){ /* fonction qui va construire les strings str1 et str2 */ };
string str1, str2;};
 
#define MACRO ( Param params ) (  ( params.ConstructStrings() ) ? params.str1 : params.str2 ; )
 
// plus tard dans le code:
string str = MACRO(  Param(/*parametres du constructeur*/)  );
 
// regardons ce qu'il se passe: le #define remplace le code, on aura donc:
string str = ( Param(/*parametres du constructeur*/).ConstructStrings() ) ? Param(/*parametres du constructeur*/).str1 : Param(/*parametres du constructeur*/).str2;

Dans cet exemple, ça ne plantera pas à l'execution, mais on n'obtiendra pas le résultat escompté.

Mais je le répète, sans connaitre le contenu de la MACRO, il me parait impossible de comprendre ce qu'il se passe.

[KindPlayer]

Dans la librairie je dispose d'une autre macro pour récupérer une string: AsString comme son nom l'indique. Cette fois la déclaration statique fonctionne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::string str = AsString(parametre); //OK

Mais quand je veux vérifier ce que contient str:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

cout << "affichage de str: " << str->c_str() << endl;

n'affiche que "affichage de str: " :s, alors que je sais tres bien ce que str est censé contenir (pas une chaine vide comme ca semble etre le cas). D'ailleurs avec l'autre macro et l'alloc dynamique jaffiche bien qq chose...
Je crois que jvais écrire un mail au programmeur de la librairie pour récupérer le code. Jcomprends riennnnnnnn

[r0d]

Réfléchis un peu aussi avant de poser des questions... tu déclare un objet et tu l'utilises comme un pointeur

[KindPlayer]

j'ai bien ecris [CODE]cout << "affichage: "<< str2.data() << endl;/CODE] dans mon code. J'aurais eu un problème à la compilation sinon. Je suis nul mais pas à ce point la quand même. Je confirme ce que j'ai dit avant donc..

[KindPlayer]

je n'avais pas vu ta remarque. En fait aprés vérification c'est pas une macro au sens C/C++ mais au sens général du terme donc ce n'est en fait qu'une fonction dont j'ai uniquement le prototype dans le .h

[Emmanuel Deloget]

Bon, je sais, il semble qu'on se répète, mais peut-on voir le code de MACRO stp ? Sans cette information, il est impossible de savoir ce qui se passe.

Pour ma part, je penche pour un problème de modèle de mémoire. Le standard C++ ne définit pas d'ABI (abstract binary interface). De fait, les compilateurs sont libres de gérer la mémoire comme ils le souhaite dans les différents modules. Si une DLL utilise un modèle de mémoire différent du programme appelant, alors il est impossible de garantir le fonctionnement d'un programme qui va détruire ce qui aura été créé par la DLL.

Ce qui pose problème pour les templates, car dans la plupart des cas le code du template est directement inséré là ou il est utilisé (il est "inliné"). De fait, si ta MACRO est (contrairement à son nom) une fonction qui crée une std::string, il ne faut pas que cette chaine soit détruite dans le programme appelant, car le destructeur sera directement inséré dans le code de l'appelant (et non pas dans la DLL). Si il y a ne serait-ce qu'une petite différence dans le modèle de mémoire, la pile ou le tas risquent fortement d'être corrompu.

Cependant, tout ceci ne reste que pure spéculation: tant que nous ne savons pas exactement ce que fait MACRO (c'est à dire tant que nous ne voyons pas son code), il nous est impossible de trouver une solution à ton problème.

Donc cherche dans les .h et livre nous les informations nécessaires - tu verras, nous allons être un brin plus efficace après ça

[Mat007]

Salut,

Il ne faut pas passer de std::string en entrée ou en sortie d'une DLL, c'est au mieux très dangereux et au pire ça fait n'importe quoi (un peu comme là quoi )...
Cf. C++, the real world, and link and binary compatibility.

MAT.

[KindPlayer]

qui commencent à m'éclairer un peu. Si je peux pas passer std::string en sortie d'une fonction d'une DLL comment puis-je récupérer l'objet renvoyé par cette fonction? Jvais voir si jtrouve une réponse dans le lien que tu as donné. Le code de la fonction je l'ai pas pour l'instant malheureusement

[Mat007]

Le code de la fonction je l'ai pas pour l'instant malheureusement

Tu dois avoir au moins la signature de cette fonction, dans un des .h fourni avec la DLL, elle ressemble à quoi ?

MAT.