squid: S2583 - Sonarqube ложно положителен при использовании переключателя

Question

Для следующего надуманного примера (реальный код на самом деле имеет смысл, я обещаю), Sonarqube 5.3 с плагином Java 2.10 даст мне страшное «Измените это условие, чтобы оно не всегда оценивать «истина»»:

public String sonarLint(DayOfWeek dow) { 
    boolean one = false; 
    boolean two = false; 
    switch (dow) { 
    case MONDAY: 
     one = true; 
    case TUESDAY: 
     two = true; 
     break; 
    default: 
     // nothing 
    } 
    return one && two ? "yes" : "no"; 
} 

насколько я могу видеть, когда dow является MONDAY, условие истинно, в то время как оно ложно в противном случае (IntelliJ соглашается, кстати, говорил мне, что two всегда whill быть правдой когда он вообще оценивается). Я ударил ошибку в Sonarqube здесь?

Answer 1

Сонар должен сообщить об ошибке и для отсутствия инструкции break.

Корпуса выключателей должны заканчиваться безусловным оператором «break» (Squid: S128).

Как только у вас есть оператор break в обоих случаях, оператор никогда не будет выполнен как истинный.