[Linkage] Ma librairie ne semble pas incluse

Version imprimable

Bonjour

Je compile sous VC++ Express un projet qui a besoin de 2 librairies que j'ai préalablement compilé et linké avec succès : cppUnit et une autre de ma composition (microLib.dll et microLib.lib).
Mais, dans mon projet, malgré les ajouts des
Code:

1 2 #pragma comment (lib, "microLib.lib")
Et le fait que j'ai ajouté le path de mes fichiers .lib dans les options de link de VC++ Express, je me prends en masse des erreurs du style :
Code:

1 2 1>StableRegionTest.obj : error LNK2019: unresolved external symbol "public: void __thiscall Spectrum::addPoint(class Dot const &)" (?addPoint@Spectrum@@QAEXABVDot@@@Z) referenced in function "private: void __thiscall StableRegionTest::addPoints(class Spectrum &)" (?addPoints@StableRegionTest@@AAEXAAVSpectrum@@@Z)
J'ai l'impression que ma librairie n'est pas incluse dans le linkage :koi:
Est-ce cela ? Comment résoudre ce problème ?

Merci d'avance de vos réponses.

@++

05/01/2007, 10h05
Médinoc

Je pense que la fonction qui manque (ou la classe qui la contient) n'est pas déclarée en dllimport/dllexport.
En effet, le nom "décoré" d'une fonction en dllimport commence toujours par __imp...
05/01/2007, 10h08
GLDavid

Bonjour Medinoc

Merci de ta réponse. Pardon de poser cette question bête mais autant je compile pratiquement tout le temps en C sous Unix mais sous VC++Express, je suis complètement débutant. Donc, comment effectuer l'opération que tu me préconises ?
Merci d'avance.

@++
05/01/2007, 10h19
Médinoc

Si tu as créé ta DLL avec les assistants de Visual, tu as sans doute un fichier d'en-tête définissant une macro NOMDELADLL_API qui vaut soit __declspec(dllexport), soit __declspec(dllimport).

Il faut alors que tu précèdes la déclaration de ta classe de cette macro, et que tu recompiles la DLL et ton programme.
J'utilise peu le C++ dans les DLL, mais ça devrait marcher.
05/01/2007, 10h34
GLDavid

Et dans le cas contraire :aie: ?
Parce que je n'ai pas utilisé d'assistant pour faire tout ça.
Dieu que Visual C++ Express est compliqué à appréhender !
Je te remercie encore de ton aide Medinoc.

@++

Dans ce cas, écrit un fichier d'en-tête de ce genre:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 /* NomDeLaDll.h */ #ifndef H_NOMDELADLL #define H_NOMDELADLL /* extern "C" : Pour les fonctions C et C++ accessibles dans les deux langages */ #ifndef EXTERN_C #ifdef __cplusplus #define EXTERN_C extern "C" #else #define EXTERN_C extern #endif #endif /* NOMDELADLL_API : Pour dllimport/export */ #ifdef BUILDING_NOMDELADLL #define NOMDELADLL_API __declspec(dllexport) #else #define NOMDELADLL_API __declspec(dllimport) #endif #endif /* ! H_NOMDELADLL */
Ce fichier est comme celui généré de base par Visual, mais avec la macro EXTERN_C en plus, car elle est pratique.

Ensuite, tu n'as plus qu'à déclarer les fonctions et classes:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 /* ClasseSpectrum.hpp */ #ifndef __cplusplus #error This file is for C++ only #endif #ifndef H_CLASSESPECTRUM #define H_CLASSESPECTRUM #include "NomDeLaDll.h" class NOMDELADLL_API Spectrum { public: void addPoint(Dot const &); }; #endif // H_CLASSESPECTRUM
Code:

1 2 3 4 5 6 7 8 9 10 11 12 /* Fonctions .h */ #include "NomDeLaDll.h #ifndef H_FONCTIONS #define H_FONCTIONS EXTERN_C NOMDELADLL_API int __stdcall UneFonction(char const *); EXTERN_C NOMDELADLL_API double __stdcall UneAutreFonction(int); #endif /* ! H_FONCTIONS */
Cela devrait suffire. Si ce n'est pas suffisant, il faudra ajouter NOMDELADLL_API devant chaque définition de fonction dans le .cpp (mais je ne crois pas que ce soit nécessaire).

À présent, le tour de magie:
Dans les options du projet de ta DLL, tu ajoutes dans "preprocessor definitions" la macro BUILDING_NOMDELADLL (et uniquement dans ce projet-là!). Ainsi, les classes et fonctions exportées seront déclarées en dllexport dans la DLL et en dllimport partout ailleurs.

05/01/2007, 11h09
Médinoc

Au fait : Je n'ai pas testé, pas compilé, donc il peut toujours y avoir des fautes de frappe...
05/01/2007, 13h26
GLDavid

Oki, merci Médinoc, je trouve la solution vraiment compliqué et très différente par rapport à ce que je faisais sous unix. Sans doute est-ce dû au fait que je passe au développement sous Visual C++ :roll:
En tout cas merci à toi.
Je vais essayer de travailler sur un code plus petit pour faire mes appels librairies. Si ça capote, je tiendrais au courant.

@++

Bon, et bien, même punition sur ce minicode :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
#pragma comment(lib, "microLib.lib")
 
#include <iostream>
#include "Spectrum.hpp"
#include <cmath>
 
int main(int argc, char*argv[]){
	srand(NULL);
	Spectrum s = Spectrum();
	for(int i=0; i<6384; i++){
		s.addPoint(Dot(rand(), rand()));
	}
	return 0;
}

Voici les erreurs au linkage :

Code:

1
2
3
4
 
1>Test.obj : error LNK2019: unresolved external symbol "public: void __thiscall Spectrum::addPoint(class Dot const &)" (?addPoint@Spectrum@@QAEXABVDot@@@Z) referenced in function _main
1>Test.obj : error LNK2019: unresolved external symbol "public: __thiscall Dot::Dot(double,double)" (??0Dot@@QAE@NN@Z) referenced in function _main
1>Test.obj : error LNK2019: unresolved external symbol "public: __thiscall Spectrum::Spectrum(void)" (??0Spectrum@@QAE@XZ) referenced in function _main

:cry: Je ne comprends plus rien !
Finalement, je me demande si j'ai bien compilé comme il faut ma librairie microLib

@++

Si quelqu'un peut m'aider, il est le bienvenue car je crois que le problème vient de la librairie que je veux inclure. Voici sa ligne de compilation :

Code:

1
2
 
/O2 /GL /I "C:\Program Files\Microsoft Platform SDK\Include" /D "WIN32" /D "NDEBUG" /D "_LIB" /D "_WINDLL" /D "_MBCS" /FD /EHsc /MD /Fo"Release\\" /Fd"Release\vc80.pdb" /W3 /nologo /c /Wp64 /Zi /TP /errorReport:prompt

Et sa ligne de linkage :

Code:

1
2
 
/OUT:"C:\Documents and Settings\dbourgais\Mes documents\Coding\µLib-Visual\µLib\Release\microLib.dll" /NOLOGO /LIBPATH:"C:\Program Files\Microsoft Platform SDK\Lib" /DLL /MANIFEST /MANIFESTFILE:"Release\microLib.dll.intermediate.manifest" /DEBUG /PDB:"c:\Documents and Settings\dbourgais\Mes documents\Coding\µLib-Visual\µLib\release\microLib.pdb" /SUBSYSTEM:WINDOWS /OPT:REF /OPT:ICF /LTCG /MACHINE:X86 /ERRORREPORT:PROMPT kernel32.lib

Merci à toute bonne âme qui m'aidera.

@++

05/01/2007, 14h42
Médinoc

Je ne vois pas le /D "BUILDING_MICROLIB" qui devrait avoir été automatiquement ajoué en ligne de commande si tu l'avais mis dans les options du projet...

Et tu as déclaré la classe Spectrum en MICROLIB_API ?
05/01/2007, 14h45
GLDavid

Euuuhhhh, je ne sais pas.
Franchement, je débute complètement sous VC++ Express et j'ai l'impression de ne rien comprendre à ce machin.
Donc, comment fait-on pour ajouter ces options que tu me décris ?
Je précise que je travaille sur un simple portage d'un code C++ Unixien vers Windows (le code étant écrit en C++ STL standard de chez standard).
Je te remercie en tout cas de ta patience, Médinoc.

@++
05/01/2007, 14h49
Médinoc

Project Properties -> Configuration Properties -> C/C++ -> Preprocessor -> Preprocessor definitions.

Et ne pas oublier de régler ça pour toutes les configurations, pas seulement Release ou Debug.

Et pour la déclaration de la classe Spectrum, il faut la modifier dans son fichier d'en-tête.

Ok, j'ai fait la modification comme tu l'as indiqué. Voici la nouvelle ligne de compilation de ma librairie :
Code:

1 2 /O2 /GL /I "C:\Program Files\Microsoft Platform SDK\Include" /D "WIN32" /D "NDEBUG" /D "_LIB" /D "_CRT_SECURE_NO_DEPRECATE" /D "BUILDING_MICROLIB" /D "_WINDLL" /D "_MBCS" /FD /EHsc /MD /Fo"Release\\" /Fd"Release\vc80.pdb" /W3 /nologo /c /Wp64 /Zi /TP /errorReport:prompt
Et malgré cela, j'ai toujours les mêmes erreurs :cry:
Comprends plus rien à rien, j'en perds mon C++

@++

05/01/2007, 14h58
Médinoc

Tu peux poster le header de la lib, ainsi que celui de la classe Spectrum ?

Tout est dans le header, dans ce genre de circonstances.

Tu es sûr que tu n'as pas deux versions différentes du header ? Ça m'est déjà arrivé, ce genre de c***erie...

PS: Si le nom décoré des fonctions de met à commencer par "__imp_", on aura résolu le problème de header.

Voici le Header de Spectrum.hpp

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
151
152
153
154
155
156
157
 
#ifndef SPECTRUM_HPP_
#define SPECTRUM_HPP_
 
#include <vector>
#include <limits>
#include <string>
#include "Dot.hpp"
#include "Curve_Generic.hpp"
#include "stdafx.h"
 
/**
 * 
 * This class will describe what is a spectrum and all the methods applied on this object.
 * 
 * @author BioXpr
 *
 */
/**All the points which make a spectrum*/
class Spectrum: public std::vector<Dot>
{
	public:
 
		typedef std::vector<Spectrum> SpectrumArray;
 
		/**Constructor of the class*/
		Spectrum();
 
		/**
                 * 
                 * @param point
                 */
		/**
                 * Adding a Dot in the spectrum
                 */
		void addPoint(const Dot& point);
 
		/**
                 * 
                 * @param index
                 * @return Dot
                 */
		/**
                 * Returns a Dot at the located index.
                 */
		Dot getPoint(size_t index) const;
 
		/**
                 * 
                 * @param index
                 * @param dot
                 */
		/**
                 * Replaces the Dot located at the given index by another Dot.
                 */
		void setPoint(size_t index, const Dot& dot);
 
		/**
                 * 
                 * @return int
                 */
		/**
                 * Returns the number of Dots.
                 */
		size_t countPoints() const;
 
		/**
                 * 
                 * @return java.util.ArrayList
                 */
		/**
                 * Returns the ArrayList which contains all the Dots of the spectrum.
                 */
		Spectrum getSpectrum() const;
 
		/**
                 * An equivalent of array_shift in PHP.
                 *
                 */
		void shiftSpectrum();
 
		/**
                 * 
                 * @param point
                 */
		/**An equivalent of array_shift in PHP.*/
		void unshiftSpectrum(Dot point);
 
		/**
                 * An equivalent of array_pop in PHP.
                 *
                 */
		void popSpectrum();
 
		/**
                 * 
                 * @param index
                 */
		/**
                 * Removes a Dot at the located index.
                 */
		void removePoint(size_t index);
 
		/**
                 * An equivalent of unset in PHP
                 *
                 */
		void destroySpectrum();
 
		/**
                 * 
                 * @param curve
                 * @return Spectrum
                 */
		/**
                 * Returns a spectrum for the given curve.
                 */
		Spectrum drawCurve(const Curve_Generic& curve) const;
 
		/**
                 * 
                 * @return Dot
                 */
		/**
                 * Returns the Dot with the higher Y coordinates
                 */
		Dot getMaxYPoint() const;
 
		size_t getMaxYPointIndex() const;
 
		/**
                 * 
                 * @param list
                 * @param cutoff
                 * @param steps
                 * @param disableAuto
                 * @return java.util.ArrayList
                 */
		/**
                 * Splits the spectrum into other spectrum for the given cut-off.<br/>
                 * Splitted spectra are stored into a returned ArrayList.
                 */
		SpectrumArray splitSpectrum(const Spectrum& list, double cutoff, std::string steps, bool disableAuto) const;
 
		/**
                 * Clone the spectrum
                 */
		Spectrum clone() const;
 
/*	
	private:
 
		std::vector<Dot> points;
*/
};
 
#endif /*SPECTRUM_HPP_*/

Voili voulou.

GLDavid, qui devient fou :fou:

@++

05/01/2007, 15h07
Médinoc

Tu n'as rien fait de ce que je t'ai conseillé, en fait.

Rajoute un header nommé MicroLib.h (ou tartempion.h si tu préfères) comme celui de mon troisième post, en respectant le nom des macros (remplace NOMDELADLL par MICROLIB).

Ensuite, inclus ce header dans spectrum.hpp et déclare ta classe Spectrum en MICROLIB_API comme de te l'ai montré dans mon troisième post...
05/01/2007, 15h09
GLDavid

Ok, je vais faire ça. Désolé, mais VC++ Express me donne l'impression de ne rien faire de standard par rapport à ce que j'ai vu auparavant. Je crois que je n'aime pas cet IDE.
Merci en tout cas et navré d'avoir mal compris.

@++

Bon, j'ai fait ce que tu m'as dit. Voici mes nouveaux fichiers dans ma lib.
microLib.hpp :

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
 
/*
MICROLIB.hpp
*/
#ifndef H_MICROLIB
#define H_MICROLIB
 
/* extern "C" : Pour les fonctions C et C++ accessibles dans les deux langages */
#ifndef EXTERN_C
#ifdef __cplusplus
#define EXTERN_C extern "C"
#else
#define EXTERN_C extern
#endif
#endif
 
/* NOMDELADLL_API : Pour dllimport/export */
#ifdef BUILDING_MICROLIB
#define MICROLIB_API __declspec(dllexport)
#else
#define MICROLIB_API __declspec(dllimport)
#endif
 
#endif /* ! H_MICROLIB */

Et la nouvelle déclaration de Spectrum.hpp :

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
151
152
153
154
155
156
157
158
159
160
161
162
163
 
#ifndef __cplusplus
#error This file is for C++ only
#endif
 
#ifndef SPECTRUM_HPP_
#define SPECTRUM_HPP_
 
#include "microLib.hpp"
 
#include <vector>
#include <limits>
#include <string>
#include "Dot.hpp"
#include "Curve_Generic.hpp"
#include "stdafx.h"
 
/**
 * 
 * This class will describe what is a spectrum and all the methods applied on this object.
 * 
 * @author BioXpr
 *
 */
/**All the points which make a spectrum*/
MICROLIB_API class Spectrum: public std::vector<Dot>
{
	public:
 
		typedef std::vector<Spectrum> SpectrumArray;
 
		/**Constructor of the class*/
		Spectrum();
 
		/**
                 * 
                 * @param point
                 */
		/**
                 * Adding a Dot in the spectrum
                 */
		void addPoint(const Dot& point);
 
		/**
                 * 
                 * @param index
                 * @return Dot
                 */
		/**
                 * Returns a Dot at the located index.
                 */
		Dot getPoint(size_t index) const;
 
		/**
                 * 
                 * @param index
                 * @param dot
                 */
		/**
                 * Replaces the Dot located at the given index by another Dot.
                 */
		void setPoint(size_t index, const Dot& dot);
 
		/**
                 * 
                 * @return int
                 */
		/**
                 * Returns the number of Dots.
                 */
		size_t countPoints() const;
 
		/**
                 * 
                 * @return java.util.ArrayList
                 */
		/**
                 * Returns the ArrayList which contains all the Dots of the spectrum.
                 */
		Spectrum getSpectrum() const;
 
		/**
                 * An equivalent of array_shift in PHP.
                 *
                 */
		void shiftSpectrum();
 
		/**
                 * 
                 * @param point
                 */
		/**An equivalent of array_shift in PHP.*/
		void unshiftSpectrum(Dot point);
 
		/**
                 * An equivalent of array_pop in PHP.
                 *
                 */
		void popSpectrum();
 
		/**
                 * 
                 * @param index
                 */
		/**
                 * Removes a Dot at the located index.
                 */
		void removePoint(size_t index);
 
		/**
                 * An equivalent of unset in PHP
                 *
                 */
		void destroySpectrum();
 
		/**
                 * 
                 * @param curve
                 * @return Spectrum
                 */
		/**
                 * Returns a spectrum for the given curve.
                 */
		Spectrum drawCurve(const Curve_Generic& curve) const;
 
		/**
                 * 
                 * @return Dot
                 */
		/**
                 * Returns the Dot with the higher Y coordinates
                 */
		Dot getMaxYPoint() const;
 
		size_t getMaxYPointIndex() const;
 
		/**
                 * 
                 * @param list
                 * @param cutoff
                 * @param steps
                 * @param disableAuto
                 * @return java.util.ArrayList
                 */
		/**
                 * Splits the spectrum into other spectrum for the given cut-off.<br/>
                 * Splitted spectra are stored into a returned ArrayList.
                 */
		SpectrumArray splitSpectrum(const Spectrum& list, double cutoff, std::string steps, bool disableAuto) const;
 
		/**
                 * Clone the spectrum
                 */
		Spectrum clone() const;
 
/*	
	private:
 
		std::vector<Dot> points;
*/
};
 
#endif /*SPECTRUM_HPP_*/

La compilation provoque des warnings mais sans conséquences sur le link final de la librairie.
En revanche, si je compile mon exécutable qui appelle cette librairie, j'ai toujours les erreurs suivantes :

Code:

1
2
3
4
5
6
7
 
1>c:\documents and settings\dbourgais\mes documents\coding\µlib-visual\spectrum.hpp(160) : warning C4091: '__declspec(dllimport)' : ignored on left of 'Spectrum' when no variable is declared
1>Compiling manifest to resources...
1>Linking...
1>Test.obj : error LNK2019: unresolved external symbol "public: void __thiscall Spectrum::addPoint(class Dot const &)" (?addPoint@Spectrum@@QAEXABVDot@@@Z) referenced in function _main
1>Test.obj : error LNK2019: unresolved external symbol "public: __thiscall Dot::Dot(double,double)" (??0Dot@@QAE@NN@Z) referenced in function _main
1>Test.obj : error LNK2019: unresolved external symbol "public: __thiscall Spectrum::Spectrum(void)" (??0Spectrum@@QAE@XZ) referenced in function _main

Il y a vraiment des jours où on a de l'amour pour Linux :cry:

@++

05/01/2007, 16h47
Laurent Gomila

Ce n'est pas "MICROLIB_API class Spectrum", mais "class MICROLIB_API Spectrum".

Et si je peux me permettre, ton héritage publique de std::vector ne devrait pas être privé (d'après le contexte) ? Aussi, tu vas devoir exporter std::vector<Dot> si tu ne veux pas avoir de problème (me souviens plus de la syntaxe).
05/01/2007, 18h16
Médinoc

Citation:

Envoyé par Laurent Gomila

Ce n'est pas "MICROLIB_API class Spectrum", mais "class MICROLIB_API Spectrum".

Oups. :oops:
Désolé, d'habitude, je n'exporte que des fonctions.
En effet, placer le MICROLIB_API avant le mot class n'exportait pas la classe elle-même, mais une (inexistante) variable de ce type.

Citation:

Et si je peux me permettre, ton héritage publique de std::vector ne devrait pas être privé (d'après le contexte) ? Aussi, tu vas devoir exporter std::vector<Dot> si tu ne veux pas avoir de problème (me souviens plus de la syntaxe).

C'est vrai que les DLLs et la STL ne font pas toujours bon ménage...

D'une manière générale, il n'y a pas de problème pour une DLL utilisant la STL dans son implémentation, mais dès qu'elle affleure dans l'interface, ça fiche un peu le bordel...
06/01/2007, 16h33
Laurent Gomila

Disons que si la même implémentation de la STL est utilisée pour la bibliothèque et tous les clients ça ira, dans le cas contraire il y aura des comportements indéfinis.
10/01/2007, 08h38
GLDavid

Pardonnez mon retard dans la réponse. J'ai résolu mon problème. J'ai recompilé mes librairies sous forme de librairies statiques et cela fonctionne bien mieux.
Je ne sais pas ce qui gênit avant. Le tout est que cela marche bien mieux maintenant.
Merci encore à Médinoc de sa patience et qu'il m'excuse si j'ai pû paraître impatient ou non courtois.
Merci aussi à Laurent pour son intervention.

@++