Discussion :

[marrad]

Salut, j'ai un problème avec une fonction de validité du date de format (JJ/MM/AAAA) cette dernière fonctionne en build target debugger mais pas en release voilà mon code

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
int str_to_long(const char *s, long *p)
{
    int ret = 0; /* code d'erreur 0 : succès. */
    if (*s == '\0')
        ret = 1; /* code d'erreur 1 : la chaine a convertir est vide. */
    else
    {
        char *end="";
        *p = strtol(s, &end, 10);
 
       if (errno != 0)
           ret = 2; /* code d'erreur 2 : la fonction strtol a rencontre une erreur. */
        else if (*end != '\0')
            ret = 3; /* code d'erreur 3 : la conversion a échoué car un caractère invalide a été détecté. */
    }
 
    return ret;
}
int valide(char *c)
{
    int day[12]={31,28,31,30,31,30,31,31,30,31,30,31};
    long n1;
    long n2;
    long n3;
    char c1[2];
    char c2[2];
    char c3[4];
    c1[0]=c[0];
    c1[1]=c[1];
    c1[2]='\0';
    int ret1 = str_to_long(c1, &n1);
    if (ret1 != 0)   return 0;
    c2[0]=c[3];
    c2[1]=c[4];
    c2[2]='\0';
    int ret2 = str_to_long(c2, &n2);
    if (ret2 != 0)   return 0;
    c3[0]=c[6];
    c3[1]=c[7];
    c3[2]=c[8];
    c3[3]=c[9];
    c3[4]='\0';
    int ret3 = str_to_long(c3, &n3);
    if (ret3 != 0)   return 0;
 
 
 
    if(c[2]!='/' || c[5]!='/' || c[10]!='\0') return 1;
 
    if(n3 % 400 == 0 || (n3 % 100 != 0 && n3 % 4 == 0))
        day[1]=29;
    if(n2>0 && n2<13 && n3>0)
    {
        if(n1>0 && n1<=day[n2-1])
        return 1;
    }
    return 0;
}

[gerald3d]

Bonjour.

J'ai l'impression qu'il y a un problème dans la gestion du pointeur end.

[Médinoc]

A-t-on la moindre garantie que strtol() règle errno à zéro en cas de succès?

[picodev]

Bonjour

Code :

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

1
2
3
4
5
6
7
8
...
    char c1[2];
...
    c1[0]=c[0];
    c1[1]=c[1];
    c1[2]='\0';
...

N'y aurait-il pas comme un problème d'allocation de taille (entre autre) ?

[Invité]

Bonsoir,

Pourquoi ne pas simplement utiliser sscanf ?

Code :

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

1
2
3
4
5
6
7
char date[] = "02/02/2015";
unsigned int jour, mois, annee;
if (sscanf(date, "%02u/%02u/%4u", &jour, &mois, &annee) == 3) {
    // Bon format
} else {
    // Mauvais format
}

Je te conseille aussi de mieux nommer tes variables pour correspondre à leur rôle. Ici c, c1-3, n1-3 ne veulent rien dire, et day n'est pas un jour, mais le nombre de jours par mois.

Pense aussi à nous préciser la nature de ce problème que tu rencontres (plantage ? Résultat non correct ? Et avec quelles valeurs). Idéalement avec un code minimal complet qui le reproduit.

[gangsoleil]

A-t-on la moindre garantie que strtol() règle errno à zéro en cas de succès?

La norme précise que les fonctions peuvent le faire, mais qu'elles ne sont pas obligées de le faire ; d'ailleurs, elles ne sont pas non plus obligées de donner une valeur non nulle à errno en cas d'erreur.

C'est pour cela qu'il faut vérifier le code de retour des fonctions et non pas errno, en plus de forcer errno à 0.

N'y aurait-il pas comme un problème d'allocation de taille (entre autre) ?

Au moins sur c1, c2 et c3...

Tu mélanges aussi du code et des déclarations, rendant le code moins lisible -- sans parler du fait que ret1, ret2 et ret3 pourraient avantageusement être remplacées par une seule variable.
str_to_long pourrait aussi vérifier que ses arguments ne sont pas nuls, ça pourrait éviter des catastrophes.

Soit dit en passant, je pense que pas mal de ces erreurs pourraient être détectées par un compilateur bien réglé.