[BSD 2006] Erreur de violation d'accès sur un ShowModal

Version imprimable

[BSD 2006] Erreur de violation d'accès sur un ShowModal et Print

Bonjour.
Alors voilà, j'ai plusieurs form dans mon projet qui sont toutes appelées à partir de ma form principales par la fonction ShowModal().
Mais voilà, la dernière form que je viens d'ajouter me mets à chaque fois une message d'erreur de Violation d'accès mémoire et je ne sais pas pourquoi.
Voici le code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 AnsiString password = ""; while(Application->MessageBox("Souhaitez-vous associer un mot de passe avec ce fichier ?", "Mot de passe", MB_YESNO + MB_DEFBUTTON2 + MB_ICONQUESTION) == ID_YES) { Application->MessageBox("ATTENTION, veillez à mettre un mot de passe dont vous vous souviendrez...", "ATTENTION", MB_OK); PasswordDlg->ShowModal(); //erreur sur cette ligne if(password == "Mot de passe de fichier annulé") { Application->MessageBox("Vous avez annulé la saisie du mot de passe.", "Annulation mot de passe", MB_OK); } else { break; } }
Voilà, si quelqu'un vois pourquoi ca plante...

14/09/2006, 17h01
bandit boy

Salut,
question idiote, mais ta Form a-t-elle déjà été créé lorsque tu l'appelles?
14/09/2006, 17h31
doudoustephane
Citation:

Envoyé par bandit boy

Salut,
question idiote, mais ta Form a-t-elle déjà été créé lorsque tu l'appelles?

salut.
tu veux dire quoi par là?
car j'ai au lancement de mon appli les lignes :
Code:

1 2 3 4 Application->CreateForm(__classid(TForm1), &Form1); Application->CreateForm(__classid(TPasswordDlg), &PasswordDlg); Application->Run();
et le code précédent sur action d'un bouton de ma form1...
14/09/2006, 17h39
kurkaine

Peut-être teste si "PasswordDlg != NULL" avant de faire l'appel au ShowModal() ? Ton code fonctionne bien chez moi. Donc à priori, je ne vois que ça.
14/09/2006, 19h04
bandit boy

En fait tu pouvais avoir ce genre de problème quand tu mets ton code sur un OnFormCreate, c'est pour ca.

Sinon je rejoins l'idée de kurkaine, teste si ta Form existe avant.
17/09/2006, 11h43
doudoustephane

BOnjour à tous.
Merci pour vos réponses.
Pour le moment, ça à l'air de marcher...
Je vais continuer à tester quand même et voir si ça marche vraiment où si c'était un passage comme dèsfois où ca marche...
je vous tiens au courant si j'ai toujours un problème.
En attendant, je mets le sujet en :resolu:...
:merci: à vous
21/09/2006, 10h07
doudoustephane
Rebonjour à tous
Alors me revoilà avec mon problème...:lol:
et donc, comme je le craignais, j'ai pourtant laisser comme vous me l'aviez dit le

Code:

if(PasswordDlg != NULL)

, ca continue toujours de planter sur le ShowModal!!!
Que faire???
Surtout que vu qu'il plante sur le ShowModal, ca signifie que PasswordDlg n'est pas NULL!!!!
Là, je suis très ennuyé du coup car ca me bloque....

Et voilà, et maintenant, ça plante aussi avec la même erreur sur un print venant de TCustomForm, je fais juste dans la fonction du clic sur un bouton :
Code:

1 2 3 4 { maForm->Print(); } //ça se met en erreur ici
si quelqu'un a aussi une idée...???:bug: :bug: :arf: :arf: :pleure: :pleure: :pleure: :piou: :piou:
21/09/2006, 13h11
henderson

Salut !

Ici c'est pas évident de trouver l'origine du bug !

Le mieux serait sans doute de construire cette dialog non pas in situ mais juste avant l'appel et la détruire au retour de l'appel.

Peut-être que ce n'est pas possible dans l'immédiat.. (?) par exemple si le programme principal utilise des données contenues dans cette dialog ... ça va te donner du travail supplémentaire...

En contre partie, on aurait la garantie que cette dialog existe.

Sinon, chercher dans le code existant tout ce qui concerne cette dialog et en particulier rechercher si elle n'est pas détruite quelque part !

Peut-être que tu vas trouver Action = caFree dans sa OnClose... ???

C'est peut-être l'explication du pointeur non NULL vers une zone mémoire qui n'est plus allouée.

A plus !
21/09/2006, 14h00
doudoustephane

qu'entends-tu alors par "créer juste avant l'appel" et surtout, comment le fais-tu?

Salut !

Rappel de ce que tu as donné au préalable :
Code:

1 2 3 Application->CreateForm(__classid(TForm1), &Form1); Application->CreateForm(__classid(TPasswordDlg), &PasswordDlg); Application->Run();
Donc ça devient :
Code:

1 2 3 4 Application->CreateForm(__classid(TForm1), &Form1); //On passe outre la construction in situ //Application->CreateForm(__classid(TPasswordDlg), &PasswordDlg); Application->Run();
L'appel :
Code:

1 2 3 4 5 6 7 8 9 TPasswordDlg *Passw = new TPasswordDlg(Owner); //Peut-être initialiser quelque chose avant de la montrer ??? Puis : if(Passw->ShowModal() == mrOk) { } else { } delete Passw;
A noter que TPasswordDlg ne doit pas se détruire dans sa OnClose (ne pas y faire Action = caFree).
On doit donc avoir lors du click sur ses boutons (qui normalement vont agir sur le ModalResult de la fiche) un simple Hide();
Celà dit, peut-être as-tu mis en place un autre mécanisme pour fermer cette dialog ?

Pour la suite du bug je ne vois pas MyForm ? Existe t-elle ?

A plus !

21/09/2006, 17h22
doudoustephane
Citation:
Envoyé par henderson

Salut !

Rappel de ce que tu as donné au préalable :

Code:

1 2 3 Application->CreateForm(__classid(TForm1), &Form1); Application->CreateForm(__classid(TPasswordDlg), &PasswordDlg); Application->Run();

Donc ça devient :

Code:

1 2 3 4 Application->CreateForm(__classid(TForm1), &Form1); //On passe outre la construction in situ //Application->CreateForm(__classid(TPasswordDlg), &PasswordDlg); Application->Run();

L'appel :

Code:

1 2 3 4 5 6 7 8 9 TPasswordDlg *Passw = new TPasswordDlg(Owner); //Peut-être initialiser quelque chose avant de la montrer ??? Puis : if(Passw->ShowModal() == mrOk) { } else { } delete Passw;

A noter que TPasswordDlg ne doit pas se détruire dans sa OnClose (ne pas y faire Action = caFree).
On doit donc avoir lors du click sur ses boutons (qui normalement vont agir sur le ModalResult de la fiche) un simple Hide();
Celà dit, peut-être as-tu mis en place un autre mécanisme pour fermer cette dialog ?

Pour la suite du bug je ne vois pas MyForm ? Existe t-elle ?

A plus !
ok, je teste ca et je te dis.
pour la suite du beug, oui, la form existe, c'est simplement qu'au début, je vous ai mis qu'une partie des initialisations...
le print est d'ailleurs fait sur le clic d'une bouton se trouvant sur cette form

sinon, je ferme ma form password avec visible = false...
21/09/2006, 17h49
doudoustephane

bon, alors en fait, je viens de refaire différent test et vu que le fait de déclarer juste avant et de mettre Hide à la place de Visible non plus, je me suis dis (après avoir fait du pas à pas et découvert qu'un des problèmes était mon code pour masquer la fenêtre), essayons sur rien... et là, ça marche et la fenêtre disparaît bien toute seule.
Cela peut-il venir du fait que j'ai utilisé (quand je l'ai créé avec fichier->nouveau) directement une form pour mot de passe et que celles-ci serait préconfiguré pour se fermer ??? et que donc, fermer deux fois et impossible ?

sinon, pour mon problème d'impression avec le print, ça ne peut pas être ça car j'ai tout créé moi-même et je ne ferme jamais la fenêtre durant cette action là. donc, je n'ai toujours pas d'idé pour ça...

[EDIT]
en fait, j'ai remarqué que comme mon autre boîte de dialogue, le fait que je clique sur mon bouton fait se fermer ma boîte de dialogue sans que je lui demande et du coup, le print qui fait un imprime-ecran plante...
[/EDIT]

Salut !

La boite de dialogue telle que me la fournit BCB (3 Pro) se charge elle-même de se fermer soit par l'intermédaire des 2 boutons (OK et CANCEL) soit avec un RETURN sur le TEdit.

Ton code devrait être pour l'appel et la décision :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 int result = PasswordDlg->ShowModal(); { //Ici le ...== "" ....n'est qu'un exemple... if((PasswordDlg->Password->Text == "") || (result == mrCancel)) { //ShowMessage("Pas de mot de passe !"); } else { //ShowMessage("Mot de passe à traiter"); } }
Jusque la, ça fonctionne chez moi !

Il n'y a rien d'autre à ajouter à cette boite de dialogue !

Au départ je pensais que tu l'avais construite par toi-même (Il m'arrive de voir des problèmes là où il n'y en a pas).

Il faudrait se remettre au diapason : qu'est-ce qui marche et qu'est-ce qui ne marche pas ?

A plus !

Citation:
Envoyé par henderson
Salut !

La boite de dialogue telle que me la fournit BCB (3 Pro) se charge elle-même de se fermer soit par l'intermédaire des 2 boutons (OK et CANCEL) soit avec un RETURN sur le TEdit.

Ton code devrait être pour l'appel et la décision :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 int result = PasswordDlg->ShowModal(); { //Ici le ...== "" ....n'est qu'un exemple... if((PasswordDlg->Password->Text == "") || (result == mrCancel)) { //ShowMessage("Pas de mot de passe !"); } else { //ShowMessage("Mot de passe à traiter"); } }
Jusque la, ça fonctionne chez moi !

Il n'y a rien d'autre à ajouter à cette boite de dialogue !

Au départ je pensais que tu l'avais construite par toi-même (Il m'arrive de voir des problèmes là où il n'y en a pas).

Il faudrait se remettre au diapason : qu'est-ce qui marche et qu'est-ce qui ne marche pas ?

A plus !
D'accord, mais ca n'explique pas le fait que mon print() plante comme ca aussi!? si?

24/09/2006, 23h05
doudoustephane

toujours aucune idée SVP ???
je continue de mon côté mais je ne trouve pas pourquoi...:( :(
25/09/2006, 11h33
henderson

Salut !

Comment veux-tu qu'on trouve le bug à partir d'une seule ligne :

Citation:

MyForm->Print();

Mettre un breakpoint sur la ligne pour tenter d'y voir un peu plus clair au niveau du pointeur.
La solution se trouve forcément au niveau du pointeur (MyForm) et au niveau de l'objet qui se trouve au bout :
- soit il n'y a pas d'objet (NULL)
- soit il n'y a plus d'objet (le pointeur n'est pas NULL mais la mémoire a été libérée)
- soit l'objet n'hérite pas de TCustomForm (ça peut arriver en cas de transtypage barbare)
- soit la mémoire allouée pour cette form n'est pas dans le scope de l'exécutable (une histoire de dll ???)

Il n'y a que toi qui peux le trouver !

A plus !
27/09/2006, 18h58
doudoustephane

Citation:

Envoyé par henderson

Salut !

Comment veux-tu qu'on trouve le bug à partir d'une seule ligne :

Mettre un breakpoint sur la ligne pour tenter d'y voir un peu plus clair au niveau du pointeur.
La solution se trouve forcément au niveau du pointeur (MyForm) et au niveau de l'objet qui se trouve au bout :
- soit il n'y a pas d'objet (NULL)
- soit il n'y a plus d'objet (le pointeur n'est pas NULL mais la mémoire a été libérée)
- soit l'objet n'hérite pas de TCustomForm (ça peut arriver en cas de transtypage barbare)
- soit la mémoire allouée pour cette form n'est pas dans le scope de l'exécutable (une histoire de dll ???)

Il n'y a que toi qui peux le trouver !

A plus !

alors voilà en pièces jointes ce que j'ai dans le débogueur. Mais ce qui me surprent encore plus, c'est que ça avait toujours marché jusqu'à maintenant...!!!:(
et là, dans ce cas, que dois-je faire ?
27/09/2006, 20h52
henderson

Salut !

Dans un premier temps :

Code:

Print();

et non pas :

Code:

Graphe_Trace->Print();

Ca permet de fixer l'appel depuis l'instance qui est sollicitée et non pas depuis une autre instance dont on ne sait même pas si elle n'a pas été détruite (moi... je n'en sais rien) !

C'est juste une idée... parce qu'ici (du coup) je ne sais même pas si Graph_Trace est une instance de TGraph_Trace...

A suivre !

C'est space ce que t'écris car on est dans la classe TGraphe_Trace et tu as déclaré une classe dans tes attributs privés du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 class TGraphe_Trace{ private: TGraphe_Trace *Graphe_Trace; public: TGraphe_Trace (); // Constructeur void Print(); } TGraphe_Trace::TGraphe_Trace (){ Graphe_Trace = new TGraphe_Trace(); } TGraphe_Trace::Print(){ ... } ... Graphe_Trace->Print(); }
J'espère que tu rends compte que tu as dans une classe T, un attribut de type T. J'avais vu ces cas rares durant mes études, mais je ne m'en rappelle plus l'utilisation. Donc je ne peux pas te dire si ta méthode est correcte d'un point de vue c++.

Par contre, si ce n'est pas le cas, comme l'a dit henderson, dans un cas normal, tu dois juste écrire Print(), si tu veux faire appel à ta fonction print(). De plus, en C++, la magie des pointeurs fait que ça marche de temps en temps et dés fois, sans rien modifier, ça plante. Là, y'a pas de mystère c'est que t'as une "fuite de mémoire" et que ton programme ne marche pas. Du coup, la règle est que si ça marche une fois sur deux, considère que ça ne marche pas du tout.

J'espère que je ne raconte pas de bétise quand même, si c'est le cas, merci de me corriger :oops:

28/09/2006, 09h21
pottiez

Aucune bétise rassure toi.
28/09/2006, 18h21
doudoustephane

je test ça et je vous tiens au courant...
merci d'avance déjà pour vos réponses...

[EDIT]
Voilà, j'ai fait ce que vous m'avez dit, soit juste Print(); mais ça ne marche toujours pas...:( :( J'ai toujours le message comme quoi Print doit prendre un emplacement mémoire.
Sinon, vous ne connaissez pas un moyen d'imprimer ma form qui pourrait remplacer mon print ???
merci à vous.
[/EDIT]
28/09/2006, 20h48
henderson

Salut !

Avec TPrinter (voir l'aide car il doit y avoir des exemples, en particulier TPrinter::TCanvas).

Il y a deux possibilités :
Soit traiter directement de Canvas à Canvas (Form ==> Printer) à tester en premier car c'est ce qu'il y a de plus simple.
Soit on procède à l'aide d'un bitmap.
Ce dernier peut très bien être celui sur lequel on dessine (malgré le double buffer) ou bien celui obtenu par TForm::GetFormImage.
Ca dépend de ce qu'il y a à imprimer !

A plus !

Il ne faut pas renoncer devant ce bug ! Il y a forcément une raison !
29/09/2006, 13h14
doudoustephane

Citation:

Envoyé par henderson

Salut !

Avec TPrinter (voir l'aide car il doit y avoir des exemples, en particulier TPrinter::TCanvas).

Il y a deux possibilités :
Soit traiter directement de Canvas à Canvas (Form ==> Printer) à tester en premier car c'est ce qu'il y a de plus simple.
Soit on procède à l'aide d'un bitmap.
Ce dernier peut très bien être celui sur lequel on dessine (malgré le double buffer) ou bien celui obtenu par TForm::GetFormImage.
Ca dépend de ce qu'il y a à imprimer !

A plus !

Ok, merci pour l'info, je connais effectivement le canvas de TPrinter puisque je l'utilise dans une autre partie de mon code.
Mais là, je souhaite un imprime écran car il s'agit d'un graphique...
Je vais donc essayer de voir ta deuxième solution

Citation:

Envoyé par henderson

Il ne faut pas renoncer devant ce bug ! Il y a forcément une raison !

En effet, il y a forcément une raison, mais laquelle???
Personnellement, je ne vois pas, et apparement, vous ne voyez pas non plus... en tout cas, je suis ok pour trouver la solution si vous avez une idée... (si besoin de plus d'info, vous pouvez demander...)

Bref, pour le moment, je crois que je n'ai pas trop de choix qui s'offre à moi...:(
Vivement que ce problème soit résolu...
29/09/2006, 13h27
kurkaine

Je pense qu'il serait bien que tu envoie ton source complet s'il n'est pas trop long, du moins que la déclaration de ta classe, ta méthode Print, ton showModal qui plante. Bref qu'on puisse tester sur nos machines.

C'est compliqué lorsqu'on a pas tous les éléments sous les yeux. Mais en tout cas, c'est clair qu'il faut trouver la solution. Courage.
29/09/2006, 13h40
doudoustephane

Citation:

Envoyé par kurkaine

Je pense qu'il serait bien que tu envoie ton source complet s'il n'est pas trop long, du moins que la déclaration de ta classe, ta méthode Print, ton showModal qui plante. Bref qu'on puisse tester sur nos machines.

C'est compliqué lorsqu'on a pas tous les éléments sous les yeux. Mais en tout cas, c'est clair qu'il faut trouver la solution. Courage.

oui, mais le truc, c'est que c'est C++ BUilder et donc, c'est lui qui fait tout. Je ne fait qu'appeler les méthodes Print et ShowModal, je ne sais meme pas ou elles sont écrites... et c'est lui aussi qui les a déclarées...
29/09/2006, 15h25
kurkaine

Sorry, j'ai créer une appli avec un simple bouton, dont le code est Print();, lorsque j'appel cette fonction, je n'ai pas d'erreur et j'imprime ma fiche. Essaie de faire la même chose, créer une appli simple ou tu as un simple bouton. Ensuite, ajoute 1 à 1 des fichiers jusqu'à trouvé l'origine probable de l'erreur.
09/10/2006, 16h41
doudoustephane

Citation:

Envoyé par kurkaine

Sorry, j'ai créer une appli avec un simple bouton, dont le code est Print();, lorsque j'appel cette fonction, je n'ai pas d'erreur et j'imprime ma fiche. Essaie de faire la même chose, créer une appli simple ou tu as un simple bouton. Ensuite, ajoute 1 à 1 des fichiers jusqu'à trouvé l'origine probable de l'erreur.

le problème, c'est que je ne peux pas faire ça car j'ai besoin d'informations d'un autre fichier pour cette form là...
09/10/2006, 17h10
doudoustephane

je pense avoir trouvé la raison de cette erreur sur ma fonction Print() mais poouvez-vous me confirmer STP...
Pour moi, cela viendrait du fait que je cherche à imprimer sur une imprimante qui n'est pas réellement connecté à l'ordinateur et que donc, celui-ci ne peut pas envoyer les informations et plante ...
Cela vous parrait-il normale ? Est-ce que ce pourrait être la raison de mon erreur ?
Merci d'avance
10/10/2006, 14h32
kurkaine

Installe une imprimante pdf et tu verras bien. Tu peux trouver des freeware, cherche sur des sites de software. En fait, tu auras une imprimante installer sauf que dés que tu imprimes, ton document est imprimer au format pdf. Mais le comportement est le même que pour une imprimante physique.
10/10/2006, 14h36
doudoustephane

Citation:

Envoyé par kurkaine

Installe une imprimante pdf et tu verras bien. Tu peux trouver des freeware, cherche sur des sites de software. En fait, tu auras une imprimante installer sauf que dés que tu imprimes, ton document est imprimer au format pdf. Mais le comportement est le même que pour une imprimante physique.

oui, je sais, je connais, c'est justement grâce à cela que j'ai vu que ça marchait lorsque l'imprimante est connectée. Ce que je voulais savoir maintenant, c'est si je peux généraliser cela ou pas, cad : ça plante quand l'imprimante est pas présente et ça plante pas si elle est présente ?
VOilà, :merci: à toi quand même...
10/10/2006, 15h22
blondelle

Salut doudouallemand:
Une idee comme ca.
Est ce que tu ne peut pas gerer l'erreur avec try, catch, throw, qui te dit que l'imprimante est absente, mais qui ne bloque pas le programme.
10/10/2006, 15h43
doudoustephane

Citation:

Envoyé par blondelle

Salut doudouallemand:
Une idee comme ca.
Est ce que tu ne peut pas gerer l'erreur avec try, catch, throw, qui te dit que l'imprimante est absente, mais qui ne bloque pas le programme.

euh, bah si mais je n'y avais même pas pensé à vrai dire...:oops: :oops:
Super idée ça, je vais faire cela...
:merci: à toi de cette idée
14/10/2006, 09h52
doudoustephane

Citation:

Envoyé par blondelle

Salut doudouallemand:
Une idee comme ca.
Est ce que tu ne peut pas gerer l'erreur avec try, catch, throw, qui te dit que l'imprimante est absente, mais qui ne bloque pas le programme.

Salut.
Alors voilà, j'ai été long mais j'ai enfin testé le try-catch et j'ai un gros problème : L'ERREUR PASSE AU TRAVERS ET N'EST PAS INTERCEPTE PAR LE CATCH!!!!! j'ai jamais vu ça...8O 8O :lol: 8O 8O

Salut !

Peut-être en testant le nombre d'imprimantes :
Code:

1 2 3 4 5 6 7 8 if(Printer()->Printers->Count != 0) { //...donc il existe au moins un dispositif compatible imprimante... } else { ShowMessage("Aucune imprimante !"); }
Ca ne résoud pas tout (pour l'instant) puisque l'on peut avoir un FAX et pas d'imprimante !

A plus !

14/10/2006, 11h03
doudoustephane
Citation:
Envoyé par henderson

Salut !

Peut-être en testant le nombre d'imprimantes :

Code:

1 2 3 4 5 6 7 8 if(Printer()->Printers->Count != 0) { //...donc il existe au moins un dispositif compatible imprimante... } else { ShowMessage("Aucune imprimante !"); }

Ca ne résoud pas tout (pour l'instant) puisque l'on peut avoir un FAX et pas d'imprimante !

A plus !
Oui, mais ça ne répond pas non plus à mon problème qui n'est pas de savoir si l'on a une imprimante ou pas, mais de savoir si l'imprimante par défaut est connectée ou pas...
:merci: d'avance
14/10/2006, 11h51
henderson

Salut !

Peut-être en se servant des API WINDOWS :

OpenPrinter il me semble que ça concerne le matériel

GetPrinter pour la quelle il y a pusieurs PRINTER_INFO en particulier PRINTER_INFO_2

ClosePrinter

Ce qui me pose un problème est de récupérer le nom de l'imprimante par défaut... ???

A plus !
14/10/2006, 12h10
doudoustephane

Citation:

Envoyé par henderson

Salut !

Peut-être en se servant des API WINDOWS :

OpenPrinter il me semble que ça concerne le matériel

GetPrinter pour la quelle il y a pusieurs PRINTER_INFO en particulier PRINTER_INFO_2

ClosePrinter

Ce qui me pose un problème est de récupérer le nom de l'imprimante par défaut... ???

A plus !

ok, je vois ça cette aprem et je te redis, merci

Fuseau horaire GMT +2. Il est actuellement 00h27.