Discussion :

[boboss123]

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

[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 :

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
#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;
}

[gangsoleil]

Bonjour,

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

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
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 */
}
/* ... */

[boboss123]

ok, merci pour vos reponses

[pfeuh]

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.

[gangsoleil]

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

[boboss123]

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 :

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
#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 ?

[pfeuh]

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

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 :

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
#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;
}

[boboss123]

merci pour vos réponses


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

Code :

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

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 ?

[pfeuh]

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é

[quetzacoatl]

tu peux essayer de faire(non testé):

Code :

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

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;
}