C#(Visualstudio)でボタンクリックイベントを別ファイルに記述したい、でもデザイナーから飛びたい

Question

メインフォームにボタンがいっぱいあって、一つのソースファイルにクリックイベント
が大量にありクリックイベント以外のメソッドを探すのが大変です。

ボタンはある程度機能ごとのカテゴリがあるので、それごとにpartialでファイルを分割
してみましたが、今度はデザイナーからボタンをクリックしてイベントまで飛んでくれ
なくなってしまいます。

デザイナーの便利な機能を生かしつつ、ファイルを分割する良い方法はありませんか？

Comments

[KOZ6.0]

どのようなものを作ろうとしているんでしょうか？
たとえば電卓を作るとして、ボタンそれぞれにイベントハンドラを書くのは悪手です。
ひとまとめにして、何が押されたかは Text プロパティなどから判断して分岐したほうが良いです。
メニューであれば、ボタンの Tag プロパティに飛び先のフォーム名を入れておき、リフレクションを使ってアセンブリから該当するフォームクラスを呼び出して表示することもできます。

Answer 1

今度はデザイナーからボタンをクリックしてイベントまで飛んでくれ
なくなってしまいます。

きちんと？ツリーの子に含めれば、該当ハンドラにジャンプしましたよ^^
Partialクラスのファイルを元クラスの子として表示 - Qiita

<Compile Include="Form1.Hoge.cs" >
  <DependentUpon>Form1.cs</DependentUpon>
  <SubType>Code</SubType>
</Compile>

隙あれば<SubType>Form</SubType>にしてくるのですが、どうやっても抑えられませんでしたorz
（直接コードを表示しようとするとデザイナが出て、「コードの表示」のひと手間が増える）

---

.NET（Core）でもできましたが、重複エラーになりますので↓としたほうがいいでしょう。

<ItemGroup>
  <Compile Update="Form1.Hoge.cs" DependentUpon="Form1.cs" SubType="Code" />
</ItemGroup>

NETSDK1022:重複する項目が含まれていました。 - .NET CLI | Microsoft Learn

---

Form1.csにぶら下がっているForm1.Hoge.cs・Fuga.csには飛ぶ。
Form1.csにぶら下がってないPiyo.csには飛ばない。

Comments

[TN8001]

UserControlに切り出すとか、大量にならないようにするほうがまっとうな気はしますが^^;

[TN8001]

完璧なソリューションを提示したうえ、証拠動画まで取ったのに意味わかんねーな。もう近寄らんとこｗ

Answer 2

クリックイベントが大量にありクリックイベント以外のメソッドを探すのが大変

というだけの話なのであれば，IDEの機能で所望のメソッドを探すとか，#region～#endregion で不必要な時にはイベント群のコードを折りたたむとかその程度で済ませておけば良いようにも思えますが．

あるいは，「カテゴリ」がボタン群の配置的にもまとまっているようであれば，カテゴリ毎のユーザコントロールに分ければどうでしょうか．

---

追記：
有用な場面は限られるかもしれませんが，以下のようにすれば「メソッドの数」を減らすことができるかもしれません．

public Form1()
{
	InitializeComponent();

	//複数のボタンで同じハンドラを使う
	this.button1.Click += this.button_Click;
	this.button2.Click += this.button_Click;
}

//複数のボタンで使うハンドラ．
//sender を見て処理を決める．
//下記は if else でやっているが，実際の分岐方法はいろいろと考えられるかと．
//（例えば，各ボタンの Tag にそれ用の何かを仕込んでおくとか？）
//
//分岐結果として Form1 の個別のメソッドを呼ぶような話になるのであれば「メソッド数を減らす」効果を得られないが，
//例えばロジック処理を実装しているような別のやつの処理を呼ぶとかであれば，Form1 のメソッド数は減る．
private void button_Click(object sender, EventArgs e)
{
	if( sender == button1 )
	{	MessageBox.Show( "1" );	}
	else if( sender == button2 )
	{	MessageBox.Show( "2" );	}
	else
	{	MessageBox.Show( "Unknown" );	}
}

Comments

[bifizmin]

回答ありがとうございます。
その方法で使い心地をみてみたいと思います。

[fana]

ボタンのハンドラって大抵，

private void button1_Click(object sender, EventArgs e){ DoWork1(); }
private void button2_Click(object sender, EventArgs e){ DoWork2(); }

みたく，単にやりたい処理を呼ぶだけになりませんか？
（Formクラス内に処理を書く場合でも，同じ処理を他の操作（コンテキストメニューとか）からも呼びたくなることも多いので）

どれだけ大量のボタンが存在するのかわかりませんが，こんな感じに各１行なら一か所（クラスのコードの末尾とか）に纏めておくだけでそれほど場所取りでもないような…？
それでも邪魔なら，そのまとまりを #region～#endregion で挟んでおけば良いかと．
（で，その後で追加したコントロールのイベントハンドラのコードがこの #endregion の後ろに追加されることに毎度イラっとするっていう…）

[bifizmin]

この方法で進まることにします。
ありがとうございました。