有几点需要优化的
1. sp_DataReceived是事件触发的，所以不要使用sleep（9000）的方式等数据收齐。
2. 数据接收和数据解析最好分离。
否则可能由于解析耗时过长导致超时。
也就是业务处理影响了串口通信。

不要在BeginInvoke中去读取串口接收的数据。
在sp_DataReceived事件中读取串口接收数据。
Byte[] ReceivedData = new byte[sp.BytesToRead];
然后将读到的数据ReceivedData 另起一个线程进行处理。

BeginInvoke委托UI线程，但并不是马上执行，
此时你关闭了串口，然后等到执行的时候，端口早已经关闭，所以就报错了。

不要在BeginInvoke中去读取串口接收的数据。

在异步/事件通讯程序中写 Sleep 代码本身就是错误的。其它暂时都不用考虑，先把基本的通讯机制搞清楚，先把 Sleep 删除掉。有数据发来就接收，接受逻辑要处理粘包分包，这是起码的编程技术。你怎么知道要阻塞9000而不是900也不是8000也不是100也不是9900？这个卡死9秒钟的动做完全是胡乱猜测的，而且是无病也要卡死的做法。

如果你看到一些博客，如果事件/异步通讯程序里边写了 Sleep，哪怕是10毫秒，直接扔掉这类垃圾代码。否则学了危害无穷。

串口关闭再打开这个操作最好不要有吧，还是要一直开的比较好

void serialPort_DataReceived(object sender, SerialDataReceivedEventArgs e)

{



//接收数据

string str = "";

do

{

int count = serialPort.BytesToRead;

if (count <= 0)

break;

byte[] readBuffer = new byte[count];



Application.DoEvents();

serialPort.Read(readBuffer, 0, count);

str += System.Text.Encoding.Default.GetString(readBuffer);



} while (serialPort.BytesToRead > 0);

listBox1.Items.Add(str);

}

网上看的一段代码用来保证数据接收完成，不要用Thread.Sleep();

Quote: 引用 15 楼 loveljy_19901114 的回复：

串口关闭再打开这个操作最好不要有吧，还是要一直开的比较好

void serialPort_DataReceived(object sender, SerialDataReceivedEventArgs e)

{



//接收数据

string str = "";

do

{

int count = serialPort.BytesToRead;

if (count <= 0)

break;

byte[] readBuffer = new byte[count];



Application.DoEvents();

serialPort.Read(readBuffer, 0, count);

str += System.Text.Encoding.Default.GetString(readBuffer);



} while (serialPort.BytesToRead > 0);

listBox1.Items.Add(str);

}

网上看的一段代码用来保证数据接收完成，不要用Thread.Sleep();

好的 学到了

可以用try catch忽略掉