Exception après la troisième impression

Version imprimable

Bonjour,

je suis débutant en c#, j'ai fait une appli qui permet d'imprimer le contenu d'une picturebox. Une foi la photo imprimé, je fais disparaître le bouton imprimer et réapparaître le bouton de sélection de photo une foi la photo sélectionnée, le bouton imprimer réapparaît. Lorsque j'essai d'imprimer pour la troisième fois j'ai une exception sur graphics.drawing.

commentaires:

1- si je n'utilse pas de timer, j'ai des exceptions (ArgumentsNullException) idem pour Thread.Sleep
2- le if flag =0 est la car je ne sais pas pourquoi mais la première foi que j'appui sur imprimer le code se lance 1 fois, la 2eme, 2 fois, la 3eme, 3 fois,.....

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
private void Print_Click(object sender3, EventArgs e3)
        {
 
            if (flag == 0)
            {
                myTimer3.Tick += new EventHandler(Timer3);
                flag = 1;
                myTimer3.Interval = 5;
 
             }
            myTimer3.Start();
 
        }
private void Timer3(Object myObject3, EventArgs myEventArgs3)
        {
            try
            {
 
                myTimer3.Stop();
                myTimer3.Dispose();
                Recom.Visible = false;
                Print.Visible = false;
                panel1.BackgroundImage = null;
                panel1.BackgroundImage = Image.FromFile("E:/livre/conf/loading.gif");
                pictureBox2.Visible = false;
                pictureBox5.Visible = false;               
 
                Thread.Sleep(interval2);
                imprimer();
                debut();
 
 
 
 
            }
            catch (Exception ex)
            {
                MessageBox.Show( ex.ToString());
            }
 
 
        }
 
private void imprimer() 
        {
                    pd.PrintPage += new PrintPageEventHandler(pd_PrintPage); 
                    pd.PrintPage += null;
                    pd.Print();
                    pd.Dispose();
 
        }
 
private void pd_PrintPage(object senderP, System.Drawing.Printing.PrintPageEventArgs ev)
        {
            try
            {
 
                    MessageBox.Show("1");
                    ev.Graphics.DrawImage(this.pictureBox5.Image, 0, 0, 390, 300);
                    ev.HasMorePages = false;
                    ev.Graphics.Dispose();
 
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.ToString());
 
            }
        }

Citation:

Exception:
SystemAccessViolation ev.Graphics.Drawing dans la fonction pd_printpage

Merci de m'aider

10/10/2011, 10h28
SLE

C'est normal, vous ajoutez chaque fois votre Timer3 au Tick de votre timer.
Une seule fois suffit !
Idem plus bas dans votre code...
10/10/2011, 10h30
kheironn

Je pense que, si tu dois utiliser un Timer pour éviter une ArgumeentNullException, c'est qu'il y a déjà un problème à ce niveau. Idem pour le Flag (?). Avant de t'initéresser à ton bug, cherche à résoudre ces deux points. Coder, ce n'est pas bidouiller.

Je n'ai pas le temps d'analyser et de tester ton code pour mieux te conseiller sur comment régler ces points. dsl.
10/10/2011, 10h31
xiringuito

ok merci pour le timer je vois, en revanche comment ça idem en bas du code?
10/10/2011, 10h35
SLE

Oui, dans votre fonction Imprimer()...

Vous pouvez également simplifier l'écriture

Code:

myTimer3.Tick += Timer3;

et si vous devez retirer l'event

Code:

myTimer3.Tick -= Timer3;

Et effectivement, comme le souligne kheironn, il y a un problème de conception dans ce code !

Ok je pense avoir saisi pour le timer.

j'ai écouté kheironn, si j'exécute mon code en direct sans timer ni rien, je sais pourquoi il me sort un nullexception, c'est simplement qu'il lance plusieurs foi la fonction mais pourquoi le code boucle?????
quand je click sur imprimer voila comment se déroule le programme: MsgBox("Imprimer"),MsgBox("fct pd_PrintPage"),MsgBox("fct pd_PrintPage"),MsgBox("Imprimer"),MsgBox("fct pd_PrintPage"),MsgBox("fct pd_PrintPage"),MsgBox("fct pd_PrintPage"),MsgBox("fct pd_PrintPage"), Donc à chaque fois le code boucle de plus belle.
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 private void Print_Click(object sender3, EventArgs e3) { MessageBox.Show("Imprimer"); Recom.Visible = false; Print.Visible = false; panel1.BackgroundImage = null; panel1.BackgroundImage = Image.FromFile("E:/livre/conf/loading.gif"); pictureBox2.Visible = false; pictureBox5.Visible = false; imprimer(); debut(); //dans cette fonction réinitialisation de pictureBox5: pictureBox5.Image=null; } private void imprimer() { pd.PrintPage += new PrintPageEventHandler(pd_PrintPage); pd.Print(); } private void pd_PrintPage(object senderP, System.Drawing.Printing.PrintPageEventArgs ev) { try { MessageBox.Show("Fonction pd_PrintPage"); ev.Graphics.DrawImage(this.pictureBox5.Image, 0, 0, 390, 300); ev.HasMorePages = false; ev.Graphics.Dispose(); } catch (Exception ex) { MessageBox.Show(ex.ToString()); } }

10/10/2011, 11h30
SLE
Comme je l'ai dit tout à l'heure, c'est normal puisqu'à chaque passage dans Imprimer() vous ajoutez la même méthode à votre event PrintPage.

Code:

pd.PrintPage += pd_PrintPage;

ne doit être défini qu'une seule fois, en dehors de la méthode Imprimer() => dans votre constructor par exemple.

Je ne vois pas non plus pourquoi vous faites
Code:

1 2 panel1.BackgroundImage = null; panel1.BackgroundImage = Image.FromFile("E:/livre/conf/loading.gif");
alors que

Code:

panel1.BackgroundImage = Image.FromFile("E:/livre/conf/loading.gif");

est suffisant. Pourquoi vouloir affecter un NULL avant ???
10/10/2011, 11h37
MaximePalmisano
Dans la lignée de :
Code:

1 2 panel1.BackgroundImage = null; panel1.BackgroundImage = Image.FromFile("E:/livre/conf/loading.gif");
Je ne comprends pas non plus :
Code:

1 2 pd.PrintPage += new PrintPageEventHandler(pd_PrintPage); pd.PrintPage += null;
Peut être que ça a une utilité que je ne connais pas mais s'abonner à null, je ne savais même pas que c'était possible x)
10/10/2011, 12h11
kheironn
Citation:
Envoyé par MaximePalmisano

Dans la lignée de :

Code:

1 2 panel1.BackgroundImage = null; panel1.BackgroundImage = Image.FromFile("E:/livre/conf/loading.gif");

Je ne comprends pas non plus :

Code:

1 2 pd.PrintPage += new PrintPageEventHandler(pd_PrintPage); pd.PrintPage += null;

Peut être que ça a une utilité que je ne connais pas mais s'abonner à null, je ne savais même pas que c'était possible x)
:ccool: s'abonner à null ? Quelle drôle d'idée. Je veux bien savoir pourquoi.

Code:

panel1.BackgroundImage = null;

ne sert à rien puisque tu lui donnes immédiatement après la valeur qui va bien.
Tu dois bien penser à ce dont tu as besoin AVANT de coder, prendre un bout de papier, écrire, PUIS JOUER dans ta tête le code et ENFIN coder. Des fautes de codes aussi grossières sont dues à une manque de recule avant le codage.
Tu es débutant en programmation, il te manque donc encore des réflexes qui viendront avec l'expérience ;)

Merci beaucoup pour votre aide. C'est juste que j'ai pris l'habitude pour une picturebox de passer son image à null avant de la remplir, dans une appli précédente si je ne faisais pas ça ça plantait. donc ici comme il s'agissait d'une image j'ai fait pareil, sans trop réfléchir.

Merci encore pour votre aide. Maintenant le dernier problème que j'ai du code qui se lance 2 fois.

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
 
 
 
        System.Windows.Forms.Timer myTimer = new System.Windows.Forms.Timer() ;
 
 
        string backgrd="";
        PrintDocument pd = new PrintDocument();
 
 
 
        private  void Timer(Object myObject, EventArgs myEventArgs)
        {
            myTimer.Stop();
            myTimer.Tick -= Timer;
            CaptureInfo.CaptureFrame();
 
 
        }
 
        public Form1()
        {
            InitializeComponent();
            pd.PrintPage += pd_PrintPage;
 
        }
 
        private void ErrorMessage(Exception error)
        {
            MessageBox.Show(error.Message, "Webcam Test", MessageBoxButtons.OK, MessageBoxIcon.Error);
        }
 
 
 
 
 
        private void debut()
        {
 
 
 
            Flash.Movie = "E:/livre/conf/photo.swf";
            panel1.BackgroundImage = Image.FromFile(backgrd);
            pictureBox1.Visible = true;
            pictureBox2.Visible = false;
            label4.Visible = true;
            label5.Visible = true;   
            Recom.Visible = false;
            Recom.Movie = null;
            Print.Visible = false;
            Print.Movie = null;
            //pictureBox2.Image = null;
            Flash.Visible = true;
 
 
 
        }
        private void Print_Click(object sender3, EventArgs e3)
        {
 
 
                Recom.Visible = false;
                Print.Visible = false;
                panel1.BackgroundImage = Image.FromFile("E:/livre/conf/loading.gif");
                pictureBox2.Visible = false;
                pictureBox5.Visible = false;                
                imprimer();
                debut();
 
        }
        private void imprimer()
        {
            MessageBox.Show("2");         
            pd.Print();         
        }
 
        private void pd_PrintPage(object senderP, System.Drawing.Printing.PrintPageEventArgs ev)
        {
            try
            {
                    MessageBox.Show("1");
                    ev.Graphics.DrawImage(this.pictureBox5.Image, 0, 0, 390, 300);
                    ev.HasMorePages = false;                             
 
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.ToString());
            }
        }
 
 
        private void Flash_Enter(object sender, EventArgs e)
        {
            myTimer.Tick += Timer;
            myTimer.Interval = 4000;
            myTimer.Start();
 
        }
 
 
 
        private void Recom_Enter_1(object sender2, EventArgs e2)
        {
 
            MessageBox.Show("3");
            debut();
 
 
        }

le code de la méthode Recom_Enter_1 et Print_click s’exécute systématiquement 2 fois. Ces méthodes servent à des boutons flash lorsque la souris survol le code est lancé. Le truc qui colle pas c'est que ça ne le fait pas avec Flash_Enter.

10/10/2011, 15h49
kheironn

As-tu fais du pas à pas pour voir à quel moment elles se lancent ? En analysant celà tu trouveras où se fait le mauvais appel.
10/10/2011, 15h51
SLE

Retirez la ligne

Code:

myTimer.Tick += Timer;

de Flash_Enter et mettez-là dans le constructeur. Retirez aussi

Code:

myTimer.Tick -= Timer;

de votre procedure Timer.
10/10/2011, 16h55
xiringuito

j'ai déplacé myTimer, j'essai le pas a pas, mais la je doit dire je bloque. je met un msg box apres chaque ligne de ma methode "debut" et je note qu'elle se déroule entièrement et 2 fois. le prblème est bien dans Recom_Enter. mais j'ai qu'une seulle ligne dans cette méthode. Comment puis-je faire pour trouver ou ca débloque?