Discussion :

[xiringuito]

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 :

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

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

Merci de m'aider

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

[xiringuito]

ok merci pour le timer je vois, en revanche comment ça idem en bas du code?

[SLE]

Oui, dans votre fonction Imprimer()...

Vous pouvez également simplifier l'écriture

Code :

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

myTimer3.Tick += Timer3;

et si vous devez retirer l'event

Code :

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

myTimer3.Tick -= Timer3;

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

[xiringuito]

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 :

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

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

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

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 :

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

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

alors que

Code :

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

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

est suffisant. Pourquoi vouloir affecter un NULL avant ???

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