Conflits de classes - link error

Version imprimable

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 :

Citation:

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:

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:

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

10/07/2009, 21h16
ram-0000
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.
10/07/2009, 21h40
sphere369
Ok, problème résolu.Mais a quoi sert alors les directives
Code:

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...))
11/07/2009, 21h51
koala01
Salut,
Citation:
Envoyé par sphere369

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

Code:

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 ;)

Citation:

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 ;)
12/07/2009, 15h13
sphere369

Citation:

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).
12/07/2009, 17h32
Mac LAK

Citation:

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.
13/07/2009, 16h05
sphere369

C'est on ne peut plus claire.

Merci a tous pour vos éclaircissements,

Sphere369