Изменение цвета элемента из другой формы

@A1000 · Регистрация: 08.12.2015

Author24 — интернет-сервис помощи студентам

Создаю некую систему телеметрии и телеуправления. Общение происходит через COM port. Для данного дела надо создать графический интерфейс. Не смотря на очень начальные познания в этом деле, решил реализовать это самостоятельно. С премудростями общения через COM port разобрался, но споткнулся на таком вопросе. На MainWindow.xaml создан квадрат, назвал я его "Т1". В зависимости от поступающего через COM port сообщения этот квадрат должен менять цвет заливки. Для репетиции создал такое

C#

using System.Windows;
using System.Windows.Media;
 
namespace shema01
{
    /// <summary>
    /// Логика взаимодействия для MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        COM_port comm = new COM_port();
        public MainWindow()
        {
            InitializeComponent();
         }
         .............................................
         .............................................  
              
        private void button1_Click(object sender, RoutedEventArgs e)
        {
            T1.Fill = Brushes.Green;
        }
        ..............................................
        ..............................................   
        }

Как и положено, при нажатии на кнопку button1, квадрат Т1 заливается зелёным цветом. Но взаимодействие с COM port организовано в отдельном классе COM_port.cs. Пробую реализовать изменение цвета заливки квадрата Т1 при поступлении в COM_port сообщения "MVT1"

C#

using System.IO.Ports;
using System.Windows.Controls;
using System.Windows;
using System.Windows.Media;
using System.Threading;
 
namespace shema01
{
    
    class COM_port
    {
        SerialPort serialPort1 = new SerialPort();
        serialPort1.DataReceived += new SerialDataReceivedEventHandler(serialPort1_DataReceived);
      ............................................
      ............................................
     public void serialPort1_DataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            Thread.Sleep(100);
            string Otvet = "";
            Otvet = serialPort1.ReadExisting();
 
            
            if (Otvet.Contains("MVT1"))
            {
                T1.Fill = Brushes.Green;
 
            }
            serialPort1.DiscardInBuffer();
        }

Естественно, на меня ругается "Имя "Т1" не существует в текущем контексте". Понимаю, что тут ошибка. Класс COM_port.cs даже не подозревает, что существует MainWindow и тем более какой-то квадрат "Т1". Скорее всего в классе COM_port.cs надо как-то объявить объект "Т1". Подскажите как это сделать.

@novikov.ea · 09.12.2015, 17:50

Сообщение от A1000

COM_port.cs даже не подозревает, что существует MainWindow и тем более какой-то квадрат "Т1"

Вы абсолютно правы)

Сообщение от A1000

Скорее всего в классе COM_port.cs надо как-то объявить объект "Т1"

Не, не надо объявлять объект "Т1" в COM_port. COM_port - это класс бизнес логики, нафига ему знать о каких-то там квадратах? COM_port будет очень не доволен, если помимо его непосредственных обязанностей его ещё и будут заставлять квадраты раскрашивать. "А потом заборы начнут заставлять красить" - думает COM_port. Пусть вон MainWindow и занимается своими квадратами, раз он их и создаёт.
Вот MainWindow, у него есть квадрат T1. А ещё у него есть COM_port. Вот живёт себе MainWindow, поживает, и думает, как бы ему узнать, когда нужно изменить цвет квадрата? Неплохо было бы, чтобы COM_port сам тыкнул MainWindow, когда нужно. Но как? Конечно же через события.
Создадим событие DataReceived у COM_port:

C#

class COM_port
{
    SerialPort serialPort1 = new SerialPort();
    serialPort1.DataReceived += new SerialDataReceivedEventHandler(serialPort1_DataReceived);
 
    public void serialPort1_DataReceived(object sender, SerialDataReceivedEventArgs e)
    {
        Thread.Sleep(100);
        string responce = serialPort1.ReadExisting();
        serialPort1.DiscardInBuffer();
        RaiseDataReceived(responce);
    }
 
    public event EventHandler<DataReceivedEventArgs> DataReceived;
    RaiseDataReceived(string responce)
    {
        var handler = DataReceived;
        if (handler != null)
            handler(this, new DataReceivedEventArgs(responce));
    }
}
 
public class DataReceivedEventArgs : EventArgs
{
    public string Responce { get; private set; }
 
    public DataReceivedEventArgs(string responce)
    {
       Responce = responce;
    }
}

При получении данных из SerialPort'а, инициирует своё событие DataReceived;
А MainWindow пускай подпишется на это событие:

C#

public partial class MainWindow : Window
{
    COM_port comm = new COM_port();
    public MainWindow()
    {
        InitializeComponent();
        comm.DataReceived += comm_DataReceived;
    }
    
    public void serialPort1_DataReceived(object sender, DataReceivedEventArgs e)
    {
        if (e.Responce.Contains("MVT1"))
        {
            T1.Fill = Brushes.Green;
        }
    }
}

Вот что-то типа того)

@A1000 · 10.12.2015, 01:33 **[ТС]**

Большое спасибо за ответ. Но при попытке применить Ваш код Visual Studi заругалась в двух местах.
В COM_port на строке 14

C#

1	public event EventHandler<DataReceivedEventArgs> DataReceived;

написало "Метод должен иметь тип возвращаемого значения"
В MainWindow на строке 7

C#

1	comm.DataReceived += comm_DataReceived;

подчеркнуло comm_DataReceived и написало "Имя "comm_DataReceived" не существует в данном контексте"
Скажу честно, с моими познаниями это на грани моего понимания, помогите исправить.

Добавлено через 2 часа 52 минуты
В MainWindow вроде разобрался. 7 строка теперь выглядит так

C#

1	comm.DataReceived += serialPort1_DataReceived;

теперь не ругается.
А вот где грабли в COM_port хоть убей, не пойму. Может кто подскажет?

@novikov.ea · 10.12.2015, 10:02

Сообщение от A1000

Скажу честно, с моими познаниями это на грани моего понимания, помогите исправить

Почитайте книжечку Троелсена про C#. Будет легче разобраться.

Сообщение от A1000

В COM_port на строке 14

C#

1	public event EventHandler<DataReceivedEventArgs> DataReceived;

Я код писал прямо здесь в этом окошке, а не в студии. Поэтому за ошибки компиляции не отвечаю. Рассказывал просто об идее. Но если Вы посмотрите на следующую строку, то заметите, что я просто забыл прописать возвращаемый тип у метода RaiseDataReceived. Об этом самом, кстати, Вам и написал компилятор:

Сообщение от A1000

"Метод должен иметь тип возвращаемого значения"

Сообщение от A1000

В MainWindow на строке 7

C#

1	comm.DataReceived += comm_DataReceived;

Метод не правильно назвал. Хотел его назвать comm_DataReceived, но забыл переименовать.

Короче, почитайте вот здесь. Та же книга Троелсена, плюс комментарии и дополнения от товарища Ерохина.

@A1000 · 12.12.2015, 12:01 **[ТС]**

Что-то я совсем запутался. Исправил ошибки, всё скомпилировалось и запустилось:
MainWindow

C#

using System.Windows;
using System.Windows.Media;
 
namespace shema01
{
    /// <summary>
    /// Логика взаимодействия для MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        COM_port comm = new COM_port();
        public MainWindow()
        {
            InitializeComponent();
            comm.DataReceived += serialPort1_DataReceived; 
         }
         .............................................
         .............................................  
              
        public void serialPort1_DataReceived(object sender, DataReceivedEventArgs e)
        {
            if (e.Responce.Contains("MVT1"))
            {
                T1.Fill = Brushes.Green;
            }
        }
        ..............................................
        ..............................................   
        }

COM_port.cs.

C#

class COM_port
{
    SerialPort serialPort1 = new SerialPort();
    serialPort1.DataReceived += new SerialDataReceivedEventHandler(serialPort1_DataReceived);
 
        ...................................................
        ...................................................
 
    public void serialPort1_DataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            Thread.Sleep(100);
            string responce = serialPort1.ReadExisting();
            serialPort1.DiscardInBuffer();
            RaiseDataReceived(responce);
        }
        public event  EventHandler<DataReceivedEventArgs> DataReceived;
        public void RaiseDataReceived(string responce)
        {
            var handler = DataReceived;
            if (handler != null)
                handler(this, new DataReceivedEventArgs(responce));
        }
 
    }
    public class DataReceivedEventArgs : EventArgs
    {
        public string Responce { get; private set; }
 
        public DataReceivedEventArgs(string responce)
        {
            Responce = responce;
        }

Но при отправке в COM port "MVT1" начинает ругаться "Вызывающий поток не может получить доступ к данному объекту, так как владельцем этого объекта является другой поток." Выделяется строка

C#

1	T1.Fill = Brushes.Green;

Что теперь ему не так; Цвет ведь пытаемся изменить из MainWindow которому принадлежит квадрат Т1

@novikov.ea · 12.12.2015, 21:56

Не пугайтесь) Всё нормально. Этот код просто выполняется из другого потока. Его нужно вызвать в основном потоке приложения.

C#

Application.Current.Dispatcher.Invoke(() =>
{
    T1.Fill = Brushes.Green;
});

@A1000 · 12.12.2015, 22:45 **[ТС]**

Слава Богу заработало. Большое спасибо за помощь!

@A1000 0 / 0 / 0 Регистрация: 08.12.2015 Сообщений: 6
	12.12.2015, 22:45 [ТС]	7
	Слава Богу заработало. Большое спасибо за помощь! 0