Discussion :

[odsen.s]

Bonjour à tous,

Je suis en train de réaliser un petit projet d'agenda en C.
Quelques tests sur certaines fonctions construites m'ammènent à constater un problème.

Voici un premier exemple de code, qui fonctionne :

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
#include <stdio.h>
#include <time.h>
 
#define FMT_DATE "%d/%m/%Y"
 
int main (void)
{
   char s[100];
   struct tm *t_now = NULL;
 
   time_t tmt = time (NULL);
   t_now = localtime (&tmt);
 
   t_now->tm_mon -= 500;
   mktime(t_now);
 
   t_now->tm_mon -= 500;
   mktime(t_now);
 
   strftime (s, sizeof s, "Time = "FMT_DATE, t_now);
   puts(s);
 
   return 0;
}

Résultat:

Code :

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

Time = 16/05/1924

Nous avons retranché 500 mois, puis à nouveau 500 mois à la date actuelle, et le résultat est bon. Si on essaie de retrancher 1000 mois d'un seul coup (ce qui revient normalement au même) il y a un problème :

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
#include <stdio.h>
#include <time.h>
 
#define FMT_DATE "%d/%m/%Y"
 
int main (void)
{
   char s[100];
   struct tm *t_now = NULL;
 
   time_t tmt = time (NULL);
   t_now = localtime (&tmt);
 
   t_now->tm_mon -= 1000;
   mktime(t_now);
 
   strftime (s, sizeof s, "Time = "FMT_DATE, t_now);
   puts(s);
 
   return 0;
}

Résultat :

Code :

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

Time = 16/-991/2007

Ici, on obtient un bogue.

Quelqu'un saurait-il m'expliquer le problème ?
Par avance, merci.

[Gruik]

Salut,

Si la structure transmise ne peut pas être convertie, mktime() renvoie la valeur (time_t)(-1) et ne modifie pas les membres tm_wday et tm_yday.

Je pense que dans les 2 cas, mktime echoue (et ne modifie donc pas la structure passée).
Dans le premier cas, le premier mktime() reussit et positionne la date au 16/05/1924, le deuxieme echoue et laisse la date au 16/05/1924.
Dans le deuxieme cas, mktime() echoue directement et ne met pas à jour les champs de la stcuture, ce qui signifie qu'on a un mois entre -1000 et -989 (0 à 11, moins 1000)

[odsen.s]

Dans le premier cas, le premier mktime() reussit et positionne la date au 16/05/1924, le deuxieme echoue et laisse la date au 16/05/1924.

Je ne suis pas sûr, car 1000 mois correspondent à 83 ans, donc 1924 est bien la date attendue après les deux mktime.

[Gruik]

Ok, dans ce cas, il ya peut etre une limite dans les valeurs "hors bornes" que peut prendre mktime

[odsen.s]

Ok, dans ce cas, il ya peut etre une limite dans les valeurs "hors bornes" que peut prendre mktime

A priori, ça doit être ça.
Dans la doc, je ne vois rien d'indiqué à ce sujet.
Quoi qu'il en soit, mktime renvoit -1 (échoue) dans le second cas, il est alors possible d'agir en conséquence.
Je trouvais simplement le comportement de la fonction curieux.

Merci pour l'aide apportée

[Emmanuel Delahaye]

Je suis en train de réaliser un petit projet d'agenda en C.
Quelques tests sur certaines fonctions construites m'ammènent à constater un problème.

• Je conseille de travailler sur une copie et non sur la structure interne de localtime().
• Il est impératif de s'assurer que mktime() a bien fait son travail en vérifiant la valeur retournée (-1 = Echec).
• Apparamment, il y a une limite au nombre de mois que l'on peut retirer et cette limite semble dépendre de l'implémentation. Chez moi :

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
#include <stdio.h>
#include <time.h>
 
#define FMT_DATE "%d/%m/%Y"
 
int main (void)
{
   char s[100];
   struct tm tm_now = {0};
 
   time_t tmt = time (NULL);
   tm_now = *localtime (&tmt);
   strftime (s, sizeof s, "Time = "FMT_DATE, &tm_now);
   printf("%s (%16ld)\n",s, (long)tmt);
 
   tm_now.tm_mon -= 200;
   tmt = mktime(&tm_now);
   strftime (s, sizeof s, "Time = "FMT_DATE, &tm_now);
   printf("%s (%16ld)\n",s, (long)tmt);
 
   tm_now.tm_mon -= 500;
   tmt = mktime(&tm_now);
   strftime (s, sizeof s, "Time = "FMT_DATE, &tm_now);
   printf("%s (%16ld)\n",s, (long)tmt);
 
   return 0;
}

donne

Code :

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

1
2
3
4
5
Time = 16/09/2007 (      1189976430)
Time = 16/01/1991 (       664059630)
Time = 16/05/1991 (              -1)
 
Press ENTER to continue.

[odsen.s]

Je conseille de travailler sur une copie et non sur la structure interne de localtime().

Ok.

# Il est impératif de s'assurer que mktime() a bien fait son travail en vérifiant la valeur retournée (-1 = Echec).
# Apparamment, il y a une limite au nombre de mois que l'on peut retirer et cette limite semble dépendre de l'implémentation. Chez moi :
<...>

Merci pour ces infos Emmanuel