Discussion :

[jackk]

Bonjour,

je travaille avec Borland C++ 5.0 et j'ai un petit problème d'effet de bord du apparemment à un passage de paramètre.

Je veux travailler sur des bytes et modifier et tester des bits.
Je joins un bout de programme qui met en évidence mon problème.
La fonction bclr fait un "bit clear" du bit N°n de octet.

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
#include <stdio.h>
#define byte unsigned char
 
void bclr(byte n, byte &octet);
 
void main(void) {
byte data = 255;
byte num;
 
   printf("\n data = %d",data);
   bclr(5,data);
   scanf("%hd",&num);
   printf("\n data = %d",data);
}
 
void bclr(byte n, byte &octet) {
    octet = octet & (~(0x01 << n));
}

Le problème: avant le scanf, data vaut bien 0xdf et au retour, si je rentre 7 pour num, data vaut maintenant 0 et num = 7

Si maintenant je rentre 260 pour num, num vaut 4 et data vaut 1. sachant que 260 = 0x104, il apparait que bien que je manipule des unsigned char, le scan remonte une valeur sur 16 bits qui écrase donc data.

Si je ne fais pas l'appel de la fonction bclr, tout fonctionne bien.

Pouvez-vous m'expliquer ou je me suis planté?

Merci

[Feriaman]

Salut,

Pouvez-vous m'expliquer ou je me suis planté?

Non, personnellement, je n'y connais pas grand chose à printf et scanf.

Pourquoi n'utilises-tu pas les opérateurs de flux de la std : cin et cout ?


[EDIT]Après une recherche rapide dans la doc de scanf, je m'apperçois que %d ne doit pas être utilisé pour les données larges de 1 byte, c'est donc normal que cela déborde.

Utilise plutot %c ou %C.

Note : tu n'aurais pas eu le problème avec les opérateurs de flux de la stl dont les paramètres sont typés.
[/EDIT]

[jackk]

merci pour ta réponse

Pourquoi n'utilises-tu pas les opérateurs de flux de la std : cin et cout ?

Parce que j'utilise le C avec des élèves en électronique et que la programmation objet sort un peu de la formation.

Après une recherche rapide dans la doc de scanf, je m'apperçois que %d ne doit pas être utilisé pour les données larges de 1 byte, c'est donc normal que cela déborde.

je pensais que le "scanf("%hd",&num);" résolvait le problème (le hd devant normalement traiter des short int).

Et pourquoi, le scanf fonctionne-t-il sans l'appel de bclr?

A+

[Feriaman]

Parce que j'utilise le C avec des élèves en électronique et que la programmation objet sort un peu de la formation.

Dans ce cas, tu trouveras de meilleurs conseils sur le forum C.
Je ne m'y connais franchement pas en scanf.

Bonne chance.

[jackk]

Peut-être en effet.

J'ai posté en C++ car j'utilise une syntaxe de passage de paramètre par adresse qui n'existe pas en C.

Avec la syntaxe classique, le problème est le même.

Si je n'obtiens pas de réponse, je posterai dans le forum C.

A moi qu'un modérateur ne passe par là et déplace ce fil dans le forum C car je suppose qu'il n'est pas très bien vu de poster la même chose dans 2 forums.

A+

[Mat.M]

s'il ya un problème avec scanf c'est que le format n'est pas bon...
c'est certain qu'avec cin c'est moins se casser la tête

[Emmanuel Delahaye]

Code :

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

void bclr(byte n, byte &octet);

Heu, si tu fais du C++, tu t'es trompé de forum...

En supposant que tu fais du C, ton code commenté et corrigé :

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
 
#include <stdio.h>
 
/* -ed- ajoute pour controles... */
#include <assert.h>
 
/* -ed-
#define byte unsigned char
 
Preferer le style 'typedef'. En fait, probablement inutile...
*/
 
/* -ed-
Il n'est pas utile de separer le prototype
       si la fonction est dans la meme unite de compilation.
       Il suffit de respecter le principe 'definir avant d'utiliser'.
 
void bclr(byte n, byte &octet);
*/
 
/* -ed- Deplace...
void bclr(byte n, byte *octet) {
    octet = octet & (~(0x01 << n));
}
 
byte etant un type char, cette fonction gerere du code unutilement
complexe. En effet, en C, les parametres de type char sont automatiquement
convertis en int. Il est donc inutile de perdre son temps a utiliser char,
car ca ne fait qu'ajouter du code inutile.
 
De plus, il serait bien plus rentable d'utiliser le retour de la fonction...
 
*/
unsigned bclr (unsigned octet, unsigned n)
{
   /* -ed- simple precaution. La methode est barbare mais efficace... */
   assert (n < 8);
   return octet & (~(0x01u << n));
}
 
/* -ed-
void main(void)
 
main() retourne int. Toujours...
*/
int main (void)
{
   unsigned data = 255;
   unsigned num;
 
/* -ed- un affichage en hexadecimal est plus clair quand on manipule
des bits...
 
La place du '\n' est en fon de ligne, pas en debut...
*/
   printf ("data = %02X\n", data);
 
/* -ed- la logique de ce code est absurde...
   bclr(5,data);
   scanf("%hd",&num);
   printf("\n data = %d",data);
*/
 
   scanf ("%u", &num);
   /* -ed- scanf () est une fonction difficile a utiliser correctement.
      Il faudrait traiter les cas d'erreur...
 
      Le C, n'est pas un langage de debutant. Trop de subtilites...
    */
 
/* -ed- il faudrait s'assurer qu'on ne va pas decaler de plus 7 ... */
   data = bclr (data, num);
   printf ("data = %02X\n", data);
 
   return 0;
}

Code :

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

1
2
3
4
5
6
 
data = FF
5
data = DF
 
Press ENTER to continue.

Tu devrais méditer là-dessus :

http://emmanuel-delahaye.developpez..../ed/inc/bits.h

Evidemment, c'est plutôt une approche industrielle...