Discussion :

[charliejo]

Vu tous les problèmes que je rencontre avec les AnsiString dans ma dll... je me penche sur les char *... mais là aussi j'ai peur de la gestion de l'allocation mémoire...
Donc, la question est :
La différence pour l'utilisation, l'allocation mémoire et surtout le risque d'écraser ma donnée en mémoire selon les 2 déclarations suivantes???

Code :

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

1
2
        char * varchar;
        varchar ="init";

Code :

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

1
2
3
        char * varchar2;
        varchar2 =  new char[5];
        strcpy(varchar2,"init");

Sachant que je vais ensuite passer des char * en valeur de retour de mes fonctions de ma dll ????


Comment également concaténer 2 variables de type char * pour copier dans une troisième?

Merci

[bakaneko]

Si je me rapelle mes cours de C, ta première méthode ne fonctionnera pas vue que tu déclares un pointeur et pas une chaine.
La deuxième devrait fonctionner.

Pour concaténer 2 chaines dans une troisième, tu as strcat.

[charliejo]

En C, je suis d'accord,cela ne marche pas ... mais en C++???
Et en C++ Builder, dans la déclaration du

Code :

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

varchar = "init";

ne fait il pas implicitement un new char[5] ????

[bakaneko]

En C, je suis d'accord,cela ne marche pas ... mais en C++???
Et en C++ Builder, dans la déclaration du

Code :

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

varchar = "init";

ne fait il pas implicitement un new char[5] ????

je ne crois pas...

[Laurent Gomila]

Code :

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

1
2
char * varchar;
        varchar ="init";

Ici, "init" est un tableau de caractère placé dans une zone mémoire en lecture seule. Son type correct est donc const char * ; utiliser un char * et le modifier ensuite mènerait à un comportement indéterminé.

Code :

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

1
2
3
char * varchar2;
        varchar2 =  new char[5];
        strcpy(varchar2,"init");

Ici tu alloues bien une chaîne de 4 caractères (+ 0 terminal) sur le free store, et tu y copies "init". C'est la manière correcte de procéder si tu veux que ta chaîne soit modifiable et persistante en mémoire. Par contre il ne faudra pas oublier de la désallouer le moment venu.

Et en C++ Builder, dans la déclaration du

Code :

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

varchar = "init";

ne fait il pas implicitement un new char[5] ????

Non. Peut-être confonds-tu avec ceci :

Code :

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

char var[] = "init";

Qui alloue une chaîne de taille correcte (mais pas sur le free store) et y places "init".

Par contre le mieux serait encore de régler ces problèmes d'Ansistring et de continuer à utiliser cette classe, les manipulation de char* sont rarement conseillées en C++.

Ou alors si tu ne fais que renvoyer des const char * (ie. pas de chaîne modifiable), ça peut être reglé très rapidement.

[charliejo]

Merci pour l'explication.
Pour les ansistring, à priori ce sont des problèmes d'emplacement mémoire (mémoire interne à la dll)
Autre solution : les string ...

J'ai donc dans ma dll

Code :

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

1
2
3
4
5
6
7
8
class testdll{
 testdll(){};
 int TestStr(string & refstr){
     string tmp = "test valeur";
     refstr = tmp;
     return 1;
 }
}

Et dans mon application appelant ma dll :

Code :

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

1
2
3
4
5
        testdll  * ptestdll;
        ptestdll = new  testdll();
        string strtmp = "init";
        int i = ptestdll->TestStr(strtmp);
        delete ptestdll;

A priori ça marche, mais je voudrais être sûr que ce code est bon et qu'il n'y a problème avec la mémoire ....
Quand je fais dans ma dll

Code :

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

     string tmp = "test valeur";

Dans quelle mémoire la valeur tmp est stocké?
Et ensuite

Code :

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

refstr = tmp;

Est ce que cet emplacement sera toujours accesible dans mon application appelante???

merci

[Marco85]

Salut !!!

Autre solution : les string ...

Il ne s'agit pas d'une "autre solution", il s'agit de LA solution C++ que tu DOIS utiliser ...

Marco85

[Fiquet]

Vu tous les problèmes que je rencontre avec les AnsiString dans ma dll...

Je ne suis pas un pro des DLL, mais j'ai cru comprendre que l'utilisation de classe de string posait des problèmes avec les DLL non ? Ou alors j'ai mal compris.

Par exemple dans ces fils :
http://www.developpez.net/forums/viewtopic.php?t=422407
http://www.developpez.net/forums/viewtopic.php?t=431709

[charliejo]

Ok, mais du coup ma fonction et mes appels sont corrects ???
Quand je fais :

Code :

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

refstr = tmp;

Il recopie la valeur de tmp dans refstr ???
Merci

[charliejo]

Petite précision j'utilise

Code :

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

std::string

En début de code j'ai

Code :

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

using namespace std;

[loufoque]

Utilise std::string.