Discussion :

[jagboys]

Bonjour,

Je cherche à comprendre ce que je fais. J'ai cree un dll mfc qui permettera de retourner la date et heure courante. Voyant que le premier cas ne marchait pas, j'ai modifié cette fonction et là ça fonctionne bien !
Est ce que c'est un problème de partage mémoire dll / programme ?

1er cas :

Code :

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

1
2
3
4
5
6
7
8
__declspec(dllexport) void JGetCurrentDateTime(char* TimeFormat, char* retValue)
{
   char str_date_time[50];
   time(&n_DateTime);
   strftime(str_date_time,50,n_TimeFormat,localtime(&n_DateTime));
   retValue = (char*) malloc(strlen(str_date_time) + 8);
   strcpy(retValue,str_date_time);
}

2ieme cas :

Code :

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

1
2
3
4
5
6
7
8
9
10
__declspec(dllexport) char* JGetCurrentDateTime2(char* TimeFormat)
{
   char str_date_time[50];
   time(&n_DateTime);
   strftime(str_date_time,50,n_TimeFormat,localtime(&n_DateTime));
   char *retValue;
   retValue = (char*) malloc(strlen(str_date_time) + 8);
   strcpy(retValue,str_date_time);
   return retValue;
}

Dans le second cas, le programme principale reçoit bien la valeur, alors que dans le premier cas la valeur de retour reste toujours vide.

Est ce que quelqu'un aurait un expliquation ?

Merci d'avance.

[stephdim]

Salut,

Dans ton 1er cas, il faut que tu donnes une référence sur le pointeur retValue et non la valeur du pointeur retValue.

tu peux faire comme ça :

Code :

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

1
2
3
4
5
6
7
 
__declspec(dllexport) void JGetCurrentDateTime(char* TimeFormat, char** retValue)
{
  ...
  *retValue=(char*) malloc(strlen(str_date_time) + 8);
  strcpy(*retValue,str_date_time);
}

dans ce cas, c'est un pointeur sur un pointeur : tu donnes l'endroit ou ranger le pointeur de la nouvelle chaine

ou alors comme ça :

Code :

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

1
2
3
4
5
 
__declspec(dllexport) void JGetCurrentDateTime(char* TimeFormat, char*& retValue)
{
  // c'est pareil que ton code
}

ou là tu donnes une référence sur ton pointeur. d'un point de vue code généré par le compilateur, c'est exactement la meme chose.

@+

[Gabrielly]

bonjour,

Si tu utilises bien les MFC tu as la classe COleDateTime pour recupérer la date courante au lieu de te mêler les pinceaux avec time(), localtime et consor...

Code :

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

1
2
3
4
5
6
 
extern "C" void WINAPI AfxGetCurrentTime(CString* pstrCurTime)
{
        COleDateTime now = COleDateTime::GetCurrentTime();
        *pstrCurTime = now.Format("%d %b %Y");
}

et puis ben..., une fonction d'exportation pour récupérer la date courante n'est pas très utile je trouve, car la méthode static "COleDateTime::GetCurrentTime()" peut être appelée n'importe dans ton code que ce soit dans ta dll ou ton exe.

[Iznogood12]

Salut

Si dans ton premier cas retValue est nul c'est parce qu'il te manque & (comme te dis stephdim).

En fait & veut dire paramètre d'entrée/sortie. Alors que si tu ne met rien la variable est considérée comme un paramètre d'entré.

Voila deux petit exemple pour que tu comprenne bien :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
// Programme principal
x = 1;
Test (x);
cout<<"x = "<<x;
// Sous programme
void Test (int X)
{
x = x + 1;
}

dans cet exemple tu aura à l'écran x =1

alors que dans l'exemple qui suit :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
// Programme principal
x = 1;
Test (x);
cout<<"x = "<<x;
// Sous programme
void Test (int& x)
{
x = x + 1;
}

Tu aura à l'écran x = 2