C# と VB.NET の質問掲示板

こんにちは、Form に配置された textBox1〜3 の入力チェックをしたいと思ってます。
Formには textBox1〜4 の4つが配置されており今後も数個増えていく予定です。

まずtextBox1〜3 の入力チェックをしたいのでテスト的に文字列0 及び null でメッセージ
を出す、プロシージャを作りました。以下となります

        public static void checkTextBox(Control c)
        {
            // c 内コントロールを列挙
            foreach (Control t in c.Controls)
            {
                do
                {
                    if (!(t is TextBoxBase)) { break; }　←ここの書き方も不安です・・・
                    if (t.Name.ToString() == "textBox4") { break; }
                    if (t.Text == null || t.Text.Length == 0)
                        {
                            MessageBox.Show(t.Name);
                            break;
                        }
                } while (true);
            }
        }
        private void button2_Click(object sender, EventArgs e)
        {
            checkTextBox(this);
        }

今後、foreach内でネストが深くなるのを避けたいと思っております。
また、今後textBox1〜3だけではなくなる事も有りますので、どの様に組めば
効率的なのか悩んでいます。
誠に恐縮ですが、アドバイス頂けたら幸いかと思っております。

何卒、宜しくお願い致します。

■No47460 (msnr さん) に返信
> 今後、foreach内でネストが深くなるのを避けたいと思っております。

そもそも、毎回、Controls から列挙しようとするのではなく、
ループで操作しやすいよう、事前に Load イベントの時点で
　private List<TextBox> textBoxes;
などに蓄えておく事をおすすめします。

その方が、今後、TextBox の名前が変更された場合にも有利ですし、あるいは
「TextBox1,3,4」と「TextBox2,5,7」を分けて管理したいといった場合でも、
都合が良いと思います。


> foreach (Control t in c.Controls)
t.Controls も含めて、再帰的に探索しなくても大丈夫でしょうか?

今のコードは、指定されたコントロールの直下にある物しか見ていないので、

　Form1
　├panel1
　│├textBox1
　│├panel2
　││└textBox2
　│└textBox3
　└textBox4

のような構造の時に『checkTextBox(panel1)』を行うと、
textBox1, textBox3 は得られますが、textBox2 は操作されません。



> foreach (Control t in c.Controls)
> {
>     do
>     {
>         if (!(t is TextBoxBase)) { break; }　←ここの書き方も不安です・・・
>         if (t.Name.ToString() == "textBox4") { break; }
>         if (t.Text == null || t.Text.Length == 0)
>         {
>             MessageBox.Show(t.Name);
>             break;
>         }
>     } while (true);
> }

そのコードだと、3 つの break 条件いずれにもヒットしなかった場合、
無限ループに陥ってしまいますよ。


> ←ここの書き方も不安です・・・
内側のループは不要だと思います。TextBox のみを対象としたいなら、
　foreach (Control t in c.Controls)
　{
　　if(!(t is TextBoxBase)) continue;
　　　　：
　}
とすればよいかと。


また、C# 2008 以降であれば if 判定も不要で、
　foreach (TextBoxBase t in c.Controls.OfType<TextBoxBase>())
　{
　　　　：
　}
で済ます事もできます。


> if (t.Text == null || t.Text.Length == 0)
前者の条件は、あまり意味が無いと思います。

TextBox の場合、Text プロパティが null を返すことはなく、
null をセットしても "" として扱われる仕様になっていますので、
この場合には、
   if( string.IsNullOrEmpty(t.Text) )
でどうでしょうか。

あるいは、Control.Text.Length で取得する代わりに、
TextBoxBase.TextLength プロパティを使うという手もあります。

皆さま、いつもお世話になっております。

この度は、皆さまに色々なアドバイスをもらいました結果
とりあえず、実現したい事は出来ました。

＞“なにを”教授して欲しいのでしょうか？フォームを設計した時点で入力エリアは
＞決まっています。それを列挙することの、何が解らないのでしょうか？
Jitta on the way　様のおっしゃる通り、フォームの設計を見直し
コントロールにパネルを１つ配置しチェックしたいコントロールを
中に入れ、ループを回す方法に変えました。
現在の私の力ではこの方法が１番安易で効率的だと考えた結果です。
導いて頂き、ありがとうございます。

また前述とは違い、戻り値を空白のテキストボックス数に変更しました。
コードは以下となります。

大変厚かましいですが、何かアドバイス頂けると非常に嬉しいです。。。
自信がないので・・・
お時間の許す限り、お付き合いください。
宜しくお願いいたします。　m(__)m

        public static int checkTextBox(Control c)
        {
            int cnt = 0;
            // c 内コントロールを列挙
            foreach (Control t in c.Controls)
            {
                if (!(t is TextBoxBase)) continue;
                if (string.IsNullOrEmpty(t.Text))
                {
                    cnt++;
                }
            }
            return cnt;
        }
        private void button2_Click(object sender, EventArgs e)
        {
            int i = checkTextBox(this.panel1);
            MessageBox.Show(i.ToString());
        }