problème de Link

Bonjour

J'ai un fichier header.h dans lequel je définis un namespace:

Code:

---

1 2 3 4 5 6 7 8 9 10

#pragma once #include <vector> #include <stdlib.h> #include <stdio.h>   using namespace std;   namespace { ..... }

---

Ensuite, j'ai deux fichier cpp dans lequel j'ai les deux déclarations:

#include header.h

A la compilation, j'obtiens une erreur comme quoi des symboles ont été deja définis.
Bien sur, je comprends d'où ca vient, mais je n'arrive pas à le corriger, meme avec des " include guards", comme pragma...

Je voudrais conserver la définition de Donnees dans header.h, de plus je veux que les deux fichiers utilisent tous les deux des éléments définis dans Donnees.

alors comment peut on faire pour éliminer ce pb de Link.
merci

Bonjour,

A quoi ressemble tes guards ?

j'avais mis des trucs du style:

#Ifndef H_HEADER
#define H_HEADER


#endif

Bizarre, comment lances tu ta compil, quel messages as-tu ? Quel compilateur utilises-tu ?

Salut,

Quelques remarques:

1. #pragma once n'est pas portable... Il est préférable de l'éviter ;)
2. <stdlib.h> et <stdio.h> sont des fichiers d'en-tête C, il vaut mieux les éviter (tu peux avantageusement les remplacer par la seule inclusion de <iostream> ;))
3. L'utilisation de la directive usign namespace est très largement déconseillée dans un fichier d'en-tête
4. Fais attention au fait qu'un espace de nom anonyme n'a une portée que très limitée
5. La définition des structures et des classes doit se faire dans un fichier d'en-tête, l'implémentation des fonctions et méthodes dans un fichier .cpp correspondant
6. évite les variable globales, et, si tu dois vraiment les utiliser, définis les dans un fichier d'implémentation et déclares les "extern" dans le fichier d'en-tête

Sinon, les symboles déjà définis ne sont pas des problème de "link" (d'édition de liens) mais bien des problèmes de compilation...

Il faudrait que tu nous fournisse au minimum le code complet du fichier header.h pour que nous soyons en mesure de situer exactement le problème ;)

voila l'erreur que je recois:

1>Édition des liens en cours...
1>LINK : warning LNK4076: fichier d'état incrémentiel non valide 'C:\Users\Pierre\Favorites\Documents\Visual Studio 2008\Projects\nouveauprojet\Debug\nouveauprojet.ilk'*; édition de liens non incrémentielle
1>Template.obj : error LNK2005: "long Donnees::nb_point" (?nb_point@Donnees@@3JA) déjà défini(e) dans Mesure Binomiale aleatoire.obj
1>Template.obj : error LNK2005: "double Donnees::Gamma_c" (?Gamma_c@Donnees@@3NA) déjà défini(e) dans Mesure Binomiale aleatoire.obj
1>Template.obj : error LNK2005: "double Donnees::H" (?H@Donnees@@3NA) déjà défini(e) dans Mesure Binomiale aleatoire.obj
1>Template.obj : error LNK2005: "double Donnees::Gamma_a" (?Gamma_a@Donnees@@3NA) déjà défini(e) dans Mesure Binomiale aleatoire.obj
1>Template.obj : error LNK2005: "double Donnees::Gamma_b" (?Gamma_b@Donnees@@3NA) déjà défini(e) dans Mesure Binomiale aleatoire.obj
1>Template.obj : error LNK2005: "double Donnees::Gamma_c_b" (?Gamma_c_b@Donnees@@3NA) déjà défini(e) dans Mesure Binomiale aleatoire.obj
1>Template.obj : error LNK2005: "class std::vector<double,class std::allocator<double> > Donnees::temps" (?temps@Donnees@@3V?$vector@NV?$allocator@N@std@@@std@@A) déjà défini(e) dans Mesure Binomiale aleatoire.obj
1>C:\Users\Pierre\Favorites\Documents\Visual Studio 2008\Projects\nouveauprojet\Debug\nouveauprojet.exe : fatal error LNK1169: un ou plusieurs symboles définis à différentes reprises ont été rencontrés

Voici les entetes de tous les fichiers:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

#ifndef HEADER_H #define HEADER_H #pragma once #include <vector> #include <stdlib.h> #include <stdio.h>   using namespace std;   #define Pi 3.141592653589793238462643383279502884197   namespace Donnees{ double H; long nb_point; const long N=10000; double Gamma_a; double Gamma_b; double Gamma_c; double Gamma_c_b; vector<double> temps; }   double Gamma(double x); #endif

---

Mesure binomiale:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#include <stdlib.h> #include <stdio.h> #include "header.h" #include <math.h> #include <iostream> #include <time.h> #include <string> #include <limits> #include <boost/tr1/random.hpp> #include <boost/math/special_functions/round.hpp> #include <vector>   using namespace std; using namespace boost::math; using boost::math::round;

---

fichier main:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#include <cstdlib> #include <boost/tr1/random.hpp> #include <cmath> #include <vector> #include <iostream> #include "header.h" #include <stdlib.h> #include <stdio.h> #include <boost/math/special_functions/round.hpp> #include <boost/random/lagged_fibonacci.hpp> #include <boost/math/special_functions/factorials.hpp> #include <fstream> #include <ctime> #include <boost/random/mersenne_twister.hpp> #include <boost/random/normal_distribution.hpp> #include <boost/random/variate_generator.hpp> #include <boost/random.hpp> using namespace boost; using namespace std;

---

j'utilise pas tous les header. certains ne devraient pas être la

Juste au cas où, je pense que tu devrais éviter les noms de fichiers comprenant des espaces (ou des accents d'ailleurs).

Et pour Pi, peut-être qu'un "const double Pi" serait plus prudent qu'un "#define Pi".

Quel est ce fichier Template ?

Citation:

---

# L'utilisation de la directive usign namespace est très largement déconseillée dans un fichier d'en-tête

---

Qu'est ce que l'utilisation directive?
L'ai-je fait?
Il vaut mieux utiliser std::cout que
1)using namespace std;
2)puis cout.

Citation:

---

# Fais attention au fait qu'un espace de nom anonyme n'a une portée que très limitée

---

C'est ce que j'ai fait??

Citation:

---

# évite les variable globales, et, si tu dois vraiment les utiliser, définis les dans un fichier d'implémentation et déclares les "extern" dans le fichier d'en-tête

---

Je sais, mais il ne me semble pas avoir utilisé des variables globales.

Citation:

---

Quel est ce fichier Template ?

---

C'est un fichier qui s'appelle template et qui n'a rien a voir avec les templates.
Il n'y a que des calculs dans ce fichier.


Mais sans vouloir être impoli, je n'ai pas eu de réponse à ma question. Je rame toujours dessus.

Citation:

---

Envoyé par deubelte

Qu'est ce que l'utilisation directive?

---

Ca veut dire 'using namespace std;' dans ton .h.

Citation:

---

Envoyé par deubelte

L'ai-je fait?

---

Oui

Citation:

---

Envoyé par deubelte

Il vaut mieux utiliser std::cout que
1)using namespace std;
2)puis cout.

---

Ce n'est pas ce qui a été dit. Il est déconseillé d'utiliser using namespace std; dans les .h, mais tu peux tout à fait les utiliser dans tes .cpp

Citation:

---

Envoyé par deubelte

Citation:

---

# évite les variable globales, et, si tu dois vraiment les utiliser, définis les dans un fichier d'implémentation et déclares les "extern" dans le fichier d'en-tête

---

Je sais, mais il ne me semble pas avoir utilisé des variables globales.

---

Et, bien comme Mr Jourdain, tu as fait des variables globales sans le savoir ...

Citation:

---

Envoyé par deubelte

C'est un fichier qui s'appelle template et qui n'a rien a voir avec les templates.
Il n'y a que des calculs dans ce fichier.

---

Cela ne risque pas de troubler d'autres lecteurs de ton code?

Citation:

---

Envoyé par deubelte

Mais sans vouloir être impoli, je n'ai pas eu de réponse à ma question. Je rame toujours dessus.

---

Pour répondre strictement à ta question :
header.h

Code:

---

1 2 3 4 5 6 7 8 9 10 11

namespace Donnees{ double H; long nb_point; const long N=10000; double Gamma_a; double Gamma_b; double Gamma_c; double Gamma_c_b; vector<double> temps; }

---

Ca, ce sont des définitions de variables globales. D'où ton problème de link, puisque ce fichier d'en-tête (header) va être inclu plusieurs fois.

Citation:

---

Envoyé par deubelte

Qu'est ce que l'utilisation directive?

---

Il ne s'agit pas d'un "utilisation directive", mais de l'utilisation d'une directive :D

using et namespace sont deux mots clés, qui, mis bout à bout représentent une directive (une instruction) indiquant au compilateur qu'il doit considérer l'espace de noms indiqué comme s'il faisait partie intégrante de l'espace de nom global, afin de permettre l'utilisation de cout plutôt que std::cout (par exemple)

Il faut éviter cette directive dans les fichiers d'en-tête pour la simple et bonne raison qu'elle se propage du fait du jeu des inclusions en cascade dans tous les fichiers qui incluent (même de manière indirecte) le fichier d'en-tête dans lequel elle est utilisée, ce qui fait perdre tout l'intérêt des espaces de nom ;)

Citation:

---

L'ai-je fait?

---

oui, ici:

Code:

---

1 2 3 4 5 6 7 8

#ifndef HEADER_H #define HEADER_H #pragma once #include <vector> #include <stdlib.h> #include <stdio.h>   using namespace std;

---

dane le deuxième code que tu donne et ici dans le code que tu donne dans la question (s'il s'agit du code d'un fichier d'en-tête)

Code:

---

1 2 3 4 5 6

#pragma once #include <vector> #include <stdlib.h> #include <stdio.h>   using namespace std;

---

Citation:

---

Il vaut mieux utiliser std::cout que
1)using namespace std;
2)puis cout.

---

Dans les fichier d'en-tête (*.h, *.hpp,...) oui, dans les fichier d'implémentation (*.cpp), c'est moins grave ;)

Citation:

---

C'est ce que j'ai fait??

---

Dans le code que tu donnes dans la question

Code:

---

1 2 3

namespace { ..... }

---

déclare un espace de noms anonyme ;)

Citation:

---

Je sais, mais il ne me semble pas avoir utilisé des variables globales.

---

il ne faut pas confondre espace de noms et structure ou classe...
toutes les variables déclarées dans l'espace de noms Donnees dans le code

Code:

---

1 2 3 4 5 6 7 8 9 10

namespace Donnees{ double H; long nb_point; const long N=10000; double Gamma_a; double Gamma_b; double Gamma_c; double Gamma_c_b; vector<double> temps; }

---

deviennent des variables globales, même si, du fait qu'elles appartiennent à l'espace de noms Données, il faut y accéder en préfixant leur nom de Donnees::

Une variable globale est une variable déclarée en dehors de toute structure ou de toute fonction...

Un espace de noms ne représente qu'un chemin permettant de lever l'ambigüité lorsque tu te trouve face à une situation dans laquelle deux types (ou deux fonction) différent(e)s devraient porter des noms identiques

Citation:

---

Mais sans vouloir être impoli, je n'ai pas eu de réponse à ma question. Je rame toujours dessus.

---

Maintenant que nous avons le fichier "header.h" complet, nous allons pouvoir te répondre ;)

Soit tu modifie le code de manière à ce que Donnees soit une structure (class ou struct), soit tu déclares les variables "extern" dans le fichier d'en-tête et tu les définis dans un fichier d'implémentation séparé.

Pour que cette réponse "courte" soit compréhensible, il faut peut être revenir sur deux termes génériques:
La déclaration de (le fait de déclarer) quelque chose consiste simplement à dire au compilateur que ce quelque chose existe.
La définition de (le fait de définir) quelque chose consiste à donner au compilateur le "corps" de ce quelque chose.

pour une fonction, nous avons donc

Code:

---

void foo(/*paramètres éventuels */);

---

qui est la déclaration de la fonction et

Code:

---

1 2 3 4

void foo(/*paramètres éventuels */) { /*ce qui doit être fait */ }

---

qui est la définition de la fonction.

Pour une classe ou une structure, nous avons

Code:

---

1 2 3

class MyClass; /* ou */ struc MyStruct;

---

qui est la déclaration de la classe ou de la structure et

Code:

---

1 2 3 4 5 6 7 8 9

class MyClass { /*contenu de MyClass */ }; /* ou */ struct MyStruct { /* contenu de MyStruct */ };

---

qui est la définition de la structure

En C++, il faut suivre le principe du "One Definition Rule", c'est à dire qu'une chose ne peut être définie qu'une et une seule fois dans le code.

Pour les fonctions, classes et structures, le simple ajout de "garde anti inclusion multiple" du genre de

Code:

---

1 2 3 4 5 6 7

#ifndef UN_TERME_UNIQUE #define UN_TERME_UNIQUE /* définition des classes et structures,  * déclaration des fonctions  * définition de fonctions inline  */ #endif

---

permet de s'assurer que nous suivrons cette règle de la définition unique.
La raison est simple: nous donnons un corps au type, mais nous ne demandons pas encore de réserver de l'espace mémoire pour pouvoir l'utiliser

Par contre, pour les variables globales, il en va autrement :aie:

Le problème, c'est que lorsque tu déclare une variable (qui n'est pas un pointeur ou une référence), la déclaration est aussi... la définition: le fait de déclarer une variable implique de manière automatique la réservation d'un espace mémoire suffisant pour... contenir les informations transportées par cette variable

Comme la déclaration et la définition d'une variable ne font qu'un, il faut:

1. s'assurer que la définition ne sera effectuée qu'une fois
2. demander au compilateur de simplement prendre en compte que la variable que nous déclarons (pour qu'il sache qu'elle existe) est définie "autre part", et qu'il pourra y accéder en temps utiles

Il faut donc définir, dans un fichier *.cpp la variable globale sous la forme de

Code:

---

1 2 3 4 5 6 7 8 9 10

namespace Donnees{ double H =valeurDeHInitiale; long nb_point = valeurDen_pointInitiale; const long N=10000; double Gamma_a = valeurDeGamma_aInitiale; double Gamma_b = valeurDeGamma_bInitiale; double Gamma_c = valeurDeGamma_cInitiale; double Gamma_c_b = valeurDeGamma_c_bInitiale; vector<double> temps; }

---

et préciser dans le fichier d'en-tête que la variable existe "autre part" en préfixant la déclaration du mot clé extern sous la forme de

Code:

---

1 2 3 4 5 6 7 8 9 10

namespace Donnees{ extern double H; extern long nb_point; extern const long N; extern double Gamma_a; extern double Gamma_b; extern double Gamma_c; extern double Gamma_c_b; extern vector<double> temps; }

---

NOTA: Les explications qui précèdent sont simplifiées de manière à te donner une idée du principe... Elles n'ont donc pas la prétention d'être complètes, ni celle d'utiliser les termes réellement appropriés ;) (mais ma réponse est déjà largement assez longue :P)

Bonjour !

Code:

---

1 2 3 4 5 6 7 8

#ifndef UN_TERME_UNIQUE #define UN_TERME_UNIQUE /* définition des classes et structures,  * déclaration des fonctions  * définition de fonctions inline  */ #endif

---

On aurait pu aussi ajouter quelquechose comme ceci, Dans le cas ou voudrait ajouter un compilation conditionnelle ( L'exemple de Koala1 alias l'homme qui murmurait aux oreils du C++ :) est aussi une compilation conditionnelle)

Code:

---

1 2 3 4 5 6 7 8

#ifndef UN_TERME_UNIQUE #define UN_TERME_UNIQUE /* définition des classes et structures,  * déclaration des fonctions  * définition de fonctions inline  */ #endif //UN_TERME_UNIQUE

---

Le comentaire va determiner se quel terme il s'agit, ensuite on termine par une ligne vide apres #endif //UN_TERME_UNIQUE sinon le compilateur va se pleindre "no new line at end of file". n'ajouter qu'une seul ligne, sinon ça ferait tache

Deja merci pour les reponses. Y a pas mal de trucs a étudier

Je sais qu'il faut pas utiliser les variables globales. Et je croyais justement que les namespace permettait de résoudre le problème des variables globales.

Citation:

---

mais si c'est pas bien d'utiliser les variables globales,

---

Oui, mais je m'appuie sur des namespace du style std, boost. Je pense que les codeurs qui ont fait ca avaient des raisons de le faire.

Disons que mon problème venait plutot d'un problème de compilation.
Je ne sais pas comment le compilateur compile, et dans quel ordre (est-ce que lepremier fichier est celui qui contient main?) il compile les fichiers.

mais supposons que j'ai un projet dans lequel j'ai un header "header.h" et deux fichiers cpp. tous les deux fichiers cpp contiennent
#include "header.h".

Donc s'il compile le premier cpp, il va compiler header.h.
Ensuite, il va compiler le deuxieme cpp. Il va donc recompiler header.h, sauf (il me semble) si on met des include guard.

Citation:

---

Le comentaire va determiner se quel terme il s'agit, ensuite on termine par une ligne vide apres #endif //UN_TERME_UNIQUE sinon le compilateur va se pleindre "no new line at end of file". n'ajouter qu'une seul ligne, sinon ça ferait tache

---

Les commentaires sont aussi lus par le compilateur?? :roll:

Citation:

---

Envoyé par buggen25

Bensuite on termine par une ligne vide apres #endif //UN_TERME_UNIQUE sinon le compilateur va se pleindre "no new line at end of file". n'ajouter qu'une seul ligne, sinon ça ferait tache

---

C'est vrai de gcc, mais pas de visual

Citation:

---

Envoyé par deubelte

Deja merci pour les reponses. Y a pas mal de trucs a étudier

Je sais qu'il faut pas utiliser les variables globales. Et je croyais justement que les namespace permettait de résoudre le problème des variables globales.

Oui, mais je m'appuie sur des namespace du style std, boost. Je pense que les codeurs qui ont fait ca avaient des raisons de le faire.

---

J'ai l'impression que tu mélanges différentes choses. Pour reprendre ce qu'expliquait Koala (alias "l'homme qui murmurait aux oreilles du C++ " puisque c'est son nouveau nom), namespace déclare un espace de nom, pas un espace de stockage. Boost et std utilise des espaces de nom car c'est effectivement une bonne pratique surtout si on souhaite produire des librairies à des millions de développeurs :mrgreen:

Citation:

---

Envoyé par 3DArchi

<snip>

Citation:

---

Envoyé par deubelte

Il vaut mieux utiliser std::cout que
1)using namespace std;
2)puis cout.

---

Ce n'est pas ce qui a été dit. Il est déconseillé d'utiliser using namespace std; dans les .h, mais tu peux tout à fait les utiliser dans tes .cpp
<snip>

---

Et encore...

Finalement, on s'habitue encore assez vite à écrire std::cout, std::endl voire std::string::npos :D

La directive using namespace présente par contre un intérêt majeur lorsqu'il y a imbrication des espaces de noms comme

Code:

---

1 2 3 4

using namespace boost::filesystem; using namespace std::tr1; using namespace myNs::game::sound/*::...*/;

---

parce qu'il devient rapidement lourd d'avoir à se "coltiner" le chemin complet à chaque fois :aie:

Mais, quoi qu'il en soit, ce genre de directive ne devrait jamais être utilisé dans un fichier d'en-tête pour les raisons sus-citées, et ne devrait donc être utilisé que dans les fichier *.cpp ;):D

Citation:

---

Envoyé par deubelte

Disons que mon problème venait plutot d'un problème de compilation.
Je ne sais pas comment le compilateur compile, et dans quel ordre (est-ce que lepremier fichier est celui qui contient main?) il compile les fichiers.

---

Ca n'a aucune importance. Il compile dans l'ordre qu'il veut. Ton code ne doit pas être construit en fonction de ça. D'ailleurs d'une compilation sur l'autre, l'ordre varie, ne serait-ce que parce que le compilateur ne recompile pas les différents fichiers non modifiés d'une fois sur l'autre.

Citation:

---

Envoyé par deubelte

mais supposons que j'ai un projet dans lequel j'ai un header "header.h" et deux fichiers cpp. tous les deux fichiers cpp contiennent
#include "header.h".

Donc s'il compile le premier cpp, il va compiler header.h.
Ensuite, il va compiler le deuxieme cpp. Il va donc recompiler header.h, sauf (il me semble) si on met des include guard.

---

Non, les includes guard servent à éviter les doubles définitions si le fichier est inclus 2 fois dans le même fichier.
Par exemple:
A.h
header1.h fait include "A.h"
header2.h fait incldue "A.h" et include "header1.h". Le include guard sert à ce que A.h ne soit effectivement inclus qu'une fois.

Citation:

---

Envoyé par deubelte

Les commentaires sont aussi lus par le compilateur?? :roll:

---

On ne va pas faire de subtilité: non, les commentaires ne génèrent pas de code par le compilateur.

Citation:

---

Envoyé par deubelte

Donc s'il compile le premier cpp, il va compiler header.h.
Ensuite, il va compiler le deuxieme cpp. Il va donc recompiler header.h, sauf (il me semble) si on met des include guard.
Les commentaires sont aussi lus par le compilateur?? :roll:

---

Je pense que quand on met #ifndef HEADER_H (c'est comme le if normal) le HEADER_H n'est pas encore definie, on met #define HEADER_H pour le définir. et ceci se passe lors de la première compilation de header.h. lors de passage a la compilation a partir du deuxième fichier .cpp,le compilateur va essayer de renter dans le fichier header.h il va tester si HEADER_H n'est pas définit avec #ifndef HEADER_H. Hors nous avons deja définit HEADER_H lors de premier passage dans header.h, donc il ne vas pas entrer dans le bloc :

Code:

---

1 2 3 4 5 6 7

#define HEADER_H /* définition des classes et structures,  * déclaration des fonctions  * définition de fonctions inline  */ #endif //HEADER_H

---

Citation:

---

Envoyé par buggen25

Je pense que quand on met #ifndef HEADER_H (c'est comme le if normal) le HEADER_H n'est pas encore definie, on met #define HEADER_H pour le définir. et ceci se passe lors de la première compilation de header.h. lors de passage a la compilation a partir du deuxième fichier .cpp,le compilateur va essayer de renter dans le fichier header.h il va tester si HEADER_H n'est pas définit avec #ifndef HEADER_H. Hors nous avons deja définit HEADER_H lors de premier passage dans header.h, donc il ne vas pas entrer dans le bloc :

Code:

---

1 2 3 4 5 6 7

#define HEADER_H /* définition des classes et structures,  * déclaration des fonctions  * définition de fonctions inline  */ #endif //HEADER_H

---

---

Un .h n'est pas à proprement parlé compilé. C'est le .cpp qui va être compilé et qui va provoqué la compilation du .h quand il sera inclus. Si un .h est inclus par 3 .cpp, alors il est compilé 3 fois. La directive #ifndef HEADER_H... sert à ce que le même .h ne soit pas inclus n fois dans le même cpp. Le compilateur n'a pas mémoire des #define TUTU d'un .cpp sur un autre.

Citation:

---

Envoyé par 3DArchi

Un .h n'est pas à proprement parlé compilé. C'est le .cpp qui va être compilé et qui va provoqué la compilation du .h quand il sera inclus. Si un .h est inclus par 3 .cpp, alors il est compilé 3 fois. La directive #ifndef HEADER_H... sert à ce que le même .h ne soit pas inclus n fois dans le même cpp. Le compilateur n'a pas mémoire des #define TUTU d'un .cpp sur un autre.

---

Autant pour moi. Je savait pas :aie:. Il faut que je teste tout ça !
En ce qui concèrne les 3 inclusion du meme fichier .h, j'espère que c'est une inclusion impilcite ou indirecte si on veut.