Discussion :

[delire8]

salut !!

je susi en train de developper une application qui a fortement recour aux dlls, notamment pour le systeme de plugins.
Cependant, je rencotre un probleme:

1- avec le code suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
typedef AnsiString (__stdcall *GETINFOSONDLL)(void);
 
 HINSTANCE hinstDLL;
 GETINFOSONDLL ImpFuncDLL;
 
 hinstDLL = LoadLibrary("DLL/key.dll") ;
 if ( hinstDLL )
    ImpFuncDLL = (GETINFOSONDLL)GetProcAddress(hinstDLL, "GetDLLVersion");
 
 if (ImpFuncDLL)
   Application->MessageBoxA(ImpFuncDLL().c_str(),"",0);
 
  FreeLibrary(hinstDLL);

hey bien ImpFuncDLL vaut null. Mais je ne comprend pas pourquoi : la DLL est bien chargée (elle affiche un msg) et GetDLLVersion est une fonction de ma DLL qui correspond a ceci

Code :

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

1
2
3
4
5
6
 
AnsiString __stdcall GetDLLVersion()
{
 AnsiString tmp = IntToStr(VERSION);
 return tmp;
}

De plus, j'ai suivi le tutorial de LFE et ne vosi aps ce que j'aurais sauté !

merci.

[Nicolas Gallerand]

Tu peux obtenir plus d'info en utilisant GetLastError()
Voilà un exemple de code pour obtenir un message en clair :

Code :

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

1
2
3
4
5
6
7
8
9
10
char MsgBuf[100];
	FormatMessage(
	    FORMAT_MESSAGE_FROM_SYSTEM,
    	NULL,
	   	GetLastError(),
	    MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT), // Default language
	    MsgBuf,
	    99,
	    NULL
		);

[drobert]

1) il est déconseillé de transférer des AnsiString via des DLL. Lire le message d'avertissement qui se crée automatiquement lorsqu'on crée une DLL:

//---------------------------------------------------------------------------
// Remarque importante concernant la gestion de mémoire de DLL lorsque votre DLL utilise la
// version statique de la bibliothèque d'exécution :
//
// Si votre DLL exporte des fonctions qui passent des objets String (ou des
// structures/classes contenant des chaînes imbriquées) comme paramètre
// ou résultat de fonction, vous devrez ajouter la bibliothèque MEMMGR.LIB
// à la fois au projet DLL et à tout projet qui utilise la DLL. Vous devez aussi
// utiliser MEMMGR.LIB si un projet qui utilise la DLL effectue des opérations
// new ou delete sur n'importe quelle classe non dérivée de TObject qui est
// exportée depuis la DLL. Ajouter MEMMGR.LIB à votre projet forcera la DLL et
// ses EXE appelants à utiliser BORLNDMM.DLL comme gestionnaire de mémoire.
// Dans ce cas, le fichier BORLNDMM.DLL devra être déployé avec votre DLL.
//
// Pour éviter d'utiliser BORLNDMM.DLL, passez les chaînes comme paramètres "char *"
// ou ShortString.
//
// Si votre DLL utilise la version dynamique de la RTL, vous n'avez pas besoin
// d'ajouter MEMMGR.LIB, car cela est fait automatiquement.
//---------------------------------------------------------------------------

2) Il est également déconseillé d'utiliser _stdcall (je pensais même que c'était impossible). Sinon, ta DLL ne sera compatible qu'avec des programmes Borland.

3) Si c'est une fonction C++, elle ne s'apelle pas GetDLLVersion, mais un truc comme @GetDLLVersion@iv@ (utilise ImpLib pour avoir son nom exact). Si ce n'est pas une fonction C++, il manque extern "C".

4) Tant que tu y est, essaye aussi de rajouter le mot-clé _export...

[Nicolas Gallerand]

2) Il est également déconseillé d'utiliser _stdcall (je pensais même que c'était impossible). Sinon, ta DLL ne sera compatible qu'avec des programmes Borland.

Au contraire __stdcall est impératif pour être standard. C'est __fastcall qui est spécifique Borland.

la déclaration de la fonction exportée doit être

Code :

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

1
2
 
extern "C" __declspec(dllexport) AnsiString __stdcall GetDLLVersion(void)

la même fonction importée doit être déclarée avec dllimport :

Code :

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

1
2
 
extern "C" __declspec(dllimport) AnsiString __stdcall GetDLLVersion(void)

La signature des fonctions exportée par une DLL Borland peut être vue facilement en utilisant Aperçu rapide (clic droit sur le fichier .DLL).

[drobert]

Mille excuses ! T'as entièrement raison pour le _stdcall. C'est moi qui ai lu trop fast...

[delire8]

salut !

dans le tutoriel de LFE

Code :

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

1
2
 
extern "C" __declspec(dllexport) AnsiString __stdcall GetDLLVersion(void)

c'est pou un appel statique, quand on a le dichier lib. Enfin, a ce que je crois.
Sinon, je veux bien inclure MEMMGR.LIB, sauf que je ne l'ai pas !
Ou le trouver ?

merci.

[Nicolas Gallerand]

J'ai répondu un peu vite.
C'est la déclaration dllimport qui est valable pour une liaison statique.

La déclaration dllexport est la même dans les deux cas. Une fonction déclarée de manière ordinaire ne sera pas exportée : sa portée est limitée au code interne de la DLL.

Ton code me parait correct sauf la déclaration de la fonction de la DLL.

A mon avis, le problème de libération mémoire concernant les AnsiString disparaît si tu passes une référence au lieu de créer un objet :

Code :

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

... AnsiString& GetDLLVersion(...)

Je serais content d'avoir une confirmation sur ce point.

[delire8]

effectivement, le probleme venait bien de mon oublie : dire que la fonction doit etre exporté dans la dll...

merci

cependant, je n'ai pas compris le probleme de la fuite de mémoire resolvable par "&" ???

merci.