Allouer un espace mémoire toujours à la même variable

Version imprimable

Bonjour,

Je programme un JNI en C++ et ma fonction native me génère des bugs mémoire (si j'appelle "trop" ma fonction native) et mon programme me donne de "mauvais" résultats.

Voici un bout de mon code :

Code:

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
36
37
38
39
40
41
42
JNIEXPORT jfloat JNICALL Java_appli_InterfacePI_piar_1value
  (JNIEnv *env, jobject obj, jint numpt, jint date)
{
    /* On crée une variable de stockage pour la dll */
    HINSTANCE PIAPI32;
    /* On charge la dll */
    PIAPI32 = LoadLibrary("piapi32.dll");
    if (PIAPI32 != NULL)
    {
        char *type = new char;
        long int *iStat = new long int;
        float *rval = new float;
        long int *mode = new long int;
        /* On a besoin du type du point pour connaitre la valeur à retourner */
        typedef long int (*_pipt_pointtype)(long int,char&);
        _pipt_pointtype pipt_pointtype;
        pipt_pointtype = (_pipt_pointtype)GetProcAddress(PIAPI32, "pipt_pointtype");
        pipt_pointtype(numpt,*type);
        typedef long int (*_piar_value)(long int,long int&,long int,float&,long int&);
        _piar_value piar_value;
        #define ARCVALUEBEFORE  1
        *mode=ARCVALUEBEFORE;
        /* On interface la fonction C++ avec la fonction de la dll */
        piar_value = (_piar_value)GetProcAddress(PIAPI32, "piar_value");
        /* On applique la fonction */
        piar_value(numpt,date,*mode,*rval,*iStat);
        /* On libère la dll */
        FreeLibrary(PIAPI32);
        /* On retourne la valeur courante du point suivant son type */
        delete mode;
        if (*type=='R')
        {
            return *rval;
        }
        else if (*type=='D')
        {
            /* On convertit l'entier en float */
            return static_cast<float>(*iStat);
        }
    }
}

J'ai deux questions :
1) Comment puis-je libérer la mémoire de mes variables en gras sachat que j'en ai besoin pour l'instruction return ? (Au passage ai-je bien utilisé l'instruction delete ?)
2) Sinon comment toujours allouer le même espace mémoire pour ces variables ?

Merci beaucoup, je suis complètement bloquée :/

Salut,

Te rend tu compte que tu ne libère la mémoire que de... mode :question:

tes variables type, rstat et rval ne prennent pas *énormément* d'espace mémoire, mais, à force d'allouer systématiquement de la mémoire pour eux sans la libérer, tu as une fuite mémoire qui, au final, arrivera à saturer n'importe quel système: ce sont toujours les petits ruisseaux qui font les gros fleuves :D.

Il n'y a, d'ailleurs, aucune raison de passer par l'allocation dynamique de la mémoire pour déclarer ces variables, d'autant plus que tu en arrive à... devoir passer en paramètre "ce qui est pointé" par tes pointeurs :aie:

Si tu modifie un tout petit peu ton code pour travailler avec des variables "classiques" au lieu de pointeurs, ton problème se résoudra "tout seul" :D

Au final, tu obtiendrait quelque chose comme

Code:

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
36
37
38
39
40
41
42
43
44
45
46
/* tu es toujours mieux de placer tes define hors fonction */
#define ARCVALUEBEFORE  1
/* Ceci dit, il est préférable de créer une constante:
 * const long ARCVALUEBEFORE = 1
 */
JNIEXPORT jfloat JNICALL Java_appli_InterfacePI_piar_1value
  (JNIEnv *env, jobject obj, jint numpt, jint date)
{
    /* On crée une variable de stockage pour la dll */
    HINSTANCE PIAPI32;
    /* On charge la dll */
    PIAPI32 = LoadLibrary("piapi32.dll");
    if (PIAPI32 != NULL)
    {
       /* la déclaration d'une variable suffit pour que C++ prévoie
        * l'espace mémoire suffisant pour la stocker
        */
        char type;
        long int iStat; // au fait: long iStat va très bien aussi :D
        float rval;
        long int mode;
        /* On a besoin du type du point pour connaitre la valeur à retourner */
        typedef long int (*_pipt_pointtype)(long int,char&);
        _pipt_pointtype pipt_pointtype;
        pipt_pointtype = (_pipt_pointtype)GetProcAddress(PIAPI32, "pipt_pointtype");
        pipt_pointtype(numpt,*type);
        typedef long int (*_piar_value)(long int,long int&,long int,float&,long int&);
        _piar_value piar_value;
        *mode=ARCVALUEBEFORE;
        /* On interface la fonction C++ avec la fonction de la dll */
        piar_value = (_piar_value)GetProcAddress(PIAPI32, "piar_value");
        /* On applique la fonction */
        piar_value(numpt,date,mode,rval,iStat);
        /* On libère la dll */
        FreeLibrary(PIAPI32);
        if (type=='R')
        {
            return rval;
        }
        else if (type=='D')
        {
            /* On convertit l'entier en float */
            return reinterpret_cast<float&>(iStat);
        }
    }
}

18/08/2010, 12h42
GatH7614

Oui oui je m'en rendais compte !!! D'où mon post !
Du coup, j'ai déclaré mes variables en variables globales et comme ça, le même espace mémoire est réutilisé.
Mais... car il y a toujours un mais.... :( ça ne résout pas mon problème pour autant :/

Bonjour,
Code:

1 2 Du coup, j'ai déclaré mes variables en variables globales et comme ça, le même espace mémoire est réutilisé.
Aucune raison de passer par des variables globales dans ton cas.
La solution que te propose koala01 est bien meilleure et plus "naturelle".

Il n'y a pas besoin de déclarer de pointeurs ou de faire des allocations :
Code:

1 2 3 4 5 6 7 8 /* la déclaration d'une variable suffit pour que C++ prévoie * l'espace mémoire suffisant pour la stocker */ char type; long int iStat; float rval; long int mode;
Ensuite, les paramètres d'entrée comme *pointeur doivent être remplacés par &valeur. En d'autres termes, l'appel
Code:

1 2 3 4 pipt_pointtype(numpt,*type); ... suite du code ... piar_value(numpt,date,*mode,*rval,*iStat);
devient
Code:

1 2 3 4 pipt_pointtype(numpt,&type); ... suite du code ... piar_value(numpt,date,&mode,&rval,&iStat);
Dans la réponse de koala01, les & n'apparaissent pas (à vérifier donc je me trompe peut-être).

Remarque 1 : j'ai l'impression que si PIAPI32 est NULL, tu ne renvoies rien dans ta fonction.

Remarque 2 : peut-être qu'il pourrait être judicieux de déclarer PIAPI32 de façon globale pour ne pas avoir à charger constamment ta dll si tu fais beaucoup d'appels à ta fonction.

Remarque 3 : koala01 a aussi changer ton static_cast en reinterpret_cast (voir la faq).

18/08/2010, 14h49
GatH7614
Merci pour toutes ces précisions !

Concernant :

Citation:

Envoyé par Aleph69

Remarque 2 : peut-être qu'il pourrait être judicieux de déclarer PIAPI32 de façon globale pour ne pas avoir à charger constamment ta dll si tu fais beaucoup d'appels à ta fonction.

Si je la mets en variable globale ça me génère une erreur ou alors je m'y prends mal (je la place dans l'en tête comme ceci
Code:

1 2 HINSTANCE PIAPI32;
). Comment déclarer une HINSTANCE en variable globale si je ne m'y prends pas correctement ?
18/08/2010, 16h06
Aleph69

Re,

il faut utiliser le mot-clé static pour signifier qu'une variable est globale.

Tu peux aussi faire comme lorsqu'on appelle du C++ depuis Java : charger ta dll à la construction d'une classe.
Dans ce cas tu peux faire en sorte que ta classe ne soit instanciée qu'une seule et unique fois comme expliqué dans la faq à cet endroit.
19/08/2010, 14h52
GatH7614

Merci pour ces réponses,

En fait le problème était dans le time out de mon interrogation serveur et non dans un problème mémoire, mais puisque j'utilise les mêmes variables dans des fonctions différentes, les déclarer en global est une bonne idée je pense !

Merci encore pour tout ! Je mets un résolu du coup !