Discussion :

[sandra771]

Bonjour,
J'ai réalisé une fontion qui découpe une date au format YYYY-MM-DDTHH:MM:SS.sss et range les éléments dans une structure tm. Cette fonction marche, elle est codée comme suit:

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
struct tm *fDateT_To_Struct(char *strdate) {
 
 static struct tm returndate;
 sscanf(strdate,"%4d-%2d-%2dT%2d:%2d:%2d.%3d",&(returndate.tm_year)
				             ,&(returndate.tm_mon)
				             ,&(returndate.tm_mday)
					,&(returndate.tm_hour)
					,&(returndate.tm_min)
				    	,&(returndate.tm_sec)
				             ,&(returndate.tm_isdst));
	returndate.tm_wday = 0;
  	returndate.tm_yday = 0;
 
	return &returndate ;
}

Maintenant, je souhaite inclure un code de retour d'erreur RetCode à ma fonction. J'ai modifié ma fonction comme suit:

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
RetCode fDateT_To_Struct(char *strdate, struct tm *st_dateStruct) {
 
	RetCode retCode;
	int i=0; 
	static struct tm returndate;
 
	i = sscanf(strdate,"%4d-%2d-%2dT%2d:%2d:%2d.%3d"
                                                                       ,&(returndate.tm_year)
				                    ,&(returndate.tm_mon)
				                    ,&(returndate.tm_mday)
					       ,&(returndate.tm_hour)
					       ,&(returndate.tm_min)
				    	       ,&(returndate.tm_sec)
				                   ,&(returndate.tm_isdst));
	returndate.tm_wday = 0;
  	returndate.tm_yday = 0;
	st_dateStruct = &returndate;
 
	if (i<0) goto error;
	else goto fin;
 
error:
	retCode = NOK;
fin:
	retCode = OK;
 
return(retCode);
}

Mais lorsqu'un j'appelle ma fonction, cela ne marche plus!
(bout de code du main

Code :

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

1
2
3
4
5
6
7
8
struct tm *pt_Str;
char  carac[25]; 
RetCode retcode; /*RetCode type prédéfini: accepte OK ou NOK*/
 
strcpy(carac, "2002-11-03T23:45:01.556");
 
retCode = fDateT_To_Struct(carac, pt_Str);
printf("pt_Str-> tm_year: %d\n",pt_Str->tm_year); /*par exemple*/

Il y a une erreur à l'exécution:

Code :

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

1
2
signal SEGV (no mapping at the fault address) in main at line 111 in file "mon_fichier.c"
  111                   printf("pt_Str-> tm_year: %d\n",pt_Str->tm_year);

Je ne maîtrise pas bien les pointeurs de structure. Quelqu'un aurait-t-il une idée?
Merci beaucoup
Sandra

[Médinoc]

Moi, je ferais plutôt un truc dans ce genre-là:

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
RetCode fDateT_To_Struct(char const *sDate, struct tm *pDateStruct)
{
int i=0;
 
/* Contrôle des paramètres */
if(sDate==NULL || pDateStruct==NULL)
	return NOK;
 
i = sscanf(sDate,"%4d-%2d-%2dT%2d:%2d:%2d.%3d",
 &(pDateStruct->tm_year),
 &(pDateStruct->tm_mon),
 &(pDateStruct->tm_mday),
 &(pDateStruct->tm_hour),
 &(pDateStruct->tm_min),
 &(pDateStruct->tm_sec),
 &(pDateStruct->tm_isdst)
 );
pDateStruct->tm_wday = 0;
pDateStruct->tm_yday = 0;
 
return ((i==7) ? OK : NOK);
}

[sandra771]

Ah oui merci, j'y penserai la prochaine fois.
J'ai testé ta solution, ça plante à l'exécution sur:

signal SEGV (no mapping at the fault address) in number at 0xfea8c918
0xfea8c918: number+0x085c: st %i1, [%o0]
Current function is fDateT_To_Struct
439 );

Sandra

[Médinoc]

Comment appelles-tu la fonction ?
PS: J'ai oublié un test dans le contrôle des paramètres, je viens de le rajouter.

[Fabllot]

Il me semble, que ta variable "pt_Str" de type "tm*" n'est jamais allouée... A moins que sscanf le fasse (mais j'en doute), il manque sans doute un "malloc" quelque part, non ?

[Fabllot]

Il me semble, que ta variable "pt_Str" de type "tm*" n'est jamais allouée... A moins que sscanf le fasse (mais j'en doute), il manque sans doute un "malloc" quelque part, non ?

Petite précision :
Dans le code de Sandra, je ne suis pas sûr que cela soit très "propre", mais grâce à la déclaration d'une structure "static" ça doit fonctionner (du moins pas de problèmes mémoire ici)...

Cependant, ma remarque concernait en fait la modification du code par Medinoc, qui du coup remplit une structure non allouée...

[sandra771]

Comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
RetCode retCode;
struct tm *pt_Str;
char date[25];
 
strcpy(date, "2002-05-11T22:12:45.123");
 
retCode = fDateT_To_Struct(date, pt_Str);		
if (retCode != OK) {
/*gestion d'erreur*/		
}

Sandra

[Médinoc]

C'est à dire, mal.

Code :

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

1
2
3
4
5
6
7
8
9
10
RetCode retCode;
struct tm Str;
char date[25] = ""; /* ou char date[25]={0};*/
 
strncpy(date, "2002-05-11T22:12:45.123", 25-1);
 
retCode = fDateT_To_Struct(date, &Str);		
if (retCode != OK) {
/*gestion d'erreur*/		
}

Tu passais un pointeur non-initialisé à la fonction. Plantage pratiquement garanti, avec ou sans le test que j'avais oublié. Tu espérais que la fonction modifie le pointeur, alors que les variables sont passées par valeur en C ?