Discussion :

[floopi51]

bonjour

j'ai récupéré du code où il y a des opérations de décalage sur un entier.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int a;
unsigned char *tptr;
int *imptr=tab;
 
for i=0; i<len; i++) {
        a=*tptr++;
        a=a|(a<<8)|(a<<16);
        *imptr++=a;
}

je voudrai savoir quelle opération de décalage je dois faire sur l'entier "a" après cette boucle pour retrouver sa valeur initiale ?

merci pour votre aide.

Floopi51

[ram-0000]

J'ai toujours d'énormes doutes en lisant ce genre d'instruction : *imptr++=a. A mon avis, cela mérite 2 lignes de code au lieu d'une pour aider le lecteur. Et c'est pareil pour a=*tptr++;.

Sinon, je dirai a = a >> 16 pour revenir à l'état l'initial

[floopi51]

non en fait ça fonctionne en faisant :

Code :

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

1
2
 
val=val&(val>>8)&(val>>16);

merci

[ram-0000]

Si tu regardes un peu le code,

Code :

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

val=val&(val>>8)&(val>>16);

est équivalent à

Code :

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

a = a >> 16

A la différence que il y en a un qui est plus facile à lire et à comprendre

[Médinoc]

En fait, il y a beaucoup plus simple: a = a & 0x000000FF.
Le code ne fait que répliquer l'octet lu sur les trois octets de poids faible du nombre, en clair il change 0x00000042 en 0x00424242.

Maintenant, avec un peu de refactoring et de programmation défensive :

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
size_t i;
unsigned char const *pcOctetsEntree = ...;
unsigned int *pEntiersSortie = tab;
 
for(i=0 ; i<len ; i++) {
	/*Restriction de portée: On déclare les variables dans la boucle.*/
	/*Plusieurs variables const sont plus claires qu'une variable qui change ainsi de valeur.*/
	/*Parenthèses "inutiles" ajoutées pour clarté.*/
	unsigned int const a = *(pcOctetsEntree++);
	unsigned int const b = a | (a<<8) | (a<<16);
	*(pEntiersSortie++) = b;
}