Sujet Précédent Sujet Suivant

MINI SPY (WIN32)

vecchio56 Messages postés 6535 Date d'inscription lundi 16 décembre 2002 Statut Membre Dernière intervention 22 août 2010 - 15 avril 2005 à 19:58
racpp Messages postés 1909 Date d'inscription vendredi 18 juin 2004 Statut Modérateur Dernière intervention 14 novembre 2014 - 16 avril 2005 à 18:23

Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/30766-mini-spy-win32

racpp Messages postés 1909 Date d'inscription vendredi 18 juin 2004 Statut Modérateur Dernière intervention 14 novembre 2014 17
16 avril 2005 à 18:23

Urgo > Désolé, j'ai posté mon précédent commentaire, adressé à vicchio56, avant de rafraichir la page. Je n'avais pas vu ton message.

racpp Messages postés 1909 Date d'inscription vendredi 18 juin 2004 Statut Modérateur Dernière intervention 14 novembre 2014 17
16 avril 2005 à 18:12

Oui, c'est vrai et c'est préférable dans certains cas. Mais ce n'est pas évident car la nature de l'assembleur fait qu'il est plus proche de la machine que de nous. Imaginons un exemple simple:
mov eax,10 ;mettre 10 dans le registre eax
Si on fait un commentaire pour chaque instruction, ça ne fera qu'augmenter encore la confusion. On ne peut faire des commentaires que pour marquer un bloc d'instructions. Donc ce bloc reste tres peu lisible.

cs_Urgo Messages postés 780 Date d'inscription lundi 16 décembre 2002 Statut Membre Dernière intervention 16 avril 2009 1
16 avril 2005 à 18:00

Merci à vous pour les commentaires (et les notes), la source a été mise à jour.

J'ai choisi ici l'anglais pour l'interface de l'application, mais d'habitude je préfère rester à la langue de Molière plutôt que celle de Shakespeare.
Effectivement c'est pas commenté, mais je vais essayer d'en mettre quelques uns d'ici la prochaine mise à jour.

Ce que je cherchais en utilisant la version de BruNews de itoa en ASM (bnitoa) est avant tout la rapidité (et aussi le retour du ptr sur le null final, ce qui m'évite tout strcat inutiles).

Quant à l'idée évoquée par racpp de rajouter l'interception des messages Windows, je ne vois pas comment le faire d'une façon simple. Je préfère rester à mon MINI spy qui me suffit amplement pour l'instant.

Voilà, ciao...

vecchio56 Messages postés 6535 Date d'inscription lundi 16 décembre 2002 Statut Membre Dernière intervention 22 août 2010 14
16 avril 2005 à 17:04

D'accord avec toi, mais nous pouvons toujours commenter du code en assembleur

racpp Messages postés 1909 Date d'inscription vendredi 18 juin 2004 Statut Modérateur Dernière intervention 14 novembre 2014 17
16 avril 2005 à 17:00

vecchio56 > Je n'ai rien contre l'assembleur. Je l'ai utilisé pendant une quinzaine d'années. Le problème c'est que même si on précise le rôle de la fonction, on ne comprend pas ce qui se passe au sein de cette même fonction. L'ajout de commentaires ne suffit plus. Il m'est déjà arrivé de devoir lire et relire le code en asm de fonctions que j'avais fait il y'a quelque temps pour me rappeler enfin comment ça fonctionne. C'est un problème connu en assemleur.
Dans ce site, on ne partage pas seulement des programmes qui fonctionnent (il y'a des sites pour cela) mais aussi , et surtout, des idées, des astuces et des trouvailles en programmation C/C++. On partage la connaissance. Un code source clair et bien commenté a plus de valeur car il nous permet de tout comprendre vite sans avoir à le décortiquer.

vecchio56 Messages postés 6535 Date d'inscription lundi 16 décembre 2002 Statut Membre Dernière intervention 22 août 2010 14
16 avril 2005 à 11:25

Il y a juste bnitoa d'écrit en asm. Si on dit ce que fait cette fonction (le nom déja est assez explicite) et si on admet qu'elle fonctionne corrctement, je ne vois pas en quoi ca diminue la lisibilité

BruNews Messages postés 21040 Date d'inscription jeudi 23 janvier 2003 Statut Modérateur Dernière intervention 21 août 2019
16 avril 2005 à 11:24

Tu pourrais mettre la version __fastcall de bnitoa() qui gagne encore +- 8% sur la __stdcall, devrait se trouver dans mes sources sinon je te l'envoie.

racpp Messages postés 1909 Date d'inscription vendredi 18 juin 2004 Statut Modérateur Dernière intervention 14 novembre 2014 17
16 avril 2005 à 02:03

Salut,
Bravo pour le code, excellent travail.
J'ai quelques petites remarques:
1- Si tu ajoutes aussi l'interception des messages Windows, même de façon limitée, se serait parfait.
2- Il est préférable que l'interface de cet utilitaire soit en français. Pas besoin de tous mettre en anglais. Ca fait toujours plaisir de voir des programmes en francais.
3- Ce code n'est pas commenté. Il est très important dans un code source qu'il soit bien commenté, surtout dans les passages un peu obscures, afin d'aider à mieux comprendre.
4- Certains passages du code sont peu lisibles. L'usage de l'assembleur, par exemple, nuit beaucoup à la lisibilité du code. C'est pas possible de le remplacer par du code en C?

En attendant les prochaines mises à jour... 9,99/10 :)

cs_Urgo Messages postés 780 Date d'inscription lundi 16 décembre 2002 Statut Membre Dernière intervention 16 avril 2009 1
15 avril 2005 à 20:08

Oui, en effet, l'affichage du contour ne se voit pas toujours, je vais corriger ça et compléter les styles.

vecchio56 Messages postés 6535 Date d'inscription lundi 16 décembre 2002 Statut Membre Dernière intervention 22 août 2010 14
15 avril 2005 à 19:58

Je trouve que c'est pas mal, mais il y a quand même quelques petits problèmes au niveau de l'affichage du contour des fenêtres filles

Discussions similaires

utilisation de Spy++

spy++ et hook

Utilisation de SPY++

projet gestion d'un cabinet médical

Gestion d'un cabinet medicale

Votre réponse

Discussions similaires