串口接收comGetChar(COM1,&read)使用百思不得其解

hongqi1O29

遇到一个没有想明白的事情：用串口接收指令（不定长的），但是指令是以0D0A（换行、回车）结束，结束之后再去指令的运行处理。
1. 接收是用while(comGetChar(COM1,&read))循环进行，设想串口接收缓冲器是有指令就在循环中，保存指令，碰到0D,0A,本次指令就结束，代码这样进行的：
                while(comGetChar(COM1,&read))
                {
                    if(read !='\n' && read !='\r')
                    {
                        shellMsg.msg_buf[shellMsg.idx] = read;
             shellMsg.idx++;


             PRINT_MSG("22");  //这条语句取消就接受不正常   
             
                    }
                    else
                    {
                       if(++command_end_flag == 2)
                       {
                shellMsg.msg_buf[shellMsg.idx]='\0';
                shellMsg.idx = 0;
                 shellMsg.state = SHELLMSG_WAITTING_DEAL_FINISH_COMMAND;


                  command_end_flag = 0;
                  break;
                       }


                    }
                }


奇怪在若是没有上面的PRINT_MSG("22")    这条语句取话接始终进入不了else中，也就是没法进行0D0A的结束的处理。


2. 所以加上没用的PRINT_MSG("22") ，随便打印输出什么都可以，这是一种解决方案，一直想不明白，翻看论坛，看到硬汉建议状态机，使用状态机，没有使用while，也可以解决。
                                if(comGetChar(COM1,&read))
                                {
                                        switch(ucStatus)
                                        {
                                                case 0:
                                                        if( read =='\r') //'\r' 是一个转义字符，表示回车,为0D
                                                        {
                                                                ucStatus = 1;
                                                        }
                                                        else
                                                        {
                                                                shellMsg.msg_buf[shellMsg.idx] = read;
                                                                shellMsg.idx++;
                                                       
                                                        }
                                                        break;
                                                case 1:
                                                        if(read =='\n')//'\n' 是一个转义字符，表示换行（newline）,为0A
                                                        {
                                                                shellMsg.msg_buf[shellMsg.idx]='\0';//'\0' 常用于字符串结束标志,为00;
                                                                shellMsg.idx = 0;
                                                                shellMsg.state = SHELLMSG_WAITTING_DEAL_FINISH_COMMAND;
                                                                ucStatus = 0;
                                                        }
                                                        break;
                                                default:
                                                        break;
                                                       
                                        }


3. 难道是while循环的问题？时间的问题？没有搞明白，请硬汉或者其他有经验的群友帮思考思考!

会飞的猪_2020

怀疑是comGetChar(COM1,&read)没数据的时候返回0，你就退出while造成的。

eric2013

应该是楼上群友的问题。你可以while死循环，while里面判断comGetChar(COM1,&read)

类似8266的AT串口，不过这里做了个超时，防止出错了一直在while里面

[C] 纯文本查看 复制代码

uint8_t ESP8266_WaitResponse(char *_pAckStr, uint16_t _usTimeOut)
{
	uint8_t ucData;
	uint16_t pos = 0;
	uint32_t len;
	uint8_t ret;

	len = strlen(_pAckStr);
	if (len > 255)
	{
		return 0;
	}

	/* _usTimeOut == 0 表示无限等待 */
	if (_usTimeOut > 0)
	{
		bsp_StartTimer(ESP8266_TMR_ID, _usTimeOut);		/* 使用软件定时器3，作为超时控制 */
	}
	while (1)
	{
		bsp_Idle();				/* CPU空闲执行的操作， 见 bsp.c 和 bsp.h 文件 */

		if (_usTimeOut > 0)
		{
			if (bsp_CheckTimer(ESP8266_TMR_ID))
			{
				ret = 0;	/* 超时 */
				break;
			}
		}

		if (comGetChar(COM_ESP8266, &ucData))
		{
			ESP8266_PrintRxData(ucData);		/* 将接收到数据打印到调试串口1 */

			if (ucData == _pAckStr[pos])
			{
				pos++;
				
				if (pos == len)
				{
					ret = 1;	/* 收到指定的应答数据，返回成功 */
					break;
				}
			}
			else
			{
				pos = 0;
			}
		}
	}
	return ret;
}