プログラムを簡潔で分かりやすく組むにはどのようにすればよろしいでしょうか？

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

現在、ツールボックスのコントロールと稲妻マークのイベント
を主に利用しプログラムを組んでいたのですが、だんだんわかりずらいプログラムになってきてしまったので、ツールボックスとイベントについてご教授頂けませんでしょうか？

簡単にこんなプログラムを作ってみました。

・TextboxにEnterでアクセスした際、色を紫に変えます。
・TextboxからLeaveで抜ける際、色を白に戻します。
・Buttonをクリックした際、Textboxに入力されている数値が足されていきます。

この際、まとめられそうな、箇所が何か所もあると思います。
どのようにプログラムを組むことで、簡潔で分かりやすいプログラムを組むことができるのでしょうか？
プロパティ横の稲妻マークでイベント等は起こさない方がいいのでしょうか？

初歩的な質問で申し訳ないのですが、どなた様かご教授ご鞭撻の程よろしくお願い致します。

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;

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

        int sum;


        // Enterイベント 

        private void textBox1_Enter(object sender, EventArgs e)
        {
            textBox1.BackColor = Color.Magenta;
        }

        private void textBox2_Enter(object sender, EventArgs e)
        {
            textBox2.BackColor = Color.Magenta;
        }

        private void textBox3_Enter(object sender, EventArgs e)
        {
            textBox3.BackColor = Color.Magenta;
        }

        private void textBox4_Enter(object sender, EventArgs e)
        {
            textBox4.BackColor = Color.Magenta;
        }


        // Leaveイベント

        private void textBox1_Leave(object sender, EventArgs e)
        {
            ColorReset();
        }

        private void textBox2_Leave(object sender, EventArgs e)
        {
            ColorReset();
        }

        private void textBox3_Leave(object sender, EventArgs e)
        {
            ColorReset();
        }

        private void textBox4_Leave(object sender, EventArgs e)
        {
            ColorReset();
        }


        // Clickイベント

        private void button1_Click(object sender, EventArgs e)
        {
            sum = sum + int.Parse(textBox1.Text);
            textBox5.Text = sum.ToString();
        }

        private void button2_Click(object sender, EventArgs e)
        {
            sum = sum + int.Parse(textBox2.Text);
            textBox5.Text = sum.ToString();
        }

        private void button3_Click(object sender, EventArgs e)
        {
            sum = sum + int.Parse(textBox3.Text);
            textBox5.Text = sum.ToString();
        }

        private void button4_Click(object sender, EventArgs e)
        {
            sum = sum + int.Parse(textBox4.Text);
            textBox5.Text = sum.ToString();
        }


        // 色初期化

        private void ColorReset()
        {
            textBox1.BackColor = Color.White;
            textBox2.BackColor = Color.White;
            textBox3.BackColor = Color.White;
            textBox4.BackColor = Color.White;
        }
    }
}

行動規範の内容に同意します

回答2件

ベストアンサー

それぞれのイベントを一つにし、

C#
1 private void textBox_Enter(object sender, EventArgs e)
2 {
3   TextBox s = (TextBox)sender;
4
5     switch(s.Name)
6     {
7     case "textBox1": s.BackColor = Color.Magenta; break;
8     case "textBox2": s.BackColor = Color.Magenta; break;
9     }
10 }

やることが同じであれば、

C#
1private void textBox_Enter(object sender, EventArgs e)
2 {
3     ((TextBox)sender).BackColor = Color.Magenta;
4 }

これで共通ですね。

投稿2016/06/16 11:08

mugicya

総合スコア1046

fender0131

2016/06/17 01:04

お答えありがとうございます。 private void textBox_Enter(object sender, EventArgs e) { ((TextBox)sender).BackColor = Color.Magenta; } こちらの動きを見て、なるほどと思いました。こちらでだいぶプログラムが見やすくなりそうです！！大変参考になりました。

行動規範の内容に同意します