APPAREA BUTTON XP OCX (VE2.0)

Commentaire de copyrightx le 15/11/2005 15:21:55

Il n'y a eu qu'une seule source "button" postée jusqu'à ce jour ...

Copyrightx

Commentaire de copyrightx le 16/11/2005 10:07:50

Que penses-tu alors de cette version ? Quelles améliorations proposerais-tu ?

Copyrightx

Commentaire de copyrightx le 17/11/2005 09:55:55

Hey ! Qu'est-ce qu'il ne vous plaît pas dans la source ? Remontez-moi cette note !!!!!

Copyrightx

Commentaire de DARKSIDIOUS le 17/11/2005 19:50:27 administrateur CS

Pas mal de remarques (qui j'espère te fera comprendre certains concepts clés de la programmation d'ocx) :
1/ Ca ne sert à rien de poster une version 1, puis une version 2 sur le site, mets à jour la version 1 directement, avec une compatibilité binaire. Pourquoi ? Tout simplement parce que ca va à l'encontre de la logique de compatibilité des ocx : tu compile un ocx, tu l'utilise dans ton projet, et vient une version 2 qui écrase l'ancienne version (compilé sans compatibilité, comme le veux les options de ton projet), et du coup, ca fait planter tout ton prog !
2/ Quand je vois autant de propriété, autant de variable, j'ai qu'une envie : ne pas utiliser ton ocx ! Il faudrait réorganiser tout ca en classe, avec une conception objet, on y gagne en facilité d'utilisation, mais aussi en facilité de programmation ! Quand je vois :
    PropBag.WriteProperty "ColorOutStart", OutS: PropBag.WriteProperty "ColorOverStart", OverS
    PropBag.WriteProperty "ColorDownStart", DownS: PropBag.WriteProperty "ColorUpStart", UpS
    PropBag.WriteProperty "ColorFocusStart", FocusS: PropBag.WriteProperty "ColorEnabledStart", EnabledS
    PropBag.WriteProperty "ColorOutEnd", OutE: PropBag.WriteProperty "ColorOverEnd", OverE
    PropBag.WriteProperty "ColorDownEnd", DownE: PropBag.WriteProperty "ColorUpEnd", UpE
    PropBag.WriteProperty "ColorFocusEnd", FocusE: PropBag.WriteProperty "ColorEnabledEnd", EnabledE
    PropBag.WriteProperty "ForeColorOut", ForeOut: PropBag.WriteProperty "ForeColorOver", ForeOver
    PropBag.WriteProperty "ForeColorDown", ForeDown: PropBag.WriteProperty "ForeColorUp", ForeUp
    PropBag.WriteProperty "ForeColorFocus", ForeFocus: PropBag.WriteProperty "ForeColorEnabled", ForeEnabled
il vaudrait mieux regrouper toutes les couleurs dans un seul objet !
3/ Grosse erreur : mettre un message box dans une propriété ! Il ne faut JAMAIS faire cà : imagine, un utilisateur utilise ton contrôle, le mets dans un programme qu'il compile sans trop le tester, le revend, et là, l'utilisateur tombe sur un message :
"Entrez un nombre positif et différent de 0 pour la vitesse du Fading. Plus le nombre est grand, plus il y a de couleurs affichées sur la caption et plus la vitesse est lente.", que veux-tu que l'utilisateur y comprenne ?
Il vaut mieux ne pas mettre de message du tout ! ou à la rigueur mettre ce message dans la description de la propriété et non comme messagebox !
4/ Tes fonctions DrawFocus, DrawDown, etc. sont incompréhensible et très difficile à maintenir, tu aurais mieux fait de faire une seule fonction Draw avec quelques paramètres qui simule tes fonctions DrawFocus, DrawDown, etc. ! Ainsi, en cas de bug, tu n'as plus qu'à changer dans une seule fonction et non dans 4 avec tout les soucis que cela peut engendrer !
5/ Pas de description pour les propriétés. Lorsqu'un contrôle possède autant de propriétés, ca devient très difficile de comprendre quelle propriété fait quoi !
6/ Pas de pages de propriétés, ce qui aurait été bien pratique pour justement regrouper les propriétés entre elles !
7/ Pas de propriété par défaut (exemple : la propriété caption aurait très bien pu être définie comme propriété par défaut, ca aurait été bien pratique pour l'utilisateur).
8/ Aucune propriété rattaché à un ID standard de VB, ce qui est quand même bien pratique !
9/ A quoi sert ta feuille dans ton projet du contrôle ??? Ca ne fait qu'alourdir le contrôle, il aurait plutôt fallu faire un groupe de projet avec un projet de test et le projet du contrôle !
10/ Ton contrôle ne se redimensionne pas : c'est quand même bien pratique (surtout pour un bouton) de donner la taille qu'on veut à un contrôle !

Bon je m'arrête là sinon j'y passerai des heures.

Tout ca pour te montrer que ton code est encore largement perfectible, et surtout, que tu n'utilise pas tout les avantages des ocx, ce qui est bien dommage car ca rend ton contrôle très difficile à utiliser (ou du moins, beaucoup moins facile à utiliser que les contrôles standard).

DarK Sidious

Commentaire de DARKSIDIOUS le 21/11/2005 15:33:36 administrateur CS

Tes question 2 et 3 sont regroupées en une seule : en mettant les propriété commune (exemple les couleurs) dans la même classe, cela te permet d'avoir moins de propriétés, un ordonnancement plus logique de celles-ci, et surtout, la possibilité de copier plusieurs propriétés en même temps. Surtout que c'est très simple à faire :
Tu déclare tes variables en private dans ton module de classe, puis tu fait les propriétés Get/Let toujours dans ton module de classe, et enfin, tu fait une propriété Get/Set dans ton usercontrol pour accèder à une instance de ta classe.

7, 9, 10 : voir le menu outils/Attributs de procédures.

12 : il ne se redimensionne pas (l'image du bouton) en mode création alors !

DarK Sidious

Commentaire de kerriencpp le 24/03/2006 15:05:10

Dommage que l'on ne puisse pas appliquer un Maskcolor sur l'image associé au boutton. Sinon l'ocx est excellent, je vais l'utiliser dans les applis!