C# テキストファイルの中身によってフォームの色を変更したい

Question

使用環境はwindows10、Visual Studio2019（Windows Forms アプリ、.NET Framework、バージョン4.8）です。

テキストファイル(「a.txt」、画像)を1行ずつ読み込み、１なら黄色、０なら赤色にフォームの色を変更（色が変更される様子が目で見えるように）したいです。

以下のように1行ずつ読み込むところまではできたのですが、どうやって色を変更するのかは検討がつきません。「if」「this.BackColor = color.RED」を使うのは予想できました。これをどう組み合わせればよいのでしょうか。ご教授頂けると幸いです。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.IO;

namespace _20211119_2
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            if (openFileDialog1.ShowDialog() == System.Windows.Forms.DialogResult.OK)
            {
                label1.Text = openFileDialog1.FileName;

                string file = openFileDialog1.FileName;
                StreamReader sr = new StreamReader(file, Encoding.GetEncoding("SHIFT_JIS"));
                while (sr.EndOfStream == false)
                {
                    string line = sr.ReadLine();
                    textBox1.Text += line + "\r\n";
                }
                sr.Close();
            }
        }

Comments

[BeatStar]

この仕様だとやめておいた方がいいかも。やってもいいですがおそらく無駄になる。
パパパパッと一気に切り替わり、最後のやつになるはずです。
やるなら、「次へ」とかみたいなボタンを配置して、そのボタンを押してから次のものに進むといった感じにしたほうがいいかと思います。(デバッガみたいに)

それと、プログラミングは「こう書けばいい」というものではありません。

[dodox86]

> 「if」「this.BackColor = color.RED」を使うのは予想できました。これをどう組み合わせればよいのでしょうか。

えと、プログラムのロジックが組めないということなのでしょうか。1行づつ読んでいるステップはあるのですから、そこに判定の処理を
入れれば良いのです。それが組めないというと、C#、ひいてはプログラミングの基本部分の学習不足です。

なお、実際にはただその処理を入れても簡単には色が変わる様子を観察することはできません。それを実現するのはUIの更新タイミングの話やタイマー、非同期など、別のトピックの知識が必要になります。

[退会済みユーザー]

前回の質問でも言いましたが、入門書などで基礎を固める所から始めてください。
質問サイトは、まったく何も知らない人に1から10まで懇切丁寧に指導する場所ではありません。
[質問するときのヒント] https://teratail.com/help/question-tips
[推奨していない質問] https://teratail.com/help/avoid-asking

[KOZ6.0]

「C# の最初のステップ」
https://docs.microsoft.com/ja-jp/learn/paths/csharp-first-steps/
このあたりで勉強するといいかもしれません。

[fana]

「ファイルから1とか0とかいうデータを読み込むこと」と，「フォームの色を変えること」とは独立した問題なので，後者に関して困っているのであればそこに話題を絞る形で質問すると良いのではないでしょうか．

1とか0とかいう値が１つずつ与えられるとき　→　その値に従ってフォームの色を変える

という話をすれば十分であるときには，「1とか0とかいう値はテキストファイルから読み出された」なんて話は雑音でしかないので．

Answer 1

[質問への追記・修正の依頼]に書きましたが，
とりあえず「テキストファイルがどうの」いう部分についてはどうでもいいと見えるので，
以下の話では単純に仮のデータ
var Data = new byte[] { 0, 1, 0, 1, 0, 1 };
に基づいて「フォームの色を変える」をやってみるとします．

↓のような順序で試しつつ，
そこに出てきた要素について調べる等してみればよいのではないかと思います．

(1)素直に書いてみる

まずは button1_Click メソッド内に率直に以下を書いてみます．

var Data = new byte[] { 0, 1, 0, 1, 0, 1 };  //仮のデータ
foreach (byte b in Data)
{
	this.BackColor = ( b==0 ? Color.Red : Color.Yellow );
}

実行してボタンを押すと，フォームの色が即黄色になります．
foreach でループで処理しているのに色変更は１回分だけしか観測できません．
なぜでしょう？

(2)処理速度を疑う

前記の要因として
「処理があまりにも早すぎるために人の目には途中経過が見えないからではないか？」
と疑ってみます．
そこで，「色を変えるごとに少し待つ」ようにしてみましょう．
（Data の部分は同一なので省略します）

foreach (byte b in Data)
{
	this.BackColor = ( b==0 ? Color.Red : Color.Yellow );
	System.Threading.Thread.Sleep( 250 );  //少し待たせてみる
}

実行すると…　やっぱり途中経過は見えません．
なぜでしょう？

(3)再描画させてみる

とりあえず，「色を変えろ」と言うだけではダメっぽいので，
追加で「フォームの再描画もしろ」と言ってみましょう．

foreach (byte b in Data)
{
	this.BackColor = ( b==0 ? Color.Red : Color.Yellow );  //「色を変えろ」
	this.Refresh();  //「再描画しろ」
	System.Threading.Thread.Sleep( 250 );  //少し待たせてみる
}

実行すると…　今度は途中経過が見えるかと思います．
なぜでしょう？
Refresh についてググりましょう．そこを出発点として描画の仕組み等に関するいろんな話を学べるでしょう．

さて，とりあえず色の変化経過が見えるようにはなりましたが，
色が　赤→黄色→赤→黄色→…　と変わっている間にフォームの位置を動かそうとか操作を試みてみれば，それができないことに気づくでしょう．
なぜでしょう？

試しに，this.Refresh(); を Application.DoEvents(); に書きかえてみれば，
色の変更処理中にもフォームを移動させたりすることができるようになります．
なぜでしょう？
Application.DoEvents についてググりましょう．
おそらく，相応にネガティブな話を読むことができるでしょうが，そこからメッセージドリブンがどうのこうのという動作の仕組みに関するいろんな話を学べるでしょう．
（そしてあなたは Application.DoEvents を用いることを棄却するでしょう）

(4)タイマーとか非同期処理とか…

運が良ければ，Application.DoEvents についての否定的な話が書かれている場所には，より良い方法の話が書かれているかもしれません．
仮にそういうのが見つけられないとしても，この説のタイトルにある単語（「タイマー」「非同期」）を手掛かりに調べてみてください．

---

[余談]
私自身，C#は素人同然なので「非同期ってどう書くの？」というのが気になったので今ググってみました．
その結果としてどうにかこうにか書いてみたのが以下です．
実装としての良し悪しの判断はできませんが，どこかしら参考になるかもしれないので示しておきます．

//なんか "async" って書く
private async void button1_Click(object sender, EventArgs e)
{
	button1.Enabled = false;  //処理中に再度ボタン押されたりすると面倒そうなので防止

	//非同期だそうで
	await Task.Run(
		() =>
		{
			var Data = new byte[] { 0, 1, 0, 1, 0, 1 };
			foreach (byte b in Data)
			{
				this.BeginInvoke(
					(Action<Color>)( (c)=>{	this.BackColor=c;	} ),
					( b==0 ? Color.Red : Color.Yellow )
				);
				System.Threading.Thread.Sleep( 250 );
			}
		}
	);

	button1.Enabled = true;  //処理が終わったらボタンを有効状態に戻す
}

Comments

[hono112]

できました。ご回答ありがとうございます。