Discussion :

[Freud44]

Bonjour,

dans mon soft, j'utilise une base de données mysql, aussi j'ai créé un système de dump et de restauration.

pour la restauration je passe par la commande :

Code :

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

system(const char * valeur);

Le hic, c'est que je récupère le chemin d'accès de mon fichier sql dans une variable de type String ou AnsiString.

donc ce n'est pas compatible avec ce qu'a besoin la commande "system" où il faut du Const char *.

Ma question est la suivante, comment dois je convertir mon String en Const char * ?

merci d'avance

[3DArchi]

[Freud44]

Merci !

J'ai complètement zappé de regarder dans la FAQ..... et surtout de faire sur BCB ^^

En tout cas voici le code pour ceux qui auraient été ou seront dans cet embarras:

Code :

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

1
2
3
4
5
6
 
chemin = "C:\\mysql\\bin\\mysql maBase< ";
chemin += valRoot;  //chemin d'accès du fichier sql vennant du dump
char * cmd = new char[chemin.Length()+1];
strcpy(cmd,chemin.c_str());
system(cmd);

[Médinoc]

N'aurait-il pas été plus simple de faire juste:

Code C++ :

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

1
2
3
chemin = "C:\\mysql\\bin\\mysql maBase< ";
chemin += valRoot;  //chemin d'accès du fichier sql vennant du dump
system(chemin.c_str());

?

[Freud44]

effectivement, j'ai préféré faire un code plus orienté "usine à gaz" afin, lors de mon passage en mode pas à pas, que je puisse vérifier que la conversion se passe bien, et qu'aucun caractère n'ai été altéré.

[3DArchi]

N'aurait-il pas été plus simple de faire juste:

Code C++ :

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

1
2
3
chemin = "C:\\mysql\\bin\\mysql maBase< ";
chemin += valRoot;  //chemin d'accès du fichier sql vennant du dump
system(chemin.c_str());

?

Si on n'en croit la FAQ, non, car le pointeur est temporaire:

Attention ! La méthode c_str retourne un pointeur sur une chaîne C, qui n'est valable que pour l'instruction suivant l'appel à c_str. En d'autres termes, vous ne pouvez pas faire ceci :

[Médinoc]

Les temporaires vivent au moins jusqu'au retour d'une fonction à laquelle ils sont transmis en argument, non?

En supposant que la fonction system() ne modifie pas elle-même la string, bien entendu.

[loufoque]

char * cmd = new char[chemin.Length()+1];
strcpy(cmd,chemin.c_str());

Tu peux faire un memcpy ici (voire même un memcpy aligné), ça ira plus vite.

[3DArchi]

Les temporaires vivent au moins jusqu'au retour d'une fonction à laquelle ils sont transmis en argument, non?

En supposant que la fonction system() ne modifie pas elle-même la string, bien entendu.

Pas tout à fait, la temporaire peut te donner accès à un buffer qui pourra être réutilisé plus tard. C'est pourquoi tu dois recopier la valeur tout de suite. J'avais eu des problèmes de ce genre avec ILog (j'avais pas choisi d'utiliser cette librairie, donc par avance éviter les trolls!): tu récupères une adresse et à l'appel suivant, l'adresse est encore valide mais les données ne le sont plus!

[coyotte507]

L'adresse fournie par .c_str() est l'adresse de la chaine de caractère dans l'objet std::string.

La seule chose que peut changer c_str(), c'est de rajouter le caractère nul '\0' à la fin. (pas rajouter, mais en fait changer le caractère après le dernier caractère lisible)

La string n'est pas modifiée pendant l'appel à system (ce n'est pas un programme multithread qui partage la variable chemin ??), donc il n'y a aucun risque à utiliser .c_str()...

Le pointeur a beau être temporaire, l'adresse sur laquelle il pointe ne l'est pas, et la valeur du pointeur (c'est à dire l'adresse) ne change pas, même s'il est temporaire...

[Médinoc]

Attention, il ne s'agit apparemment pas d'une std::string, mais comme le titre du thread l'indique, d'une AnsiString Borland.

[coyotte507]

Attention, il ne s'agit apparemment pas d'une std::string, mais comme le titre du thread l'indique, d'une AnsiString Borland.

Ah d'accord

Mais s'il y a un risque pour que la mémoire soit écrasée entre .c_str() et la fin de l'appel à system:

Code :

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

system(chemin.c_str());

Il y a aussi un risque pour qu'elle soit écrasée entre .c_str() et la fin de l'appel à strcpy non?

Code :

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

1
2
char * cmd = new char[chemin.Length()+1];
strcpy(cmd,chemin.c_str());

A moins que l'appel à system risque d'écraser l'emplacement mémoire et pas l'appel à strcpy

[coyotte507]

Si on n'en croit la FAQ, non, car le pointeur est temporaire:

En fait non, la FAQ dit que ce code risque de planter:

Code :

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

1
2
3
4
5
6
AnsiString Src = "Source.txt";
AnsiString Dest = "Destination.txt";
char * pSrc = Src.c_str();
char * pDest = Dest.c_str();
// instructions
CopyFile(pSrc,pDest,true);

A cause de //instructions. La solution avec le strcpy est proposée dans le cas où il y a des instructions entre l'appel à .c_str() et à la fonction concernée.

[3DArchi]

Je ne connais pas AnsiString, mais comme j'avais eu un pb similaire avec Ilog, j'ai vu la chose comme ça. D'ailleurs, si on reprend la FAQ:

Code :

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

1
2
3
4
5
AnsiString Src = "Source.txt";
AnsiString Dest = "Destination.txt";
char * pSrc = Src.c_str();
char * pDest = Dest.c_str();
CopyFile(pSrc,pDest,true);

Entre pSrc = Src.c_str() et CopyFile(pSrc,..., il y a pDest = Dest.c_str();. Donc à priori, on pourrait avoir la même adresse, la chaîne ayant changée entre pSrc.c_str et pDest.c_str, non? D'où la solution préconisée dans la FAQ de faire une copie de la chaîne obtenue avec c_str() tout de suite!