使用TimeFilter提示非所有控制路径返回值错误修复

报错根因 #

该错误表示返回值为bool类型的TimeFilter()函数存在无返回值的执行路径：代码可以在不触发任何return语句的情况下直接走到函数结尾的}，编译器无法保证所有分支都返回合法布尔值。
对应到你的代码逻辑，漏判了边界场景：

• 关闭时间过滤器（Use_TimeFilter == false）的分支有明确返回值，无问题
• 开启时间过滤器后，仅判断了StartHour1 < EndHour1、StartHour1 > EndHour1两种大小关系，遗漏了StartHour1 == EndHour1的相等场景
• 当计算后开始小时与结束小时相等时，两个if判断均不命中，代码直接执行到函数末尾，无返回值，触发报错。
  除此之外原代码还有一个逻辑bug：时区偏移后的小时值判断条件写为>24，当偏移结果刚好等于24时不会做转模处理，会出现非法的24点小时值。

原有代码参考 #

预定义基础参数 #

int StartHour = 8;
int EndHour = 20;

原问题实现 #

bool TimeFilter()
    {
    EndHour1=EndHour+GMTOffset;
    StartHour1=StartHour+GMTOffset;
    if ((StartHour+GMTOffset)<0)  {StartHour1=StartHour+GMTOffset+24;} 
    if ((EndHour+GMTOffset)<0)    {EndHour1=EndHour+GMTOffset+24;}     
    if ((StartHour+GMTOffset)>24) {StartHour1=StartHour+GMTOffset-24;} 
    if ((EndHour+GMTOffset)>24)   {EndHour1=EndHour+GMTOffset-24;}    
    

if (Use_TimeFilter==false) {comment1=""; return (true);}
      else 

if (StartHour1<EndHour1)
       {
         if (Hour()>=StartHour1 && Hour()<EndHour1)
         {
         comment1=""; 
         return (true);
         } 
         
         comment1="WARNING: Trading diluar Time Filter, No Open Position\n"; 
         return (false);
       }
 
     else
     
if (StartHour1>EndHour1)

       {
         if (Hour()>=StartHour1 || Hour()<EndHour1)
         {
         comment1=""; 
         return (true);
         } 
         comment1="WARNING: Trading diluar Time Filter, No Open Position\n"; 
         return (false);
       }

}

修复方案 #

• 补全StartHour1 == EndHour1的分支处理，用if-else if-else结构覆盖所有大小关系，保证所有执行路径都有明确返回值
• 修正时区偏移的边界判断，将>24改为>=24，避免出现24点的非法小时值
• 简化冗余代码写法，原有业务判断逻辑完全保留，不改变原功能行为

修复后完整代码：

// 若StartHour1、EndHour1未提前声明为全局变量，需补充这两行定义
int StartHour1;
int EndHour1;

bool TimeFilter()
{
    // 计算时区偏移后的起止小时
    EndHour1 = EndHour + GMTOffset;
    StartHour1 = StartHour + GMTOffset;
    
    // 24小时制归一化处理，修正边界判断条件
    if (StartHour1 < 0)  {StartHour1 += 24;} 
    if (EndHour1 < 0)    {EndHour1 += 24;}     
    if (StartHour1 >= 24) {StartHour1 -= 24;} 
    if (EndHour1 >= 24)   {EndHour1 -= 24;}    

    // 未启用时间过滤器，直接允许交易
    if (!Use_TimeFilter) {
        comment1 = ""; 
        return true;
    }

    // 普通时段：起止时间在同一天，例如8:00-20:00
    if (StartHour1 < EndHour1)
    {
        if (Hour() >= StartHour1 && Hour() < EndHour1)
        {
            comment1 = ""; 
            return true;
        } 
        comment1 = "WARNING: Trading diluar Time Filter, No Open Position\n"; 
        return false;
    }
    // 跨零点时段：起止时间跨天，例如20:00-次日8:00
    else if (StartHour1 > EndHour1)
    {
        if (Hour() >= StartHour1 || Hour() < EndHour1)
        {
            comment1 = ""; 
            return true;
        } 
        comment1 = "WARNING: Trading diluar Time Filter, No Open Position\n"; 
        return false;
    }
    // 补全相等边界：起止时间相同属于参数配置错误，默认禁止交易，可根据业务需求调整返回值
    else
    {
        comment1 = "WARNING: Invalid time filter config, start time equals end time\n";
        return false;
    }
}

验证说明 #

修改后所有分支逻辑完全覆盖：

• 关闭过滤器、普通时段、跨零点时段的逻辑与原代码完全一致，不改变原有业务行为
• 新增的相等分支处理了参数异常场景，彻底消除无返回值的执行路径，编译报错会直接消失
• 修正后的时区归一化逻辑可以正确处理偏移后刚好为0点/24点的边界场景

内容的提问来源于stack exchange，提问作者geoffreyLemaire