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

**GatH7614** · 18/08/2010, 09h50

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 :

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
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 :/

**koala01** · 18/08/2010, 10h45

Salut,

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

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

.

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

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"

Au final, tu obtiendrait quelque chose comme

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
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);
        }
    }
}

**GatH7614** · 18/08/2010, 11h42

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 :/

**Aleph69** · 18/08/2010, 12h25

Bonjour,

Code :

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

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 :

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

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 :

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

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

devient

Code :

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

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).

**GatH7614** · 18/08/2010, 13h49

Merci pour toutes ces précisions !

Concernant :

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 :

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

1
2
 
HINSTANCE PIAPI32;

). Comment déclarer une HINSTANCE en variable globale si je ne m'y prends pas correctement ?

**Aleph69** · 18/08/2010, 15h06

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.

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

C++

Vue hybride

Discussions similaires

Partager

Partager