Discussion :

[dvdavid]

Bonjour,

Je voulais savoir comment faire pour définir la propriété ReadOnly pour tous les TextBox d'une Form. J'ai essayé ceci, mais ça ne marche pas:

Code :

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

1
2
3
4
5
 foreach (Control cc in this.Controls)
           {
                 if (cc is TextBox)
                     cc.ReadOnly = booActive;
           }

Il ne reconnait pas la propriété ReadOnly pour l'objet "cc", il me met :

Erreur 1 'System.Windows.Forms.Control' ne contient pas une définition pour 'ReadOnly' et aucune méthode d'extension 'ReadOnly' acceptant un premier argument de type 'System.Windows.Forms.Control' n'a été trouvée (une directive using ou une référence d'assembly est-elle manquante ?)

Merci de votre aide

[meziantou]

il faut caster cc en TextBox

Code :

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

1
2
3
4
5
6
7
 
foreach (Control cc in this.Controls)
{
    TextBox tb = cc as TextBox;
    if (tb != null)
        tb.ReadOnly = booActive;
}

[Themacleod1980]

ou

Code :

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

1
2
3
4
5
6
 
 foreach (Control cc in this.Controls)
           {
                 if (cc is TextBox)
                     (TextBox)cc.ReadOnly = booActive;
           }

car cc est un "Control" et non un TextBox.
après ton "if (cc is TextBox)", tu sais que ce "Control" est bel et bien un TextBox, donc tu le cast.

[Guulh]

Plus précisément:

Code :

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

1
2
3
4
5
 foreach (Control cc in this.Controls)
           {
                 if (cc is TextBox)
                     ((TextBox)cc).ReadOnly = booActive;
           }

Ou si t'es en framework 3.5 ou mieux, un peu de linq:

Code :

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

1
2
3
4
foreach (TextBox textBox in this.Controls.OfType<TextBox>())
           {
                     textBox.ReadOnly = booActive;
           }

[EDIT]corrigé pour la postérité suite à la remarque de meziantou

[meziantou]

Tu as fais une petite erreur dans ton code avec Linq. Dans le foreach il faut mettre TextBox et non Control

Code :

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

1
2
3
4
foreach (TextBox textBox in this.Controls.OfType<TextBox>())
{
    textBox.ReadOnly = booActive;
}

[mioux]

Tiens, pour faire ça, je fais comme ça moi

Code C# :

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
 
private void SetReadOnly(ControlCollection controls, bool isReadOnly)
{
    foreach (Control curControl in this.Controls)
    {
        if (curControl is TextBox)
        {
            ((TextBox)curcontrol).ReadOnly = isReadOnly;
        }
        SetReadOnly(curControl.Controls, isReadOnly);
    }
}
 
private void SetReadOnly(bool isReadOnly)
{
    SetReadOnly(this.Controls, isReadOnly);
}

J'ai donc une question con cachée derrière ce code : suis-je certifié à 100% en utilisant this.Controls de parcourir l'ensemble des contrôles du formulaire en cours ? Non parce que j'ai déjà eu des surprises avec des SplitContainer et la propriété ActiveControl (je m'explique, si on sélectionne une TextBox dans un SplitContainer dans une Form, Form.ActiveControl renvoie le SplitContainer et non la TextBox).

Je ne suis pas certain à 100%, parce que si je me souviens bien (je n'ai pas de Visual Studio sous la main pour faire de test), si on ajoute un contrôle dans un GroupBox ou autre élément englobant, Visual Studio fait un "this.groupBox1.Controls.Add(this.textBox1)", et je ne suis pas persuadé que textBox1 reste dans la liste des contrôle du formulaire.

[Aeronia]

Tiens, pour faire ça, je fais comme ça moi

Code C# :

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
 
private void SetReadOnly(ControlCollection controls, bool isReadOnly)
{
    foreach (Control curControl in this.Controls)
    {
        if (curControl is TextBox)
        {
            ((TextBox)curcontrol).ReadOnly = isReadOnly;
        }
        SetReadOnly(curControl.Controls, isReadOnly);
    }
}
 
private void SetReadOnly(bool isReadOnly)
{
    SetReadOnly(this.Controls, isReadOnly);
}

J'ai donc une question con cachée derrière ce code : suis-je certifié à 100% en utilisant this.Controls de parcourir l'ensemble des contrôles du formulaire en cours ? Non parce que j'ai déjà eu des surprises avec des SplitContainer et la propriété ActiveControl (je m'explique, si on sélectionne une TextBox dans un SplitContainer dans une Form, Form.ActiveControl renvoie le SplitContainer et non la TextBox).

Je ne suis pas certain à 100%, parce que si je me souviens bien (je n'ai pas de Visual Studio sous la main pour faire de test), si on ajoute un contrôle dans un GroupBox ou autre élément englobant, Visual Studio fait un "this.groupBox1.Controls.Add(this.textBox1)", et je ne suis pas persuadé que textBox1 reste dans la liste des contrôle du formulaire.

En effet, tu ne parcourra pas l’entièreté de tes controls en faisant de cette manière.

Pour se faire, tu peux utiliser une méthode récursive qui doit être simple à mettre en place, comme ceci par exemple :

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
 
//Dans le code de ta form ou de ton USerCOntrol ou autre ...
foreach(Control ctrl in Controls)
      SetReadOnly(ctrl,booActive);
//...
 
private void SetReadOnly(Control control,bool booActive)
{
    if(control is Panel)
    {
          Panel panel = (Panel)control;
          foreach(Control ctrl in panel.Controls)
                    SetReadOnly(ctrl,booActive);
    }
   else if (control is TextBox)
   {
           TextBox textbox=(TextBox)control;
           textbox.ReadOnly = booActive;
   }
 //Autre cas ...
}

A noter qu'on peut le faire via un switch etc...

[mioux]

C'est justement ce que j'ai fais (à la différence que je passe la liste des contrôles à la fonction récursive au lieu d'appliquer la fonction récursive pour chaque élément enfant, mais c'est un détail).

Le truc, c'est que sur les solutions proposées, à chaque fois, c'était pour l'ensemble des éléments de Form.Controls, et que je pensais (apparemment à juste titre ;D) qu'en parcourant uniquement Forms.Controls, on risquait de rater des éléments, du coup je complète avec ma fonction récursive ;D

(Au passage, pas besoin de faire un

Code C# :

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

1
2
3
4
5
6
7
8
9
10
11
12
if(control is Panel)
    {
          Panel panel = (Panel)control;
          foreach(Control ctrl in panel.Controls)
                    SetReadOnly(ctrl,booActive);
    }
   else if (control is TextBox)
   {
           TextBox textbox=(TextBox)control;
           textbox.ReadOnly = booActive;
   }
 //Autre cas ...

vu que la classe Control a une propriété Controls qui contient les contrôles enfants, sauf à vouloir appliquer la propriété ReadOnly à d'autres types d'éléments ;D)

[Aeronia]

Oups en effet, je n'avais pas fait gaffe au code que tu avais mis, mea culpa x)

A noter que pour le SplitContainer, tu vas être obliger de faire un traitement spécial car tu n'as pas de liste de controls sur ton SplitContainer mais c'est bien dans Panel1.Controls et dans Panel2.Controls que tu trouveras tes controles.

D'ou ma méthode :

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
 
//Dans le code de ta form ou de ton UserControl ou autre ...
foreach(Control ctrl in Controls)
      SetReadOnly(ctrl,booActive);
//...
 
private void SetReadOnly(Control control,bool booActive)
{
    if(control is Panel)
    {
          Panel panel = (Panel)control;
          foreach(Control ctrl in panel.Controls)
                    SetReadOnly(ctrl,booActive);
    }
   else if (control is TextBox)
   {
           TextBox textbox=(TextBox)control;
           textbox.ReadOnly = booActive;
   }
   else if (control is SplitContainer)
  {
          SplitContainer split = (SplitContainer)control;
          foreach(Control ctrl in split.Panel1.Controls)
                    SetReadOnly(ctrl,booActive);
          foreach(Control ctrl in split.Panel2.Controls)
                    SetReadOnly(ctrl,booActive);
  }
 //Autre cas ...
}

[dvdavid]

Merci à tous,
je ne pensais pas qu'une simple petite question comme celle-ci passionnerait autant les foules ...

En tout cas, merci encore pour toutes ces explications