Как передать нажатую кнопку в другой метод C#?

Question

[Мария]

Подскажите пожалуйста, как в методе TakeMessage обратиться именно к той кнопке, которую я нажал т.е. вместо btnTest использовать ту которая вызвала обработчик события btnClick.

public void TakeMessage(string btnStatus, string userName)
        {


            if (userName == "")
            {
                MessageBox.Show("Введите Ваше имя!");
            }

            else if (btnStatus == "Red")
            {
                btnTest.BackColor = Color.Green;
                btnTest.Text = "Свободно";
            }
            else
            {
                btnTest.BackColor = Color.Red;
                btnTest.Text = userName;
            }
        }

        
        private void btnClick(object sender, EventArgs e)
        {
            Control btn = (Control)sender;

            if (btn.BackColor == Color.Green)
            {
                btnStatus = "Green";
            }

            else
            {
                btnStatus = "Red";
            }

            Server.SendMessageToAll(btnStatus, tbLogin.Text);
            TakeMessage(btnStatus, tbLogin.Text);
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            foreach (var button in Controls.OfType<Button>())
            {
                button.Click += btnClick;
            }
        }

Answer 1

[Андрей Шедько]

Используйте делегат.

Answer 2

[Dmitry Pavlov]

Параметр sender это и есть кнопка, которая была нажата. Передайте ее третьим параметром в TakeMessage, предварительно скастив к типу кнопки, например.

Comments

[Мария]

Пробовала - результат Exception

[Dmitry Pavlov]

Мария: в дебаггере если смотреть, в какой момент происходит исключение и какого типа на самом деле приходит объект sender?

Answer 3

[Tom Nolane]

public void TakeMessage(string btnStatus, string userName, Button btnTest)
        {


            if (userName == "")
            {
                MessageBox.Show("Введите Ваше имя!");
            }

            else if (btnStatus == "Red")
            {
                btnTest.BackColor = Color.Green;
                btnTest.Text = "Свободно";
            }
            else
            {
                btnTest.BackColor = Color.Red;
                btnTest.Text = userName;
            }
        }

private void btnClick(object sender, EventArgs e)
        {
            var btn = sender as Button; 

            if (btn.BackColor == Color.Green)
            {
                btnStatus = "Green";
            }

            else
            {
                btnStatus = "Red";
            }

            Server.SendMessageToAll(btnStatus, tbLogin.Text);
            TakeMessage(btnStatus, tbLogin.Text, btn);
        }

в вашем случае не вижу смысла создавать отдельный метод TakeMessage ... все что есть в TakeMessage, можно реализовать в одном методе обработчике: btnClick

Comments

[Мария]

Это часть кода мини клиент-серверного приложения и метод TakeMessage() используется для получения данных другим клиентом, как я понимаю.

Я доработала код по вашему примеру, но, к сожалению, получаю следующий Exception - image.prntscr.com/image/2133ff5e439847ddb592a6f5e5...

[Tom Nolane]

Мария: вы отправляете btn в Server.SendMessageToAll(btnStatus, tbLogin.Text, btn);
в моем коде я туда не отправляю 3-им аргументом ничего, я отправляю в TakeMessage(btnStatus, tbLogin.Text, btn); поскольку вы не показали, что у вас выполняется в Server.SendMessageToAll() - я его никак не трогал

[Мария]

Я сделала по аналогии, думала, что сработает(. Если Вам не сложно, взгляните пожалуйста на солюшен.
https://drive.google.com/file/d/0B18Me-xxK8kYOEk0O...

[Tom Nolane]

Мария: вот так сделайте и запустите....
и на всякий пожарный https://yadi.sk/d/8Pj1QmGS3EFeLE

[Мария]

Извините, но я неправильно, точнее не до конца раскрыла суть задачи. Нам нужно запустить два клиента (я запускаю из папки CleintServerApp(WCF)_\CleintServerApp(WCF)_\Client\bin\Debug Clinеt.exe) и на обоих кнопки должны синхронно менять цвет, за это и отвечал метод SendMessageToAll. Вот почему мне нужно было как-то передать в этот метод "нажатую кнопку". Это аналог бронирования билетов т.е. как бы два клиента (окна): на одном нажимаешь канопку она меняет цвет и то же самое происходит на втором клиенте.

[Мария]

Решила вопрос путем передачи Dictionary buttons в котором содержится статус кнопки и ее имя, а после в методе TakeMessage обращаюсь к кнопке с именем из словаря.
Спасибо большое, что откликнулись и помогли.