passage de paramètres

**jackk** · 18/12/2006, 16h24

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** · 18/12/2006, 16h53

Salut,

Envoyé par jackk

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** · 18/12/2006, 17h43

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** · 18/12/2006, 17h52

Envoyé par jackk

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** · 18/12/2006, 18h14

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** · 18/12/2006, 18h18

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

**Laurent Gomila** · 18/12/2006, 19h54

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

Les pros du scanf ne sont clairement pas ici. Comme le problème ne semble pas lié au passage par référence et que tout le reste est du C, je déplace vers le forum approprié.

Mais à mon avis la réponse est la suivante : il n'y a pas de formatteur pour unsigned char (sous forme décimale), il va falloir utiliser un autre type

**DaZumba** · 18/12/2006, 22h09

Le specificateur de format hd signifie short int. Pour un unsigned int, c'est u. Cf. man scanf() [qui est une fonction dangeureuse à apprendre à des débutants].
Edit: oups, pas vu que tu utilises unsigned char. Dans ce cas, essaie le specificateur c -- mais sans garanties.

**Médinoc** · 18/12/2006, 23h17

Dans le monde merveilleux de scanf(), il me semble bien que "%c" signifie une chaîne

"%1c" devrait convenir.
Ou tout simplement getchar().

Je crois que scanf() n'offre aucun moyen de saisir directment un nombre en décimal dans un char. Il faut au minimum passer par un short.

gl · 19/12/2006, 19h46

Envoyé par Médinoc

Dans le monde merveilleux de scanf(), il me semble bien que "%c" signifie une chaîne

Non un caractere pas une chaine. Une chaine c'est %s

**Bayard** · 19/12/2006, 20h15

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

-> %d pour du décimal
-> %x pour de l'hexa

-> %hd, j'ai du mal à me rappeler ? Quelle est cette chose étrange ?

-> Pouvez-vous essayer %d ?

**Emmanuel Delahaye** · 19/12/2006, 20h34

Envoyé par jackk

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...

**DaZumba** · 19/12/2006, 20h42

Envoyé par gl

Non un caractere pas une chaine. Une chaine c'est %s

Disons que %c est utilisé pour une suite de caractère (espaces compris). %s est utilisé pour une chaine (espaces négligés par défauts, chaine terminée par '\0').

gl · 19/12/2006, 21h03

Envoyé par DaZumba

Disons que %c est utilisé pour une suite de caractère (espaces compris). %s est utilisé pour une chaine (espaces négligés par défauts, chaine terminée par '\0').

Non, %c c'est pour un caractere. Si on attends une suite de caractere, il faut le preciser.

Envoyé par n1124

Matches a sequence of characters of exactly the number specified by the field
width (1 if no field width is present in the directive).246)

**DaZumba** · 19/12/2006, 23h34

Envoyé par gl

Non, %c c'est pour un caractere. Si on attends une suite de caractere, il faut le preciser.

Ooky dooky -- merci pour la précision. A force de ne jamais utiliser scanf(), on finit par ne pas la connaître, cette fonction...

gl · 20/12/2006, 19h38

Envoyé par DaZumba

A force de ne jamais utiliser scanf(), on finit par ne pas la connaître, cette fonction...

Approche tres saine du probleme a laquelle j'adhere egalement

passage de paramètres

C

Discussions similaires

Partager

Partager