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.