Discussion :

[Flow_75]

Bonjour,

Je vous expose mon probleme:
Je recois via liaison Série une trame de données 24bits venant d'un convertisseur.
Voulant exploiter ces données je voudrais les convertir en étendant le bit de signe.
Le probleme est que après avoir faire le programme, un collègue m'a dit que mes nombre sont pas corrects......(cohérents mais pas corrects).

Je vous montre mon bout de code. Je ne sais pas votre avis dessus (c'est d'ailleurs l'occasion de me le donner).

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
char message[200]; // buffer de reception
int ech;
char *pech;
 
pech = (char*)&ech;
 
/* Reception des echantillons */
 
*(pech+1)     = message[0];
*(pech+2)     = message[1];
*(pech+3)     = message[2];
 
if(*(pech+1) &0x80) *pech = 0xFF; // Extension bit de signe
else *pech = 0x00;

Je ne sais pas si c'est la bonne méthode.

Si quelqu'un a des suggestions.... Elles seront les bienvenues.

Merci

[JeitEmgie]

il faudrait d'abord que vous vérifiez si les nombres négatifs que vous recevez du convertisseur sont vraiment exprimés en complément à 2 ou à 1 ou si le bit de signe est indépendant du reste du nombre :

en 24 bits en complément à 2 -1 == 0xFFFFFF
mais en complément à 1, -1 = 0xFFFFFE
et avec bit indépendant -1 = 0x800001

en fonction de cette information vous pourrez écrire correctement le code…
et obtenir un nombre négatif exprimé correctement en complément à 2 du côté PC…

accessoirement, il faut aussi que vous vérifiez que "int" est bien un entier de 32 bits en fonction de vos options de compilation ou des valeurs par défaut sur votre environnement…
ce serait trop bête de perdre du temps pour une erreur à ce niveau…

(et pensez à utiliser "union" en C c'est fait pour çà…

Code :

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

1
2
3
4
union {
 int ivalue ; // if int is really a 32 bits integer…
 unsigned char bytes[4] ;
}

)

et aussi, vérifiez l'ordre dans lequel les bytes arrivent : du moins vers le plus signifiant ou l'inverse…

[Flow_75]

salut,

Merci de ta réponse.
Donc, d'apres ta réponse (et la datasheet) c'est un nombre complément à 2 (0xFFFFFF = -1).
Sur mon pc, j'avais eu le meme doute que toi, j'avais donc fait un sizeof(int) pour m'en assurer. Et c'est bien un 32bits (ou bien 4 octets).

[JeitEmgie]

salut,

Merci de ta réponse.
Donc, d'apres ta réponse (et la datasheet) c'est un nombre complément à 2 (0xFFFFFF = -1).
Sur mon pc, j'avais eu le meme doute que toi, j'avais donc fait un sizeof(int) pour m'en assurer. Et c'est bien un 32bits (ou bien 4 octets).

et j'oubliais évidemment :

il faut tester si votre architecture CPU côté PC est little endian ou big endian…
pour savoir dans quel ordre mettre les bytes dans l'array de l'union…

[Flow_75]

et j'oubliais évidemment :

il faut tester si votre architecture CPU côté PC est little endian ou big endian…
pour savoir dans quel ordre mettre les bytes dans l'array de l'union…

Bien vu l'aveugle.
J'ai mis les octets dans l'autre sens et ca a marché.

Merci de ton aide...> Résolu