C++

Redefinition de namespace

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Redefinition de namespace

nathan_g

Bonjour,

Je rencontre un comportement curieux à la compilation de plusieurs fichiers.

J'ai une fichier Article.H :

Code :

class Article
{
public:
double prix;
double tva();
};

Un fichier Article.C :

Code :

#include "Article.H"
double Article::tva()
{
return (0.196*prix);
}

et un fichier main.C :

Code :

#include <iostream>
using namespace std;
#include "Article.H"
int main()
{
Article pomme;
pomme.prix = 5;
cout << pomme.tva() << endl;
}

Quand je compile séparément les deux fichiers Article.C et main.C puis que j'en réalise l'édition de liens, cela fonctionne parfaitement malgré l'apparition du code de Article.H à deux occasions dans l'executable final (Au début de Article.C et de main.C ).

Par contre, si je rajoute un namespace dans Article.H en l'écrivant ainsi :

Code :

class Article
{
public:
double prix;
double tva();
};
namespace Fruit
{
double poids;
}

Je trouve une erreur de compilation :

Code :

main.o(.bss+0x0): multiple definition of `Fruit::poids'
Article.o(.bss+0x0): first defined here
collect2: ld returned 1 exit status

Pourtant si la première version de Article avec une redéfiniton des déclarations de Article.H est acceptée à la compilation, la seconde version devrait fonctionner, non ?

Quelqu'un pourrait-il m'indiquer ou est le problème et la raison pour laquelle la première version est compilable et pas la seconde ?
Quelle serait la solution à adopter ?

D'avance merci de votre aide,

Publicité

tuxce

salut,
il faudrait ajouter:

Code :

#ifndef ARTICLE_H
#define ARTICLE_H
/* ton code de article.h
*/
#endif

je crois que la différence de comportement vient du fait que poids est alloué en mémoire, alors qu'avant, ce n'était que des définitions.

EDIT: trop vite sans faire attention, je le corrige quand meme.

Message édité par tuxce le 05-10-2007 à 17:55:18

nathan_g

La commande est :

Code :

#ifndef ARTICLE_H

et non :

Code :

#ifundef ARTICLE_H

J'ai déja indiqué ces lignes sans que cela change quelquechose. Je ne les avais pas précisées pour alléger ma question.
Le problème n'est pas résolu avec cette modification.

Est-ce que vous connaitriez une solution ?

Cordialement,

Taz

bisounours-codeur

définition != déclaration

tuxce

ce que tu peux faire, c'est déclarer les eventuelles fonctions/classes dans le .h, et les variables, dans le .cc (c'est du c++ )

Code :

class Article
{
public:
double prix;
double tva();
};
namespace Fruit{};

Code :

#include "article.h"
namespace Fruit
{
double poids=0;
};
double Article::tva()
{
return (0.196*prix);
}

Code :

#include <iostream>
#include "article.h"
using namespace std;
extern double Fruit::poids;
int main()
{
Article pomme;
pomme.prix = 5;
cout << pomme.tva() << endl;
cout << poids << endl;
}

mais par contre, je suis pas sur que ca soit indispensable de déclarer une variable global de cette facon!

nathan_g

Merci de ces réponses.
J'aimerais comprendre plus précisément quel était mon erreur.

Est-ce qu'il y avait une double déclaration de la variable poids du namespace et que cela posait problème ?
Pourtant, quand je déclare la variable prix de la classe Article, elle apparait deux fois à la compilation (dans Article.o et main.o), sans que cela pose problème à l'édition de liens.

Pourquoi ce comportement différent ?

tuxce

la variable prix n'est pas allouée, alors que poids si.
aucune instance de Article n'est déclaré dans le article.h.

nathan_g

Pour mieux comprendre, ce que tu dis, l'écriture

Code :

class Article
{
public:
double prix;
double tva();
};

ne correspondrait pas à une allocation de poids

alors que :

Code :

namespace Fruit
{
double poids;
}

correspondrait à allouer la variable poids ?

Est-ce bien cela que tu indiques ?

Pourquoi cette différence de comportement ?

tuxce

Je suis pas expert, mais je sais que:

Code :

#include <iostream>
class A
{
public:
int x=0;
};
int main ()
{
std::cout << A::x;
return 0;
}

provoque une erreur car il n'y a aucune instance de A, ce n'est que la définition de la classe.

alors que

Code :

#include <iostream>
namespace A
{
int x=0;
};
int main ()
{
std::cout << A::x;
return 0;
}

fonctionne car x est alloué en mémoire, le namespace ne fait que définir un espace pour les déclarations.

Message cité 1 fois

nathan_g

Bonjour,

Merci de ces différentes précisions.

J'ai essayé le code donné plus haut avec les trois fichiers mais cela ne semble pas accepter. D'une part, je ne parviens à définir le namespace en deux parties (d'une part dans Article.H, d'autre part dans Article.cpp. Ensuite, en déplaçant les lignes de code, je conserve un problème sur la non-déclaration ou des déclarations multiples de variables.

Message cité 1 fois

Publicité

Taz

bisounours-codeur

tuxce a écrit :

Je suis pas expert, mais je sais que:

Code :

#include <iostream>
class A
{
public:
int x=0;

Illégal.

J'aimerais qu'on arrête de traiter le même problème dans 36 sujets. Déclaration != définition. Quelque soit votre membre statique, variable globale dans un namespace ou pas, il vous FAUT:
- .h qui contient la déclaration
- .cpp qui contient la définition et initialisation

nathan_g

Je veux bien essayer de revenir à ce sujet mais, dans ce cas, peut-tu donner un exemple simple d'utilisation d'une variable dans un namespace (et non pas d'une méthode). J'ai l'impression qu'il me faut passer par la redefinition d'une classe dans mon namespace pour cela.

Taz

bisounours-codeur

redéfinition = illégal

a.h
namespace A { extern int x; }

a.cpp
namespace A { int x = 42; }

et voilà tu peux utiliser A::x partout.

Message édité par Taz le 08-10-2007 à 16:25:31

tuxce

sauf si a.h est inclus dans 2 .cpp différents.

nathan_g a écrit :

J'ai essayé le code donné plus haut avec les trois fichiers mais cela ne semble pas accepter. D'une part, je ne parviens à définir le namespace en deux parties (d'une part dans Article.H, d'autre part dans Article.cpp. Ensuite, en déplaçant les lignes de code, je conserve un problème sur la non-déclaration ou des déclarations multiples de variables.

tu peux poster la sortie?
chez moi, ca fonctionne (avec g++ -Wall: pas d'erreur)

nathan_g

Tuxce : merci de ce détail (sauf...) qui doit changer pas mal de chose.

En l'occurence, de manière simplifiée, mon code qui ne fonctionne pas est :

Fichier Article.H

Code :

#ifndef _ARTICLE_
#define _ARTICLE_
namespace Article
{
double poids;
};
#endif

Fichier Article.C:

Code :

#include "Article.H"
namespace Article
{
double poids = 0.3;
};

Fichier main.C

Code :

#include <iostream>
using namespace std;
#include "Article.H"
using namespace Article;
int main()
{
cout << Article::poids << endl;
return (1);
};

La compilation ne fonctionne pas et me sort le message :

Code :

Article.C:5: error: redefinition of `double Article::poids'
Article.H:6: error: `double Article::poids' previously declared here

Comment donner à tout cela une écriture correcte ?
J'ai du rater quelque chose mais je ne vois pas comment donner une écriture correcte à ce code.

tuxce

ca d'accord...
mais tu as essayé le code avec extern, je t'avais remis les 3 fichiers:
http://forum.hardware.fr/forum2.ph [...] 0#t1620020

Message édité par tuxce le 08-10-2007 à 16:01:33

Taz

bisounours-codeur

putain je suis fatigué moi... désolé

Message cité 1 fois

tuxce

Taz a écrit :

putain je suis fatigué moi... désolé

t'aurais pu rajouter à coté du "fatigué" comme quoi tu as édité le post d'avant en ajoutant le extern
en gros, ca revient à ce que j'ai posté avant!

Message édité par tuxce le 08-10-2007 à 16:43:42

nathan_g

Merci Tuxce de ton implication dans mon problème.

A moins que j'ai mal compris, j'ai modifié le précédent fichier main.C afin d'y inclure le " extern " ce qui donne le fichier main.C suivant , les deux autres n'étant pas modifiés :

Code :

#include <iostream>
using namespace std;
#include "Article.H"
using namespace Article;
extern double Article::poids;
int main()
{
cout << Article::poids << endl;
return (1);
};

Malheureusement, cela ne résouds en rien mon problème car le message d'erreur n'est pas modifié. Désolé !

tuxce

il faut modifier les autres aussi:
article.h:

Code :

namespace Fruit {};

article.c:

Code :

namespace Fruit
{
double poids=0;
};

Message édité par tuxce le 08-10-2007 à 18:23:11

nathan_g

Bon, sans doute que j'ai un peu de mal
Maintenant, le code est :

Article.H

Code :

namespace Article {};

Article.C :

Code :

#include "Article.H"
namespace Article
{
double poids = 0.3;
};

et main.C :

Code :

#include <iostream>
using namespace std;
#include "Article.H"
using namespace Article;
extern double Article::poids;
int main()
{
cout << Article::poids << endl;
return (1);
};

Malheureusement, ca ne compiletoujours pas. Le message est maintenant :

Code :

main.C:9: error: `Article::poids' should have been declared inside `Artcle'
main.C: In function `int main()':
main.C:13: error: `poids' is not a member of `Article'
/usr/lib/gcc/i586-mandrake-linux-gnu/3.4.1/../../../crt1.o(.text+0x18):In function `_start':
../sysdeps/i386/elf/start.S:98: undefined reference to `main'
collect2: ld returned 1 exit status

Qu'est ce que j'ai raté ?
Ou est-le problème dans cette nouvelle écriture ?

tuxce

faudra que tu m'expliques pourquoi tu n'as pas pris tel quel l'exemple que je t'ai donnée!!!

pour ton programme, il faut modifier:

Code :

cout << Article::poids << endl;

par

Code :

cout << poids << endl;

nathan_g

Bonsoir,

Merci de cette nouvelle réponse, mais ce que tu proposes ne change absolument rien.
Le message d'erreur reste identique !

Peut-tu me confirmer que, de ton coté, la compilation de ces trois fichiers est acceptée et donne un executable fonctionnant normalement ?

Taz

bisounours-codeur

Voir mon exemple corrigé

Code :

$ cat a.cc
#include "a.hh"
namespace A { int x = 42; }
$ cat a.hh
namespace A { extern int x; }
$ cat main.cc
#include "a.hh"
int main() { return A::x; }
$ g++ -Wall main.cc a.cc
$ ./a.out; echo $?
42

nathan_g

OK !!!!!!

Merci beaucoup Taz de cet exemple qui fonctionne parfaitement .

Il ne me reste plus qu'à analyser tout cela pour comprendre mes problèmes .

Merci également Tuxce du temps passé.

Je dormirais mieux ce soir.

Publicité

FORUM HardWare.fr

Programmation

C++

Redefinition de namespace

Sujets relatifs
Question à la con : parcourir un namespace...	the type or namespace 'Office' does not exists in 'Microsoft'
Problème de namespace	Template, namespace, autre ?
[RESOLU] VBS : Problème de redéfinition de variable	Problème de Namespace (EXSLT)
[C++] include et redefinition de type	Rédéfinition de sujet
redefinition incomprehensible :'(	Probleme classe heritant vector : redéfinition erase()
Plus de sujets relatifs à : Redefinition de namespace

Page générée en 0.076 secondes