C++

[C++] Classes, méthodes et DLLs

Recherche :

Mot : Pseudo : Filtrer
Page : 1 2 3 Page Suivante Page Précédente Bas de page
Auteur	Sujet : [C++] Classes, méthodes et DLLs

godbout

Génial.

J'ai un problème vraiment très bizarre...

J'ai une classe avec différentes méthodes. Dans des une méthodes, une DLL est chargée (l'instance est gardée dans une variable membre de la classe) et quelques fonctions d'initialisation de la DLL sont appelées. Dans d'autres méthodes, j'appelle d'autres fonctions de la DLL.
Dès que j'appelle, dans ces autres méthodes, une fonction qui passe des paramètres, j'ai un plantage à la sortie de la méthode :??:
Ca arrive souvent lorsqu'un pointeur n'est pas deleté, mais là j'ai beau cherché, je ne trouve pas

Un peu d'aide serait donc la bienvenue [:ocube]

Voilà le code de la classe :
Y a des trucs moisis dedans, mais c'est parce que j'fais des tests pour trouver le pb
Le code normal est entre commentaires, mais je vous l'ai viré.

Code :

//---------------------------------------------------------------------------
TWB_Extension::TWB_Extension( AnsiString var_DllPath, AnsiString var_DllName )
{
ZeroMemory( this, sizeof( TWB_Extension ) );
PathAndFilename = var_DllPath + "\\" + var_DllName + ".dll";
Name = var_DllName;
}
//---------------------------------------------------------------------------
TWB_Extension::~TWB_Extension()
{
if ( Dependencies != NULL )
{
delete Dependencies;
Dependencies = NULL;
}
if ( MainMenuHead != NULL )
{
delete MainMenuHead;
MainMenuHead = NULL;
}
}
//---------------------------------------------------------------------------
bool TWB_Extension::Init()
{
bool var_Ret = false;
InstDll = LoadLibrary( PathAndFilename.c_str() );
if ( InstDll != NULL )
{
typedef __int32(*LPFNINITMODULE)();
LPFNINITMODULE lpfnInitModule = NULL;
lpfnInitModule = (LPFNINITMODULE)GetProcAddress( InstDll, "InitModule" );
if ( lpfnInitModule != NULL )
{
int var_Res = lpfnInitModule();
if ( var_Res == 0 )
{
var_Ret = true;
}
}
}
return var_Ret;
}
//---------------------------------------------------------------------------
bool TWB_Extension::Load()
{
bool var_Ret = false;
if ( InstDll != NULL )
{
typedef bool(*LPFNGETMODULETYPE)(int);
LPFNGETMODULETYPE lpfnGetModuleType = NULL;
lpfnGetModuleType = (LPFNGETMODULETYPE)GetProcAddress( InstDll, "EnableCommand" );
if ( lpfnGetModuleType != NULL )
{
ModuleType = lpfnGetModuleType(1);
}
}
return var_Ret;
}
//---------------------------------------------------------------------------
bool TWB_Extension::Unload()
{
bool var_Ret = false;
if ( InstDll != NULL )
{
typedef void(*LPFNCLEANUP)();
LPFNCLEANUP lpfnCleanUp = NULL;
lpfnCleanUp = (LPFNCLEANUP)GetProcAddress( InstDll, "Cleanup" );
if ( lpfnCleanUp != NULL )
{
lpfnCleanUp();
var_Ret = true;
}
FreeLibrary( InstDll );
}
InstDll = NULL;
return var_Ret;
}

Le code de déclaration :

Code :

TWB_Extension* var_Extension = new TWB_Extension( "N:\\modules\\Bin", "Project1" );
var_Extension->Init();
var_Extension->Load();
delete var_Extension;

Et ça plante dans la rtl60.bpl, en appelant une fonction qui s'appelle CallDynaInst.

Publicité

chrisbk

t'as le code des fonctions des dll qui sont appelées ? (ou juste leur prototype ?)

me demande si t'as pas un probleme de convention d'appel

Citation :

ZeroMemory( this, sizeof( TWB_Extension ) )

Alors ca c'est meme pas interdit, c'est pire que ca ! A la revolution on t'aurais decapité sur le champ. Mets a NULL/0 membre par membre, mais surtout fait pas ca ! (Tu risques d'ecraser ton pointeur de V-Table)

Sinon d'une maniere generale :

Code :

TWB_Extension* var_Extension = new TWB_Extension( "N:\\modules\\Bin", "Project1" );
var_Extension->Init();
var_Extension->Load();
delete var_Extension;

tu tiens reelement a faire une alloc dynamique alors qu'une statique te simplifierait la vie ? (encore que vu les noms de variables j'ai comme un doute [:slackerbitch] )

Message cité 1 fois
Message édité par chrisbk le 03-11-2005 à 11:41:49

chrisbk

(pis t'appelle meme pas ton beau unload [:pingouino] )

Message cité 1 fois

godbout

Génial.

chrisbk a écrit :

t'as le code des fonctions des dll qui sont appelées ? (ou juste leur prototype ?)

me demande si t'as pas un probleme de convention d'appel

C'est moi qui les ai faites
Enfin pour l'instant, en test.

Code :

extern "C" __int32 __stdcall WINEXT_API InitModule()
{
return 0;
}

Code :

extern "C" bool WINEXT_API __stdcall EnableCommand(int id)
{
return true;
}

Code :

extern "C" void WINEXT_API __stdcall Cleanup()
{
}

chrisbk a écrit :

Citation :

ZeroMemory( this, sizeof( TWB_Extension ) )

Je ne suis qu'un piètre codeur

chrisbk a écrit :

Sinon d'une maniere generale :

Code :

TWB_Extension* var_Extension = new TWB_Extension( "N:\\modules\\Bin", "Project1" );
var_Extension->Init();
var_Extension->Load();
delete var_Extension;

tu tiens reelement a faire une alloc dynamique alors qu'une statique te simplifierait la vie ? (encore que vu les noms de variables j'ai comme un doute [:slackerbitch] )

Qu'est ce qu'elles ont mes variables? [:ocube]
Ouais, j'suis obligé. Là j'ai fait un truc pourri pour tester, mais le vrai code n'est pas comme ça. Je réupère les extensions dynamiquement, et je les fous dans une liste chainée, et je parcours la liste quand j'dois faire du carnage. M'enfin tout ça c'est déjà dans l'analyse, c'est codée, y a juste le p'tit truc là qui m'emmerde
Donc j'ai réduit au maximum pour trouver d'où venait le pb, d'où le code un peu pourri que tu vois ci dessus [:ocube]

Message cité 1 fois

godbout

Génial.

chrisbk a écrit :

(pis t'appelle meme pas ton beau unload [:pingouino] )

Je l'appelle dans le vrai code mais là j'm'en fou ça plante avant :mad:
J'vais finir par le foutre si tu continues [:ocube]

(en plus, le unload ne correspond même plus à l'inverse du load vu que j'ai du foutre un init à la con entre temps, pitoyable [:itm] )

Message édité par godbout le 03-11-2005 à 11:51:17

chrisbk

godbout a écrit :

C'est moi qui les ai faites

bon bin la tu les declares en stdcall alors que par defaut la convention est cdecl

donc soit tu les modifies dans la dll, soit tu modifies ton typedef (ce qui me semble le mieux)

typedef __stdcall __int32 (*LPMONTRUC)();

jte garanti pas que ca sauve le tout mais deja ca sera un soucis en moins

Citation :

Je ne suis qu'un piètre codeur

bin améliore toi Vire ca c'est super super dangeureux

Citation :

Qu'est ce qu'elles ont mes variables? [:ocube]

rien, juste le nom ... :whistle:

Message cité 1 fois
Message édité par chrisbk le 03-11-2005 à 11:54:12

godbout

Génial.

chrisbk a écrit :

J'peux que modifier le typedef ouais, vu que la dll d'exemple que j'ai faite est pompée sur les "vraies" dlls, qui elles ne doivent pas être modifiées.

chrisbk a écrit :

bin améliore toi Vire ca c'est super super dangeureux

J'crois que tu sais que j'ai du mal d'améliorer quoique que ce soit avec un boss qui dit que les typedef c'est has been [:itm]

chrisbk a écrit :

rien, juste le nom ... :whistle:

Tu me proposes quoi ? [:ocube]
Quel con j'avais pas calé le coup des var_ [:rofl]
Une innovation technique :sol:

Message cité 1 fois
Message édité par godbout le 03-11-2005 à 12:05:42

chrisbk

Citation :

Tu me proposes quoi ? [:ocube]

Nan mais justement au vu du nom jpensais que ce code etait pas de toi [:dawao]

Message cité 1 fois

blackgoddess

vive le troll !

godbout a écrit :

J'crois que tu sais que j'ai du mal d'améliorer quoique que ce soit avec un boss qui dit que les typedef c'est has been [:itm]

Au bûcher.

Message cité 1 fois

---------------
-( BlackGoddess )-

godbout

Génial.

chrisbk a écrit :

Citation :

Tu me proposes quoi ? [:ocube]

Nan mais justement au vu du nom jpensais que ce code etait pas de toi [:dawao]

J'ai tenté une rebellion mais ça n'a pas marché, et on m'a bien fait comprendre la dernière fois que j'étais pas apte à décider [:cupra]
Donc je met des var_ pour les variables [:god]
Bon, je tiens quand même le coup avec mes typedef, mais je sais pas pour combien de temps [:ocube]

Publicité

chrisbk

bon alors ca améliore les choses mon truc ?

Message cité 1 fois

godbout

Génial.

blackgoddess a écrit :

Au bûcher.

Arrêtez voir avec ça, ça me rappelle ce qu'on me dit quand je joue de la guitare [:mlc]

Euh, sinon j'ai changé, mais rien n'y fait.

chrisbk

c'est embetant [:el g]

godbout

Génial.

chrisbk a écrit :

bon alors ca améliore les choses mon truc ?

Je me sens meilleur codeur, mais ça chie toujours autant [:ocube]

chrisbk

ca plante apres quoi ? l'init ? le load ?

godbout

Génial.

Attends, ça marche maintenant
J'avais viré le ZeroMemory, mais j'avais pas initialisé toutes les variables de la classe à NULL (juste celle qui sont utilisées dans le code). Apparemment ça viendrait de là.

C'était toujours après le Load, juste au moment de quitter la fonction.

chrisbk

moi jpense que ca vient aussi de mon histoire de convention d'appel [:icon11] (ce qui est appuyé par le lieu du plantage)

Message édité par chrisbk le 03-11-2005 à 12:15:46

godbout

Génial.

Ah ben nan ça venait pas de ça :??: :heink:

godbout

Génial.

Attends je vire, on verra bien
Parce que j'avais bien mis tes conventions, mais ça chiait toujours.

chrisbk

chui perdu

ca marche ou pas ?
ca venait d'ou ?

godbout

Génial.

J'ai viré tes conventions, ça marche :mad:

godbout

Génial.

J'en sais rien du tout
Peut être le ZeroMemory?
Pourtant en le virant ça chiait toujours dans la colle.

godbout

Génial.

J'ai remis le ZeroMemory ça marche [:rofl]

Bon écoute, j'réessaye dans mon "vrai" code et pas ma petite appli test pourrie, et j'te dis [:ocube]

Message édité par godbout le 03-11-2005 à 12:19:41

chrisbk

[:le kneu]

jpige pu rien [:pingouino]

refait voir le code apres modif

Message édité par chrisbk le 03-11-2005 à 12:20:51

blackgoddess

vive le troll !

jcomprends pas comment ca marche avec des conventions d'appel differentes oO

puis le ZeroMemory est drolement scabreux.

Message cité 2 fois

---------------
-( BlackGoddess )-

chrisbk

blackgoddess a écrit :

jcomprends pas comment ca marche avec des conventions d'appel differentes oO

sur une fonction avec 0 argument, ca change rien (le stdcall / cdecl), par contre sa fonction avec un entier normalement ca devrait cacater

Message cité 1 fois

godbout

Génial.

Code :

//---------------------------------------------------------------------------
TWB_Extension::TWB_Extension( AnsiString var_DllPath, AnsiString var_DllName )
{
PathAndFilename = var_DllPath + "\\" + var_DllName + ".dll";
Name = var_DllName;
InstDll = NULL;
Configure = 0;
ModuleType = 0;
Dependencies = new TStringList();
ShowName = "";
Version = 0;
LogMessage = NULL,
MainMenuHead = NULL;
OrderMenuHead = NULL;
}
//---------------------------------------------------------------------------
TWB_Extension::~TWB_Extension()
{
if ( Dependencies != NULL )
{
delete Dependencies;
Dependencies = NULL;
}
if ( MainMenuHead != NULL )
{
delete MainMenuHead;
MainMenuHead = NULL;
}
}
//---------------------------------------------------------------------------
bool TWB_Extension::Init()
{
bool var_Ret = false;
InstDll = LoadLibrary( PathAndFilename.c_str() );
if ( InstDll != NULL )
{
typedef __stdcall __int32(*LPFNINITMODULE)();
LPFNINITMODULE lpfnInitModule = NULL;
lpfnInitModule = (LPFNINITMODULE)GetProcAddress( InstDll, "InitModule" );
if ( lpfnInitModule != NULL )
{
int var_Res = lpfnInitModule();
if ( var_Res == 0 )
{
var_Ret = true;
}
}
}
return var_Ret;
}
//---------------------------------------------------------------------------
bool TWB_Extension::Load()
{
bool var_Ret = false;
if ( InstDll != NULL )
{
typedef __stdcall bool(*LPFNGETMODULETYPE)(int);
LPFNGETMODULETYPE lpfnGetModuleType = NULL;
lpfnGetModuleType = (LPFNGETMODULETYPE)GetProcAddress( InstDll, "EnableCommand" );
if ( lpfnGetModuleType != NULL )
{
ModuleType = lpfnGetModuleType(1);
}
}
return var_Ret;
}

Ca a l'air de marcher dans ma vraie appli aussi, merci :love: [:super chinois]

Par contre, en remettant crado comme avant, j'ai pas réussi à refaire planter [:god]

Au fait, si tu vois un autre truc à améliorer dans le code, n'hésite pas [:ocube]

godbout

Génial.

blackgoddess a écrit :

puis le ZeroMemory est drolement scabreux.

Moi qui pensait avoir trouvé un tip super [:mlc]

A votre avis ça passe sur le site où on se fout de la gueule des erreurs des autres ?

Message cité 1 fois

godbout

Génial.

chrisbk a écrit :

sur une fonction avec 0 argument, ca change rien (le stdcall / cdecl), par contre sa fonction avec un entier normalement ca devrait cacater

Quelle maîtrise. J'en savais rien du tout. On me file la DLL avec les conventions là, j'ai pas cherché à savoir, honte à moi.
Par contre effectivement, ça plantait seulement avec une function à paramètres.

Message cité 1 fois

chrisbk

bin les Dependencies = NULL; dans le destructeur ca sert a rien [:pingouino]

j'aime pas trop les var_Ret, je prefere mettre un return, meme au plein milieu d'une fonction (et c'est mon choix(tm)), et plutot que de faire

Code :

if (InstDll != NULL)
{
plein de truc
}

je prefere faire

Code :

if (InstDll == NULL)
return false;

bon, la c'est question de gout. Sinon t'as des lignes qui servent a rien mais c'est pas trop grave

Message cité 1 fois

chrisbk

godbout a écrit :

A votre avis ça passe sur le site où on se fout de la gueule des erreurs des autres ?

hein ?

godbout a écrit :

Quelle maîtrise. J'en savais rien du tout. On me file la DLL avec les conventions là, j'ai pas cherché à savoir, honte à moi.

bah pas grand monde se penche sur ce genre de blague enfin la convention d'appel c'est important, si y'a rien mets rien si y'a qqchose bin mets la meme. Jte ferais un cours sur les conventions d'appel si ca te chante [:klem3i1]

Message cité 1 fois

godbout

Génial.

chrisbk a écrit :

bin les Dependencies = NULL; dans le destructeur ca sert a rien [:pingouino]

Ouais, je sais que ça sert à rien, mais j'préfère faire tout le temps le même schémà, et ne pas oublier de mettre à NULL une fois ou j'en aurai besoin que l'inverse [:ocube]
Mais sinon je vire hein
Je veux devenir un vrai codeur comme Bill Gates.

chrisbk a écrit :

j'aime pas trop les var_Ret, je prefere mettre un return, meme au plein milieu d'une fonction (et c'est mon choix(tm)), et plutot que de faire

Code :

if (InstDll != NULL)
{
plein de truc
}

je prefere faire

Code :

if (InstDll == NULL)
return false;

Ouais, alors ça j'ai mis du temps à me décider. J'me suis dit que les return c'était dès fois un peu traitre, genre quand t'as ton LoadLibrary plus bas... A moins de refoutre un LoadLibrary devant chaque return, mais que tu viens pour modifier le code, tu le vois pas forcément.

chrisbk a écrit :

bon, la c'est question de gout. Sinon t'as des lignes qui servent a rien mais c'est pas trop grave

Genre genre genre dis moi tout [:ocube]
Si tu parles des grands if () au lieu de foutre sur une ligne, c'est pour la lisibilité [:aloy]

godbout

Génial.

chrisbk a écrit :

Ouais, j'avais pas fait gaflle, le con [:ocube]

Pour le cours, ça va attendre, je sais pas si j'vais continuer longtemps dans cette voie là [:dawa]

chrisbk

[quotemsg=1237380,32,19632
Genre genre genre dis moi tout [:ocube]
Si tu parles des grands if () au lieu de foutre sur une ligne, c'est pour la lisibilité [:aloy] [/quotemsg]

bin genre ca

Code :

LPFNGETMODULETYPE lpfnGetModuleType = NULL;
lpfnGetModuleType = (LPFNGETMODULETYPE)GetProcAddress( InstDll, "EnableCommand" )

chui une niasse, je concatene

Code :

LPFNGETMODULETYPE lpfnGetModuleType = (LPFNGETMODULETYPE)GetProcAddress( InstDll, "EnableCommand" )

Message cité 1 fois

blackgoddess

vive le troll !

je suis interressé moi
mais en fait jvais googler au lieu de demander betement, jreviendrais si je comprends pas, mci

Message cité 1 fois

---------------
-( BlackGoddess )-

chrisbk

blackgoddess a écrit :

je suis interressé moi
mais en fait jvais googler au lieu de demander betement, jreviendrais si je comprends pas, mci

y'a un article de la msdn assez causant la dessus

en deux mots, la convention change :
-ou tu mets les parametres et dans quel ordre (pile ? registre ?)
-qui fait le menage dans les parametres a la fin de la fonction (appelant ? appelé ?)

godbout

Génial.

chrisbk a écrit :

[quotemsg=1237380,32,19632
Genre genre genre dis moi tout [:ocube]
Si tu parles des grands if () au lieu de foutre sur une ligne, c'est pour la lisibilité [:aloy]

bin genre ca

Code :

LPFNGETMODULETYPE lpfnGetModuleType = NULL;
lpfnGetModuleType = (LPFNGETMODULETYPE)GetProcAddress( InstDll, "EnableCommand" )

chui une niasse, je concatene

Code :

LPFNGETMODULETYPE lpfnGetModuleType = (LPFNGETMODULETYPE)GetProcAddress( InstDll, "EnableCommand" )

[/quotemsg]
Ah ouais celui là il est flagrant [:dawao]

chrisbk

bin ya cui la aussi

Code :

int var_Res = lpfnInitModule();
if ( var_Res == 0 )
{
var_Ret = true;
}

ca pourrait se concatener en

Code :

if ( lpfnInitModule() == 0 )
return true;

c'est super verbeux ton truc

Message cité 1 fois

godbout

Génial.

Oui mais nan, parce que le vrai code c'est ça :

Code :

int var_Res = lpfnInitModule();
if ( var_Res == 0 )
{
typedef __stdcall int(*LPFNSETCONFIGDATA)( const TModuleConfigData* );
LPFNSETCONFIGDATA lpfnSetConfigData = NULL;
lpfnSetConfigData = (LPFNSETCONFIGDATA)GetProcAddress( InstDll, "SetConfigData" );
if ( lpfnSetConfigData != NULL )
{
lpfnSetConfigData( NULL );
}
ModuleType = 0;
typedef __stdcall int(*LPFNGETMODULETYPE)();
LPFNGETMODULETYPE lpfnGetModuleType = NULL;
lpfnGetModuleType = (LPFNGETMODULETYPE)GetProcAddress( InstDll, "GetModuleType" );
if ( lpfnGetModuleType != NULL )
{
ModuleType = lpfnGetModuleType();
}
Dependencies->Clear();
typedef __stdcall int(*LPFNWE_GETMODULEDEPENDENCIES)( TCsString *& );
LPFNWE_GETMODULEDEPENDENCIES lpfnWE_GetModuleDependencies = NULL;
lpfnWE_GetModuleDependencies = (LPFNWE_GETMODULEDEPENDENCIES)GetProcAddress( InstDll, "WE_GetModuleDependencies" );
if ( lpfnWE_GetModuleDependencies != NULL )
{
TCsString* var_CsString;
lpfnWE_GetModuleDependencies( var_CsString );
int i = 0;
while ( (var_CsString[i] == NULL) == false )
{
Dependencies->Add( AnsiString( var_CsString[i].c_str() ) );
i++;
}
typedef __stdcall void(*LPFNWE_FREESTRINGS)( TCsString* );
LPFNWE_FREESTRINGS lpfnWE_FreeStrings = NULL;
lpfnWE_FreeStrings = (LPFNWE_FREESTRINGS)GetProcAddress( InstDll, "WE_FreeStrings" );
if ( lpfnWE_FreeStrings != NULL )
{
lpfnWE_FreeStrings( var_CsString );
}
}
ShowName = Name;
typedef const char*(*LPFNGETSHOWNAME)();
LPFNGETSHOWNAME lpfnGetShowName = NULL;
lpfnGetShowName = (LPFNGETSHOWNAME)GetProcAddress( InstDll, "GetShowName" );
if ( lpfnGetShowName != NULL )
{
ShowName = AnsiString( lpfnGetShowName() );
}
Version = 0;
typedef __stdcall unsigned long(*LPFNVERSION)();
LPFNVERSION lpfnVersion = NULL;
lpfnVersion = (LPFNVERSION)GetProcAddress( InstDll, "Version" );
if ( lpfnVersion != NULL )
{
Version = lpfnVersion();
}
Configure = 0;
typedef __stdcall int(*LPFNCONFIGURE)( HWND, TModuleConfigData* );
LPFNCONFIGURE lpfnConfigure = NULL;
lpfnConfigure = (LPFNCONFIGURE)GetProcAddress( InstDll, "Configure" );
if ( lpfnConfigure != NULL )
{
Configure = 1;
}
var_Ret = true;
}

[:aloy]

Mais bon, c'était bien le coup des __stdcall merci [:super chinois]

Par contre j'ai juste un p'tit problème avec une fonction qui retourne un const char*, ça compile pas avec le __stdcall. J'peux le virer sans pb? [:ocube] (le mauvais [:god] )

edit: Si j'met le __stdcall après le type (int, const char*, etc...), ça compile. D'ailleurs c'est comme ça dans les dlls, ça marche aussi ? :??:

Message cité 1 fois
Message édité par godbout le 03-11-2005 à 13:57:38

godbout

Génial.

chrisbk a écrit :

c'est super verbeux ton truc

C'est pour que ce soit plus lisible pour quand je patirai [:cupra]

Message cité 1 fois

Publicité

Page : 1 2 3

Page Suivante

Page Précédente

Haut de page

FORUM HardWare.fr

Programmation

C++

[C++] Classes, méthodes et DLLs

Sujets relatifs
Classes	[C++] Probleme avec la stl et les classes templates
[Résolu] Expressions régulières et classes	les classes Date et Timestamp
APPLET embarquant des classes natives	Classes, Vecteurs et Tri...
Design applicatif & methodes statiques	[C#] Problème d'instanciation de classes et usage de la class Debug
[Visual C++] Classes filles de CException	prob acces membres de classes pour fichier inclus en path absolu
Plus de sujets relatifs à : [C++] Classes, méthodes et DLLs

Page générée en 0.153 secondes