Discussion :

[panthere noire]

Hello

j' essaye de me faire une fonction qui permet dans le main de mettre du std::string qui est le plus courant en c++ .

je reprend du code C qui lui travaille en char

donc je me disai que de passer une conversion const std::string var en const char * var devrai passer avec l'ajout de .c_str() qui permet de convertir en char , seulment il manque le pointeur.
donc je fait comment ?

voila un bout de code (non compilable)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
using namespace std;
 
 
void MD5File(const char*, char* md5);
 
int main()
{
	const std::string sources; 
	std::string md ;
	sources="/home/user/nohup.out";
	MD5File(sources.c_str(),md.c_str());
	std::cout << "MD5" << md << std::endl;
return 0;
}
void MD5File(const char* filename, char* md5)
{
    cFileMD5 filemd5;
    filemd5.md5File(filename, md5);
    //return md5;
}

l'erreur est claire : main.cpp:: error: invalid conversion from ‘const char*’ to ‘char*’

simple a première vue mai je vois pas comment faire la conversion ?? j'ai rien vu dans la faq qui concerne la conversion avec une const char *

Merci d'avance

[AF_2.8]

la fonction c_str() de la class String te renvoie un pointeur sur un const char

Code :

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

const char* c_str ( ) const;

Ton problème est que ta fonction MD5File veut un char * en deuxième argument (ce qui sous-entend qu tu peux modifier ce pointeur) or tu lui passes un const char * qui lui ne peut en aucun être modifé. Le compilateur ne pouvant assurer que le pointeur const que tu lui passes ne sera pas modifé il sort en erreur.
Utilise un

Code :

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

reinterpret_cast<char *>

ou mieux un

Code :

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

const_cast<char *>

sur ton deuxième argument.

[screetch]

ce code ne marchera pas, tu ne sais pas vraiment ce sur quoi pointe md.c_str()
en particulier, tu ne sais pas si il a assez d'espace pour stocker le MD5 et d'ailleurs, en l'occurrence c'est non.
De plus, si le pointeur renvoyé par c_str() pointe sur des données constantes, il y a bien une raison. Utiliser un reinterpret_cast ou un const_cast pourrait tout simplement mettre en l'air la chaîne de caracteres, un const_cast ou reinterpret_cast est souvent deconseillé (surtout le const_cast qui a pratiquement aucun cas d'utilisation valable)

Bref, a la place de ca, utilise un tableau temporaire de caracteres:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
void MD5File(const char*, char* md5);
 
int main()
{
	const std::string sources="/home/user/nohup.out";
        char md[33]; // un MD5 est sur 32 char? ou 16? chais plus
	MD5File(sources.c_str(),md.c_str());
	std::cout << "MD5" << md << std::endl;
return 0;
}
void MD5File(const char* filename, char* md5)
{
    cFileMD5 filemd5;
    filemd5.md5File(filename, md5);
    //return md5;
}

[AF_2.8]

Code :

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

1
2
3
 
        char md[33]; // un MD5 est sur 32 char? ou 16? chais plus
	MD5File(sources.c_str(),md.c_str());

Ben vu que la fonction c_str() appartient à la classe string, elle ne risque pas non plus de beaucoup fonctionner sur un char.

[jblecanard]

Ben vu que la fonction c_str() appartient à la classe string, elle ne risque pas non plus de beaucoup fonctionner sur un char

Vrai mais c'est juste une faute d'inattention équivalente à une faute de frappe, et au moins c'est détecté à la compilation. Alors que la mauvaise manipulation des cast peux provoquer des problèmes bien plus vicieux.

[AF_2.8]

const char* c_str ( ) const;

Get C string equivalent
Generates a null-terminated sequence of characters (c-string) with the same content as the string object and returns it as a pointer to an array of characters.

A terminating null character is automatically appended.

The returned array points to an internal location with the required storage space for this sequence of characters plus its terminating null-character, but the values in this array should not be modified in the program and are only granted to remain unchanged until the next call to a non-constant member function of the string object.

Parameters
none

Return Value
Pointer to an internal array containing the c-string equivalent to the string content.

Tout le problème réside sur ce que fait la fonction MD5 si elle ne modifie pas la variable le plus simple et de modifier le prototype avec deux const char *.
Sinon il faut dupliquer le const char *

Code :

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

1
2
3
4
5
 
  char * cstr;
  string str ("Ma jolie phrase");
  cstr = new char [str.size()+1];
  strcpy (cstr, str.c_str());

[dragonjoker59]

La meilleure solution reste pour moi celle proposée par screetch (un MD5 est bien sur 32 caractères). Celle-ci a l'avantage de ne pas avoir à réecrire la fonction MD5File (si jamais tu n'y as pas accès).
@andnicam : La fonction MD5File modifie certainement le second paramètre pour y stocker le MD5 du fichier dont l'adresse est passée en paramètre. Et je ne comprends de ce fait pas l'intérêt de la duplication de "ma jolie phrase".

[panthere noire]

Merci pour vos réponse


bon alors oui je récupère le md5 dans la variable md a 32 caractère .

j'ai donc fait :

Code :

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

1
2
3
4
5
 
	const std::string sources="/home/user/nohup.out";
	char md[33]; 
	MD5File(sources.c_str(),md);
	std::cout << " MD5 : " << md << std::endl;

si je comprend bien il faut éviter un cast. je vais dédier une fonction histoire de voir ce que sa donne.