detection d'erreur sur atoi()

Bonjour,

Je voudrais convertir une chaine ASCII en nombre => donc j'utilise la fonction atoi().
Le problème est que je ne sais pas comment faire pour détecté s'il y a eu une erreur pendant la conversion (par exemple s'il y a des lettres dans la chaine).

Comment dois-je faire ?

Remarque : compîlo => version modifié de gcc pour µControlleur PIC32

Salut,

Il faut utiliser une autre fonction, sscanf par exemple. Le résultat est le nombre de variables scannées avec succès, tu peux donc t'en servir comme détection d'erreur.

A+

Pfeuh

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
#include <stdio.h>
 
int main(int argc, char **argv)
{
    int value;
    int result;
    char textinput1[] = "12345";
    char textinput2[] = "blabla";
    result = sscanf(textinput1, "%i", &value);
    if(result != 1)
    {
        printf("erreur\n");
    }
    else
    {
        printf("%i\n", value);
    }
    result = sscanf(textinput2, "%i", &value);
    if(result != 1)
    {
        printf("erreur\n");
    }
    else
    {
        printf("%i\n", value);
    }
 
 
    return 0;
}

Bonjour,

Pourquoi ne pas utiliser strtol, qui est faite pour ca, plutot que sscanf, qui est beaucoup plus complexe a manier ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
long ma_valeur;
/* --- */
errno = 0;
ma_valeur = strtol (chaine, (char**)NULL, 10);
 
if ((errno == ERANGE && (ma_valeur == LONG_MAX || ma_valeur == LONG_MIN)) ||
   (errno != 0 && val == 0))
{
/* traitement de l'erreur */
}
else
{
/* suite du code */
}
/* ... */

ok, merci pour vos reponses

Citation:

Envoyé par gangsoleil

Pourquoi ne pas utiliser strtol, qui est faite pour ca, plutot que sscanf, qui est beaucoup plus complexe a manier ?

Parce que sur son µC, il veut peut-être passer par des int et non des long, il n'y a pas forcément beaucoup de ram. Et il me semble d'autre part que errno n'est pas sûr si on utilise des threads, ce qui est enviseageable sur les gros µC Microchip. En fait, c'est surtout parce que je ne connaissais pas strtol. ;) Qui a des avantages certains, dont le choix de la base.

Et moi je n'avais pas lu qu'il etait sur micro-controlleur... Tout va bien :)

Citation:
Envoyé par pfeuh
Salut,

Il faut utiliser une autre fonction, sscanf par exemple. Le résultat est le nombre de variables scannées avec succès, tu peux donc t'en servir comme détection d'erreur.

A+

Pfeuh
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 #include <stdio.h> int main(int argc, char **argv) { int value; int result; char textinput1[] = "12345"; char textinput2[] = "blabla"; result = sscanf(textinput1, "%i", &value); if(result != 1) { printf("erreur\n"); } else { printf("%i\n", value); } result = sscanf(textinput2, "%i", &value); if(result != 1) { printf("erreur\n"); } else { printf("%i\n", value); } return 0; }
en faite cette methode ne fonctionne pas => si je rentre "1a", ça me trouve un nombre ("a1" => ne donne pas de nombre par contre)
Y a t-il quand même une solution avec sscanf ?

tu peux essayer de faire(non testé):

Code:

1
2
3
4
5
6
7
8
9
 
int nbr_char;
....
result = sscanf(textinput1, "%i%n", &value,&nbr_char);
if(nbr_char!=strlen(textinput1))
{
   puts("Erreur: ...");
   return -1;
}

Citation:

Envoyé par boboss123

cette methode ne fonctionne pas

Elle ne fait pas ce que tu voudrais qu'elle fasse, c'est différent. Elle parse ta chaine jusqu'au premier caractère non numérique. Ça permet de sortir plusieurs données d'une même chaine. Le premier caractère rencontré dans ton cas est un '1', elle a donc bien une chaine qu'elle peut évaluer jusqu'au deuxième caractère 'a'.

Citation:

Envoyé par boboss123

Y a t-il quand même une solution avec sscanf ?

il y a la solution proposée par quetzacoatl. Sinon, tu peux rajouter un test avant le sscanf que je te propose ci-dessous, isnumerictext Du coup, tu auras vu au moins une fois des tests unitaires.
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 #include <stdlib.h> #include <ctype.h> #include <assert.h> #include <stddef.h> typedef struct { char* pattern; int result; } test_record; test_record test_datas[] = { {"a1", 0}, {"1a", 0}, {"123", 1}, {"123 ", 0}, {"", 0}, {" 123", 0}, {NULL, 0} }; int isnumerictext(char* text) { if(*text=='\0') return 0; if (!isdigit(*text)) return 0; while(1) { if(*text=='\0') return 1; if (!isdigit(*text)) return 0; text++; } } int main(int argc, char **argv) { test_record* record_ptr = test_datas; while(record_ptr->pattern != NULL) { printf("\"%s\" : %i\n", record_ptr->pattern, record_ptr->result); assert(isnumerictext(record_ptr->pattern)==record_ptr->result); record_ptr++; } return 0; }

merci pour vos réponses

sinon, j'ai trouvé encore une autre solution :
Code:

1 2 3 4 5 6 7 unsigned int val; int result; unsigned char str[2]; result = sscanf(buf, "%u%1s", &val, str); if(result != 1){ // erreur }
=> par contre, c'est normal que la chaine "-1" me renvoi la valeur 4294967295 (avec result = 1) ? c'est un bug de ma lib ?

C'est normal, tu as demandé à formater en unsigned (%u) le nombre est donc -1 codé en 32 bits à savoir 4294967295. car 0xffffffff représente 4294967295 en non signé et -1 en signé

ok merci (perso, je trouve pas ça très normal car tu demande qu'il te récupère un entier non signé et il récupère quand même un entier signé)