一次死循环调试

最新推荐文章于 2023-09-25 16:45:57 发布

原创

最新推荐文章于 2023-09-25 16:45:57 发布 · 498 阅读

0 ·

CC 4.0 BY-SA版权

在项目中遇到站机软件与分机通信时，若某分机连接失败，UI界面会卡死，原因是OnTimer函数调用的DealHeatinglist存在潜在死循环。死循环产生于 DealHeatinglist 的条件判断错误，GetReadyHeatingcmd返回的非空加热命令不保证i_cnt自增，导致无法退出循环。解决方案包括使用for循环替代while循环，或者修改GetReadyHeatingcmd为返回命令列表并遍历处理。

一次死循环调试

一种隐含死循环

一种隐含死循环

现象

在项目中，站机软件与多台分机通信，发现如果一台分机连接不上后，UI界面基本卡死。于是，开始查找什么原因导致UI无响应，后来发现OnTimer函数调用了DealHeatinglist，DealHeatinglist有可能造成死循环。

void c_jyq_dlg::DealHeatinglist(CTLinkList<struct s_heating_cmd> *p_heatinglist, unsigned ui_heatingcancnt)
{
	//deal heating cmd first, then deal ready cmd
	int i_cnt = 0;
	POS pos = p_heatinglist->GetHeadPosition();
	while (NULL != pos)
	{
		struct s_heating_cmd* p_cmd = p_heatinglist->GetNext(pos);
		if (E_HCMDS_READY == p_cmd->by_cmdstatus) continue;

		DealHeatingcmd(p_cmd);
		if (E_HCMDS_SUBDONE != p_cmd->by_cmdstatus)
		{
			i_cnt++;
		}
	}
	myassert(i_cnt <= theApp.m_sysparam.ui_heatingcnt_perline);

	i_cnt = 0;
	myassert(ui_heatingcancnt <= theApp.m_sysparam.ui_heatingcnt_perline);
	while (i_cnt < ui_heatingcancnt)
	{
		struct s_heating_cmd* p_cmd = GetReadyHeatingcmd(p_heatinglist);
		if (NULL == p_cmd)
			break;

		Dea