Discussion :

[cypher.sephiroth]

Bonjour,

Pour mon interface GTK, j'ai besoin de générer un nom de fichier du genre "aammjjliste.txt" avec aa : numéro de l'année en cours, mm : numéro du mois en cours, jj : numéro du jour en cours.
Tout se passe bien lors de la création de la chaîne aammjj, mais dès que j'essaie d'ajouter la partie "liste.txt", j'ai une SegFault.

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
#include <stdio.h>
#include <time.h>
 
int main(void)
{
    time_t Timbre_Date;
    struct tm * Date_Jour;
    int Annee;
    int Mois;
    int Jour;
    int Heure;
    char *Date_Compactee;
    char *Nom_Fichier;
 
    Timbre_Date = time(NULL);
    Date_Jour = localtime(&Timbre_Date);
    Annee = (Date_Jour-> tm_year-100);
    Mois = (Date_Jour -> tm_mon);
    Jour = (Date_Jour -> tm_mday);
    Heure = (Date_Jour -> tm_hour);
 
    /* Affiche la date et l'heure courante (format francais) */
    printf("Annee : %02u\n Mois : %u\n Jour : %u\n Heure : %u\n", Annee, Mois, Jour, Heure);
    sprintf(Date_Compactee, "%02u%02u%02u%02u", Annee, Mois, Jour, Heure);
    printf("Date compactée : %s", Date_Compactee);
    sprintf(Nom_Fichier, "%sListe.txt", Date_Compactee);
 
    return 0;
}

Pourriez-vous m'indiquer où je fais l'erreur ?
Merci beaucoup

[Médinoc]

Le pointeur Date_Compactee ne pointe sur rien du tout.
Tu ne peux pas faire un sprintf() sur du vide comme ça, du moins pas en C standard.

[Mimidoux]

Bonjour.

je pense qu'il faut allouée de la mémoire avec malloc()

si tu alloues pas la mémoire il veut remplir le texte dans date_compact qui est un pointeur non initialisé. sprintf() écrit alors dans un endroit ou il n'a surement pas le droit d'écrire. => segfault.

[Médinoc]

Par contre, en utilisant les bonnes fonctions, on doit pouvoir faire ce que tu veux.

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
/* Version avec va_list */
char * alloc_vsprintf(char const *format, va_list args)
{
	size_t longueur;
	char* ret = NULL;
 
	/* 1. Calcul de la taille à allouer */
	#ifdef _MSC_VER
	/* Version Microsoft: snprintf() n'existe pas */
	longueur = _vscprintf(format, args);
	#else
	/* Version utilisant snprintf() */
	longueur = vsnprintf(NULL, 0, format, args);
	#endif
 
	/* 2. Allocation de mémoire */
	ret = malloc((longueur+1) * sizeof *ret);
	if(ret != NULL)
	{
		/* 3. Écriture */
		vsprintf(ret, format, args);
	}
	return ret;
}
 
/*Version avec arguments*/
char * alloc_sprintf(char const *format, ...)
{
	char * retVal;
	va_list args;
	va_start(args, format);
	retVal = alloc_vsprintf(format, args);
	va_end(args);
	return retVal;
}

Ne pas oublier de faire un free() sur le pointeur retourné quand tu n'en as plus besoin...

[Melem]

Code :

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

1
2
3
4
5
6
7
8
9
 
	/* 1. Calcul de la taille à allouer */
	#ifdef _MSC_VER
	/* Version Microsoft: snprintf() n'existe pas */
	longueur = _vscprintf(format, args);
	#else
	/* Version utilisant snprintf() */
	longueur = vsnprintf(NULL, 0, format, args);
	#endif

En fait, parce que je sens déjà la question venir, la CRT (MS) contient la fonction _snprintf et semblables mais elle c'est juste un sprintf avec un argument buf_size en plus. Ce n'est pas une vraie implémentation de snprintf. C'est pourquoi il faut utiliser _vscprintf(format, args) à la place de vsnprintf(NULL, 0, format, args).

[cypher.sephiroth]

Bon...
Et sinon, une simple modification du genre :

Code :

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

1
2
char Date_Compactee [6];
char Nom_Fichier [20];

ça peut suffire ?

[Médinoc]

Déjà, c'est huit caractères et non six, mais oui, si les données d'entrées sont vérifiées et ne dépassent jamais la taille limite, ça passe.

Mais je trouve qu'allouer la bonne longueur à coup sûr est plus sécurisant.

[cypher.sephiroth]

Déjà, c'est huit caractères et non six, mais oui, si les données d'entrées sont vérifiées et ne dépassent jamais la taille limite, ça passe.

Ben c'est sûr que le jour où les mois feront plus de 99 jours et les années plus de 99 mois, ça pourra être problématique.

[gl]

Déjà, c'est huit caractères et non six

9 caractères avec le 0 terminal.

[Médinoc]

Ben c'est sûr que le jour où les mois feront plus de 99 jours et les années plus de 99 mois, ça pourra être problématique.

Pour être franc, je n'avais pas vu que l'entrée venait de localtime() . Je pensais à un fichier ou une saisie utilisateur...

[cypher.sephiroth]

Pour être franc, je n'avais pas vu que l'entrée venait de localtime() . Je pensais à un fichier ou une saisie utilisateur...

Y'a pas de mal, il m'arrive aussi de lire en diagonale, parfois.
La version avec les va_list me semble intéressante, je l'analyse demain.

Bonne journée !

[cypher.sephiroth]

J'ai trouvé une traduction pour mon 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
22
23
24
25
 
char *concat (int Nombre_Args, ...)
{
    va_list Liste_A_Traiter;
    int Compteur_Args;
    size_t Taille_Chaine;
    char *Resultat_Concatene;
 
    va_start(Liste_A_Traiter, Nombre_Args);
    for (Taille_Chaine = Compteur_Args = 0; Compteur_Args < Nombre_Args; Compteur_Args++)
            Taille_Chaine += strlen(va_arg(Liste_A_Traiter, char *));
    va_end(Liste_A_Traiter);
 
    Resultat_Concatene = (char *) malloc(Taille_Chaine + 1);
    if (NULL != Resultat_Concatene)
    {
        va_start(Liste_A_Traiter, Nombre_Args);
        strcpy(Resultat_Concatene, "");
        for (Compteur_Args = 0; Compteur_Args < Nombre_Args; Compteur_Args++)
            strcat(Resultat_Concatene, va_arg(Liste_A_Traiter, char *));
        va_end(Liste_A_Traiter);
    }
 
    return Resultat_Concatene;
}

[Médinoc]

Essaie d'en faire une version qui prend directement une va_list en paramètre.
Pour la parcourir deux fois, tu devrais pouvoir faire une copie de ladite va_list...

[cypher.sephiroth]

Essaie d'en faire une version qui prend directement une va_list en paramètre.

C'est-à-dire ? Là, je ne vois pas grand-chose.

Pour la parcourir deux fois, tu devrais pouvoir faire une copie de ladite va_list...

Est-ce que ça ne serait pas trop gênant au niveau consommation des ressources ?

[Médinoc]

Une va_list généralement est un simple pointeur, donc au niveau copie ça ne bouffe rien.