[c#] groupbox et picturebox

Version imprimable

06/03/2006, 20h11
topolino

[c#] groupbox et picturebox
Bonjour,

dans un groupbox j'ai généré 16 pictureBox.

Pour un tratement je fais un
Code:

1 2 foreach(PictureBox oPictureBox in this.groupBox1.Controls){ // code }
il me compte les 8 1er picture box et les autres <valeur indefinies> ?!?

Pourquoi ?

et que dois je faire pour les avoirs tous ?

merci
06/03/2006, 21h36
NiamorH

je ne sais pas d'ou peut venir ton erreur mais c'est toujours un risque de plantage de faire un for each comme ça.

Il vaut mieux déclarer une variable de type Control et tester si le type = picturebox au moins pas de plantage possible si il tombe sur un autre type.

Maintenant je ne pense pas que ça résoudra ton problème, mais tu peux toujours essayer.
Peut- etre que tu as oublié qqc car ça parait étrange qu'il se trompe en comptant ?!
06/03/2006, 21h50
topolino

Bizarre oui.
Lorsuqe je fais un groupBox.Controls.Count j'ai le bon nombre de pictoreBox, mais lorsque je fais un for classique, meme pb.

Je ne sais plusquoi faire
06/03/2006, 23h13
SLE

Peux-tu montrer le reste du 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
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
 
		private void createThumb(string cPictureName){
			System.Drawing.Bitmap oBitmap;
			System.Drawing.Bitmap oBitmapTarget;
 
			if(groupBox1.Controls.Count>0) groupBox1.Controls.Clear();
			if(groupBox2.Controls.Count>0) listView1.Clear();
 
			string[] arrayPictureNames;
			arrayPictureNames = cPictureName.Split(new char [] {';'});
 
			listView1.CheckBoxes = true;
			listView1.View = View.Details;
			listView1.Scrollable = true;
			listView1.Sorting = SortOrder.Ascending;
 
			// Creation des colonnes
 
			ColumnHeader header1, header2; 
			header1 = new ColumnHeader(); 
			header2 = new ColumnHeader(); 
 
			header1.Text = "Aperçu"; 
			header1.TextAlign = HorizontalAlignment.Left; 
			header1.Width = 500; 
 
			header2.TextAlign = HorizontalAlignment.Left; 
			header2.Text = "Date"; 
			header2.Width = 150; 
 
			//Ajout des colonnes
			listView1.Columns.Add(header1); 
			listView1.Columns.Add(header2); 
 
			System.IO.DirectoryInfo di = new System.IO.DirectoryInfo(@"C:\Inetpub\application\test\wallpaperManagment\fondEcranImage\");
 
			int nI = 0;
			int nX = 0;
			int nY = 0;
			int nSep = 5;
			int nLocationX = 120;
			int nLocationY = 72;
			int nJ = 0;
 
			foreach (System.IO.FileInfo fi in di.GetFiles()) {
				foreach(string cName in arrayPictureNames){
					if(cName!=fi.Name){
						// pictureBox1
						this.pictureBox1 = new System.Windows.Forms.PictureBox();
						this.pictureBox1.Location = new System.Drawing.Point(16+nX, 24+nY);
						this.pictureBox1.Name = fi.Name;
						this.pictureBox1.Size = new System.Drawing.Size(nLocationX, nLocationY);
						this.pictureBox1.TabIndex = 0;
						this.pictureBox1.TabStop = false;
						this.groupBox1.Controls.Add(this.pictureBox1);
 
						oBitmap = new Bitmap(fi.FullName);
						oBitmapTarget = new Bitmap(pictureBox1.Width-2,pictureBox1.Height-2);
						Graphics oGraphics = Graphics.FromImage(oBitmapTarget);
						oGraphics.DrawImage(oBitmap,new System.Drawing.Rectangle(0, 0, pictureBox1.Width,pictureBox1.Height)); 
 
						pictureBox1.Name = fi.Name;
						pictureBox1.Image = (Image) oBitmapTarget;
						pictureBox1.Click += new System.EventHandler(this.pictureBox1_Click);
 
						ListViewItem item = new ListViewItem(fi.Name.ToString());
						item.SubItems.Add(fi.CreationTime.ToString()); 
 
						listView1.Items.Add(item); 
 
						nI++;
 
						if(nI<6){
                                                        nX = nLocationX*nI+nSep*nI;
                                                }else{
                                                        nI=0;
                                                        nJ++;
                                                        nX=0;
                                                        nY = nLocationY*nJ+nSep*nJ;
                                                }
					}
				}
			}
			for(int nM=0;nM<=groupBox1.Controls.Count;nM++){
                                groupBox1.Controls[nJ].Dispose();
                        }
 
		}

Le code bizarre :

Code:

1
2
3
4
 
for(int nM=0;nM<=groupBox1.Controls.Count;nM++){
            groupBox1.Controls[nJ].Dispose();
         }

Il me compte bien le bon nombre de controls mais j'en perd la moitié si je fais par exemple : groupBox1.Controls[nJ].Name

07/03/2006, 11h59
SLE
OK J'ai compris ce qui ne vas pas...

Tu dois faire ton dispose() dans le sens inverse (sinon le tableau se tasse et tu sautes 1 ligne sur 2). Donc il faut commencer par la fin du tableau de contrôles, comme ceci :
Code:

1 2 3 for(int nM=groupBox1.Controls.Count - 1; nM>=0;nM--){ groupBox1.Controls[nM].Dispose(); }
@+
07/03/2006, 13h51
topolino

Ton exemple marche et merci à toi.

Mais je ne comprends pas, car si au lieu de ".dispose()" je fais simplement un ".Name", en gardans ma solution et bien j'ai le meme pb.

Et pourquoi j'ai le meme pb avec un foreach ?!?

En tout les cas ta solution marche tout le temps, c'est seulement que j'ai envie de comprendre.
07/03/2006, 14h00
SLE

:koi: Alors là je pige pas non plus !

J'avais déjà eu ce genre de problème, que j'avais résolu en parcourant le tableau de la fin vers le début, et l'explication que je t'ai donné me semblait la plus logique, mais si .Name ne fonctionne pas non plus alors je suis très intéressé d'avoir l'explication aussi !?!?!

@+
07/03/2006, 14h04
topolino

En tout cas merci.

Si deja grace a ton astuce on regle ce bug, c'est deja mieux que rien.

Merci
07/03/2006, 15h02
Harry_polin

J'ai remarqué un truc,

quand on fait un foreach, le système considère que la collection qu'on parcoure n'est pas modifiée en terme de nombre d'éléments. Donc si on la modifie dans le foreach, soit ca plante, soit il lève une exception.

En revanche, avec un for(int etc... ca marche bien).
bien sur il faut partir de la fin, car si on supprime en itérant positivement, en réalité on en supprime 1 sur 2.
je supprime i,
i+1 devient donc i,
je fais i++,
je supprime i, (donc en fait je suis en train de supprimer le i+2 du i précédent)
07/03/2006, 16h52
topolino

je suis d'accord avec toi si l'on veut supprimer un element.

Mais dans mon cas, meme en ne faisant que element[i].Name j'en perdais la moitié !!!!
07/03/2006, 17h05
Harry_polin

Je ne comprends pas trop le lien entre "truc.Name" et la suppression d'un item...
07/03/2006, 17h17
SLE

Citation:

Envoyé par Harry_polin

Je ne comprends pas trop le lien entre "truc.Name" et la suppression d'un item...

:koi: Ben justement c'est bien ce qu'il dit !!! Même s'il fait un simple MessageBox.Show(c.Name) dans le foreach au lieu d'un Dispose(), le tableau de controls semble se tasser aussi, alors qu'il n'y a pas de suppression...

Je vais tester pour voir si j'ai le même phénomène... Bizarre quand même !