共通イベントハンドラを使用した多オブジェクトに対するコードの短縮化について

Question

いつもお世話になっています。

前提・実現したいこと

前提1：（UserControl内にtableLayoutPanelで)ボタンが60個ある。
前提2：それらのボタンはドラッグアンドドロップでファイルを受け付けてファイル名を(各ボタン名).textに表示している。

実現したいこと：60個のボタンに対応するコードを短縮させたい。

※具体的には

string[] filename = (string[])e.Data.GetData(DataFormats.FileDrop, false);
input1_1.Text = Path.GetFileName(filename[0]);
input1_1.BackColor = Color.Chartreuse;

を1つのメソッドに置き換える事で達成できればと思っています。
(でも、if文の中にあるボタンのNameをどのようにすればよいのかわかっていません)

該当のソースコード

input1_1やinput1_2はボタンのNameです。

        private void DragDrop(object sender, DragEventArgs e)
        {
            if (sender.Equals(input1_1)) {
                string[] filename = (string[])e.Data.GetData(DataFormats.FileDrop, false);
                input1_1.Text = Path.GetFileName(filename[0]);
                input1_1.BackColor = Color.Chartreuse;
            }
            else if (sender.Equals(input1_2)) {
                string[] filename = (string[])e.Data.GetData(DataFormats.FileDrop, false);
                input1_2.Text = Path.GetFileName(filename[0]);
                input1_2.BackColor = Color.Chartreuse;
            }
            else if (sender.Equals(input1_3)) {
                string[] filename = (string[])e.Data.GetData(DataFormats.FileDrop, false);
                input1_3.Text = Path.GetFileName(filename[0]);
                input1_3.BackColor = Color.Chartreuse;
            }
            else if (sender.Equals(input1_4)) {
                string[] filename = (string[])e.Data.GetData(DataFormats.FileDrop, false);
                input1_4.Text = Path.GetFileName(filename[0]);
                input1_4.BackColor = Color.Chartreuse;
            }
        }

補足情報（FW/ツールのバージョンなど）

VisualStudio2017 (Community)
.NET Framework Ver 4.7.03056

Answer 1

６０個を全て一つのイベントハンドラを登録して、Senderで区別すればいいです

動かしてないけどこんな感じ

        private void DragDrop(object sender, DragEventArgs e)
        {
            Button btn = Sender as Button;    // これで発動元のボタンが取れる
            string name = btn.Name;           // ボタンの名前

Answer 2

if(senderが60個のボタンのうちのどれかであれば){
    senderを操作する(ボタンにキャストして)
}

senderが60個のボタンのうちのどれかであればはボタンのリストなり配列なり作ればいいです。

Answer 3

senderをキャストすればいいんでは。キャスト先がTextBoxでいいのかわからんけど。

        private void DragDrop(object sender, DragEventArgs e)
        {
　　　　　　TextBox input = (TextBox)sender;
           string[] filename = (string[])e.Data.GetData(DataFormats.FileDrop, false);
           if( input != null　&& filename.Any())
           {
                input.Text = Path.GetFileName(filename[0]);
                input.BackColor = Color.Chartreuse;
           } 
　　　　　｝

Comments

[P5_USER]

kiichi54321さん
ありがとうございます。
私の場合は、Buttonをキャスト先にする必要がありました。

教えてもらったそのコードを一部改変すると、
望み通りの動きになってくれました。

というのも、Buttonと打っただけでは、IntelliSenseの候補の中に、
自分の設定したnamespaceのButtonと
System.Windows.Forms.Buttonと
VisualStyleElement.Buttonがあり、
真ん中のSystem.Windows.Forms.Buttonにしなければいけなかったからです。

Answer 4

ボタンから派生した新たなコントロールを作るのが良いのではないかと思います。