一次死循环调试

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

一种隐含死循环

现象

在项目中,站机软件与多台分机通信,发现如果一台分机连接不上后,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)
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值