Discussion :

[flamme34]

Bonjour,

Je code actuellement un programme sous Borland C++ Builder6.

lors d'une vérification de fichier je dois faire un bilan afin de faire savoir à l'utilisateur que certain de ces fichiers ne sont pas valide.

Or lorsque j'appelle souvent ma fenêtre de bilan je fini par avoir une exception "Ouf of ressource"
je ne comprend pas pourquoi car je créée ma fenêtre puis la détruis après.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
if ( bilan)
{
        TForm_DeclanchementPAsGerer * p_fen1 = new  TForm_DeclanchementPAsGerer(this);
 
        isInPause = true;
        p_fen1->ShowModal();
        isInPause= false;
 
        delete p_fen1;
}
p_fen1 = NULL

lorsque je n'affiche pas de bilan je n'ai pas de soucis.
Ce "out of resource" n'est pas régulié. Autant au bout de 1 affichage j'ai le message d'erreur et autant une autre fois après 10 affichage toujours rien.

Merci d'avance pour votre aide .

[gbdivers]

Bonjour

Je vois 3 sources potentielles :
* la création d'une instance de TForm_DeclanchementPAsGerer alloue d'autres ressources, qui ne sont pas correctement libéré. Faut voir le code de cette classe
* es tu sur que la source du problème soit bien à cette endroit ?
* le programme quitte ta fonction entre le new et le delete... mais tu le saurais je suppose si tu avais une exception lancée

En tout cas, je t'encourage à utiliser les pointeurs intelligents (unique_ptr ici)

[flamme34]

tout d'abord Merci d'avoir pris le temps de me répondre.

Je viens de revérifier afin d'être sûr de chez sûr, il n'y a aucune ressource qui pourrait ne pas être libérer lors du delete de ma classe. Cette fenêtre contient uniquement du texte, un bouton OK pour fermer la fenêtre, et un TMemo ou je stock également du texte.

En faisant du pas à pas j'ai pu observer que mon code passe bien par le delete.

Je ne suis pas tout a fait sûr que le problème vienne de là mais j'ai constaté que lorsque je n'affichais pas de bilan je n'observais pas ce message de OutOfRessource.

Ce problème n'étant pas répétable a un instant donnée, j'ai du mal a savoir en pas à pas quand celui ci va se produire et par la même observé a quelle moment exacte surgit le problème.

EDIT : Je viens par hasard de tomber sur le moment exact où le problème se produit. Le message m’apparaît au moment de l'instanciation de la fenêtre.

EDIT2 : Je viens également de tester l'implémentation avec des std::unique_ptr mais malgrès le fait que j'ai fait l'include <memory> le compilateur me dis qu'il ne connais pas

Code :

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

TForm_DeclanchementPAsGerer * p_fen1 = new  TForm_DeclanchementPAsGerer(this);

[Bousk]

Bonjour,

Je trouve ton code pour le moins étrange.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if ( bilan)
{
        TForm_DeclanchementPAsGerer * p_fen1 = new  TForm_DeclanchementPAsGerer(this);
 
        isInPause = true;
        p_fen1->ShowModal();
        isInPause= false;
 
        delete p_fen1;
}
p_fen1 = NULL

Bon, passons sur le ; oublié à la fin, mais... ça compile ça ?!
p_fen1 n'est pas sensé être connu en sorti du bloc puisque sa portée est limité à ce bloc.
Pourquoi utiliser une allocation dynamique ?
p_fen1 ne serait-il pas déclaré également ailleurs ? Aucun warning, aucune erreur ?
A quoi servent les flag de pause ? Une fenêtre modal a de toutes façons la priorité jusqu'à sa fermeture. S'agit-il d'un contexte multi-thread ? Des allocations sont faîtes dans ces autres threads ?

[flamme34]

il s'agit d'une erreur de recopi.

J'ai enlevé des lignes n'ayant pas de rapport avec mes fenêtres.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if ( bilan)
{
        TForm_DeclanchementPAsGerer * p_fen1 = new  TForm_DeclanchementPAsGerer(this);
 
        isInPause = true;
        p_fen1->ShowModal();
        isInPause= false;
 
        delete p_fen1;
p_fen1 = NULL;
}

Voici le code propre.

il s'agit bien d'une application multithread. les flags de pause me permette de synchroniser avec un autre thread.

Cette fenêtre n'est utilisé/instantié que dans cette fonction.

[gbdivers]

@Bousk
Bien vu...

J'ai enlevé des lignes n'ayant pas de rapport avec mes fenêtres.

Ca commence à faire beaucoup de chose que l'on voit pas, sans être sur que la source du problème ne vient pas de là


Concernant unique_ptr, comme tu n'as pas un compilateur à jour, il faut utiliser boost