同じボタンを押すたびに同じラベルに違う文字列を表示するには？

Question

よろしくお願いします。
初心者なのですがC# において一つのフォームの中に一つのボタンと一つのラベルがあって、
例えばボタンを1回押すとラベルの表示が初期表示のaからbになる、２回目押すと
cになって、もう１度押すとaに戻ってきて、それを繰り返す、
という処理は出来ますか？
例えばボタンクリックの処理の中に

        private void button1_Click(object sender, EventArgs e)
        {
            
            label1.Text = "b";
            if (label1.Text == "b")
                label1.Text = "c";


        }

と書くといきなりラベルにcが表示されます。
とりあえず書いてみた全体のコードです。

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

namespace aiueo
{


    public partial class Form1 : Form
    {


        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            
            label1.Text = "b";
            if (label1.Text == "b")
                label1.Text = "c";


        }

    }

}

while(true) や配列の処理を使うのでしょうか？

Comments

[退会済みユーザー]

質問する際は何を何で作っているかを質問の一行目に書きましょう。（例: Windows Forms アプリを Visual Studio 2022 でフレームワークを .NET Framework 4.8 として作っています・・・とか）

[dodox86]

> と書くといきなりラベルにcが表示されます。
...
> label1.Text = "b";
> if (label1.Text == "b")
> label1.Text = "c";

いや、いきなり自分で”b”をセットしているのですから当然なのですが、、、処理の流れを考えましょう。それとも自分では分からないので、「どのようにコードを書いたら要望通りの動きになりますか？」と言う質問でしょうか。

[sgsgsg1983]

二人の追記者ありがとうございます。
Windows Forms アプリを Visual Studio 2022で作ってました。
舌足らずですみませんでした。
あと上記のような処理をそもそもできるのかと、出来れば実装のコードも
知りたくて質問させていただきました。

Answer 1

まずなぜいきなりCが表示されかですが、
label1.Text = "b";
上記でlabel1.Textにｂを設定しています。
if (label1.Text == "b")
上記では、label1.Textにbが設定されているため、必ずIF文が成立します。
label1.Text = "c";
よって、上記が必ず実行されます。
なので、毎回Cが表示されることになります。

べたな書き方ですが、理解しやすい実装としては下記になります。

        public Form1()
        {
            InitializeComponent();
            label1.Text = "a";
        }

        private void button1_Click(object sender, EventArgs e)
        {
            if (label1.Text == "a"){
                label1.Text = "b";
            }else if(label1.Text == "b"){
                label1.Text = "c";
            }else{
                label1.Text = "a";
            }
        }

Comments

[sgsgsg1983]

教えてくださったコードでとりあえず動きました。
ありがとうございました。

[Zuishin]

低評価した時はちゃんと理由を書きましょう。

[Zuishin]

ちゃんと低評価しやすいよう、隙のある回答をしたんだから書けるでしょう？

[kikukiku]

ちなみにコメントせずに低評価することはしませんよ。
勝手に犯人扱いしないでください。
通報しますよ

[Zuishin]

ああここはあなたの回答でしたか。自分のと間違えました。

しかし、ここには質問者もコメントしているのに、自分に言われたことと確信があるようですね。
だから何だというわけではありませんが。

[sgsgsg1983]

>Zuishinさん
ちなみに僕は低評価ボタン押してませんよ。

[Zuishin]

最近、何の間違いもない回答に無言で嫌がらせの低評価をする人がいるので、時々こうして自分の回答を編集したりコメントしたりしてアピールしています。

今回は他のことをしている間にフォーカスの位置が変わったので間違えただけで、誰か特定の人のことを言っているわけではありません。

私の回答は、元々質問の回答として間違ってはいないが質問者向けではないとわかって書いたものですから、使えるよう勉強する必要もありません。「このような方法もある」と第三者に示したかっただけです。
色物ですが、応用範囲が広いことだけは間違いありません。

kikukiku さんの回答は確かに動きますが、あまりにも泥縄式で、実際に使うと笑われる類いのものだし、教科書で教えられることもないでしょう。

初心者がイベントドリブンを学ぶときには、プロパティやフィールドを使った状態管理から習うはずです。(fana さんの二つ目のコード)

fana さんの一つ目のコードはそれより一歩進んだもので、GUI とロジックを分離したエレガントで実用的な解法です。私も高評価していますが、同じ学ぶならこれを学ぶべきです。これを志すか否かで、生産性が天と地ほど変わってきます。

Answer 2

簡素なやつを．

例えばこんなのでも用意すれば…

//順繰りに文字列をどうのする作業者
class CyclicalStr
{
	private readonly string[] m_Str;
	private int m_CurrIndex;

	public CyclicalStr( IEnumerable<string> strs, int init_index=0 )
	{	m_Str = strs.ToArray();	m_CurrIndex = (init_index % m_Str.Length );	}

	public string Current{	get{	return m_Str[m_CurrIndex];	}	}
	public CyclicalStr ToNext(){	m_CurrIndex = ( (m_CurrIndex+1) % m_Str.Length );	return this;	}
}

Form1 の実装はこんな感じか．

private CyclicalStr m_StrDecider;

public Form1()
{
	InitializeComponent();

	m_StrDecider = new CyclicalStr( new string[]{ "a", "b", "c" } );
	label1.Text = m_StrDecider.Current;
}

private void button1_Click(object sender, EventArgs e)
{
	label1.Text = m_StrDecider.ToNext().Current;
}

---

[追記]
同じことを全部 Form1 に書いたらこうなる．

public partial class Form1 : Form
{
	//m_Str は，ラベルに表示する3つの文字列を持っています．
	//m_CurrIndex は，現在ラベルに何番目の文字列を表示しているのかを覚えておく変数です．
	//（つまり，ラベルには m_Str[ m_CurrIndex ] を表示するということ）
	private readonly string[] m_Str = new string[3]{ "a", "b", "c" };
	private int m_CurrIndex = 0;

	public Form1()
	{
		InitializeComponent();

		//ラベルに初期の文字列をセット
		label1.Text = m_Str[ m_CurrIndex ];
	}

	private void button1_Click(object sender, EventArgs e)
	{
		//m_CurrIndexを更新
		//（ここを実行するたびに 0->1->2->0->1->2->0-> ... と変わる）
		m_CurrIndex = (m_CurrIndex+1) % 3;
		//ラベルの文字列をセット
		label1.Text = m_Str[ m_CurrIndex ];
	}
}

Comments

[sgsgsg1983]

回答ありがとうございました。
自分はまだここまで複雑なコードを理解できません。
勉強してみます。

[fana]

最も「複雑じゃない」形を目指して書いたつもりだったのだが…

CyclicalStrクラスが持っているデータやメソッドを全部ダイレクトに Form1 クラス側に持たせる形の方がわかりやすいのだろうか？
↓
一応，そういうのも追記しておいた．

[fana]

何を「複雑」と述べているのか，的な意味合いが違うのだろう．
追記した版は「文法や文字数の観点では平易（＝複雑じゃない）」形と言えるかな．

[BeatStar]

fanaさん

横から失礼します。
多分、なぜ追記のようなコードになったのか書いた方がいいかもしれません。
私も入門者の頃は何も考えずに丸暗記しようとしていました。理由を説明するとなおわかりやすいかもしれません…

[fana]

えっと…理由……？
「本当にやりたいこと」はこうかな？と推測し，→それを素直に実装したから…かな．

例えば，「本当にやりたいこと」が，
・【ボタンを押す毎にラベルの表示を順繰りに変えたい】
・【「現在のラベルの表示内容に基づいて次のラベルの表示内容を決定して」ラベルの表示を変える】
のどちらなのかによって実装は変わるでしょう．

私が推測したのは前者の側だったから，
別の回答にあるような
> if( label1.Text == "a" )
という実装にはならなかった．（この形は後者の側に対応している実装である）

要は(?)，「現在のラベルの表示内容」というのを「アプリケーションの状態を表すデータ」だとして扱うかどうか，というところが違う．
私は「ラベルとはいわゆる View でしかなく，アプリケーションのデータを保持する存在ではない」…という考えの元にコードを書いた，とでも言えばよいのかな．

[fana]

そういう話ではなくて，
「配列とその配列のindex用の値　みたいなデータを持っている形」だとか「なにやら剰余を用いている」だとかいう実装詳細についての理由を問うているのであれば，
それは単に「これが最も短く書けるように思ったから」程度の話でしかないので，そこは「こだわらなくていい/別の実装形態でもよい」部分かと．
別の回答内で述べられている
> プロパティやフィールドを作って状態管理する
という例を提示しているにすぎないので．

[BeatStar]

すみません、紛らわしかったですね…

私が意図していたのは「なぜ余剰や配列を用いているのか」です。流れというか、なぜそうなるのかがわかりにくいのです。初心者って。
たとえば数学的に証明するとかでしょうか。

[fana]

ボタンが押された回数を数えるカウンタでも用意して，その値に基づいて表示文字列を決めることを考えるとする．
カウンタ値は最初は0で，ボタンを押すごとに 1,2,3,... と増えていく，と．
↓
カウンタ値に対する表示文字列を決める if でも何でも書けばいいよね．じゃあそこの条件ってどうなるのか？
0のときに"a", 1のときに"b", 2のときに"c" と表示するとして，次の3からはどうするのか？ また "a" から繰り返したいということは…
↓
・0, 3, 6, 9, ... のときは"a"
・1, 4, 7, 10, ... のときは"b"
・2, 5, 8, 11, ... のときは"c"
っていう話だ．
これが「ボタンを押した回数カウンタの値を3で割った余り　が{0,1,2}のどれなのか？　っていう場合分けである」ということはさすがにちょっと考えればわかるっしょ，としか言えないなぁ．小学生レベルの算数の話なのだし．
（示したサンプルでは，保持している値（この話で言うところのカウンタ値に相当するもの）自体を常に3で割った余りにしているけども，話の内容的には一緒）

配列を使う/使わない　は本当にどうでもいい実装形態の話でしかない．使いたくないなら使わなければいい．３パターンの分岐程度なら if や swith を使って書いてもいい．
何で使ったか？って問われても困る．「0,1,2,... っていう形の素直な値に対して文字列を決めるのに利用できる最もシンプルなデータ構造だと思ったから…？？」

[fana]

「剰余」に考えが及ばないとしても，そこで詰む話でもないよね．
【0,1,2,0,1,2,0, ... って値を繰り返すような実装があれば事足りる】のだから，素朴に

m_CurrIndex++;
if( m_CurrIndex>2 )m_CurrIndex=0;

みたく書いたっていい．
コレよりも剰余を使う記述が優れているとも思わないし．
（むしろコレの方が素直な表現である分だけ良いかもしれない．）

[fana]

正直，どうでもいい誤差程度の話に理由を問われても困る．
（せめてもうちょっと，アルゴリズム的な，というか，やり方で優劣的な差が出る内容じゃないと…）

Answer 3

実行できるサンプルコードを記載します。

「イベントを await で受け取る」に記載のコードを使うと、イベントスパゲティを簡単にほどき、次のように for 文で扱うことができるようになります。

なお、初心者の学習向けのコードではありません。
誰か他の人が回答してくれるでしょうが、フォームにプロパティやフィールドを作って状態管理するのが定石です。

using System;
using System.Threading.Tasks;
using System.Windows.Forms;
using Zuishin.WaitEvent;

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

			// ToggleButtonTextWithClick に button1 を登録
			_ = ToggleButtonTextWithClick(button1);
		}

		// 引数で指定されたボタンが押されるたびにテキストを変える
		private async Task ToggleButtonTextWithClick(Button button)
		{
			var buttonTexts = new[] { "a", "b", "c" };

			for (int i = 0; ; i = (i + 1) % buttonTexts.Length)
			{
				button.Text = buttonTexts[i];
				await button.WaitEvent<EventArgs>(nameof(button.Click));
			}
		}
	}
}

これを動かすためには、リンク先で記載されているコードが必要です。
プロジェクトに新しいクラスを追加して次のように書き換えてください。

using System;
using System.Collections.Concurrent;
using System.Linq.Expressions;
using System.Reflection;
using System.Threading;
using System.Threading.Tasks;

namespace Zuishin.WaitEvent
{
	static class WaitEventExtensions
	{
		private static readonly ConcurrentDictionary<Type, Delegate> handlerFactoryCache = new ConcurrentDictionary<Type, Delegate>();

		public static Task<(object Sender, TEventArgs EventArgs)> WaitEvent<TEventArgs>(
			this object sender,
			string eventName,
			CancellationToken? cancellationToken = null)
		{
			if (sender is null) throw new ArgumentNullException(nameof(sender));
			if (eventName is null) throw new ArgumentNullException(nameof(eventName));

			var targetEvent = sender
				.GetType()
				.GetEvent(eventName, BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic)
				?? throw new ArgumentException($"{nameof(sender)} has no event named {eventName}.");

			var completionSource = new TaskCompletionSource<(object, TEventArgs)>();
			cancellationToken?.Register(() => completionSource.SetCanceled());

			var handler = GetHandler(targetEvent.EventHandlerType, completionSource);
			targetEvent.AddEventHandler(sender, handler);

			return completionSource.Task.ContinueWith(task =>
			{
				targetEvent.RemoveEventHandler(sender, handler);
				return task.Result;
			});
		}

		private static Delegate GetHandler<TEventArgs>(Type eventHandlerType, TaskCompletionSource<(object, TEventArgs)> completionSource)
		{
			var factory = handlerFactoryCache.GetOrAdd(eventHandlerType, _ =>
			{
				var taskCompletionSourceType = typeof(TaskCompletionSource<(object, TEventArgs)>);
				var taskCompletionSourceExpression = Expression.Parameter(taskCompletionSourceType);
				var setResult = taskCompletionSourceType.GetMethod(nameof(TaskCompletionSource<(object, TEventArgs)>.SetResult));
				var sExpression = Expression.Parameter(typeof(object));
				var eExpression = Expression.Parameter(typeof(TEventArgs));
				var tupleConstructor = typeof(ValueTuple<object, TEventArgs>).GetConstructor(new[] { typeof(object), typeof(TEventArgs) });
				var expression = Expression.Lambda<Func<TaskCompletionSource<(object, TEventArgs)>, Delegate>>(
					Expression.Lambda(
						eventHandlerType,
						Expression.Call(
							taskCompletionSourceExpression,
							setResult,
							Expression.New(tupleConstructor, sExpression, eExpression)),
						sExpression,
						eExpression),
					taskCompletionSourceExpression);
				return expression.Compile();
			});
			return ((Func<TaskCompletionSource<(object, TEventArgs)>, Delegate>)factory)(completionSource);
		}
	}
}

Comments

[Zuishin]

ここではラベルではなくボタンのテキストを変更していますが、応用は容易でしょう。

[sgsgsg1983]

回答ありがとうございました。
自分はまだここまで複雑なコードを理解できません。
勉強してみます。