Discussion :

[tatayoyo]

bonjour,

je cherche une methode simple pour faire ca.
exemple:
si on teste "124543" ca renvoie true
si on teste "12ef34e" ca renvoie false
je pensais tester les caracteres un par un, les convertir en chiffre, et verifier que le resultat est bien compris entre 0 et 9. mais si je converti une lettre en chiffre (fonction atoi()) j'obtiens 0, ca fausse ma methode.
auriez vous une meilleure idee ?

[tatayoyo]

je precise que la chaine de caractere en question a tester a une longueur aleatoire. ca peut etre "12e" ou "12345" ou "12345rt23"

[DaZumba]

Utilise la fonction isdigit(), declaree dans ctype.h. Tu testes chaque caractere, des que isdigit() renvoie faux, tu arretes et renvoies faux.

[Trap D]

Plutôt qu'utiliser atoi (qui est obsolète) utilise strtol:

long strtol( const char *nptr, char **endptr, int base );
Parameters
nptr
Null-terminated string to convert
endptr
Pointer to character that stops scan
base
Number base to use
Libraries
All versions of the C run-time libraries.

Return Value
strtol returns the value represented in the string nptr, except when the representation would cause an overflow, in which case it returns LONG_MAX or LONG_MIN. strtol returns 0 if no conversion can be performed. wcstol returns values analogously to strtol. For both functions, errno is set to ERANGE if overflow or underflow occurs.

En testant au sortir de la fonction le pointeur endptr tu peux savoir si ton mot a été entièrement transformé en nombre (endptr = nptr + strlen(nptr)) ou non.
Un des intérêts est que si tu veux tester un nombre en hexa tu changes simplement la base de conversion.

[Marcin]

Tu peux aussi tester les codes ASCII. Il suffir de vérifier si le code de chacun de tes caractères est compris entre 48 (le '0') et 57 (le '9')

[Trap D]

Tu peux aussi tester les codes ASCII. Il suffir de vérifier si le code de chacun de tes caractères est compris entre 48 (le '0') et 57 (le '9')

Non portable, ce ne sont pas les mêmes codes en EBCDIC, il faut utiliser la macro isdigit comme signalé plus haut.

[tatayoyo]

ok, merci.

[Emmanuel Delahaye]

si on teste "124543" ca renvoie true
si on teste "12ef34e" ca renvoie false

strtol()
sscanf()
http://emmanuel-delahaye.developpez....tes.htm#saisie

Fait de ton mieux et poste ton code si il ne fonctionne pas.

[DaZumba]

strtol()
sscanf()

Qu'est-ce que vous avez tous contre isdigit() ? Elle est ANSI-C, cette brave petite fonction, et le code devient evident:

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
 
int all_digits(const char *s)
{
    const char *p = s;
 
    while(*p)
    {
       if(!isdigit(*p))
       {
         return 0;
       }
       p++;
    }
    return 1;
}

Evidemment, la fonction ci-dessus suppose que la chaine de caracteres s est NULL-terminated.

[Trap D]

DaZumba >> Je n'ai rien contre isdigit, j'ai simplement indiqué une autre possibilité, c'est tout

[Emmanuel Delahaye]

Qu'est-ce que vous avez tous contre isdigit() ?

Rien du tout. Déjà citée. (Omission ne veut pas dire opposition)

[DaZumba]

Omission ne veut pas dire opposition

Joliment dit. C'est a croire que je deviens paranoiaque. Il va falloir que je corrige cela!

[Nasky]

Joliment dit. C'est a croire que je deviens paranoiaque. Il va falloir que je corrige cela!

C'est juste qu'on est habitué à ce qu'Emmanuel donne toutes les solutions

Par contre, pour faire mon relou, il vaut mieux éviter plusieurs return dans une fonction même si dans ce cas précis, ça ne pause pas de problème.

Nas'

[DaZumba]

Par contre, pour faire mon relou, il vaut mieux éviter plusieurs return dans une fonction même si dans ce cas précis, ça ne pause pas de problème.

Oui, on peut en effet respecter ce principe. Ca devient alors:

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
 
int all_digits(const char *s)
{
    int ret = 1;
    const char *p = s;
 
    while(*p)
    {
       if(!isdigit(*p))
       {
         ret = 0;
         break;
       }
       p++;
    }
    return ret;
}

[Nasky]

Pour faire encore mon relou, il vau mieux éviter l'utilisation de break

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int all_digits(const char *s)
{
    int ret = 1;
    int i=0 ;
    const char *p = s;
 
    while(*p && isdigit(*p))
    {
       i++;
       p++;
    }
    if(i == strlen(s)) ret = 1 ; else ret = 0 ;
    return ret;
}

C'est moche je sais mais bon, c'était juste histoire de chipoter

Nas'

[DaZumba]

Pour faire encore mon relou, il vau mieux éviter l'utilisation de break

Je m'y attendais... Dans ce cas, on va compresser tout ca:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int all_digits(const char *s)
{
    const char *p = s;
 
    while(*p && isdigit(*p))
    {
       p++;
    }
    return *p == '\0';
}

[Nasky]

Ah oui, pas mal l'astuce du return. Je n'y avais pas pensé
Bon ba voilà, on a une jolie fonction maintenant

Nas'

[gege2061]

Et maintenant rendons ça illisible :

Code :

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

int all_digits(const char *s){while(*s++&&isdigit(*s));return!*s;}

Sans rire, les optimisations c'est bien mais il ne faut pas perdre la illisibilité surtout que le compilateur sais surement mieux le faire que vous!

[edit]Encore un effort et on remporte l'IOCCC de cette année [/edit]

[DaZumba]

Sans rire, les optimisations c'est bien mais il ne faut pas perdre la illisibilité surtout que le compilateur sais surement mieux le faire que vous!

On n'optimise rien, on voulait simplement:
- un return unique
- eviter un break qui etait, je l'admets, tout a fait inelegant

Par contre, ton idee de te passer de la variable locale *p et boucler directement sur s est tres bonne ! Et je ne sais pas, mais je trouve cela lisible (d'autant que le nom de la fonction, avec eventuellement un petit commentaire, aide beaucoup a la comprehension).

[gege2061]

On n'optimise rien, on voulait simplement:
- un return unique
- eviter un break qui etait, je l'admets, tout a fait inelegant

C'était un brin ironique, effectivement je n'ai rien à redire sur vos modifications, c'était juste pour vous prévenir de ne pas tomber dans l'extrème!

Par contre, ton idee de te passer de la variable locale *p et boucler directement sur s est tres bonne !

Merci

Et je ne sais pas, mais je trouve cela lisible

Peut être parce que tu as vu la version propre avant, ou l'expérience mais un débutant risque de s'amuser!

avec eventuellement un petit commentaire

Ah non l'une des premières règles, si tu veux faire tu code illisible c'est de ne pas mettre de commentaires (ou des commentaires qui servent à rien).