Discussion :

[AMP29]

Bonjour,

J'ai une série de PictureBox dont les images changent lorsque l'on clique dessus (on/off), chacune individuellement.
Les PictureBox représentent des numéros, et, au clic, en plus du changement d'image, le numéro correspondant est également soit ajouté soit retiré d'une liste string (nommée Combi dans le code ci-dessous).

J'utilise une méthode commune pour traiter l'état on/off de chaque image suite au clic

Le principe :
Les noms des PictureBox comportent le numéro de l'image qu'elles doivent afficher.
Par exemple "pictureBox6", affichera soit l'image "6.png" soit l'image "6b.png"
Dans le code ci-dessous le clic sur l'image affichera alternativement Regular_Image ou Regular_Image + "b" suivant l'image déjà affichée.

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
 
        private void Common_Number_Click(object sender, EventArgs e)
        {
 
 
            Shift_Number(sender);
            // d'autres trucs 
 
        }
 
        private void Shift_Number (object sender)
        {
            string name = (sender as System.Windows.Forms.Control).Name;
 
 
            string Regular_Image = name.Substring(10); // extraction du numéro d'image à partir du nom de la PictureBox
            string ShiftImage = Regular_Image + "b";
 
            var pictureBox = sender as PictureBox;
 
            if (pictureBox.Image.Tag.ToString() == Regular_Image)
            {
 
                    Bitmap image = (Bitmap)Tyest.Properties.Resources.ResourceManager.GetObject(ShiftImage);
                    pictureBox.Image = image;
                    pictureBox.Image.Tag = Regular_Image + "b";
 
                    Combi.Add(Regular_Image);
 
 
            }
            else
            {
 
                Bitmap image = (Bitmap)Tyest.Properties.Resources.ResourceManager.GetObject(Regular_Image);
                pictureBox.Image = image;
                pictureBox.Image.Tag = Regular_Image;
 
                Combi.Remove(Regular_Image);
 
            }
        }

Jusque là tout va bien.

Mais, si je souhaite à l'aide d'un bouton "Clear" mettre toutes les image sur "OFF" en une seule fois, comment faire pour entrer dans ces méthodes avec le bon objet "sender" ?
Elles sont toutes deux basées sur un type d'expéditeur PictureBox.

Le code suivant ne fonctionne évidemment pas

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
        private void EraseBtn_Click(object sender, EventArgs e)
        {
 
            foreach (string Numero in Combi) // Combi est la liste des numéros sur ON (variable globale)
            {
                var Picture = sender as PictureBox;
                Picture.Name = "pictureBox" + Numero;
                Common_Number_Click(Picture, e);
            }
        }

Le "sender" qui est le bouton "Clear" n'est pas castable en PictureBox.

Dans "EraseBtn_Click" le truc plante à

Code :

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

1
2
 
Picture.Name = "pictureBox" + Numero;

Il y-a-t'il un moyen de créer dans ma méthode un object PictureBox, puis le renommer avec le nom de la bonne PictureBox (réelle) dont je veux changer l'image ?
Ou une façon quelconque de parvenir à changer toutes les images en une seule fois ?

Merci pour votre aide

[PixelJuice]

Bonjour,

C'est très simple, au lieu de stocker dans ta liste le nom de chaque Picture Box, et bien tu stock directement la Picture Box .

Comme ça tu as directement accès au nom pour tes histoires de noms, et à l'appel de la méthode EraseBtn_Click, tu n'auras qu'à utiliser Clear() directement.

Tu peux même stocker ça dans un Dictionary<string, PictureBox> ou la clé est le nom, si jamais tu veux récupérer ça rapidement / facilement.

Evidemment tu auras pu aussi faire une correspondance avec le nom et tous les contrôles de ta Form, de Type Picture Box, mais c'est fastidieux pour rien.

[AMP29]

Génial !

J'ai créé

Code :

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

1
2
 
        public Dictionary<string, PictureBox> NumPic = new Dictionary<string,PictureBox>();

Que je rempli et vide pareil que ma liste Combi qui comporte le numéro des images

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
 
            if (pictureBox.Image.Tag.ToString() == Regular_Image)
            {
                if (NBNum < 5)
                {
                    Bitmap image = (Bitmap)Tytest.Properties.Resources.ResourceManager.GetObject(ShiftImage);
                    pictureBox.Image = image;
                    pictureBox.Image.Tag = Regular_Image + "b";
                    NBNum++;
                    if (NBNum > 4)
                    {
                        TestCombiBtn.Visible = true;
                        ScorePanel.Visible = true;
 
                    }
                    Combi.Add(Regular_Image);
                    NumPic.Add(Regular_Image, pictureBox); //<<<<<<
 
                }
            }
            else
            {
 
                Bitmap image = (Bitmap)Tytest.Properties.Resources.ResourceManager.GetObject(Regular_Image);
                pictureBox.Image = image;
                pictureBox.Image.Tag = Regular_Image;
                NBNum--;
                if (NBNum < 5)
                {
                    TestCombiBtn.Visible = false;
                    ScorePanel.Visible = false;
                }
                Combi.Remove(Regular_Image);
                NumPic.Remove(Regular_Image); //<<<<<<<<<<
 
            }

Ma méthode "Erase" devient

Code :

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

1
2
3
4
5
6
7
8
 
        private void EraseBtn_Click(object sender, EventArgs e)
        {
             foreach (PictureBox Pic in NumPic.Values)
            {
                 Common_Number_Click(Pic, e);
            }
        }

Je n'y aurais jamais pensé.
(Il y a moyen d'améliorer le code en supprimant ou modifiant les trucs en trop, mais j'avais trop hâte de tester !)

Merci ++++++ PixelJuice

[umfred]

Tu aurais aussi pu naviguer dans les PictureBox de la collection de contrôles de ta form

Code :

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

1
2
3
4
5
6
7
foreach (Control pic in Form1.Controls)
{
    if(pic is PictureBox)
    {
        Common_Number_Click(Pic, e);
    }
}

Après, si tu stockes le nom du contrôle, tu peux aussi accéder au contrôle via Form1.Controls("PictureBox1")
(Form1 le nom de ta Form, mais tu peux le remplacer par this)
donc tu pourrais aussi faire

Code :

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

1
2
3
4
5
 
for(i=NBMIN; i<=NBMAX; i++)
{
    Common_Number_Click(this.Controls("pictureBox" + i),e)
}

avec NBMIN la valeur Mini (0 par exemple si tu as pictureBox0) et NBMAX la valeur max (9 si tu as pictureBox9 par exemple)

[AMP29]

Merci umfred.

Dans le cas présent je n'ai pas besoin de parcourir tous les éléments PictureBox de ma form car ils ne sont pas tous à "inverser".
Seuls ceux qui sont sur ON doivent l'être.
Ceux que je place dans ma liste.

Mais cette manière d'obtenir des informations sur tous les éléments d'un ensemble de contrôles me plait bien.

Je retiens

Code :

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

1
2
 
Common_Number_Click(this.Controls("pictureBox" + i),e)

qui répond également à ce que je n'arrivais pas à faire, avec "i" le numéro des images dans ma liste initiale (Combi).
C'est une manière de faire peut-être plus parcimonieuse que de devoir créer une variable globale "Dictionnary" pour l'occasion.
Super !