warning: deprecated conversion from string constant to ‘char*’Résolu

Question

Bonjour à tous,

j'ai réalisé un programme gcc qui se compile parfaitement, or j'ai des "warning" que je souhaiterais supprimer.

celui qui m'énerve mad le plus (façon de parler big_smile) c'est :

15 : warning: deprecated conversion from string constant to &#8216;char*&#8217;

ma ligne de code est :

 
Code:

char *test[4] = { "A1", "A2", "A3", NULL };



j'ai compris (à priori) que ma fonction demande un pointeur vers un char non constant,
j'ai donc essayé de faire :

 
Code:
[b]
const char *test[4] = { "A1", "A2", "A3", NULL };/b



et là mon programme ne se compile plus yikes


Quelqu'un peut m'aider à enlever le warning svp !!!

Merci d'avance

cptpingu · Answer

En preneur le code suivant: (0 au lieu de NULL pour éviter un include):


int main(void)
{
  const char *test[4] = { "A1", "A2", "A3", 0 };

  return 0;
}


Ligne de compilation (Je ne sais pas si c'est du C ou du C++)

C:
gcc -W -Wall -ansi -pedantic test.c
Tout fonctionne (à part un warning normal parce que je n'utilise pas la variable "test")

C++:
g++ -W -Wall -Wabi -pedantic test.cc
Tout fonctionne (à part un warning normal parce que je n'utilise pas la variable "test")

Quel est la version de ton gcc ? Peux-tu tester la même chose que moi ?

flateur18 · Answer

Merci pour ta réponse

j'ai le gcc4.3

et mon fichier est .cpp je ne peux malheureusement pas le changer en .c car on travail en groupe.

j'ai essayé ta manip, mais j'ai toujours le warning et maintenant avec une erreur qui le laisse pas compiler

cptpingu · Answer

j'ai le gcc4.3 
De même, j'ai gcc4.3.3

et mon fichier est .cpp
L'extension du fichier n'a pas d'importance. J'en déduis que tu fais du C++.

j'ai essayé ta manip, mais j'ai toujours le warning et maintenant avec une erreur qui le laisse pas compiler 
Qu'as tu essayé, exactement ?
Peux tu mettres le code ci-dessous dans un fichier:

#include 

int main(void)
{
  const char *test[4] = { "A1", "A2", "A3", 0 };

  for (int i = 0; i < 4; ++i)
    std::cout << test[i] << std::endl;

  return 0;
}


Et le compiler comme ceci:

g++ -W -Wall -Wabi -ansi -pedantic TONFICHIER 

Dis moi si ce simple fichier est correctement compilé.

flateur18 · Answer

oui ce simple fichier est compilé.

ce que j'ai essayé :

#include 

const char *test[4] =
  { "xxxxxxxx", "yyyyyyyyyyyyy", "zzzzzzzzzz", 0 };
for (int i = 0; i < 4; ++i)
    std::cout << test[i] << std::endl;

J'explique plus profondément : ce qui est dans test[] est censé être utilisé comme paramètres pour un ButtonBox que je crée par la suite.
j'écris:

const char *test[4] =
  { "xxxxxxxx", "yyyyyyyyyyyyy", "zzzzzzzzzz", 0 };

Mon_bouton =
    new puButtonBox (10, 90, 10, 15, test, TRUE);


test contient les labels à mettre.

Avec ta suggéstion j'ai fait :

const char *test[4] =
  { "xxxxxxxx", "yyyyyyyyyyyyy", "zzzzzzzzzz", 0 };

for (int i = 0; i < 4; ++i)
    std::cout << test[i] << std::endl;

Mon_bouton =
    new puButtonBox (10, 90, 10, 15, test, TRUE);

mais j'arrive pas à compiler comme ça.

Merci pour ton interrêt

cptpingu · Answer

new puButtonBox (10, 90, 10, 15, test, TRUE)
Je comprends déjà mieux le problème.
Peux-tu me faire voir le prototype du constructeur de "puButtonBox()" ?
Je suis à peu près sur que le 5ème arguments attends un "char*[]" et non un "const char *[]". Si c'est une classe à toi, ajoute le "const" dans le constructeur de ta classe, surtout si tu n'as pas vocation à modifier les éléments du tableaux.


là mon programme ne se compile plus yikes
Quelle est l'erreur ? (Copie pas pas tout, juste ce qui est intéressant).

flateur18 · Answer

Prototype du constructeur: puButtonBox my_box = new puButtonBox ( x1, x2, y1, y2, labels, <0|1> ) ; labels = "char*[]" en effet. je rappel que je veux juste enlever le warning : warning: deprecated conversion from string constant to ‘char*’ mon programme ne compile plus quand je fais : const char *test[4] = { "A1", "A2", "A3", NULL }; l'erreur est : error: no matching function for call to ‘puButtonBox::puButtonBox(int, int, int, int, const char* [4], int)’ /usr/include/plib/pu.h:1488: note: candidates are: puButtonBox::puButtonBox(int, int, int, int, char**, int) /usr/include/plib/pu.h:1478: note: puButtonBox::puButtonBox(const puButtonBox&)

flateur18 · Answer

Prototype du constructeur: puButtonBox my_box = new puButtonBox ( x1, x2, y1, y2, labels, <0|1> ) ; labels = "char*[]" en effet. je rappel que je veux juste enlever le warning : warning: deprecated conversion from string constant to ‘char*’ mon programme ne compile plus quand je fais : const char *test[4] = { "A1", "A2", "A3", NULL }; l'erreur est : error: no matching function for call to ‘puButtonBox::puButtonBox(int, int, int, int, const char* [4], int)’ /usr/include/plib/pu.h:1488: note: candidates are: puButtonBox::puButtonBox(int, int, int, int, char**, int) /usr/include/plib/pu.h:1478: note: puButtonBox::puButtonBox(const puButtonBox&)

cptpingu · Answer

puButtonBox my_box = new puButtonBox ( x1, x2, y1, y2, labels, <0|1> ) ; Mais c'est pas ça un prototype ! C'est la déclaration si tu préfères. error: no matching function for call to ‘puButtonBox::puButtonBox(int, int, int, int, const char* [4], int)’ /usr/include/plib/pu.h:1488: note: candidates are: puButtonBox::puButtonBox(int, int, int, int, char**, int) /usr/include/plib/pu.h:1478: note: puButtonBox::puButtonBox(const puButtonBox&) D'après l'erreur: j'en déduis que ton prototype est: puButtonBox(int, int, int, int, char**, int) Or tu ne lui passes pas un "char**", mais un "const char* [4]". D'où l'erreur. Pour corriger cela, change le prototype par: puButtonBox(int, int, int, int, const char**, int) gcc te fait un warning parce que tu fais quelque chose de dangereux: tu passes des chaînes en dur, à une fonction/méthode qui à le droit de modifier celle-ci.

flateur18 · Answer

J'ai trouvé !!! il faut utiliser le tableau de "const char *". Tel quel ça provoque une erreur à la compilation (conversion de "const char **" à "char **" impossible). Pour résoudre ceci, j'ai utilisé un "const_cast(test)" à la place de test dans l'appel de fonction. const char *test[4] = { "A1", "A2", "A3", NULL }; Mon_bouton = new puButtonBox (10, 90, 10, 15, const_cast(test), TRUE); et là plus de warning et je compile. Merci à tous

cptpingu · Answer

Pour résoudre ceci, j'ai utilisé un "const_cast<char**>(test)" à la place de test dans l'appel de fonction. 

Ah mais non !!! Surtout pas !
Je t'explique: g++ te fait un warning pour te prévenir d'une action dangereuse de ta part (passage d'un const à une méthode qui veut du non const, et donc qui pourrait modifier une zone read-only).
Tu n'as pas résolu le problème. Tu as forcé le compilateur à ne pas t'afficher le warning. Le problème est toujours présent.
Dans le constructeur de puButtonBox, tu peux potentiellement modifier "test", ce qui ne devrait pas être autorisé. La solution est plus simple que cela, je te l'ai déjà donné: Mettre un const dans le prototype du constructeur "puButtonBox".

D'une manière générale, si tu te retrouves à utiliser const_cast, pose toi de sérieuses questions sur la qualité de ton code.

flateur18 · Answer

Non, c_str() est censé retourner un "const char*".  Donc on tourne en rond. Et de plus ce n'est pas "char *" qu'attend le constructeur, mais un "char **".

Je reconnais volontier que le const_cast n'est pas ce qui ce fait de plus beau.

Donc il reste la solution de copier les chaînes "const" dans des éléments alloués dynamiquement et non-const. (un travail d'ouvrier car j'ai plus de 500 ligne à modifier - et c'est ce que j'essayais d'éviter depuis le début )

cptpingu · Answer

Non, c_str() est censé retourner un "const char*". Donc on tourne en rond. Et de plus ce n'est pas "char *" qu'attend le constructeur, mais un "char **".

Quelle rapport avec c_str() et un tableau de chaines de caractères (cf variable test) ?

Je reconnais volontier que le const_cast n'est pas ce qui ce fait de plus beau. 
Mieux vaut laisser le warning dans ce cas, au moins l'erreur n'est pas masquée (parce que ça revient à ignorer l'erreur).

Pourquoi ne pas changer le constructeur de puButtonBox ?
Je peux voir un peu plus de code, en rapport avec cela ? (juste quelques extraits).

flateur18 · Answer

voici une partie de mon code: const char *test1[4] = { "Kalman", "Runge Kutta", "Euler", 0 }; const char *test2[3] = { "Capteurs US", "GPS", 0 }; GGP1 = new puButtonBox (10, 90, 10, 15, const_cast (test1), TRUE); GGP2 = new puButtonBox (10, 70, 10, 15, const_cast (test2), TRUE); comment pourrai-je faire la méthode c_str()? car je ne sais pas du tout comment m'y prendre (j'ai pas l'habitude) merci encore

cptpingu · Answer

Je réitère ma question: Pourquoi ne pas changer le constructeur de puButtonBox ? 
Peux-tu me faire voir le code du constructeur de puButtonBox ?

Warning: deprecated conversion from string constant to ‘char*’

14 réponses

Votre réponse

Discussions similaires