souci de transtypage

Version imprimable

Voir 40 message(s) de cette discussion en une page

13/09/2007, 17h29
simoryl

ok ok, mais comment je fais pour avoir la valeur qui est pointée et non pas son adresse mémoire?
Frappez pas trop fort :aie: je commence à tout mélanger...
Quand j'aurai compris le truc et que mon proto sera ok je pense que ça ira déjà mieux (j'espère...)

Ben, comme pour tout pointeur, quand tu veux accéder à la valeur et non à l'adresse à laquelle elle se trouve
Code:

1 2 3 4 5 6 7 8 9 10 11 12 /* transformer un entier en un pointeur sur entier */ int entier; int *ptrent; ptrent=&entier; /* récupérer un entier au départ d'un pointeur sur entier int recup; recup=*ptrent; /* particularité pour les tableaux */ int tab[10]; ptrent=tab; recup=tab[3];
[EDIT]Ceci dit, ce genre de question nous inciterait presque à te conseiller de réviser ou de relire très sérieusement ton cours ou ton livre à la partie "pointeurs et tableaux" ;):D

13/09/2007, 18h12
simoryl

aaaaahhhhhhh
Ce que j'avais fait été donc bon... Mes souvenirs sont pas encore trop pourri...
Arf Pas de bol, maintenant, j'ai Firefox qui explose encore quand j'arrive au niveau de mon transtypage... De quoi se pendre...
Je vais aller bouffer, ça ira pitète mieux après....

:cry: J'ai trop l'impression de toucher le fond là, y a pas plus pire que moi...
J'ai fait ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
NS_IMETHODIMP CSpecialThing::Add(PRInt32 a, PRInt32 b, PRInt32 *_retval)
{
	PRInt32 test;
	typedef char* (_stdcall *ProcDllPtr)(char* monString);
	char* reponseDLL;
	HMODULE hinstDLL=LoadLibraryA((LPCSTR)"C:\\DLMDLL.dll");
	if (hinstDLL) {
		ProcDllPtr ProcExportDll = (ProcDllPtr) GetProcAddress(hinstDLL, "Ajouter_article");
		reponseDLL=ProcExportDll((char*)("affiche moi"));
	} else {
		//traitement erreur
	}
	FreeLibrary((HINSTANCE)hinstDLL);
 
	PRInt32  * testa = reinterpret_cast<PRInt32 *>(reponseDLL);
	test=*testa;
 
	*_retval = test;
	return NS_OK;
}

ça compile nickel mais mon Firefox plante juste au niveau du transtypage... Je sent que j'ai encore du faire une connerie...

Citation:
Envoyé par simoryl
:cry: J'ai trop l'impression de toucher le fond là, y a pas plus pire que moi...
J'ai fait ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 NS_IMETHODIMP CSpecialThing::Add(PRInt32 a, PRInt32 b, PRInt32 *_retval) { PRInt32 test; typedef char* (_stdcall *ProcDllPtr)(char* monString); char* reponseDLL; HMODULE hinstDLL=LoadLibraryA((LPCSTR)"C:\\DLMDLL.dll"); if (hinstDLL) { ProcDllPtr ProcExportDll = (ProcDllPtr) GetProcAddress(hinstDLL, "Ajouter_article"); reponseDLL=ProcExportDll((char*)("affiche moi")); } else { //traitement erreur } FreeLibrary((HINSTANCE)hinstDLL); PRInt32 * testa = reinterpret_cast<PRInt32 *>(reponseDLL); test=*testa; *_retval = test; return NS_OK; }
ça compile nickel mais mon Firefox plante juste au niveau du transtypage... Je sent que j'ai encore du faire une connerie...
je pense que tu libère ta dll trop tôt. il y as de forte chance que lorsque tu la libère, la memoire pointer par reponseDLL soit libéré, d'ou ton crash.
essaie cela
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 NS_IMETHODIMP CSpecialThing::Add(PRInt32 a, PRInt32 b, PRInt32 *_retval) { PRInt32 test; typedef char* (_stdcall *ProcDllPtr)(char* monString); char* reponseDLL; HMODULE hinstDLL=LoadLibraryA((LPCSTR)"C:\\DLMDLL.dll"); if (hinstDLL) { ProcDllPtr ProcExportDll = (ProcDllPtr) GetProcAddress(hinstDLL, "Ajouter_article"); reponseDLL=ProcExportDll((char*)("affiche moi")); } else { //traitement erreur } PRInt32 * testa = reinterpret_cast<PRInt32 *>(reponseDLL); test=*testa; *_retval = test; FreeLibrary((HINSTANCE)hinstDLL); return NS_OK; }
Le mieux serait un class qui interface ta dll

Déjà, on pourrait commencer par te suggérer d'adopter une politique d'indentation rigoureuse... N'oublie jamais qu'un code est beaucoup plus souvent lu qu'il n'est écrit, et que, s'il te semble limpide aujourd'hui, il peut te sembler bien plus nébuleux dans une semaine ou dans trois mois :P

Ainsi, et sans avoir apporté la moindre correction, ton code serait bien plus lisible sous un forme proche de
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 NS_IMETHODIMP CSpecialThing::Add(PRInt32 a, PRInt32 b, PRInt32 *_retval) { PRInt32 test; typedef char* (_stdcall *ProcDllPtr)(char* monString); char* reponseDLL; HMODULE hinstDLL=LoadLibraryA((LPCSTR)"C:\\DLMDLL.dll"); if (hinstDLL) { ProcDllPtr ProcExportDll = (ProcDllPtr) GetProcAddress(hinstDLL, "Ajouter_article"); reponseDLL=ProcExportDll((char*)("affiche moi")); } else { //traitement erreur } FreeLibrary((HINSTANCE)hinstDLL); PRInt32 * testa = reinterpret_cast<PRInt32 *>(reponseDLL); test=*testa; *_retval = test; return NS_OK; }
(préfère l'indentation sous forme d'espaces (4, c'est pas mal :D) à celle sous forme de tabulation, et veille autant que possible à respecter une regle simple: une ligne= une instruction ;))

Maintenant, les remarques qui font mal:
Méfie toi des typedef à l'intérieur des fonctions... ils ne sont en effet disponible qu'à l'intérieur de cette fonction, et on peut penser que, si tu les trouve intéressants dans une fonction, tu leur trouvera sans doute une utilité quelconque ailleurs ;)

Tu utilises beaucoup trop le cast "C style" à mon goût: les instructions

LoadLibraryA((LPCSTR)"C:\\DLMDLL.dll");
(ProcDllPtr) GetProcAddress(hinstDLL, "Ajouter_article");
ProcExportDll((char*)("affiche moi"));
FreeLibrary((HINSTANCE)hinstDLL);

sont autant d'instructions dans lesquelles tu utilise un cast "C style" alors que tu pourrais t'en passer assez facilement

Sauf erreur, LPCSTR n'est qu'un alias de const char*, ce qui te permettrait de travailler sous une forme plus générique de
Code:

1 2 3 std::string str="c:\\DLMDLL.dll"); /* pourrait très bien être un argument ;) */ LoadLibraryA(str.c_str());
La même remarque peut s'appliquer à ProcExportDll((char*)("affiche moi"));

Pour les deux autres (ProcDllPtr et HINSTANCE), tu devrais peut être préférer, au pire le reinterpret_cast, au mieux (si il fonctionne) le static_cast...

Cette suggestion a pour seul but de te permettre de t'y retrouver plus facilement dans le code ;)

En effet, en effectuant une recherche sur "(HINSTANCE)", tu peux tomber, au choix,

sur une fonction qui reçoit HINSTANCE en paramètre
sur un transtypage de quelque chose en HINSTANCE

Par contre, si tu travailles à coup de static_/reinterpret_/const_/dynamic_ cast, et en utilisant au besoin les expressions régulières, tu sera certain, lors d'une recherche sur "_cast<HINSTANCE>" ou sur "_cast<un type donné> (HINSTANCE)" que les seuls résultats obtenus seront... des transtypages de quelque chose en HINSTANCE (ou de HINSTANCE en autre chose)

14/09/2007, 11h59
simoryl

hèhè, oui en effet, mon indentation est partie en live... A force de commenter, décommenter, ajouter, supprimer... Faut que je remette ça au propre :lol:

J'ai commencé le vc++ il y a à peine 1 semaine, je suis pas encore à fond dedans, tes éclaircissements sur les methodes de transtypage sont très intéréssantes et plus digeste que la plupart des tutaux que j'ai lu (c'est quand on a les pieds dedans qu'on comprend mieux généralement).

Sinon, en effet, je libérais ma dll trop tôt, maintenant ça crash plus mais je récupère toujours un chiffre qui ne correspond pas à celui envoyé par ma dll qui est 666 (j'aurai pas du choisir celui-là mdr). Je pense que je récupère encore un pointer au lieu de la valeur...
Reste plus qu'à me sacrer roi des boulets si ça continue! Mais ce qui me rassure c'est que je comprend petit à petit et que je commence à imprimer le vc++ dans mon cerveau
Faut pas le dire mais je suis blond lol...
14/09/2007, 12h09
koala01

Ceci dit, ne confond pas...:

VC++ est un outil ( pour être précis: un EDI (Environnement de Développement Intégré) voir un RAD (Rapid Application Development) pour les versions "pro") et est l'abréviation de Visual C++ (Visual faisant référence à la suite d'outils Visual Studio de microsoft)

Le langage, c'est C++, et il n'y a pas de V dedans ;):mouarf:
14/09/2007, 13h59
simoryl

c'est un abus de langage de ma part.
Avec Delphi, j'ai pas ce souci là lol... Et j'ai pas non plus de souci de pointeur comme en ce moment avec ce fichu PRInt32 à la :evilred:
Je deteste rester coincer sur 2 lignes de code pendant des jours... ça me donne l'impression d'être plus bête que je ne le suis déjà mdr
14/09/2007, 14h18
yan

juste pour info, tu recupere quoi comme valeur au lieu de 666 ??
14/09/2007, 14h43
simoryl

Je récupère :3552822. C'est fun, c'est toujours le même contrairement à une époque... Je parie que je récupère le bon chiffre mais pas au bon format...
14/09/2007, 14h46
Médinoc

En hexa, ça donne 0x363636.
Ce qui en little-endian (notamment sur un processeur x86) donne les octets:

Code:

{ 0x36, 0x36, 0x36, 0}

Ce qui équivaut à :

Code:

{ '6', '6', '6', '\0'}

---> Donc, il est possible que tu aies transformé ton 666 en chaîne de caractères...
14/09/2007, 14h48
yan

Citation:

Envoyé par Médinoc

En hexa, ça donne 0x363636.
Ce qui en little-endian (notamment sur un processeur x86) donne les octets:

Code:

{ 0x36, 0x36, 0x36, 0}

Ce qui équivaut à :

Code:

{ '6', '6', '6', '\0'}

---> Donc, il est possible que tu aies transformé ton 666 en chaîne de caractères...

:king: comme quoi ma question n'était pas si bête.
Mais je pensais plustôt à un problème endian ou de décalage
14/09/2007, 14h50
Médinoc

Réessaie avec 42, on verra bien si tu obtiens 0x**003234...
14/09/2007, 15h04
simoryl

en effet si je remplace mon 666 par 42 j'obtient bien 3234
:cry: J'ai l'impression que chaque problème résolu en créé un autre...
Bon, il me reste plus qu'à voir comment transformer mon hexa en decimal
14/09/2007, 15h14
Laurent Gomila

J'ai pas tout suivi, mais là tu récupères de ta fonction un char* (donc "666") et tu voudrais récupérer l'entier 666 ? C'est ça ?

Si oui, alors pourquoi être parti sur un reinterpret_cast ? Ce qu'il te faut c'est une conversion chaîne --> entier, et c'est très bien expliqué dans la FAQ.
14/09/2007, 15h18
yan

Citation:

Envoyé par Médinoc

Réessaie avec 42, on verra bien si tu obtiens 0x**003234...

ou

Code:

std::string reponseDLL=ProcExportDll((char*)("affiche moi"));

si c'est bien ça, reponseDLL aurai ton nombre sous forme de string.
14/09/2007, 15h50
simoryl
YAHOOOOOOOOOOO!!!!!!!!!!!!!!!

:yaisse2:

ça marche!!!!!

:yaisse2:

C'est la fête, Merci à tous!!!!

Vous êtes des :king:

Pour ceux que ça intérésse, voici mon code (pas très beau mais qui marche!) :
Code:

1 2 3 4 5 std::istringstream iss(reponseDLL); int nombre; iss >> nombre; test=nombre;
Comme d'hab', c'est un truc à la c... sur lequel on passe des heures...
Pas grave, ça MARCHE!!!!!!

:yaisse2:

MERCI, 1000 MERCI

:yaisse2:
14/09/2007, 15h53
yan

Citation:

Envoyé par simoryl

c'est un abus de langage de ma part.
Avec Delphi, j'ai pas ce souci là lol... Et j'ai pas non plus de souci de pointeur comme en ce moment avec ce fichu PRInt32 à la :evilred:

Si ce que tu récupère est bien une string, je ne voit pas comment cela pouvait marcher avec delphi
14/09/2007, 17h27
simoryl

c'était par rapport à la remarque c++ et vc++, pour Delphi le langage c'est Delphi et l'IDE c'est Delphi aussi lol

Voir 40 message(s) de cette discussion en une page