Discussion :

[Nyxemis]

bonjour,

je suis débutant en prog C/C++ et j'aimerai savoir si quelqu'un pouvait m'expliquer en détails (avec des mots que je puisse comprendre lol ) le code suivant (mot par mot si quelqu'un a assez de patience) :

Code :

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

1
2
3
4
5
6
7
8
9
10
{
    time_t     now = time(0);
    struct tm  tstruct;
    char       buf[80];
    tstruct = *localtime(&now);
 
    strftime(buf, sizeof(buf), "%X", &tstruct);
 
    return buf;
}

D'avance, merci.

[diogene]

Des maladresses et une erreur fatale.
Pour plus de détails, consulter la documentation des différentes fonctions utilisées

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
   time_t     now = time(0); 
// La fonction time() retourne la date et l'heure courante dans un format de type time_t (un type arithmétique adéquat). 
// Son argument est un pointeur, ici le pointeur NULL. Il aurait été préférable de le souligner :
// time_t     now = time(NULL);
    struct tm  tstruct;
// Création d'une structure tm pouvant contenir la date et l'heure sous un format plus accessible grace à ses 
// différents champs (année, mois, jour du mois, heure, minutes, secondes,...) 
    char       buf[80];
// création d'un tableau de 80 char
    tstruct = *localtime(&now);
// localtime() convertit une donnée de date&heure du format time_t en un format struct tm. La donnée de format time_t 
// à convertir (now) doit être communiquée à la fonction via son adresse (d'où l'argument &now).
// Elle renvoie l'adresse d'une variable statique de type struct tm contenant les informations (ou NULL en cas d'échec)
// Cette variable statique est obtenue en prenant le contenu de l'adresse renvoyée (espérons qu'elle n'est pas NULL) par
// *localtime(&now) et copiée dans la structure tm tstruct (ce qui est d'ailleurs inutile)
    strftime(buf, sizeof(buf), "%X", &tstruct);
// transforme le contenu de la structure tstruct, dont l'adresse est passée en argument (&tstruct) en une chaine 
// de caractères exprimant la date et l'heure suivant  le format "%X" (voir la doc de la fonction pour les détails 
// sur ce format) et la place dans le tableau buf.
// Mais pourquoi avoir créé tstruct pour ça alors qu'il suffisait pour faire la même chose de 
// strftime(buf, sizeof(buf), "%X", localtime(&now));
   return buf;
// ERREUR fatale :On ne doit jamais retourner l'adresse d'une variable locale puisque cette variable est détruite
// en sortant de son bloc de définition et que cette adresse ne signifie alors plus rien. 
// Or buf est, dans ce contexte, l'adresse du premier élément du tableau local buf[] défini dans ce bloc.

[Sve@r]

Code :

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

1
2
3
4
5
6
7
8
9
10
{
    time_t     now = time(0);
    struct tm  tstruct;
    char       buf[80];
    tstruct = *localtime(&now);
 
    strftime(buf, sizeof(buf), "%X", &tstruct);
 
    return buf;
}

Code c :

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

1
2
3
4
5
 
   return buf;
// ERREUR fatale :On ne doit jamais retourner l'adresse d'une variable locale puisque cette variable est détruite
// en sortant de son bloc de définition et que cette adresse ne signifie alors plus rien. 
// Or buf est, dans ce contexte, l'adresse du premier élément du tableau local buf[] défini dans ce bloc.

Bonjour

On peut arranger ce dernier détail

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
{
    time_t     now = time(NULL);
    struct tm  tstruct;
    char       *buf;
    tstruct = *localtime(&now);
 
    buf=malloc(80 * sizeof(char));
    if (buf == NULL) return NULL;
 
    strftime(buf, sizeof(buf), "%X", &tstruct);
 
    return buf;
}

La différence avec le code précédent, c'est que maintenant, buf contient l'adresse d'une zone mémoire allouée de façon explicite, et qui ne se trouve pas dans la zone mémoire appelée "pile" mais "tas" et donc qui ne sera pas détruite automatiquement en fin de fonction.
Donc l'adresse de cette zone persistante est récupérée par l'appelant qui peut s'en servir comme il l'entend. Toutefois ce sera à lui de s'assurer que la zone est libérée via un free() lorsqu'il n'en aura plus besoin.

[Nyxemis]

Bonjour,

Et bien merci beaucoup pour la correction et les explications
Est ce que quelqu'un saurait répondre a ma question initiale svp ?

D'avance, merci beaucoup !

[ram-0000]

Est ce que qqun saurait repondre a ma question initiale svp ?

Heu ... là, je ne comprends plus.

La réponse de diogene a été plus que complète et surtout détaillé et pédagogique. Maintenant, c'est aussi à toi de faire un effort pour comprendre ce qui a été dit.

Prends un bon livre sur le C, relis tes cours de programmation, regarde les différents tutoriels mais je vois difficilement ce que l'on peut faire de plus.

[Obsidian]

Et bien merci beaucoup pour la correction et les explications
Est ce que quelqu'un saurait répondre a ma question initiale svp ?

Les commentaires — en gris — insérés dans le code de Diogène sont les explications que tu recherches. Si ça ne suffit pas, dis-nous sur quels points en particulier tu butes.

[Mipwiq]

Les explications de Diogène sont très détaillées. Je suis sûr que même un débutant comme toi peut les comprendre.

Si tu as un problème en particulier avec une ligne ou une fonction dit nous précisément ce qui te dérange.

[Nyxemis]

oups ! Désolé... j'étais sur le téléphone quand j'ai répondu, et je n'avais pas tout vu. Effectivement, la réponse est très complète et je t'en remercie diogene !!!


Merci de votre participation a mon apprentissage !

[Nyxemis]

Bonjour

On peut arranger ce dernier détail

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
{
    time_t     now = time(NULL);
    struct tm  tstruct;
    char       *buf;
    tstruct = *localtime(&now);
 
    buf=malloc(80 * sizeof(char));
    if (buf == NULL) return NULL;
 
    strftime(buf, sizeof(buf), "%X", &tstruct);
 
    return buf;
}

La différence avec le code précédent, c'est que maintenant, buf contient l'adresse d'une zone mémoire allouée de façon explicite, et qui ne se trouve pas dans la zone mémoire appelée "pile" mais "tas" et donc qui ne sera pas détruite automatiquement en fin de fonction.
Donc l'adresse de cette zone persistante est récupérée par l'appelant qui peut s'en servir comme il l'entend. Toutefois ce sera à lui de s'assurer que la zone est libérée via un free() lorsqu'il n'en aura plus besoin.

j'ai essayer ton code, et voici l'erreur qui en ressort...

In function 'const string currentDateTime()'
error: invalid conversion from 'void*' to 'char*' [-fpermissive]|
||=== Build finished: 1 errors, 0 warnings (0 minutes, 1 seconds) ===|

[Bktero]

Tu ne compilerais pas en mode C++ toi ?

[Nyxemis]

effectivement, tu as raison

[diogene]

Ce code est faux :

Code :

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

1
2
3
4
5
    char       *buf;
...
    buf=malloc(80 * sizeof(char));
    if (buf == NULL) return NULL;
    strftime(buf, sizeof(buf), "%X", &tstruct);

buf est un pointeur et sizeof (buf) donnera la taille d'un pointeur, pas du tout la taille de la zone allouée.

Code :

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

1
2
3
    buf=malloc(80);
    if (buf != NULL) strftime(buf, 80, "%X", &tstruct);
    return buf;

[Mipwiq]

buf est un pointeur et sizeof (buf) donnera la taille d'un pointeur, pas du tout la taille de la zone allouée.

Dans ce cas là, si il doit y avoir utilisation de sizeof, pour avoir la taille il faudrait faire

Code :

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

80 * sizeof(*buf)

je trouve plus compréhensible de présenter la chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
 
    char       *buf;
    int          size = (80 * sizeof(*buf));
 
    buf = malloc(size);
    if (buf == NULL) return NULL;
    strftime(buf, size, "%X", &tstruct);
    return buf;

[Sve@r]

Code :

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

1
2
3
4
5
    char       *buf;
...
    buf=malloc(80 * sizeof(char));
    if (buf == NULL) return NULL;
    strftime(buf, sizeof(buf), "%X", &tstruct);

buf est un pointeur et sizeof (buf) donnera la taille d'un pointeur, pas du tout la taille de la zone allouée.

Exact, erreur d'inattention de ma part.

Ce code est faux

Mouais. Je pense que tu aurais pu modérer un peu ton expression.

Dans ce cas là, si il doit y avoir utilisation de sizeof, pour avoir la taille il faudrait faire

Code :

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

80 * sizeof(*buf)

je trouve plus compréhensible de présenter la chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
 
    char       *buf;
    int          size = (80 * sizeof(*buf));
 
    buf = malloc(size);
    if (buf == NULL) return NULL;
    strftime(buf, size, "%X", &tstruct);
    return buf;

[Nyxemis]

comment devrais je faire si je voudrai demander d’écrire une heure au format hh:mm:ss ? Le but etant d'en demander plusieurs et de les enregistrer...


ps: tu a vraiment assurer pour l'explication que je viens une nouvelle fois de regarder ! Merci encore !!!

[edgarjacobs]

Et pourqoui pas ceci:

Code :

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

1
2
3
4
5
6
7
time_t curtime;
struct tm *ti;
char tmp[50];
 
curtime=time(NULL);
ti=localtime(&curtime);
sprintf(tmp,"%02d:%02d:%02d %02d/%02d/%04d\n",ti->tm_hour,ti->tm_min,ti->tm_sec,ti->tm_mday,ti->tm_mon+1,ti->tm_year+1900);

Qui place dans tmp une chaine de caractère du style "01:42:02 31.07.2013"