K8063 COMMANDE D'UN DIGIT JUMBO, AFFICHAGE HEURE, DATE EN ALTERNÉ,...

Adn56 Messages postés 1172 Date d'inscription jeudi 24 mai 2007 Statut Membre Dernière intervention 28 septembre 2013 - 10 sept. 2009 à 10:02
Adn56 Messages postés 1172 Date d'inscription jeudi 24 mai 2007 Statut Membre Dernière intervention 28 septembre 2013 - 10 sept. 2009 à 10:02
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/50537-k8063-commande-d-un-digit-jumbo-affichage-heure-date-en-alterne

Adn56 Messages postés 1172 Date d'inscription jeudi 24 mai 2007 Statut Membre Dernière intervention 28 septembre 2013 1
10 sept. 2009 à 10:02
Salut, tout d'abord pourquoi un niveau initié ?
je trouve ton code trop lourd à lire, regarde par exemple les "If Check3.Value 1 And zz 3 Then" tu aurais pu faire une fonction à l'appeler en lui passant la valeur de zz ( ou faire un case au moins), regarde aussi le nombre de fos ou tu écris "169.checksum = 256 - (13 + addr + 66 + 2) Mod 256
170.commandstring = Chr$(13) & Chr$(addr) & Chr$(66) & Chr$(2) & Chr$(checksum)
171.Comm.Output = commandstring" ceux la aussi mérite une ptie fonction.
Pour ce qui est de la VM110 (K8055) tu n'utilisse pas la derniére DLL, la nouvelle version inclus la recherche automatique des cartes, l'affichage de la version et surtout la gestion multicarte.
Regarde aussi la form K8055, tu redéclare les appels à la dll !
Fait un seul appel en public, au lieu de le faire plusieurs fois en private.
Bon j'arrêtte là, mais ya plein de chose qui font que le codage me parait pas être un bon exemple pour les autres. Je connais trés bien les cartes velleman si tu as besoin de conseil demande par MP. POur le codage regarde d'autre source pour voir comment factoriser ton code, il en sera bien plus lisible.
Rejoignez-nous