Discussion :

[bforzeus]

Bonjour a tous,

je viens vers vous après plusieurs recherche sur le net mais sans sucées.

voila j'essaye d’exécuter un programme avec code blocks, et je me retrouve avec plusieurs message d'erreur dans mon main du type

Code cmd :

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

attention : deprecated conversion from string constant to 'char*' [-Wwrite-strings]

les lignes qui me font les erreurs contiennent tous le NEW, exemple ci-dessous :

Code c++ :

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

 sRHO = new Block("RO");

je déclare bien

Code c++ :

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

Block *sRHO;

et dans mon fichier Block.cpp je déclare comme suite:

Code c++ :

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

1
2
3
4
5
6
7
8
 
#include "Block.h"
Block::Block( char *name, Bank *b) : BlockPP(name, b)
{   
    B = 3;
    Text = ID;
    strcpy(address.name,name);
}

auriez vous une idée sur ce qui coince ?
est ce que cela est due au linkage et que j'ai mal configuré le code blocks,
ou c'est une erreur de syntaxe ...

merci de votre aide

[Neckara]

Bonjour,

Toutes les chaînes "aeddde" sont stockées dans le data segment, on ne doit donc pas pouvoir les modifier, il faut alors faire const char * c = "aeddde" ou void foo(const char * c);.

[bforzeus]

Merci Neckara pour ta réponse,

je n'ai pas bien compris ta réponse, que ce que tu veux dire avec "aeddde", et la ligne

Code c++ :

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

1
2
const char * c = "aeddde" 
 void foo(const char * c)

je doit la mettre a quelle niveau.

[Neckara]

Merci Neckara pour ta réponse,

je n'ai pas bien compris ta réponse, que ce que tu veux dire avec "aeddde", et la ligne

Code c++ :

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

1
2
const char * c = "aeddde" 
 void foo(const char * c)

je doit la mettre a quelle niveau.

A ta ligne sRHO = new Block("RO");, tu lui donne une chaîne constante, il faut donc que le constructeur de Block prenne en paramètre un const char * ou un const std::string &

[bforzeus]

j'ai modifier le constructeur de la façon suivante :

Code c++ :

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

1
2
3
4
5
6
7
#include "Block.h"
Block::Block( const char *name, Bank *b) : BlockPP(name, b)
{   
    B = 3;
    Text = ID;
    strcpy(address.name,name);
}

mais j'ai toujours les même erreurs.

Code cmd :

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

attention : deprecated conversion from string constant to 'char*' [-Wwrite-strings]

saurai tu pour quoi, n'y a til pas d'autre moins de contourné le probleme

[Neckara]

BlockPP(name, b) => as-tu pensé à modifier aussi le constructeur de BlockPP ?

[bforzeus]

Oui, j'ai modifier le blockpp.cpp comme suite :

Code c++ :

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

1
2
3
4
BlockPP::BlockPP(const char *n, bank *b) 
{  b = b;
    d = d;
}

et j'ai toujours la même erreur

[bforzeus]

tu sais pas s'il ya un autre moyen pour contourné cette problématique des

Code cmd :

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

attention : deprecated conversion from string constant to 'char*' [-Wwrite-strings]

[ternel]

Attention, l'affectation n'est pas l'initialisation.
Lors d'un construction d'une variable d'un type donné, il y a plusieurs phases:

• allocation de la mémoire: par le compilateur ou un new
• initialisation de la mémoire
• exécution du constructeur a proprement parler

L'allocation n'est pas du ressort de la classe, mais les autres étapes si.
Pour ce faire, on utilise les constructeurs:

Code :

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

1
2
3
4
5
6
class C {
private:
    int n;
public:
    C(int value) : n(value) { std::cout <<n<<std::endl;}
};

La partie entre les accolades est le corps du constructeur, appelé une fois la variable initialisée.
La partie juste après le ':', bien qu'optionnelle, est composée d'appels aux constructeurs des membres et classes parentes (ou des (grands-)parents virtuels).
Chaque membre non explicitement construit est construit via un constructeur par défaut.

[bforzeus]

j'ai bien déclaré mon constructeur, sauf que l'appelle a ce dernier avec une chaine de caractère constante qui pose probleme, et je comprend vraiment pas pourquoi

[dragonjoker59]

De quel type est ton address.nameJ'ai l'impression que tu essaies de faire quelque chose de pas joli joli avec cette ligne :

Code :

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

strcpy(address.name,name);

là ou un simple address.name = name, si tant est que address.name est bien un std::string

[bforzeus]

elle est bien de type std::string.
je pense pas que mon probleme viens de la,
parce que j'ai plusieurs erreur de ce type et dans les autre classe je fais des appelle simple.
le seul point en commun c'est que les fonction qui coince c'est les constructeur avec un char, et moi je fais appelle a eux avec des chaine de caractère dans mon main.
je ne voix vraiment pas ce qui dérange de faire appelle avec une chaine de caractère constante

[Bousk]

Bonsoir,

le problème est stipulé plus haut : les chaînes "en dur" dans le code sont des const char* définis particulièrement dans l'exécutable, le data_segment.
Si une méthode demande un char* et non un const char*, rien n'assure qu'elle ne modifie pas ce paramètre. Puisqu'il n'est pas constant.

[bforzeus]

merci a tous,

c’était bien ça le probleme, j'ai oublier de mettre le const char* dans le fichier .h, maintenant c'est nikel.
SOl:
il faut remplacer char* par const char* dans le .cpp et .h

ca fais plaisir de voir qu'il y a moins d'erreurs et de warning dans son programme