реклама на сайте
подробности

 
 
2 страниц V  < 1 2  
Reply to this topicStart new topic
> VS2010 прием по TCP, доступ к главной форме из потока
XVR
сообщение Dec 2 2015, 15:05
Сообщение #16


Гуру
******

Группа: Свой
Сообщений: 3 123
Регистрация: 7-04-07
Из: Химки
Пользователь №: 26 847



Чтение у вас сделано мягко говоря необычно sm.gif

Ваш код с комментариями

Код
                            byte[] bytes = new byte[tcpСlient.ReceiveBufferSize];
// Заказываете буфер для чтения неизвестно какого размера -
//  то, что вернет tcpСlient.ReceiveBufferSize тут и то, что он вернет 3мя строками позже не обязательно совпадают

                            while (tcpStream.DataAvailable == true)
// Если данные есть - читаем их, если нет - то не читаем, а выдаем пустой buffer
// (returndata во 2й раз будет не null, а String.Empty)
                            {
                                tcpStream.Read(bytes, 0, (int)tcpСlient.ReceiveBufferSize);
                                returndata = Encoding.UTF8.GetString(bytes);
// Это зачем ???
                                count++;
                            }
                            if (returndata != null)
// Это просто жесть :)
                            {
                                Invoke(rdd, new object[] { bytes });
                                str = returndata;
                                returndata = String.Empty;
                            }

Код жуткий, кроме того, он выдаст нечто нарезанное далеко не по границам пакетов
Пакет нужно собирать в приемной нити, и отдавать целиком в Display
Go to the top of the page
 
+Quote Post
smk
сообщение Dec 2 2015, 18:19
Сообщение #17


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



Спасибо. Что-то я догадываюсь как исправить, но "жесть" даже не понимаю куда двигаться. Приложение нужно уже вчера. Может поможете?

А как сделать так чтоб обычно? Главное надежность. Посылка приходит каждые 30-40 мС.


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post
XVR
сообщение Dec 2 2015, 20:38
Сообщение #18


Гуру
******

Группа: Свой
Сообщений: 3 123
Регистрация: 7-04-07
Из: Химки
Пользователь №: 26 847



returndata и все манипуляции вокруг него не нужны вообще
Вызывать Invoke(rdd,...) нужно сразу после tcpStream.Read и передавать в него разиер реально прочтенных данных, но лучше все же собирать пакет после tcpStream.Read, и только готовый отдавать на показ

Какой формат принимаемых пакетов?

Go to the top of the page
 
+Quote Post
smk
сообщение Dec 3 2015, 07:50
Сообщение #19


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



Цитата(XVR @ Dec 2 2015, 22:38) *
returndata и все манипуляции вокруг него не нужны вообще
Вызывать Invoke(rdd,...) нужно сразу после tcpStream.Read и передавать в него разиер реально прочтенных данных, но лучше все же собирать пакет после tcpStream.Read, и только готовый отдавать на показ

Какой формат принимаемых пакетов?

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

Вот так сейчас сделал. Счетчик тикает, данные идут, отображения всеравно нет.


Код
        private void ReceiveRun()
        {
            
            UpdateReceiveDisplayDelegate rdd = new UpdateReceiveDisplayDelegate(Display);
            try
            {
                while (true)
                {
                        if (tcpStream.CanRead)
                        {
                            byte[] bytes = new byte[256];
                            while (tcpStream.DataAvailable == true)
                            {
                                tcpStream.Read(bytes, 0, (int)256);
                                count++;
                            }
                            Invoke(rdd, new object[] { bytes });
                         }
                        else
                        {
                            label1.Text = "Прием невозможен";
                            th.Abort();
                            tcpСlient.Close();
                            tcpStream.Close();
                            return;
                        }
                 }
            }
            catch
            {
                label1.Text = "ошибка";
            }
        }

        delegate void UpdateReceiveDisplayDelegate(Byte [] mydata);
        private void Display(Byte[] mydata) //вывод на форму
        {
            Int32 temp;
            temp = ((mydata[3] << 24) + (mydata[2] << 16) + (mydata[1] << 8) + mydata[0]);
            label1.Text = temp.ToString("D");
            label3.Text = Convert.ToString(count);
                //if (mydata[0] > 0) label2.Text = "OK";
                //else label2.Text = "0";
        }


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post
XVR
сообщение Dec 3 2015, 12:36
Сообщение #20


Гуру
******

Группа: Свой
Сообщений: 3 123
Регистрация: 7-04-07
Из: Химки
Пользователь №: 26 847



Цитата
Хочу спросить, что значит "собирать пакет"?
Это значит, что TCP соединение не является datagram ориентированным. Т.е. это просто поток байтов. И никто не гарантирует, что приемная сторона будет получать эти байты кусками такого же размера, как передавала передающая сторона. Т.е. вы свои 256 байт можете получить в несколько чтений (и мелкой нарезкой).

Цитата
Как это делается?
Принимаются данные и накапливаются в буфере, пока не наберется полный пакет.

Цитата
while (tcpStream.DataAvailable == true)
{
tcpStream.Read(bytes, 0, (int)256);
count++;
}
Invoke(rdd, new object[] { bytes });
Это неправильно. У вас в буфере будет мешанина из кусков принятых данных. Надо как то так
Код
byte[] bytes = new byte[256];
...

int size = 0;
while(size<256)
{
  while (tcpStream.DataAvailable == true)
   {
     size+=tcpStream.Read(bytes, size, 256-size);
   }
  if (size<256) Thread::Sleep(100);
}
count++;

Invoke(rdd, new object[] { bytes });

Go to the top of the page
 
+Quote Post
smk
сообщение Dec 3 2015, 14:03
Сообщение #21


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



Судя по режиму отладки функция Display выполняется очень часто. И действительно в буфере нули. Если и приходит что-то, то оно сразуже затирается нулями при следующем вызове Display. Как красиво сделать так чтоб Display вызывалась только если приняты свежие данные?
Пока сделал вот так:
Код
if(count_old != count)Invoke(rdd, new object[] { bytes });

Но всеравно в Display передаются нули.


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post
XVR
сообщение Dec 3 2015, 14:15
Сообщение #22


Гуру
******

Группа: Свой
Сообщений: 3 123
Регистрация: 7-04-07
Из: Химки
Пользователь №: 26 847



Нужно правильно принимать. См сообщение №20
Go to the top of the page
 
+Quote Post
smk
сообщение Dec 3 2015, 14:31
Сообщение #23


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



Кажись заработало. пока-что... Тестирую. Спасибо! Поглядывайте пока в тему пожалуйста... пока тестирую.


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post
smk
сообщение Dec 3 2015, 20:45
Сообщение #24


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



Вот что получается. Если я кнопкой отправляю 16 запросов то после 16-го все последующие не обрабатываются. Ну или еще один с большой задержкой. Если я командую прибору отвечать без запроса - молотит без сбоев. Если геркулесом отправляю запросы - тоже никаких проблем, отвечает хоть на 50-й. Даже не представляю каким способом вычислить этот затык. Можно что-то придумать? Спасибо.

Содержание пакета не имеет значения. Просто перестает отправлять или вообще подвисает. На мой непрофессиональный взгляд все вполне должно работать.
Код
       private void button3_Click(object sender, EventArgs e)//Отправить
        {

            if (tcpStream.CanWrite)
            {
                outBuffer[0] = 1;
                if (checkBox1.Checked)
                {
                    outBuffer[1] = 2;
                    outBuffer[2] = 1;
                }
                else
                {
                    outBuffer[1] = 2;
                    outBuffer[2] = 0;
                }
  
                tcpStream.Write(outBuffer, 0, outBuffer.Length);
            }
            else
            {
                label1.Text = "Передача невозможна";
                th.Abort();
                tcpСlient.Close();
                tcpStream.Close();
                return;
            }
        }


Как я успел понять - программа ничего не отсылает. Почему - непонятно.


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post
smk
сообщение Dec 4 2015, 08:13
Сообщение #25


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



похоже, что проблема со стороны прибора. проверю - напишу.


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post
XVR
сообщение Dec 4 2015, 10:15
Сообщение #26


Гуру
******

Группа: Свой
Сообщений: 3 123
Регистрация: 7-04-07
Из: Химки
Пользователь №: 26 847



А чему равен outBuffer.Length ?
Go to the top of the page
 
+Quote Post
smk
сообщение Dec 4 2015, 13:03
Сообщение #27


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



Цитата(XVR @ Dec 4 2015, 12:15) *
А чему равен outBuffer.Length ?

всегда 256. проверял. Насколько я успел понять, то с приемной стороны забивается окно приема. Выправил, но не проверил еще.


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post
smk
сообщение Dec 4 2015, 23:05
Сообщение #28


Гуру
******

Группа: Свой
Сообщений: 2 246
Регистрация: 17-03-05
Из: Украина, Киев
Пользователь №: 3 446



Все отлично работает. Прием и передача ОК. Огромное спасибо!


--------------------
Живи днем так, чтобы ночью ты спал спокойно.
Go to the top of the page
 
+Quote Post

2 страниц V  < 1 2
Reply to this topicStart new topic
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0

 


RSS Текстовая версия Сейчас: 14th July 2025 - 09:23
Рейтинг@Mail.ru


Страница сгенерированна за 0.01486 секунд с 7
ELECTRONIX ©2004-2016