struct type redefinition avec VS 2005

Version imprimable

Bonjour,

J'essaye de compiler mon projet avec VS 2005. Malheuresement, VS 2005 n'aime pas mon projet (plutot j'ai fait une connerie) :D .

L'erreur :
error C2011: 'IFont' : 'struct' type redefinition

Si j'ai bien compris qu'il y a 2 definition de mon class (ou struct) IFont, mais je n'arrive pas a comprendre d'ou vient la deuxieme definition et comment la resoudre ,surtout que mon projet etais developper avec VS 2003 est ca compilé tres bien ( NO ERRORS AVEC VS 2003).

Voici le fichier source de l'erreur :

1. header file :

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
#pragma once
#include <afxwin.h> // MFC core and standard components
#include <afxtempl.h>
 
 
	class IFont
	{
	public:
 
		/*IFont(const char*, const char*, const char*);
		IFont(const char*, const char*, int);*/
		IFont();
		~IFont();
		float Height();
		float Baseline();
		float Width(const char* s, int length);
		void Extents(const char* s,int i, Text_Extent* e);
	};
 
 
template <class T>
	class MapIPaint {
 
	protected:
		// void Init(const char*);
		// void DefineEntries(const char*);
		// void DefineInitial(const char*);
		// T* CreateEntry(const char*);
 
		int initial;		// denotes which entry is used on startup
		CArray<T*> m_ptArray;
		int m_current;
 
	public:
		T* FindOrAppend(const CString& str);
		T* FindOrAppend(unsigned char r, unsigned char g, unsigned char b);
		T* FindOrAppend(bool none, int p, float w, int l, int r, int z=false);
		INT_PTR Size() const;
		bool AtEnd();
		T* First();
		T* Last();
		T* Prev();
		T* Next();
		int GetCurrentIndex();
		T* GetCur();
		T* Index(int index) const;
		int Find(const T* obj) const;
		void setInitial(int i);
		T* GetInitial();
	};

2. cpp file
[

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#include <stdio.h>
#include "mapipaint.h"
 
/////////////////////// IFont ///////////////////////
// j'ai pas donné l'implementation ,parceque je suis certains que ca ne vient pas d'ici
 
IFont::IFont() {} 
IFont::~IFont() {}
float IFont::Height() {
	return 0.0;
}
float IFont::Baseline() {
	return 0.0;
}
float IFont::Width(const char* s, int length) {
	return 0.0;
}
 
void IFont::Extents(const char* s , int i , Text_Extent* e) {}
 
// bien sur ici il y a la definition de class template

Comme je l'avais dis , il compile bien avec VS 2003.

quelqu'un aurra une idée ?

Merci en avance

14/08/2007, 14h25
Alp

Essayer d'utiliser les garde d'inclusions : #ifndef, #define, #endif tel que dans :
http://c.developpez.com/faq/cpp/inde...ASS_header_cpp
14/08/2007, 14h28
kass28

Citation:

Envoyé par Alp

Essayer d'utiliser les garde d'inclusions : #ifndef, #define, #endif tel que dans :
http://c.developpez.com/faq/cpp/inde...ASS_header_cpp

En fait je crois que le probleme vient de ces deux includes :
#include <afxwin.h> // MFC core and standard components
#include <afxtempl.h>

et puis comme je l'avais dis , ca compile avec VS 2003 donc ce n'est pas un probleme d'inclusion multiple des fichiers header. Mais il y a vraiment une nouveaute dont la class IFont !!

il est clair qu'il faut toujours mettre des #ifdef....

ca devient donc

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
#ifndef __CLASS_IFRONT__
#define __CLASS_IFRONT__
#include <afxwin.h> // MFC core and standard components
#include <afxtempl.h>
 
 
	class IFont
	{
	public:
 
		/*IFont(const char*, const char*, const char*);
		IFont(const char*, const char*, int);*/
		IFont();
		~IFont();
		float Height();
		float Baseline();
		float Width(const char* s, int length);
		void Extents(const char* s,int i, Text_Extent* e);
	};
 
 
template <class T>
	class MapIPaint {
 
	protected:
		// void Init(const char*);
		// void DefineEntries(const char*);
		// void DefineInitial(const char*);
		// T* CreateEntry(const char*);
 
		int initial;		// denotes which entry is used on startup
		CArray<T*> m_ptArray;
		int m_current;
 
	public:
		T* FindOrAppend(const CString& str);
		T* FindOrAppend(unsigned char r, unsigned char g, unsigned char b);
		T* FindOrAppend(bool none, int p, float w, int l, int r, int z=false);
		INT_PTR Size() const;
		bool AtEnd();
		T* First();
		T* Last();
		T* Prev();
		T* Next();
		int GetCurrentIndex();
		T* GetCur();
		T* Index(int index) const;
		int Find(const T* obj) const;
		void setInitial(int i);
		T* GetInitial();
	};
 #endif

Pour ce qui est de la class IFont, verifie dans tes headers qu'il ne manque pas une directive de compilation dans le meme principe #ifdef... mais normalement l'ensemble de ces fichiers utilisent cette logique...

Citation:

Envoyé par rikau2

il est clair qu'il faut toujours mettre des #ifdef....

ca devient donc

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
#ifndef __CLASS_IFRONT__
#define __CLASS_IFRONT__
#pragma once
#include <afxwin.h> // MFC core and standard components
#include <afxtempl.h>
 
 
	class IFont
	{
	public:
 
		/*IFont(const char*, const char*, const char*);
		IFont(const char*, const char*, int);*/
		IFont();
		~IFont();
		float Height();
		float Baseline();
		float Width(const char* s, int length);
		void Extents(const char* s,int i, Text_Extent* e);
	};
 
 
template <class T>
	class MapIPaint {
 
	protected:
		// void Init(const char*);
		// void DefineEntries(const char*);
		// void DefineInitial(const char*);
		// T* CreateEntry(const char*);
 
		int initial;		// denotes which entry is used on startup
		CArray<T*> m_ptArray;
		int m_current;
 
	public:
		T* FindOrAppend(const CString& str);
		T* FindOrAppend(unsigned char r, unsigned char g, unsigned char b);
		T* FindOrAppend(bool none, int p, float w, int l, int r, int z=false);
		INT_PTR Size() const;
		bool AtEnd();
		T* First();
		T* Last();
		T* Prev();
		T* Next();
		int GetCurrentIndex();
		T* GetCur();
		T* Index(int index) const;
		int Find(const T* obj) const;
		void setInitial(int i);
		T* GetInitial();
	};
 #endif

Quand on utilise

Code:

#pragma once

ca suffirra pour eviter les problemes d'inclusion

14/08/2007, 15h04
Alp

De plus #pragma once n'est pas du tout standard et ne marche pas sur un bon nombre de compilateurs.

Les gardes d'inclusions (#ifndef & compagnie) marchent à tous les coups, eux.

Tu es sur que ce n'est pas ça la cause de ton problème de compilation ?
Desfois que VS2003 ait des habitudes bizarres ...
14/08/2007, 15h21
kass28
Citation:

Envoyé par Alp

De plus #pragma once n'est pas du tout standard et ne marche pas sur un bon nombre de compilateurs.

Les gardes d'inclusions (#ifndef & compagnie) marchent à tous les coups, eux.

Tu es sur que ce n'est pas ça la cause de ton problème de compilation ?
Desfois que VS2003 ait des habitudes bizarres ...

Je suis sur que ce n'est pas un probleme d'inclusion.

En fait , vu que j'inclus les deux fichiers :
Code:

1 2 3 #include <afxwin.h> // MFC core and standard components #include <afxtempl.h>
il y a inclusion du fichier : ocidl.h et dans ce fichier aparament il y a declaration d'une classe IFont : la meme chose que moi.

Et je crois que mon probleme vient de là ? mais j'en sais rien comment le resoudre.

Merci
14/08/2007, 15h24
rikau2

possibilité de mettre un namespace ....

http://c.developpez.com/faq/cpp/?page=namespace

Citation:

Envoyé par rikau2

possibilité de mettre un namespace ....

http://c.developpez.com/faq/cpp/?page=namespace

J'ai deja essayer de faire un namespace comme par exemple :
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 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 #pragma once #include <afxwin.h> // MFC core and standard components #include <afxtempl.h> namespace ImportDll { class IFont { public: /*IFont(const char*, const char*, const char*); IFont(const char*, const char*, int);*/ IFont(); ~IFont(); float Height(); float Baseline(); float Width(const char* s, int length); void Extents(const char* s,int i, Text_Extent* e); }; template <class T> class MapIPaint { protected: // void Init(const char*); // void DefineEntries(const char*); // void DefineInitial(const char*); // T* CreateEntry(const char*); int initial; // denotes which entry is used on startup CArray<T*> m_ptArray; int m_current; public: T* FindOrAppend(const CString& str); T* FindOrAppend(unsigned char r, unsigned char g, unsigned char b); T* FindOrAppend(bool none, int p, float w, int l, int r, int z=false); INT_PTR Size() const; bool AtEnd(); T* First(); T* Last(); T* Prev(); T* Next(); int GetCurrentIndex(); T* GetCur(); T* Index(int index) const; int Find(const T* obj) const; void setInitial(int i); T* GetInitial(); }; }// mportDll
et dans le .cpp :
Code:

1 2 3 4 5 #include "mapipaint.h" // qui contient le namespace using namespace ImportDll; ...........
et quand j'ai compile j'ai eu cette erreur :
d:\kaledo\kaledocollection_vs2005_kassem\6_product\kaledostyle\importdll\utilities\mapipaint.cpp(81) : error C2872: 'IFont' : ambiguous symbol
could be 'c:\program files\microsoft visual studio 8\vc\platformsdk\include\ocidl.h(3533) : IFont' //
or 'd:\kaledo\kaledocollection_vs2005_kassem\6_product\kaledostyle\importdll\utilities\mapipaint.h(60) : ImportDll::IFont'

14/08/2007, 15h59
r0d

La seule solution que je vois est de changer le nom de ta classe.
En effet, contrairement aux libs qui sont bien programmées (houuu pas bô le troll), ma MFC déclare tout dans le namespace global.
14/08/2007, 16h01
rikau2

Justement ne pas mettre de using namespace car cela masque l'interet de celui ci.

Il faut utiliser ta classe en faisant par exemple:

ImportDll::IFont MyIFont = new ImportDll::IFont(...

le using namespace t'evite de faire le mynamespace::maclasse ... tu cherches justement a faire le contraire !

Une autre solution, mais la a verifier, si tu veux utiliser les elements qui ne font pas parti de ta classe, essayes ::IFont pour l'appel a la standard, et IFont simplement pour l'appel a ta classe
14/08/2007, 16h03
kass28

Citation:

Envoyé par r0d

La seule solution que je vois est de changer le nom de ta classe.
En effet, contrairement aux libs qui sont bien programmées (houuu pas bô le troll), ma MFC déclare tout dans le namespace global.

Mais je n'arrive pas a comprendre pourquoi il y a une erreur ambiguous symbol
alors que je fait :

Code:

using namespace importDll // le name space qui contient la classe

En fait pourquoi il n'arrive pas a faire la difference ?
14/08/2007, 16h13
r0d
En fait rikau2 a raison.
Par exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 #pragma once namespace Dum { class IFont { // code }; };
suivi de

Code:

Dum::IFont myFont = Dum::IFont();

ça fonctionne (il ne faut pas utiliser de using, sinon il se perd).
14/08/2007, 16h19
rikau2

lorsque l'on declare un namespace c'est pour "cacher" des données derriere. Ca permet alors de definir plusieurs fois le meme non de methode, variable, classe, ... ou plutot si plusieurs personnes travaillant sur un meme projet developpent dans leur coin, si toutes les deux, par le plus grand des hasard , a le meme nom de classe, le compilateur ne sais pas lequel choisir.

Dans ton cas c'est exactement ca, tu as deux elements qui possedent le meme nom, ton compilateur t'injure car il ne sait pas faire la difference. C'est la que le namespace arrive.........

la directive using namespace std (par exemple) t'evite d'ecrire devant chaque element du namespace std, exemple cout, cin, cerr... l'element std::cout, std::cerr, ...

Le probleme apparait si deux personnes declarent un namespace std, le compilateur ne peut faire de differences.

lorsque tu mets using namespace ImportDll, tu masques l'utilitées du namespace, et tous les elements de ton namespace apparaissent pour le compilateur, sous la directive using namespace..., comme des elements ne necessitant pas le prefixe de ton namespace, et la on tourne en rond....

Donc pour que ca marche, laisse ton namespace, enleve le using namespace ..., et devant chaque element du namespace, tu met monNamespace:: et la le compilo sera content
14/08/2007, 17h25
farscape

IFont est une classe COM d'ou le problème ...
14/08/2007, 17h31
kass28

Citation:

Envoyé par farscape

IFont est une classe COM d'ou le problème ...

D'accord et c'est quoi la solution ? Il faut absolument alors faire un namespace ? ou il y a d'autre moyen?

Et quand vous dites c'est un objet COM d'ou le probleme , ca signifie quoi ?

Merci en avance
14/08/2007, 17h39
Alp

Ca veut dire que via des includes on dispose de cette classe, et quand le compilateur rencontre ton IFont, il nous signale qu'il a deux IFont.
Donc 2 solutions :
- Tu mets ton IFont dans une namespace
- Tu changes le nom de ton IFont
14/08/2007, 17h48
farscape

ta classe rentre en conflit avec celle définie dans un header ole (ocidl.h),
La seule solution viable c'est le namespace pour ta classe.