Discussion :

[GLDavid]

Bonjour

Mon problème fait suite à un précédent thread du forum C++. En effet, je porte actuellement un code de librairie C++ venant de Linux vers Windows en utilisant Visual C++ Express 2005.
Je précise que sous Linux, j'utilise g++ et un makefile, la compilation se passe sans erreurs ni warnings et que l'édition de liens est très correcte.
En revanche sous Visual C++, c'est autre chose. En effet, après avoir résolu les problèmes de syntaxes, mon code compile sans erreurs ni warnings. L'ennui vient de l'édition de liens. Je me prend des erreurs telles celles-ci :

Code :

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

1
2
3
 
1>moncode.obj : error LNK2005: "public: __thiscall TSetSeq::TSetSeq(void)" (??0TSetSeq@@QAE@XZ) already defined in TSetSeq.obj
1>moncode.obj : error LNK2005: "public: __thiscall TSetSeq::~TSetSeq(void)" (??1TSetSeq@@QAE@XZ) already defined in TSetSeq.obj

Pouvez-vous m'aider à résoudre ce problème ?
Merci d'avance.

@++

[farscape]

salut,
a priori tu n'as pas du utiliser des ifdef anti-inclusion voir faq:
http://c.developpez.com/faq/vc/?page=IDE#PbWithHeaders
avec vc2005 on peut utiliser un #pragma once à la place

Code :

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

1
2
3
 
// header.h 
#pragma once

[GLDavid]

Hello farscape

Merci de ta réponse, mais j'avais déjà inclus les ifdef :

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
22
23
24
25
26
 
#ifndef _TSETSEQ_H_
#define _TSETSEQ_H_
 
#include <vector>
#include <string>
 
#include <math.h>
//#include <cstdlib>
 
#include "TSeq.h"
class TScoMat;
class TBox;
 
 
/**
 * class TSetSeq
 * 
 * This class defines a sequence set
 */
class TSetSeq
{
//Blasblas
} ;
 
#endif

Et malgré ça, j'ai toujours mes erreurs au linkage.
De plus, où doit-on mettre le #pragma once ?
Merci d'avance de tes réponses ainsi qu'aux autres forumeurs.

@++

[farscape]

re,
alors si ce n'est pas les includes c'est que tes classes sont définies deux fois .
tu as vérifié les sources : TSetSeq.obj et moncode.obj ?

[Gabrielly]

De plus, où doit-on mettre le #pragma once ?

A mettre au début du *.h, mais comme Farscape le dit, il y a des doublons de définitions. Il faut scruter le code et particulièrement tes includes.

[farscape]

une possibilité d'erreur possible :

Code :

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

1
2
3
4
5
6
 
class TSetSeq
{
TSetSeq(){} // definit dans le .h 
~TSetSeq(){}
};

et dans le .cpp:

Code :

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

1
2
3
4
5
6
7
8
9
 
TSetSeq::TSetSeq() // redefinit dans le .cpp.
{
 
}
TSetSeq:: ~TSetSeq()
{
 
}

[hiko-seijuro]

je crois que c'est l'erreur qu'il souhaitait corriger lol

[GLDavid]

Pardon, mais je ne vois toujours pas l'erreur . Voici un extrait de mon header :

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
22
 
class TSetSeq
{
private:  
 
   /**Number of sequences to store in the set**/
   unsigned long int NumSeq;
   /**Maximum value of sequences to store in the set**/
   unsigned long int MaxNumSeq;
   /**Pointer to a sequence**/
   TSeq::PTSeq Seq;
 
public:
 
   /**A vector a sequence pointers**/
   typedef std::vector<TSetSeq> PTSetSeq;
 
   /**Constructor**/
   TSetSeq();
   /**Destructor**/
   ~TSetSeq();
};

et voici le code c++ :

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
22
23
24
 
#include "TSetSeq.h"
 
TSetSeq::TSetSeq()
{
  Init();
}
 
TSetSeq::~TSetSeq()
{
  Free();
}
 
void TSetSeq::Init()
{
   NumSeq=0;   
   MaxNumSeq=0;
}
 
void TSetSeq::Free()
{
   Seq.resize(0);
   Init();   
}

Je ne vois vraiment pas où peut être mon erreur.

@++

[farscape]

ça correspond au source TSetSeq.ccp ?
et dans le source moncode.cpp il doit y avoir une reference au constructeur et au destructeur de cette classe ..

[GLDavid]

Bon, j'ai trouvé, apparemment, c'était des inclusions de headers en trop que j'avais.
Bon, encore 7 erreurs de linkages à résoudre et on pourra s'en aller

@++ et merci de vos indications.