Discussion :

[Bartuk]

Bonjour,

j'ai déclaré une variable de manière globale dans mon programme :

Code :

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

1
2
 
char * Reponse2=(char *)calloc(1000, sizeof(char));

J'ai plusieurs fonctions qui remplissent cette chaine de caractères avec différentes informations. Je voudrais qu'à chaque début de fonction, la chaine Reponse2 soit totalement vide. Je rentre mes infos dans Reponse2 de la manière suivante :

Code :

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

1
2
 
sprintf(Reponse2,"%s",temp);

Mon problème est que Reponse2 conserve les informations stockées dans Reponse2 des autres appels de fonction (etant donné que Reponse2 est globale). Je ne veux pas que Reponse2 soit une variable locale, elle doit rester globale.

J'ai essayer plusieurs méthodes pour la vider mais visiblement rien de bon... Je m'en remet à vous...
Merci par avance pour votre aide

[Trap D]

Tu alloues une fois au début de programme Reponse2, puis après l'avoir utilisé *Reponse2 = 0; . Mais bon c'est un peu curieux cette variable globale qui doit être réinitialisée à chaque fois ; problème de conception
D'autre part au lieu du

Code :

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

sprintf(Reponse2,"%s",temp);

tu pzeux faire un

Code :

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

strcpy(Reponse2, temp);

[hegros]

Code :

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

memset(Reponse2,0,1000)

[fearyourself]

Il y a tout de même un grand problème de conception comme l'a signalé TrapD mais je vais insister là-dessus:

- Une variable globale ou non qui est alloué dynamiquement mais a une taille fixe? Il vaut mieux mettre:

Code :

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

1
2
 
char Reponse2[1000];

Ensuite, utilisé directement la valeur 1000?

Utilises une macro! Lorsque tu va vouloir modifier la taille 1000 à 2000, tu vas parcourir tout ton code??

Enfin:

Je voudrais qu'à chaque début de fonction, la chaine Reponse2 soit totalement vide.

Effectivement avec un memset c'est faisable, mais il y a un problème de conception de programme, cela sera lourd et pas du tout optimal. Tu ferais cela avec moi, tu en entendrais parler...

[Bartuk]

Ok merci à tous les deux.
Le memset marche parfaitement, je l'avais visiblement mal utilisé auparavant

Pour la conception... il est possible que ce ne soit pas top top... Le problème est qu'il s'agit d'une dll appelée en permanence par un service. De nombreuses fonctions de cette dll sont utilisées. Au début j'utilisais des variables locales, mais comme je devais les renvoyer à la fin de la fonction je ne pouvais pas libérer la mémoire avec free et du coup au bout d'un certain nombre d'appel de fonction : saturation de mémoire. Du coup j'ai opté pour une variable globale unique qui prend pour chaque fonction la valeur associée à la fonction. Le problème que j'ai eu alors c'est que si une chaine plus grande que celle que je vais mettre a été affectée avant, alors il y aura des restes en fin de chaine, d'où le besoin de la vider à chaque appel de fonction. C'est curieux je sais, il faudra un jour que je me remette au C

En tout cas merci à tous les deux

[fearyourself]

Ok merci à tous les deux.
Le memset marche parfaitement, je l'avais visiblement mal utilisé auparavant

Pour la conception... il est possible que ce ne soit pas top top... Le problème est qu'il s'agit d'une dll appelée en permanence par un service. De nombreuses fonctions de cette dll sont utilisées. Au début j'utilisais des variables locales, mais comme je devais les renvoyer à la fin de la fonction je ne pouvais pas libérer la mémoire avec free et du coup au bout d'un certain nombre d'appel de fonction : saturation de mémoire. Du coup j'ai opté pour une variable globale unique qui prend pour chaque fonction la valeur associée à la fonction. Le problème que j'ai eu alors c'est que si une chaine plus grande que celle que je vais mettre a été affectée avant, alors il y aura des restes en fin de chaine, d'où le besoin de la vider à chaque appel de fonction. C'est curieux je sais, il faudra un jour que je me remette au C

En tout cas merci à tous les deux

Je n'ai jamais fait de dll mais une variable globale dans une dll, n'est-ce pas une hérésie?

Ensuite, pour ton problème, assure toi de mettre un '\0' à la fin de la chaîne et tu n'auras plus besoin de mettre tout le tableau à 0.

Jc

[Bartuk]

Je n'ai jamais fait de dll mais une variable globale dans une dll, n'est-ce pas une hérésie?

Effectivement c'est ce que je pense... c'est ma première dll, elle communique avec mon application J2EE. Malheureusement, comme je l'ai expliqué, je n'arrive pas à libérer la mémoire des variables locales entre le "return" de celle-ci et la fin de la fonction... mon post précédent à ce sujet était resté sans réponse... si vous avez une idée
Du coup j'utilise une variable globale pour éviter la saturation en mémoire, même si c'est loin d'êter optimal et qu'il m'est impossible de gérer l'espace alloué correctement...

[Emmanuel Delahaye]

Je n'ai jamais fait de dll mais une variable globale dans une dll, n'est-ce pas une hérésie?

Effectivement c'est ce que je pense...

Le principe de base est de définir le tableau au niveau utilisateur et de passer l'adresse du premier élément et le nombre d'élements à la fonction (fusse-t-elle en DLL)

Eventuellement, un code retour indique si la place a été suffisante ou non.

Sinon, la DLL pourrait très bien allouer le bloc (malloc()) et ce serait alors à la fonction appelante de prendre les précautions nécessaires pour libérer le bloc après usage.

Autre solution.

Une fonction d'évaluation qui retourne la taille nécessaire, puis la fonction 'normale' à qui on passe addresse du bloc l'alloué (et sa taille, pour contrôle éventuellement...)