Visual Studio SerialPort_приём данных

[NicksonWer 1]

Доброго времени!
За несколько дней пришлось в спешке освоить C# и по примерам накидать приложение.
По большому счёту всё работает, но есть небольшая проблема, которую нужно исправить.

Описание проекта. 
- микроконтроллер подключён к ПК через COM-порт;
- программа раз в секунду шлёт МК символы поочерёдно "0","1"или "2";
- МК в зависимости от пришедшего символа даёт соответствующий ответ: если "0", то возвращает "333", "1" - "322", "2" - "655";
- Программа получив ответы, разлаживает их в соответствующие элементы: Если было отправлено "0", то вернувшийся ответ "333" положить в label14;          "1" - "322" = label15;          "2" - "655" = label10

Вот такая тестовая незамысловатая логика для теста работоспособности.

Проблема в том, что значения не всегда выводятся полным значением. Как можно видеть на скрине, первое значение вместо "333" вывелось только "33", "322" вывелось полностью, и "655" только "55".
При следующей итерации может уже получиться другой вывод....

Краткое описание кода программы:
- код имеет подключенный Таймер, прерывание которого срабатывает раз в 1000 мс
- после соединения по КОМ-порту включается таймер и раз в секунду отправляется значение "0" "1" либо "2".
- в функции serialPort1_DataReceived происходить приём данных в переменную типа string dataIN
- в функции ShowData происходит работа с содержимым dataIN. В зависимости от отправленного символа, ответ помещается в соответствующий элемент

Скрытый текст

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;
using System.IO.Ports;

namespace AstroCanon
{
    public partial class Form1 : Form
    {
        string dataIN;
        private int counter_timer; // хранит передаваемое число плате для получения данных
        private int q;
        private int count;
        private int label_2, label_3;
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            string[] myPort;

            myPort = System.IO.Ports.SerialPort.GetPortNames();
            comboBox1.Items.AddRange(myPort);
        }

        private void button1_Click(object sender, EventArgs e)
        {
            serialPort1.BaudRate = 115200;
            serialPort1.PortName = comboBox1.Text.ToString();
            if (serialPort1.IsOpen == false)
                serialPort1.Open();

            button1.Enabled = false;
            timer1.Enabled = true;

            label1.Text = "ON";
            label1.ForeColor = Color.Green;            // изменить цет на зелёный 

            
        }

        private void button2_Click(object sender, EventArgs e)
        {
            serialPort1.Close();
            button1.Enabled = true;

            label1.Text = "OFF";
            label1.ForeColor = Color.DarkRed;

            timer1.Enabled = false;
        }

        private void serialPort1_DataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            dataIN = serialPort1.ReadExisting();
            this.Invoke(new EventHandler(ShowData));



     


        }

        private void ShowData(object sender, EventArgs e)
        {
            richTextBox1.Text += dataIN;

             if (count == 0)
            {
                // int data0 = Convert.ToInt32(str);
                label14.Text = dataIN;
                return;
            }
            if (count == 1)
            {
                //  int data1 = Convert.ToInt32(str);
                label15.Text = dataIN;
                return;
            }
            if (count == 2)
            {
                //  int data2 = Convert.ToInt32(str);
                label10.Text = dataIN;
                return;
            } 

           
        }
            
        private void timer1_Tick(object sender, EventArgs e)
        {

            if (count <= 3) count++;                    // инкрементирует значение для дальнейшей передачи в устройство;
            if (count >= 3) count = 0;                 

            string str_с = Convert.ToString(count);
            serialPort1.Write(str_с);

           

            

           // label7.Text = data0;


            //  string str = Convert.ToString(counter_timer);

            //  serialPort1.Write(str);

            //  richTextBox1.Text += dataIN;
            //  int a = Convert.ToInt32(dataIN);
            //  return;
            //label2.Text = Convert.ToString(a);

            // label2.Text = Convert.ToString(counter_timer);
            // if (count == 3)   label3.Text = Convert.ToString(label_3);

        }

        private void button3_Click(object sender, EventArgs e)
        {
            count = 2;
            string str = Convert.ToString(count);
            serialPort1.Write(str);



            // string str = Convert.ToString(counter_timer);
            //serialPort1.Write((string)counter_timer);
            // serialPort1.Write("3");
            //label2.Text = Convert.ToString(dataIN);


        }

        private void button4_Click(object sender, EventArgs e)
        {
            count = 3;
            string str = Convert.ToString(count);
            serialPort1.Write(str);
        }

        private void button6_Click(object sender, EventArgs e)
        {

        }

        private void label16_Click(object sender, EventArgs e)
        {

        }

        private void label20_Click(object sender, EventArgs e)
        {

        }

        private void label2_Click(object sender, EventArgs e)
        {

        }
    }
}

 

 

[Adver 220]

PCBWay - всего $5 за 10 печатных плат, первый заказ для новых клиентов БЕСПЛАТЕН (Экспресс изготовление за 24 часа!)

Сборка высококачественных печатных плат от $30 + БЕСПЛАТНАЯ доставка по всему миру + трафарет

Онлайн просмотрщик Gerber-файлов от PCBWay!

Мы принимаем формат Sprint-Layout 6! Экспорт в Gerber из Sprint-Layout 6.

Студенческое спонсорство

[Yurkin2015 544]

Тут надо знать, что функция serialPort1_DataReceived() вызывается на каждый принятый байт. К примеру, посылаете от МК на компьютер строку "655". Строка поылается символ за символом, т.е. сначала принимается символ "6". Приняли только первый символ, как тут же вызывается serialPort1_DataReceived(), и в переменную dataIN записывается "6".

Затем принимаемся следующий байт "5". Но Windows не система реального времени, в ней вызов функций может происходить с задержкой. Поэтому пока вызывается serialPort1_DataReceived() может успеть приняться следующий байт. В буфере КОМ-порта уже будет сидеть "55", и в переменную dataIN это и запишется, предыдущая "6" пропадёт.

После этого всё успокоится, и настанет время выводить данные в label, и нарисуется только последний принятый обрывок строки, т.е."55".

Надо поступить так, например. Перед посылкой в МК команды, надо очистить переменную

dataIN = "";

Затем при приёме в переменной dataIN накапливать полученные байты операцией +=

dataIN += serialPort1.ReadExisting();

Как только приняли 3 байта - можно выводить  накопленную строку в label

if(dataIN.Length >= 3 ) this.Invoke(new EventHandler(ShowData));

 

 

Edited Friday at 10:54 AM by Yurkin2015

[Adver 220]

Изготовление 2-х слойных плат от 2$, а 4-х слойных от 5$!

Быстрое изготовление прототипа платы всего за 24 часа! Прямая доставка с нашей фабрики!

Смотрите видео о фабрике JLCPCB: https://youtu.be/_XCznQFV-Mw

Посетите первую электронную выставку JLCPCB https://jlcpcb.com/E-exhibition чтобы получить купоны и выиграть iPhone 12, 3D-принтер и так далее...

[NicksonWer 1]

2 часа назад, Yurkin2015 сказал:

Тут надо знать, что функция serialPort1_DataReceived() вызывается на каждый принятый байт. К примеру, посылаете от МК на компьютер строку "655". Строка поылается символ за символом, т.е. сначала принимается символ "6". Приняли только первый символ, как тут же вызывается serialPort1_DataReceived(), и в переменную dataIN записывается "6".

Затем принимаемся следующий байт "5". Но Windows не система реального времени, в ней вызов функций может происходить с задержкой. Поэтому пока вызывается serialPort1_DataReceived() может успеть приняться следующий байт. В буфере КОМ-порта уже будет сидеть "55", и в переменную dataIN это и запишется, предыдущая "6" пропадёт.

После этого всё успокоится, и настанет время выводить данные в label, и нарисуется только последний принятый обрывок строки, т.е."55".

Надо поступить так, например. Перед посылкой в МК команды, надо очистить переменную

dataIN = "";

Затем при приёме в переменной dataIN накапливать полученные байты операцией +=

dataIN += serialPort1.ReadExisting();

Как только приняли 3 байта - можно выводить  накопленную строку в label

if(dataIN.Length >= 3 ) this.Invoke(new EventHandler(ShowData));

 

 

Замечательно!
Спасибо. Вечером обязательно попробую реализовать и после тпишусь.
А пока еще небольшой вопрос. 
В коде, в функции serialPort1_DataReceived есть такая стока как:
 

this.Invoke(new EventHandler(ShowData));

Что она делает??? не могу понять

[Adver 220]

Высокая надежность SiC! Как они этого добились?

За несколько лет кропотливых исследований и совершенствования технологии компания Infineon смогла довести показатели надежности и стабильности параметров высоковольтных и быстродействующих карбид-кремниевых транзисторов линейки CoolSiC практически до уровня их кремниевых собратьев.

Подробнее

[Yurkin2015 544]

Ваша программа состоит из нескольких частей-потоков, которые работают параллельно, то есть якобы одновременно выполняются разные части программы. Один поток рисует лэйблы на экране, другой принимает данные из КОМ-порта. В C# нельзя просто так использовать один ресурс из разных потоков. Функция приёма байтов может активизироваться внезапно при поступлении байта в порт, из неё нельзя рисовать лэйбл, потому что , может, в этот же момент другой поток тоже что-то рисует на этой же лэйбле. Возникает конфликт "кто главнее". 

Вместо прямого рисования при поступлении байта функция генерирует событие ShowData в системе, мол, обнови метку на экране. Это событие помещается в общую очередь событий. И когда у рисовального потока дойдут руки и он среагирует на это событие, тогда метка спокойно обновится на экране новой строкой.

[Adver 220]

Материалы вебинара «STM32L5. Секреты оценки энергопотребления»

Опубликованы запись и материалы вебинара КОМПЭЛ, посвященного первому семейству МК STM32L5 на ядре Cortex-M33. На вебинаре было рассказано о самых распространенных ошибках при расчете энергопотребления микроконтроллеров и о специальном тесте ULPMark, позволяющем дать наиболее объективную оценку энергоэффективности. Измерения проводились на демонстрационной платформе STM32L562E-DK.

Подробнее

[NicksonWer 1]

2 минуты назад, Yurkin2015 сказал:

Ваша программа состоит из нескольких частей-потоков, которые работают параллельно, то есть якобы одновременно выполняются разные части программы. Один поток рисует лэйблы на экране, другой принимает данные из КОМ-порта. В C# нельзя просто так использовать один ресурс из разных потоков. Функция приёма байтов может активизироваться внезапно при поступлении байта в порт, из неё нельзя рисовать лэйбл, потому что , может, в этот же момент другой поток тоже что-то рисует на этой же лэйбле. Возникает конфликт "кто главнее". 

Вместо прямого рисования при поступлении байта функция генерирует событие ShowData в системе, мол, обнови метку на экране. Это событие помещается в общую очередь событий. И когда у рисовального потока дойдут руки и он среагирует на это событие, тогда метка спокойно обновится на экране новой строкой.

Спасибо за объяснение. Да, принцип другой нежели у микроконтроллеров, где один поток, да прерывания....

[Adver 235]

Снижена цена на AC/DC и DC/DC преобразователи Mornsun в Компэл!

Компэл и компания Mornsun снизили цены на преобразователи AC/DC-преобразователи семейств LS и LDE. По привлекательной цене также предлагаются DC/DC-преобразователи изолированных семейств поколений R2 и R3 различного конструктивного исполнения.

Подробнее

[NicksonWer 1]

8 часов назад, Yurkin2015 сказал:

Тут надо знать, что функция serialPort1_DataReceived() вызывается на каждый принятый байт. К примеру, посылаете от МК на компьютер строку "655". Строка поылается символ за символом, т.е. сначала принимается символ "6". Приняли только первый символ, как тут же вызывается serialPort1_DataReceived(), и в переменную dataIN записывается "6".

Затем принимаемся следующий байт "5". Но Windows не система реального времени, в ней вызов функций может происходить с задержкой. Поэтому пока вызывается serialPort1_DataReceived() может успеть приняться следующий байт. В буфере КОМ-порта уже будет сидеть "55", и в переменную dataIN это и запишется, предыдущая "6" пропадёт.

После этого всё успокоится, и настанет время выводить данные в label, и нарисуется только последний принятый обрывок строки, т.е."55".

Надо поступить так, например. Перед посылкой в МК команды, надо очистить переменную

dataIN = "";

Затем при приёме в переменной dataIN накапливать полученные байты операцией +=

dataIN += serialPort1.ReadExisting();

Как только приняли 3 байта - можно выводить  накопленную строку в label

if(dataIN.Length >= 3 ) this.Invoke(new EventHandler(ShowData));

 

 

Сделал всё так как Вы описали.
Всё заработало. 
Большое Вам спасибо!

[NicksonWer 1]

9 часов назад, Yurkin2015 сказал:

Тут надо знать, что функция serialPort1_DataReceived() вызывается на каждый принятый байт. К примеру, посылаете от МК на компьютер строку "655". Строка поылается символ за символом, т.е. сначала принимается символ "6". Приняли только первый символ, как тут же вызывается serialPort1_DataReceived(), и в переменную dataIN записывается "6".

Затем принимаемся следующий байт "5". Но Windows не система реального времени, в ней вызов функций может происходить с задержкой. Поэтому пока вызывается serialPort1_DataReceived() может успеть приняться следующий байт. В буфере КОМ-порта уже будет сидеть "55", и в переменную dataIN это и запишется, предыдущая "6" пропадёт.

После этого всё успокоится, и настанет время выводить данные в label, и нарисуется только последний принятый обрывок строки, т.е."55".

Надо поступить так, например. Перед посылкой в МК команды, надо очистить переменную

dataIN = "";

Затем при приёме в переменной dataIN накапливать полученные байты операцией +=

dataIN += serialPort1.ReadExisting();

Как только приняли 3 байта - можно выводить  накопленную строку в label

if(dataIN.Length >= 3 ) this.Invoke(new EventHandler(ShowData));

 

 

Если строка  if (dataIN.Length >= 3) this.Invoke(new EventHandler(ShowData)); ожидает пока не наберётся масссив на три элемента , то как сделать так, чтобы оператор if ждал пока не появится стоп-символ? Например, "%"

[Adver 220]