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.