「爲」用Java迴路斷線不工作

Question

我有下面的代碼：

private boolean result = false; 
public boolean checkHourExist(int hourfrom, 
           int minutefrom, 
           int hourto, 
           int minuteto, 
           int day, 
           int templateId) 
{ 
    List<templates> t = getAllTemplatesWithDays(); 

    for (templates tem : t) 
    { 
     if(day == -1 | tem.Temp_DayId == day) 
     { 
      if(tem.Temp_HourFrom >= hourfrom & tem.Temp_HourTo == hourto) 
      { 
       if(tem.Temp_HourFrom == hourfrom) 
       { 
        if(tem.Temp_MinFrom == minutefrom) 
        { 
         result= true; 
         break; 
        } 
        else if(minutefrom < tem.Temp_MinFrom & minuteto >= tem.Temp_MinTo) 
        { 
         result = true; 
         break; 
        } 
       } 
       else 
       { 
        result = true; 
        break; 
       } 
      } 
      else if(tem.Temp_HourFrom == hourfrom & tem.Temp_HourTo < hourto) 
      { 
       result = true; 
       break; 
      } 
      else if(tem.Temp_HourFrom > hourfrom & tem.Temp_HourTo < hourto) 
      { 
       result = true; 
       break; 
      } 
     } 
    } 
    return result; 
} 

當代碼去我if S的一個，該循環永遠不會打破，並一直持續到結束，我變量結果不會取真值。我認爲這可能是愚蠢的，但我看不出什麼錯誤。

Answer 1

您必須在if s中將|和&加倍。

||裝置或
&&裝置和

|和&是bitewise運算符。

要澄清我的答案，您的break條件對我來說似乎很好，我在代碼中看到的唯一真正的問題是我提到的操作符。

Answer 2

如何返回而不是打破。從我的角度來看，這使得它更具可讀性

private boolean result = false; 
public boolean checkHourExist(int hourfrom,int minutefrom,int hourto,int minuteto,int day,int templateId) 
{ 

    List<templates> t = getAllTemplatesWithDays(); 

    for (templates tem : t) 
    { 
     if(day == -1 | tem.Temp_DayId == day) 
     { 
      if(tem.Temp_HourFrom >= hourfrom & tem.Temp_HourTo == hourto) 
      { 
       if(tem.Temp_HourFrom == hourfrom) 
       { 
        if(tem.Temp_MinFrom == minutefrom) 
        { 
         return true; 
        } 
        else if(minutefrom < tem.Temp_MinFrom & minuteto >= tem.Temp_MinTo) { 
         return true; 
        } 
       } else 
       { 
        return true; 
       } 
      } 
      else if(tem.Temp_HourFrom == hourfrom & tem.Temp_HourTo < hourto) { 
       return true; 
      } 
      else if(tem.Temp_HourFrom > hourfrom & tem.Temp_HourTo < hourto) { 
       return true; 
      } 
     } 
    } 

    return false; 
}