IllegalArgumentException возвращает значение, указанное в объявлении объекта

Question

Ниже приведен код, который должен вызывать исключение IllegalArgumentException, если заданное значение вне диапазона. Но вместо этого, если заданные числа в setTime() вне диапазона, он вернет соответствующее значение, когда объект был создан в основном методе вместо желаемого сообщения об ошибке! что является причиной

вот код:

public class MyTime { 

    private int hour = 0; 
    private int minute = 0; 
    private int second = 0; 

    public static void main (String [] args) { 
// when the value is out of range in setTime(), the value given bellow in t1 is returned 
     MyTime t1 = new MyTime (10,10,10); 
     t1.setTime(26, 23, 14); 
     System.out.println("toString(): " + t1); 
    } 

    public MyTime (int hour, int minute, int second) { 
     this.hour = hour; 
     this.minute = minute; 
     this.second = second; 
    } 
    public void setTime (int hour, int minute, int second) { 
     try { 
      if (hour > 0 && hour < 23) { 
       this.hour = hour; 
      } 

      if (minute > 0 && minute < 59) { 
       this.minute = minute; 
      } 

      if (second > 0 && second < 59) { 
       this.second = second; 
      }    

     } 
     catch (IllegalArgumentException exception) { 
      System.out.println("Invalid entry"); 
     } 
    } 

Answer 1

Вы сказали, что он должен бросок исключение. Таким образом, вы не должны catch это внутри метода. Удалите попробовать {...} поймать {..} вокруг если «s

public void setTime (int hour, int minute, int second) { 

    if (hour > 0 && hour < 23) { 
     this.hour = hour; 
    } else { 
     throw new IllegalArgumentException(); 
    } 

    if (minute > 0 && minute < 59) { 
     this.minute = minute; 
    } else { 
     throw new IllegalArgumentException(); 
    } 

    if (second > 0 && second < 59) { 
     this.second = second; 
    } else { 
     throw new IllegalArgumentException(); 
    }  

} 

Answer 2

Вы должны бросить исключение.

public void setTime (int hour, int minute, int second) { 
    try { 
     if (hour > 0 && hour < 23) { 
      this.hour = hour; 
     }else{ 
      throw new IllegalArgumentException("Invalid Hour Value"); 
     } 

     if (minute > 0 && minute < 59) { 
      this.minute = minute; 
     }else{ 
      throw new IllegalArgumentException("Invalid Minutes Value"); 
     } 

     if (second > 0 && second < 59) { 
      this.second = second; 
     }else{ 
      throw new IllegalArgumentException("Invalid Seconds Value"); 
     }    

    } 
    catch (IllegalArgumentException exception) { 
     System.out.println("Invalid entry"); 
    } 
} 

Answer 3

вам необходимо обновить свой код ручки за пределами диапазона значений

например:

if (hour > 0 && hour < 23) 
    this.hour = hour; 
else 
    throw new IllegalArgumentException();