CHECKSUM - CALCUL DU CRC-16 OU 32

Signaler
Messages postés
23
Date d'inscription
samedi 22 mars 2003
Statut
Membre
Dernière intervention
21 août 2003
-
Messages postés
1154
Date d'inscription
samedi 14 août 2004
Statut
Membre
Dernière intervention
5 avril 2012
-
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/12617-checksum-calcul-du-crc-16-ou-32

Messages postés
1154
Date d'inscription
samedi 14 août 2004
Statut
Membre
Dernière intervention
5 avril 2012
2
Deux remarques :

1 - Des commentaires en anglais : Reprendre un programme d'un autre langage, pourquoi pas (quoi-que), mais alors autant aller au bout de la démarche et traduire également les commentaires et voir de les étoffer ( "The following is a little cryptic" Un commentaire comme celui là suppose une non compréhension du code qui suit... Donc si le but était d'aligner des lignes de code, le challenge est gagné ;)

2 - Des tableaux de constantes : Ok, ça marche !!! cependant il aurait été préférable de les créer dynamiquement, ce qui aurait permis d'expliquer (rappelons que le niveau est débutant) comment on est arrivé à ces valeurs, mais également d'alléger le code.

N.B. Ces remarques se veulent constructives et non destructives... À bon entendeur !

Cordialement.
Messages postés
23
Date d'inscription
samedi 22 mars 2003
Statut
Membre
Dernière intervention
21 août 2003

hum je n'ai pas vraiment lu toute la partie du calcul du crc, mais je pense que le Create de cette classe n'est pas nécéssaire, il suffirait de mettre les deux constantes de la procédure comme deux constantes de l'unitée.

Sinon, utiliser une classe d'objet pour simplement appeler une fonction n'est pas 'efficace'. En fait, il suffirait d'avoir les deux fonctions CRC16 et CRC32 en tant que fonctions de l'unitées, cela déchargerait le programme de toutes la gestion inhérentes à la classe TObject