Почему не работают события?

Question

[Николай]

Подскажите, как заставить работать выделенные в коде события:

Form1.cs

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 Курсовой_Тест
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        // Events

        private void button1_MouseMove(object sender, MouseEventArgs e)
        {
            button1.BackColor = Color.LightGray;
        }

        private void button1_Click(object sender, EventArgs e)
        {
            Close();
        }

        private void button3_Click(object sender, EventArgs e)
        {
            WindowState = FormWindowState.Minimized;
        }

        private void button2_MouseLeave(object sender, EventArgs e)
        {
            button2.ForeColor = Color.FromArgb(230, 126, 34);
        }

        private void button4_MouseLeave(object sender, EventArgs e)
        {
            button2.ForeColor = Color.FromArgb(230, 126, 34);
        }

        private void button2_MouseMove(object sender, MouseEventArgs e)
        {
            button2.ForeColor = Color.FromArgb(100, 100, 100);
        }

        private void button4_MouseMove(object sender, MouseEventArgs e)
        {
            button2.ForeColor = Color.FromArgb(100, 100, 100);
        }

        private void button2_MouseDown(object sender, MouseEventArgs e)
        {
            button2.FlatAppearance.BorderColor = Color.DimGray;
            button2.BackColor = Color.DimGray;
        }

        private void button4_MouseDown(object sender, MouseEventArgs e)
        {
            button2.FlatAppearance.BorderColor = Color.DimGray;
            button2.BackColor = Color.DimGray;
        }

        private void button2_MouseUp(object sender, MouseEventArgs e)
        {
            button2.FlatAppearance.BorderColor = Color.FromArgb(230, 126, 34);
        }

        private void button4_MouseUp(object sender, MouseEventArgs e)
        {
            button2.FlatAppearance.BorderColor = Color.FromArgb(230, 126, 34);
        }

        private void button2_Click(object sender, EventArgs e)
        {
            this.Controls.Remove( button2 );
            this.Controls.Remove( label1 );
            this.Controls.Remove( textBox1 );
            this.Controls.Remove( textBox2 );
            this.Controls.Remove( label3 );
            this.Controls.Remove( label4 );

            // Form №2

            label2.Text = "Вопрос первый!";

            // Ответ 1
            Button button4 = new Button();
            button4.Text = "Вариант ответа 1";
            button4.Size = new System.Drawing.Size(350, 50);
            button4.Location = new Point(120, 370);
            button4.FlatStyle = System.Windows.Forms.FlatStyle.Flat;
            button4.Font = new System.Drawing.Font("Verdana", 14.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(204)));
            button4.FlatAppearance.BorderColor = System.Drawing.Color.FromArgb(((int)(((byte)(230)))), ((int)(((byte)(126)))), ((int)(((byte)(34)))));
            button4.FlatAppearance.BorderSize = 3;
            button4.UseVisualStyleBackColor = true;
            button4.Name = "button4";
            button4.FlatAppearance.MouseOverBackColor = System.Drawing.Color.FromArgb(((int)(((byte)(230)))), ((int)(((byte)(126)))), ((int)(((byte)(34)))));
            button4.ForeColor = System.Drawing.Color.FromArgb(((int)(((byte)(230)))), ((int)(((byte)(126)))), ((int)(((byte)(34)))));

button4.MouseLeave += new System.EventHandler(this.button4_MouseLeave);
button4.MouseDown += new System.Windows.Forms.MouseEventHandler(this.button4_MouseDown);
button4.MouseMove += new System.Windows.Forms.MouseEventHandler(this.button4_MouseMove);
button4.MouseUp += new System.Windows.Forms.MouseEventHandler(this.button4_MouseUp);

this.Controls.Add(button4);

            // Ответ 2

            Button button5 = new Button();
            button5.Text = "Вариант ответа 2";
            button5.Size = button4.Size;
            button5.Location = new Point(button4.Location.X + button4.Width + 20, 370);
            button5.FlatStyle = button4.FlatStyle;
            button5.Font = button4.Font;
            button5.FlatAppearance.BorderColor = button4.FlatAppearance.BorderColor;
            button5.FlatAppearance.BorderSize = button4.FlatAppearance.BorderSize;
            button5.UseVisualStyleBackColor = button4.UseVisualStyleBackColor;
            button5.FlatAppearance.MouseOverBackColor = button4.FlatAppearance.MouseOverBackColor;
            button5.ForeColor = button4.ForeColor;

            this.Controls.Add(button5);

            // Ответ 3

            Button button6 = new Button();
            button6.Text = "Вариант ответа 3";
            button6.Size = button4.Size;
            button6.Location = new Point(120, button4.Location.Y + button4.Height + 20);
            button6.FlatStyle = button4.FlatStyle;
            button6.Font = button4.Font;
            button6.FlatAppearance.BorderColor = button4.FlatAppearance.BorderColor;
            button6.FlatAppearance.BorderSize = button4.FlatAppearance.BorderSize;
            button6.UseVisualStyleBackColor = button4.UseVisualStyleBackColor;
            button6.FlatAppearance.MouseOverBackColor = button4.FlatAppearance.MouseOverBackColor;
            button6.ForeColor = button4.ForeColor;

            this.Controls.Add(button6);

            // Ответ 4

            Button button7 = new Button();
            button7.Text = "Вариант ответа 4";
            button7.Size = button4.Size;
            button7.Location = new Point(button5.Location.X, button6.Location.Y);
            button7.FlatStyle = button4.FlatStyle;
            button7.Font = button4.Font;
            button7.FlatAppearance.BorderColor = button4.FlatAppearance.BorderColor;
            button7.FlatAppearance.BorderSize = button4.FlatAppearance.BorderSize;
            button7.UseVisualStyleBackColor = button4.UseVisualStyleBackColor;
            button7.FlatAppearance.MouseOverBackColor = button4.FlatAppearance.MouseOverBackColor;
            button7.ForeColor = button4.ForeColor;

            this.Controls.Add(button7);

        }
    }
}

Answer 1

[Espleth]

Ой ужас, объедините кнопки в массив и избавьтесь от дублирования кода
Как заставить кнопки реагировать на события я вам точно сейчас не скажу, но скажу как посмотреть:
Добавьте на форму простую кнопку. Сделайте даблклик по ней. Создастся в коде метод для события, куда вас сразу же перекинет. Нас это не интересует, нас интересует то, что создалось в автоматически сгенерированном коде (там отдельный файл, находящийся в форме). Там для вашей кнопки будет что-то типа this.button1.onClick += button1_Click; вот это именно то, что вам и нужно.
Вообще вы так можете смотреть в этот автоматически-сгенерированный код как в шпаргалку для того, чтобы узнавать, как реализовывать такие вещи.

Comments

[Николай]

List buttons = new List();

for ( int i = 0; i < 4; ++i )
{
Button b = new Button();
b = new Button();
b.Text = "Вариант ответа " + Convert.ToString(i+1);
b.Size = new System.Drawing.Size(350, 50);

if (i == 0)
{
b.Location = new Point(120, 370);
}
else if (i == 1)
{
b.Location = new Point(140 + b.Size.Width, 370);
}
else if (i == 2)
{
b.Location = new Point(120, 390 + b.Size.Height);
}
else
{
b.Location = new Point(140 + b.Size.Width, 390 + b.Size.Height);
}

b.MouseDown += new MouseEventHandler(button_MouseDown);
b.MouseLeave += new EventHandler(button_MouseLeave);
b.MouseMove += new MouseEventHandler(button_MouseMove);
b.MouseUp += new MouseEventHandler(button_MouseUp);
b.FlatStyle = System.Windows.Forms.FlatStyle.Flat;
b.Font = new System.Drawing.Font("Verdana", 14.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(204)));
b.FlatAppearance.BorderColor = System.Drawing.Color.FromArgb(((int)(((byte)(230)))), ((int)(((byte)(126)))), ((int)(((byte)(34)))));
b.FlatAppearance.BorderSize = 3;
b.UseVisualStyleBackColor = true;
b.FlatAppearance.MouseOverBackColor = System.Drawing.Color.FromArgb(((int)(((byte)(230)))), ((int)(((byte)(126)))), ((int)(((byte)(34)))));
b.ForeColor = System.Drawing.Color.FromArgb(((int)(((byte)(230)))), ((int)(((byte)(126)))), ((int)(((byte)(34)))));
this.Controls.Add(b);
}

[Espleth]

tryvols: ну теперь лучше =)
События, главное, нормально работают?
Координаты кнопок по-моему как-то корявенько задаются: можно было бы выявить какой-то алгоритм размещения и пользоваться им. Хотя если больше четырех кнопок не планируется никак, то это ухудшило бы читаемость кода.
Еще у вас странно цвета задаются: куча лишних скобок, интовое число переводится в байтовое, а потом обратно в интовое что ли? Вам наверное проще было бы задавать примерно так:
b.BackColor = System.Drawing.Color.Blue; (или еще лучше в начале написать using System.Drawing; а потом это не писать)
Ну и кнопку переименовать с b в button, например. Ну это уже так, чисто на вкус.

[Николай]

Артем: окей, спасибо, поработаю над этим)

[Николай]

а события работают нормально)