GCC et les entiers :]

Bonjour,

J'aimerais obtenir des précisions sur le comportement de ce programme compilé avec GCC :

Code:

---

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

#include <stdio.h> #include <math.h> #include <ctype.h>   short int ctoi(const int c) { if (c == '0') return 0; else if (c == '1') return 1; else if (c == '2') return 2; else if (c == '3') return 3; else if (c == '4') return 4; else if (c == '5') return 5; else if (c == '6') return 6; else if (c == '7') return 7; else if (c == '8') return 8; else if (c == '9') return 9; else return c; }   int funcn(void) { char c; int n; int i, j; i = n = 0; while ((c = getchar()) != '\n' && c != EOF) { if (isdigit(c)) { n += (ctoi(c) * pow(10, i)); ++i; } } printf("---+ %d\n", n); return n; }     int main(void) { unsigned int nl, nc; printf("NUM1> "); nl = funcn(); printf("\n\nNUM2> "); nc = funcn(); printf("%u %u\n", nl, nc); return 0; }

---

Le programme a pour SIMPLE but d'inverser les chiffres composant le nombre que l'on fourni en entrée au programme.

Le problème, c'est qu'avec GCC, le nombre inversé n'est pas exactement l'inverse du nombre concerné. Généralement, c'est comme si le programme soustrayais 1 ou 2 au nombre inversé.

Exemple:

Code:

---

1 2 3 4 5 6 7 8

NUM1> 1234 ---+ 4320     NUM2> 56789 ---+ 98763 4320 98763

---

Les réponses devraient évidemment être 4321 et 98765...

Qu'est qui cause ces erreurs?
Comment palier à ce problème?

Notez que ce problème fait son apparition SEULEMENT avec GCC.
Compilé avec ICC (Intel) et CL (Microsoft), le programme se comporte comme il se doit.


Merci,

Sincèrement,

Array

Salut,

à priori, je ne vois pas pourquoi cela ne marcherait pas.
Avec gcc version 4.1.2, il n'y a pas de problème.

Citation:

---

NUM1> 1234
---+ 4321


NUM2> 456789
---+ 987654
4321 987654

---

Essaie de voir ce que pow te renvoie pour pow(10,0).
Ma version râle si je ne compile pas avec l'option -lm
(à cause de #include <math.h>) ....
Peut-être que l'option -Wall peut te donner plus d'informations.....

Sinon, faire plus de print :)

euh.....

Une valeur entre ' ' est un CARACTERE, pas un entier...

Code:

---

1 2 3 4 5 6

short int ctoi(const int c) { if (c == 0) return 0; else if (c == 1) return 1;

---

marcherait beaucoup mieux....

Citation:

---

Envoyé par souviron34

Une valeur entre ' ' est un CARACTERE, pas un entier...

---

C'est un entier du type int

Salut,

Des problèmes de type un peu partout dans ce code ...mais le plus sérieux est celui-ci :

Code:

---

1 2	n += (ctoi(c) * pow(10, i));

---

"pow" retourne un double !! ...tandis que le "n" est de type int sachant que la conversion float vers int provoque une perte de précision voilà ce que tu obtiens pour chaque tour de boucle :

Code:

---

1 2 3 4 5

1=(1) 21=1+(20) 320=1+20+(299) 4320=1+20+299+(4000)

---

Essaie avec :

Code:

---

1 2 3 4 5 6

<...> double n; <...> printf("---+ %d\n", (int)n); <...>

---

Citation:

---

Envoyé par souviron34

euh.....

Une valeur entre ' ' est un CARACTERE, pas un entier...

---

On est bien d'accord, 'A' est une constante entière de type int dont la valeur est 65 (encodage ASCII), non?

Thierry

Citation:

---

Envoyé par Thierry Chappuis

On est bien d'accord, 'A' est une constante entière de type int dont la valeur est 65 (encodage ASCII), non?

Thierry

---

Absolument....

Mais en entree de sa fonction, le PO a un int..

qu'il compare ensuite a la valeur ASCII.... hum humm.....

Code:

---

1 2 3 4 5

if (c == '0') return 0; else if (c == '1') return 1;

---

si c (entier) est 1, son caractere ASCII est 49... et reciproquement...

c == '1' voudrait dire que c = 49.

Inversement

c == 1 signifie c = SOH en ASCII...

Citation:

---

Envoyé par diogene

C'est un entier du type int

---

oui mais pas identique entre valeur numerique et valeur du caractere (voir ci-dessus)...

http://www.asciitable.com/

ok j'ai relu son code...

C'est ok parce qu'il prend la representation..

Mais personellement, j'aurais trouve beaucoup plus clair de mettre

Code:

---

1 2 3 4

int ctoi ( char c ) { .... }

---

ou alors

Code:

---

1 2 3 4 5 6 7

int ctoi (int c ) { if ( c < 48 || c > 57 ) return NaN ; else return ( c - 48 ); }

---

Citation:

---

Envoyé par souviron34

ok j'ai relu son code...

C'est ok parce qu'il prend la representation..

Mais personellement, j'aurais trouve beaucoup plus clair de mettre

Code:

---

1 2 3 4

int ctoi ( char c ) { .... }

---

---

Et les fonctions isalpha(), isnum(), isspace(), etc, elles ont quel prototype? Naturellement, leur utilisation implique en prinicpe un cast explicite en unsigned char lorsqu'on leur passe un argument de type char:

Code:

---

1 2 3 4 5 6

char c = 'A'; /*...*/ if (isalpha((unsigned char) c)) { /*...*/ }

---

Thierry

Citation:

---

Envoyé par Thierry Chappuis

Et les fonctions isalpha(), isnum(), isspace(), etc, elles ont quel prototype? Naturellement, leur utilisation implique en prinicpe un cast explicite en unsigned char lorsqu'on leur passe un argument de type char:

---

Pourquoi ce cast est-il nécessaire ?

Parce qu'on n'est pas supposé passer un nombre négatif à ces fonctions, à part EOF.
Par contre, il me semble que le problème ne se pose pas pour iswalpha(), car wchar_t n'est pas signé. Mais je ne sais pas ce que dit précisément la norme là-dessus...

Edit: D'après la n1124, ça a l'air bon, ce qui donne:

• En char: isalpha((unsigned char)c)
• En wchar_t: iswalpha(c)
• En TCHAR (windows): _istalpha((_TUCHAR)c)

Citation:

---

Envoyé par diogene

Pourquoi ce cast est-il nécessaire ?

---

De mémoire il me semble que ces fonctions sont chargées de tronquer l'int reçu en char donc je ne voyais pas le problème non plus mais comme le dit Médinoc un char signé qui vaut 0xFF pourrait être pris pour EOF un fois converti en int.