我正在尝试使用WinForms和Modbus 485协议读取3个温度设备。
基本上我必须定期向每个设备写一个命令,等待响应,当我得到响应时,处理它。每个设备都有一个独特的通信地址。为了定期发送命令,我正在使用计时器。Timer1.interval=100;
这就是我发送命令的方式以及处理响应的位置:
private void ProcessTimer_Tick(object sender, EventArgs e)
{
switch (tempState)
{
case TempTimerState.sendCommDevice1:
if (!tempSerial.IsOpen)
{
tempSerial.Open();
}
tempSerial.DiscardInBuffer();
communication.tempCommand[0] = 0x01; //device adress
communication.tempCommand[6] = 0xA5; //CRC
communication.tempCommand[7] = 0xC2; //CRC
tempSerial.Write(communication.tempCommand, 0, 8);
tempState = TempTimerState.recievedDevice1;
communication.waitTime = 0; //time to wait before throw a timeout exception
communication.dataRecievedTemp = false; //flag for response recieved
break;
case TempTimerState.recievedDevice1:
communication.waitTime++;
if (communication.dataRecievedTemp)
{
communication.waitTime = 0;
if(CheckCRC(communication.tempResponse)) //CRC checking
{
//process response
}
else
{
//handle CRC Failure error
}
}
if(commcommunication.waitTime>=maxWaitTime)
{
//handle Timeout exception
}
tempState=TempTimerState.sendCommDevice2;
break;
}
}
等每个设备。这是我的序列端口数据接收事件:
private void tempSerial_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
{
SerialPort sp = (SerialPort)sender;
sp.Read(communication.tempResponse, 0, sp.BytesToRead);
communication.dataRecievedTemp = true; //flag for data recieved
}
所以我的沟通应该是:
send command device1
recieve response device1
send command device2
recieve command device2
send command device3
recieve command device3
然后再次send command device1
。问题是我有时会得到通信超时错误,我确信所有设备都响应非常快,每次都是。由于我预设sp.ReceivedBytesThreshold=8
我也开始得到CRC错误。我的回复应该总是8个字节。
我认为问题出在串口数据接收事件中,但我看不出有什么问题。
P.S。我也尝试将定时器间隔设置为1000毫秒,但这并没有解决我的问题
答案 0 :(得分:1)
依赖ReceivedBytesThreshold是非常脆弱的,当你一次失去同步时,节目结束了。您的代码也非常容易受到DataReceived可能触发的其他原因的影响,您没有检查e.EventType属性。对于二进制协议,当然可以是SerialData.Eof。
只需编写不依赖于EventType和可用字节数的健壮代码。像这样:
private byte[] rcveBuf = new byte[8];
private int rcveLen;
private void tempSerial_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
{
SerialPort sp = (SerialPort)sender;
rcveLen += sp.Read(rcvebuf, rcveLen, rcveBuf.Length - rcveLen);
if (rcveLen == rcveBuf.Length) {
Array.Copy(rcveBuf, communication.tempResponse, rcveBuf.Length);
communication.dataRecievedTemp = true;
rcveLen = 0;
}
}
并在超时时将rcveLen重置为零。并确保超时不会太低,如果你的程序被换掉,你可能会失去很多秒,使用10秒是安全的。