besoin d aide

**nemo69500** · 06/04/2006, 16h46

Pouvez vous me dire , svp , si vous voyer une erreur dans ce code?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
 
#include <math.h>
 
unsigned char bit_test(unsigned char,unsigned int);
unsigned char bit_set(unsigned char,unsigned int);
unsigned char bit_clear(unsigned char,unsigned int);
unsigned char generate_8bit_crc(unsigned char*,unsigned int,unsigned char);
 
 
unsigned char generate_8bit_crc(unsigned char* data,unsigned int length, unsigned char pattern)
{
   unsigned char   *current_data;
   unsigned char   crc_byte;
   unsigned int   byte_counter;
   unsigned int   bit_counter;
 
   current_data = data;
   crc_byte = *current_data++;
 
   for(byte_counter=0; byte_counter < (length-1); byte_counter++)
   {
      for(bit_counter=0; bit_counter < 8; bit_counter++)
      {
         if(bit_test(crc_byte,7)==0)
         {
            crc_byte <<= 1;
            if(bit_test(*current_data, 7 - bit_counter)==128)
            {crc_byte=bit_set(crc_byte,0);}
            else
            {crc_byte=bit_clear(crc_byte,0); }
            continue;
         }
         crc_byte <<= 1;
         if(bit_test(*current_data, 7 - bit_counter)==128)
         {
           crc_byte=bit_set(crc_byte,0);
         }
         else
         {
         crc_byte=bit_clear(crc_byte,0);
         }
      crc_byte ^= pattern;
      }
   current_data++;
   }
   for(bit_counter=0; bit_counter < 8; bit_counter++)
   {
      if(bit_test(crc_byte,7)==0)
      {
         crc_byte <<= 1;
         continue;
      }
      crc_byte <<= 1;
      crc_byte ^= pattern;
   }
   return crc_byte;
}
 
unsigned char bit_test(unsigned char X,unsigned int Y)
{
   unsigned char RES;
   unsigned char Z=pow(2,Y);
   RES=X & Z;
   return (RES);
}
 
unsigned char bit_set(unsigned char X,unsigned int Y)
{
 
   unsigned char Z=pow(2,Y);
   return (X | Z);
}
 
unsigned char bit_clear(unsigned char X,unsigned int Y)
{
   unsigned char Z=pow(2,Y);
   return (X & (~Z));
}

Merci d avance de votre aide

**barthelv** · 06/04/2006, 16h48

Tu veux pas plutot nous dire ce qui ne marche pas ?

**niglo** · 06/04/2006, 16h49

Lis les règles du forum avant de poster !!

**nemo69500** · 06/04/2006, 16h51

Je recoi une trame de 7 caractere + 2 de crc sur le port com de mon pc .

Je dois verifier le premier caractere pour le debut de la trame , mais le probleme se situ au niveau du CRC, je n arrive pas a le recalculer. J ai pri la meme fonction que pour l envoyer sauf que je ne l ai pas faite . C est un PIC qui m envoi la trame . Bref je n arrive pas a recalculer mon CRC

Voila le probleme

Pouvez vous m aider?

PS : dsl pour le code lacher en vrac mais je suis legerement sous pression a cause de ce code , ca fais depuis ce matin que je suis dessus , dsl encore une fois

**nemo69500** · 06/04/2006, 17h25

Personne ne peux m aider svp?

**Emmanuel Delahaye** · 06/04/2006, 17h40

Envoyé par nemo69500

Pouvez vous me dire , svp , si vous voyer une erreur dans ce code?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

   unsigned char Z=pow(2,Y);

Il est absurde d'utiliser pow() pour une puissance de 2. Il suffit de décaler un bit d'autant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

   unsigned char Z = 1u << Y;

Plus besoin de <math.h>
Pour le reste c'est pas facile. Tu as les specs, un exemple de donnée avant / après ?

Autre chose. Il est inutile de passer des paramètres de type 'char' à une fonction (idem pour le type retourné). Ca fait générer du code en plus pour rien du tout. Le type par défaut est int ou unsigned[ int]. Idem pour les variables, (sauf exception rare, comme scanf() avec "%c").

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
static unsigned bit_test(unsigned X, unsigned int Y)
{
   unsigned RES;
   unsigned Z = 1 << Y;
   RES = X & Z;
   return RES;
}

etc.

Et les majuscules, c'est traditionellement pour les constantes...

**PRomu@ld** · 06/04/2006, 17h41

Personne ne peux m aider svp?

Evite les ups ...

J ai pri la meme fonction que pour l envoyer sauf que je ne l ai pas faite

J'ai du mal à comprendre alors, si tu as pris la même fonction que pour le calcul du crc il ne devrait pas y avoir de problème ...

**nemo69500** · 06/04/2006, 18h39

Il est absurde d'utiliser pow() pour une puissance de 2. Il suffit de décaler un bit d'autant :

la puissance c est Y et pas deux

et non ca ne marche pas meme si j ai pris le meme code , c est pour cela que je demande si y a pas une erreur

**PRomu@ld** · 06/04/2006, 18h48

la puissance c est Y et pas deux

Emmanuel disait que c'est 2 à la puissance Y, c'est donc bien une puissance de deux. Ou alors ton erreur vient de là ...

**nemo69500** · 06/04/2006, 18h50

c possible je regarde et je vous dis

**nemo69500** · 06/04/2006, 18h59

fonction pow(base,exposant)

voici une citation de la faq c++ donc non je me suis pas tromper c est bien 2^Y

**PRomu@ld** · 06/04/2006, 19h19

J'ai peur que nous n'ayons pas le même vocabulaire que toi ...

Pour moi une puissance de 2 c'est 2^X ...

Ou alors tu n'as pas compris ce qu'Emmanuel voulait te dire :

En binaire, lorsque tu décalles sur la gauche (en faisant abstraction des débordements), tu multiplie par 2.

0010 -> 2 (10)
0100 -> 4 (10)
...

Pour calculer une puissance de deux du type 2^Y, il faut décaller Y fois sur la gauche d'où le :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
unsigned char Z = 1u << Y;

**nemo69500** · 06/04/2006, 19h49

excuser moi j avais mal compris , en attendant j ai toujours un probleme

**Gnux** · 06/04/2006, 20h25

Envoyé par nemo69500

excuser moi j avais mal compris , en attendant j ai toujours un probleme

Ravi de le savoir mais si tu pouvais juste un peu préciser quel était ce problème dans le code ça serait pas mal et cela te permettrait d'éditer ton message pour y mettre un titre explicite

"Besoin d'aide" "problème/pb/pbl en C" "ça beug/bog/buggue" "pourquoi ça marche pas?" ne sont pas des titres

PS : dsl pour le code lacher en vrac mais je suis legerement sous pression a cause de ce code , ca fais depuis ce matin que je suis dessus , dsl encore une fois

Moi c'est précisément ce qui me met sous pression si l'on peut dire
MErci d'éviter ça à l'avenir

**nemo69500** · 07/04/2006, 07h23

Ravi de le savoir mais si tu pouvais juste un peu préciser quel était ce problème dans le code ça serait pas mal et cela te permettrait d'éditer ton message pour y mettre un titre explicite

Je recoi une trame de 7 caractere + 2 de crc sur le port com de mon pc .

Je dois verifier le premier caractere pour le debut de la trame , mais le probleme se situ au niveau du CRC, je n arrive pas a le recalculer. J ai pri la meme fonction que pour l envoyer sauf que je ne l ai pas faite . C est un PIC qui m envoi la trame . Bref je n arrive pas a recalculer mon CRC

**PRomu@ld** · 07/04/2006, 07h51

Bref je n arrive pas a recalculer mon CRC

As tu le bon algorithme au moins ?

**Emmanuel Delahaye** · 07/04/2006, 08h07

Envoyé par nemo69500

mais le probleme se situ au niveau du CRC, je n arrive pas a le recalculer.<...> Bref je n arrive pas a recalculer mon CRC

Ca veux dire quoi exactement 'j'arrive pas à recalculer ? Quel est le problème constaté ?

**nemo69500** · 07/04/2006, 08h22

Cela signifie que je n arrive pas a avoir le meme crc en ayant la meme trame d octet donc je ne peu pas verifier si la transmission a ete corect

**Emmanuel Delahaye** · 07/04/2006, 09h37

Envoyé par nemo69500

Cela signifie que je n arrive pas a avoir le meme crc en ayant la meme trame d octet donc je ne peu pas verifier si la transmission a ete corect

Quel est l'algo qui a servi a généré le crc ? Est-il portable (indépendant de la machine).

**nemo69500** · 07/04/2006, 09h42

La fonction qui a generer le crc est la fonction "generate_crc_8" qui est dans le premier message dans la partie code

besoin d aide

C

Vue hybride

Discussions similaires

Partager

Partager