Conflits de classes - link error

**sphere369** · 10/07/2009, 22h07

Bonjour, je suis entrain de debuger un programme qui me donne un message d'erreur que je n'arrive pas a résoudre.

Ci-dessous le message d'erreur recu et les codes concerner :

1>Édition des liens en cours...
1>testBSPDE1.obj : error LNK2005: "double (__cdecl* BlackScholesOneFactorIBVP::sigma)(double,double)" (?sigma@BlackScholesOneFactorIBVP@@3P6ANNN@ZA) déjà défini(e) dans FDMDirector.obj
1>testBSPDE1.obj : error LNK2005: "double (__cdecl* BlackScholesOneFactorIBVP::f)(double,double)" (?f@BlackScholesOneFactorIBVP@@3P6ANNN@ZA) déjà défini(e) dans FDMDirector.obj
1>testBSPDE1.obj : error LNK2005: "double (__cdecl* BlackScholesOneFactorIBVP::mu)(double,double)" (?mu@BlackScholesOneFactorIBVP@@3P6ANNN@ZA) déjà défini(e) dans FDMDirector.obj
1>testBSPDE1.obj : error LNK2005: "double (__cdecl* BlackScholesOneFactorIBVP::IC)(double)" (?IC@BlackScholesOneFactorIBVP@@3P6ANN@ZA) déjà défini(e) dans FDMDirector.obj
1>testBSPDE1.obj : error LNK2005: "double (__cdecl* BlackScholesOneFactorIBVP::BCR)(double)" (?BCR@BlackScholesOneFactorIBVP@@3P6ANN@ZA) déjà défini(e) dans FDMDirector.obj
1>testBSPDE1.obj : error LNK2005: "double (__cdecl* BlackScholesOneFactorIBVP::BCL)(double)" (?BCL@BlackScholesOneFactorIBVP@@3P6ANN@ZA) déjà défini(e) dans FDMDirector.obj
1>testBSPDE1.obj : error LNK2005: "double (__cdecl* BlackScholesOneFactorIBVP::b)(double,double)" (?b@BlackScholesOneFactorIBVP@@3P6ANNN@ZA) déjà défini(e) dans FDMDirector.obj
1>C:\sphere369\DistanceLearning\Code\Part1_Cpp_Essential_Skills\Debug\Chap10_BSPDE1.exe : fatal error LNK1169: un ou plusieurs symboles définis à différentes reprises ont été rencontrés
1>Le journal de génération a été enregistré à l'emplacement "file://c:\sphere369\DistanceLearning\Code\Part1_Cpp_Essential_Skills\Chap10_BSPDE1\Debug\BuildLog.htm"
1>Chap10_BSPDE1 - 8 erreur(s), 2 avertissement(s)

testBSDE1.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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
 
#include "fdmdirector.cpp"
#include "arraymechanisms.cpp"
 
#include <iostream>
#include <string>
using namespace std;
 
#include "ExcelDriver.hpp"
 
void printOneExcel(Vector<double, long> & x,						
					Vector<double, long>& functionResult,
					string& title)
{ 
	// N.B. Excel has a limit of 8 charts; after that you get a run-time error
 
	cout << "Starting Excel\n";
 
	ExcelDriver & excel = ExcelDriver::Instance();
 
	excel.MakeVisible(true);		// Default is INVISIBLE!
 
	excel.CreateChart(x, functionResult, title, "X", "Y");
 
 
}
 
// Excel output as well
void printInExcel(const Vector<double, long>& x,							// X array
					const list<string>& labels,							// Names of each vector
					const list<Vector<double, long> >& functionResult)	// The list of Y values
{ // Print a list of Vectors in Excel. Each vector is the output of
  // a finite difference scheme for a scalar IVP
 
	cout << "Starting Excel\n";
 
	ExcelDriver & excel = ExcelDriver::Instance();
 
	excel.MakeVisible(true);		// Default is INVISIBLE!
 
 
	// Don't make the string names too long!!
	excel.CreateChart(x, labels, functionResult, string("FDM Scalar IVP"),
						string("Time Axis"), string ("Value"));
 
 
}
 
 
 
double mySigma (double x, double t)
{
 
	double sigmaS = 0.30 * 0.30;
 
	return 0.5 * sigmaS * x * x;
}
 
double myMu (double x, double t)
{
	double r = 0.06;
	double D = 0.00;
 
	return (r - D) * x;
}
 
double myB (double x, double t)
{
	double r = 0.06;
 
	return  -r;
}
 
double myF (double x, double t)
{
	return 0.0;
}
 
double myBCL (double t)
{
 //return 0.0; // C
 
	double K = 10;
	double r = 0.06;
	double T = 1.0;
	return K * ::exp(-r * (T - t));
}
 
double myBCR (double t)
{
	double Smax = 100;
//	return Smax;
 
 
	return 0.0; // P
}
 
double myIC (double x)
{
	double K = 10;
 
	if (x < K)
		return -(x - K);
 
	return 0.0;
 
}
 
 
int main()
{
	using namespace BlackScholesOneFactorIBVP;
 
	// Assignment of functions
	sigma = mySigma;
	mu = myMu;
	b = myB;
	f = myF;
	BCL = myBCL;
	BCR = myBCR;
	IC = myIC;
 
	double T = 1.0;	int J= 200; int N = 4000-1;
	double Smax = 100.0;
	FDMDirector fdir(Smax, T, J, N);
	fdir.Start();
 
L1:
	fdir.doit();
	//	print(fdir.current());
 
	if (fdir.isDone() == false)
		goto L1;
 
	Vector<double, long> xresult(J+1, 1);
	xresult[xresult.MinIndex()] = 0.0;
	double h = Smax / (double) (J);
 
	for (long j = xresult.MinIndex()+1; j <= xresult.MaxIndex(); j++)
	{
			xresult[j] = xresult[j-1] + h;
	}
 
	print (xresult);
	//print(fdir.current());
 
	printOneExcel(xresult, fdir.current(), string("Value"));
 
	return 0;
}

FDMDirector.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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
#ifndef FDMDirector_CPP
#define FDMDirector_CPP
 
#include "mesher.cpp"
#include "fdm.cpp"
#include "arraymechanisms.cpp"
 
#include <iostream>
using namespace std;
 
class FDMDirector
{
 
private:
	FDMDirector ()
	{
 
 
	}
 
public:
 
	double T;
	double Xmax;
 
	double k;
	long J, N;
	double tprev, tnow;
	Mesher m;
	Vector<double, long> xarr; // Useful work array
	FDM fdm;
 
	FDMDirector (double XM, double TM, long JSteps, long NSteps)
	{
 
		T = TM;
		J = JSteps;
		N = NSteps;
		Xmax = XM;
		tprev = tnow = 0.0;
		fdm = FDM();
 
		m = Mesher(0.0, XM, T);
	}
 
 
	Vector<double, long> current()
	{
		return fdm.current();
	}
 
	void Start() // Calculate next level
	{
		// Steps 1, 2: Get stuff from Mesher
		k = m.timeStep(N);
		xarr = m.xarr(J);
 
		cout << "k, xarr\n";
		cout << k << endl;
 
		// Step 3: Update new mesh array in FDM scheme
		fdm.initIC(xarr);
 
	}
 
	void doit()
	{
		// Step 4, 5: Get new coefficient arrays + solve
 
		tnow = tprev + k;
		fdm.calculateCoefficients(xarr, tprev, tnow);
		fdm.solve(tnow);
		tprev += k;
	}
 
 
	bool isDone()
	{
		if (tnow < T)
		{
			return false;
		}
 
		cout << "done";
		return true;
	}
};
 
#endif

**ram-0000** · 10/07/2009, 22h16

C'est pas bien de définir une classe dans un cpp, les .h sont fait pour cela.

Ton projet se compose de 2 cpp (au moins).

FDMDirector.cpp
testBSDE1.cpp

FDMDirector.cpp définit la classe FDMDirector et ses fonctions membre.
testBSDE1.cpp inclut le fichier FDMDirector.cpp et donc redéfinit la classe FDMDirector et ses fonctions membre d'où ton conflit de link.

Renomme le fichier FDMDirector.cpp en FDMDirector.h
Vire le fichier FDMDirector.cpp de ton projet (de toute façon, il n'existe plus après le renommage
Cela devrait aller mieux.

En général, dans le fichier .h, on déclare une classe et dans le fichier cpp, on définit les fonctions membres.
Certaines fonctions très courtes (1 instruction ou pour des raisons de performance mais c'est souvent une fausse raison) peuvent aussi être définies dans le .h, ce sont les fonction inline.

**sphere369** · 10/07/2009, 22h40

Ok, problème résolu.Mais a quoi sert alors les directives

Code :

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

1
2
3
4
5
6
7
 
#ifndef FDMDirector_CPP
#define FDMDirector_CPP
 
.....
 
#endif

?

Etant donne que les fonctions membres ont été définies en ligne j'avais pensé que dans ce cas cela n'avais pas d'impact que ce soit un fichier .cpp ou .h (pour les templates on inclut les .h a la place des .h afin d'éviter les problèmes de lien (en tout cas c'est comme cela que je l'ai appris dans les livres...))

**koala01** · 11/07/2009, 22h51

Salut,

Envoyé par sphere369

Ok, problème résolu.Mais a quoi sert alors les directives

Code :

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

1
2
3
4
5
6
7
 
#ifndef FDMDirector_CPP
#define FDMDirector_CPP
 
.....
 
#endif

?

Il s'agit de ce que l'on appelle des "gardes anti inclusion multiple".

Le fait est que les fichiers d'en-têtes sont susceptibles d'être inclus, finalement, n'importe où.

Il peut donc facilement arriver un moment où, le jeu des inclusions en cascade se faisant, on puisse en arriver à une situation dans laquelle un même fichier d'en-tête est inclus plusieurs fois dans une même unité de compilation (imagine, simplement, fichier1.h et fichier2.h qui incluent tous les deux un fichier donné et qui sont eux-mêmes inclus dans un autre fichier).

Comme il existe une règle dite de la déclaration unique, le compilateur se plaindrait de trouver des déclarations identiques à deux endroits du code.

les gardes anti inclusion multiples sont là pour éviter ce genre de phénomènes

Etant donne que les fonctions membres ont été définies en ligne j'avais pensé que dans ce cas cela n'avais pas d'impact que ce soit un fichier .cpp ou .h (pour les templates on inclut les .h a la place des .h afin d'éviter les problèmes de lien (en tout cas c'est comme cela que je l'ai appris dans les livres...))

Le préprocesseur est, en définitive, quelque chose de fort simple qui, lorsqu'il rencontre une directive #include va se contenter de recopier le contenu du fichier indiqué à la place de la directive.

Le fait est que les extensions ne servent, en définitive, qu'à préciser à l'utilisateur le but d'un fichier (et, de manière très facultative, au système d'exploitation pour savoir avec quelle application ouvrir un fichier donné...)

Il ne s'agit donc que de conventions, mais, afin de faciliter la compréhension de tous, il est important de les suivre.

Ne serait-ce que parce que certains outils permettant d'automatiser la compilation vont se baser sur le fait qu'elles seront suivies

**sphere369** · 12/07/2009, 16h13

Envoyé par koala01

Salut,

Il s'agit de ce que l'on appelle des "gardes anti inclusion multiple".

Le fait est que les fichiers d'en-têtes sont susceptibles d'être inclus, finalement, n'importe où.

Il peut donc facilement arriver un moment où, le jeu des inclusions en cascade se faisant, on puisse en arriver à une situation dans laquelle un même fichier d'en-tête est inclus plusieurs fois dans une même unité de compilation (imagine, simplement, fichier1.h et fichier2.h qui incluent tous les deux un fichier donné et qui sont eux-mêmes inclus dans un autre fichier).

Comme il existe une règle dite de la déclaration unique, le compilateur se plaindrait de trouver des déclarations identiques à deux endroits du code.

les gardes anti inclusion multiples sont là pour éviter ce genre de phénomènes

Le préprocesseur est, en définitive, quelque chose de fort simple qui, lorsqu'il rencontre une directive #include va se contenter de recopier le contenu du fichier indiqué à la place de la directive.

Salut, mais la directive #ifndef FDMDirector_CPP spécifie que la recopie du fichier doit se faire uniquement si FDMDirector_CPP n'a pas encore été définie. C'est pour cela que j'ai du mal a comprendre le bug si dessus (a moins ce que cette directive ne marche pas dans les fichiers .cpp).

**Mac LAK** · 12/07/2009, 18h32

Envoyé par sphere369

Salut, mais la directive #ifndef FDMDirector_CPP spécifie que la recopie du fichier doit se faire uniquement si FDMDirector_CPP n'a pas encore été définie. C'est pour cela que j'ai du mal a comprendre le bug si dessus (a moins ce que cette directive ne marche pas dans les fichiers .cpp).

Elle marche très bien, mais dans un DEUXIÈME fichier CPP, les protections anti-inclusion du PREMIER fichier CPP ne sont plus valides... Donc, il réimplémente la classe, vu que tu avais inclus des CPP au lieu d'inclure des entêtes.

Sauf cas particuliers (entêtes précompilés, #pragma spécifiques, etc.), considère qu'à chaque fichier CPP, ton compilateur perds la mémoire de tout ce qui a été fait précédemment dans la compilation de ton programme / librairie.

Conflits de classes - link error

C++

Vue hybride

Discussions similaires

Partager

Partager