Discussion :

[akileaQuébec]

Bonjour à tous!

Tout d'abord, merci pour le droit d'admission à ce forum: il semble merveilleux!

Venons-en maintenant au fait: je tente de créer une application pour un projet personnel. Je suis tombé sur une erreur que je suis incapable de résoudre. Je me suis créé une petite application à part qui reproduit l'erreur mais je ne réussis toujours pas à comprendre ce qui se passe.

Est-ce que l'un d'entre vous aurais l'aimabilité de m'aider svp?

Voici l'erreur obtenue:

Code :

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

1
2
 
1>CClasse.obj : error LNK2005: "int (__cdecl* CClasse::ptFunct)(int,int)" (?ptFunct@CClasse@@3P6AHHH@ZA) déjà défini(e) dans main.obj

Voici le contenu des fichiers de mon application-test:

Fichier main.cpp

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
 
#include <iostream>
#include "CClasse.h"
 
int main()
{
	//Initialise the pointer of function
	//Remarquez que je n'appel aucune fonction de CClasse!
	//CClasse::Init();
 
	int poulet1=1, poulet2=2, poulet3;
	//Call the function via the function pointer
 
	//poulet3 = CClasse::ptFunc(poulet1,poulet2);
	return 0;
}

Fichier Classe.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#ifndef __CLASSE_H__
#define __CLASSE_H__
 
namespace CClasse{
	//Pointer of function
	int (*ptFunct)(int a, int b);
 
	void Init();
	int AddAndDisplay(int a, int b);
};
 
#endif //__CLASSE_H__

Fichier Classe.cpp

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
 
#include "CClasse.h"
#include <iostream>
 
using namespace std;
 
void CClasse::Init()
{
	ptFunct=AddAndDisplay;
}
 
int CClasse::AddAndDisplay(int a, int b)
{
	int temp;
	temp=a+b;
	cout<<"Additionner et afficher:"<<endl;
	cout<<a<<" + "<<b<<" = "<<temp<<endl;
	return(temp);
}

[Mat007]

Salut,

Le problème n'est pas spécialement lié au pointeur sur fonction, si tu mets un int i; ça sera aussi le cas.
Et si tu ne mets pas de namespace le problème sera aussi toujours là (en fait pas de namespace équivaut au namespace par défaut, donc il y a toujours un namespace).

Un include est une inclusion pure et simple d'un fichier, donc c'est comme si tu copiais le contenu de CClasse.h à la fois dans CClasse.cpp et dans main.cpp.
Du coup tu te retrouves avec deux fois la même variable (en l'occurence ptFunct).

Le plus sage est peut-être de mettre ça dans une classe ?

MAT.

[aoyou]

Pour résoudre temporairement ton problème, ptFunct doit être statique. Autrement, ptFunct sera défini deux fois (une fois dans main.o et une fois dans CClasse.o)

Par contre, je me demande ce que tu bricoles. Ca ne ressemble pas à du C++. On dirait que tu utilises un namespace comme une classe.
Ta fonction Init () ressemble à un constructeur.
Pourquoi un pointeur de fonction ? Ca sent de l'héritage, comme on fait de l'objet en C.

Evite __CLASSE_H__. Les noms commençant par deux underscores sont réservés au langage C++.

[akileaQuébec]

Bonjour à vous deux,

Merci pour votre avis sur la question. J'ai effectivemenet essayé avec int i et tu avais raison le même type d'erreur se produit!

J'ai donc essayé de faire une classe purement statique, mais sans succès: j'ai la même erreur. J'admet que l'utilisation du namespace est louche et que le concept de classe statique répond plus à mes besoins.

Par ailleurs, à bien y repenser, je vais aller réviser mon polymorphismes... je crois que la conception de mon application ne s'en portera que mieux.


Merci pour votre aide!

[Mat007]

Le mot clef static dans un namespace ne sert à rien, d'ailleurs il est déprécié en C++.
En gros que tu le mettes ou non c'est pareil pour le compilateur.

Par contre si tu le déclares extern dans le .h ça devrait "fonctionner" si tu veux persister dans cette voie...
(enfin je crois, j'avoue que ça fait des années que je n'ai pas utilisé extern)

MAT.

[screetch]

static dans un namespace sert a kekchose, ca sert a ne pas exporter la cible pour le linker et donc a ce que la variable soit "locale au fichier"

[Mat007]

Ok je me suis un peu embalé.
C'est néanmoins déprécié en C++ standard cf. http://www.informit.com/guides/conte...eqNum=210&rl=1

MAT.

[aoyou]

C'est néanmoins déprécié en C++ standard cf. http://www.informit.com/guides/conte...eqNum=210&rl=1

C'est bien pour cela que j'avais dit que c'était une résolution temporaire, au moins pour qu'il comprenne son problème.

Par ailleurs, à bien y repenser, je vais aller réviser mon polymorphismes... je crois que la conception de mon application ne s'en portera que mieux.

Effectivemment, il y a des choses à revoir.

[Médinoc]

Tu appelles un namespace "CClasse" ?!

De plus, pourquoi utiliser ainsi une variable globale dans un namespace, plutôt qu'une variable static dans une classe ?