Discussion :

[fabpeden]

bonjour, je suis en train de faire une dll en c que je dois appeler dans windev. Celle-ci doit me renvoyer une chaine de caractère de la forme suivante : "AAAAMMJJHHSSCC000", mais lorsque je l'exécute je ne récupère que "A", quelqu'un voit-il d'où cela pourrait venir, merci
voici mon code :

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
extern "C" __declspec(dllexport) char RecupererHeure(int nbh, time_t t)
{
#include <time.h>
 
/* Déclarations des données */
//int nbh;
char dte[100];
 
/* date et heure d'aujourd'hui */
//time_t t = time(0L);
 
/* nbh: nombre d'heures à soustraire */
t -= 3600*nbh;  
 
/* Calcul du jour recherché */
tm *tm = localtime(&t);
 
/* Concaténation des différentes valeurs (jour,mois,année,heure,minute)*/
sprintf(dte,"%04d%02d%02d%02d%02d000",tm->tm_year+1900,tm->tm_mon+1,tm->tm_mday,tm->tm_hour,tm->tm_min);
 
 
return *dte;
//system("pause");
}

[Franck.H]

Normal, regarde bien la signature de ta fonction:

Code :

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

char RecupererHeure(int nbh, time_t t)

sous cette forme elle renvoie simplement un char et non un char * donc,

Code :

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

char * RecupererHeure(int nbh, time_t t)

De même pour ton tableau, ici c'est un tableau local que tu veux renvoyer ... pas bien, il faut renvoyer un pointeur donc ton tableau au lieu de:

Code :

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

char dte[100];

sera:

Code :

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

char * dte = NULL;

il faut alors allouer cet espace soit:

Code :

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

1
2
3
4
5
6
dte = malloc (100);
 
if (dte != NULL)
{
   /* ton code */
}

[fabpeden]

Maintenant, j'ai modifié l'appel de la fonction *RecupereHeure et la valeur de retour (dte au lieu de *dte)

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
 
extern "C" __declspec(dllexport) char *RecupererHeure(int nbh, time_t t)
{
 
#include <time.h>
#include <string.h>
 
/* Déclarations des données */
//int nbh;
char dte[100];
 
/* date et heure d'aujourd'hui */
//time_t t = time(0L);
 
/* nbh: nombre d'heures à soustraire */
t -= 3600*nbh;  
 
/* Calcul du jour recherché */
tm *tm = localtime(&t);
 
/* Concaténation des différentes valeurs (jour,mois,année,heure,minute)*/
sprintf(dte,"%04d%02d%02d%02d%02d000",tm->tm_year+1900,tm->tm_mon+1,tm->tm_mday,tm->tm_hour,tm->tm_min);
 
/* Renvoi de la valeur */
cout << dte << endl;
system("pause");
return dte;
 
}

Cela ne m'envoie plus "2" mais une valeur "1240760", je pense que la mauvaise interprétation dit venir de windev et pas du C, je vais voir! Et il me donne toujours cette valeur quelque soit la date !merci beaucoup

[mchk0123]

Cela viens de comment sont gérés les chaînes de caractères en C.
En langage C, il n'y à pas de type "string". Le seul moyen est d'utiliser à la place un tableau de caractères. Le type caractère étant le "char".

Dans ton code dte est bien un tableau de char. Tout va bien. Par contre ta fonction renvoi un "char", cad un seul caractère. C'est pourquoi tu n'obtient que 'A'.

Ce que tu voudrais c'est renvoyer le tableau dte dans son intégralité. Maintenant, le problème est que dte est stockée dans une zone mémoire qui n'existe que le temps de l'appel RecupererHeure. Autrement dit, quand RecupererHeure se termine, dte n'existe plus. Donc le code suivant sera faux :

Code :

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

1
2
3
4
5
extern "C" __declspec(dllexport) char* RecupererHeure(int nbh, time_t t) {
...
 
return dte;
}

Le seul moyen d'y arriver et d'allouer une "string" depuis ton programme appellant en windev, de la passer en paramètre, de la modifier. Le paramètre sera alors en entrée/sortie de ta fonction. Exemple :

Code :

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

1
2
3
4
5
extern "C" __declspec(dllexport) void RecupererHeure(int nbh, time_t t, char *dte) {
...
sprintf(dte, ...);
// pas de return
}

EDIT : arf ... pas assez rapide

[Emmanuel Delahaye]

il faut renvoyer un pointeur donc ton tableau au lieu de:

Code :

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

char dte[100];

sera:

Code :

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

char * dte = NULL;

il faut alors allouer cet espace soit:

Code :

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

1
2
3
4
5
6
dte = malloc (100);
 
if (dte != NULL)
{
   /* ton code */
}

Je vois mal un codeur Windev s'embêter avec des blocs à libérer... Je passerais plutôt l'adresse du premier élément d'un tableau de char et sa taille à la fonction :

Code :

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

int RecupererHeure(char *text_out, size_t size, int nbh, time_t t)

et dans la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
  if (size >= sizeof "AAAAMMJJHHSSCC000")
  {
      /* OK */
      sprintf(...);
  }
  else
  {
      /* retourner un code d'erreur... */
  }

[fabpeden]

J'ai essayé et en fait ca marchouille! Lorsque je fais un trace sous windev (pour ceux qui connaissent pas c'est comme un printf), il me donne la valeur attendue, mais quand j'essaye juste après de récupérer cette valeur il me donne juste "2" et en plus cela bloque l'exécution du programme!

[Emmanuel Delahaye]

J'ai essayé et en fait ca marchouille! Lorsque je fais un trace sous windev (pour ceux qui connaissent pas c'est comme un printf), il me donne la valeur attendue, mais quand j'essaye juste après de récupérer cette valeur il me donne juste "2" et en plus cela bloque l'exécution du programme!

Tu as essayé quoi ? Parce que si tu t'obstines à retourner l'adresse d'une variable locale, le comportement est indéfini. C'est une certitude. Il peut arriver n'importe quoi. On t'a indiqué 2 alternatives.