IllegalArgumentException gibt den in object declueration angegebenen Wert zurück

Question

Der folgende Code ist Teil eines Programms, das eine IllegalArgumentException auslösen soll, wenn der angegebene Wert außerhalb des gültigen Bereichs liegt. Wenn jedoch die angegebenen Zahlen in setTime() außerhalb des gültigen Bereichs liegen, wird der entsprechende Wert zurückgegeben, wenn das Objekt in der main-Methode anstelle der gewünschten Fehlermeldung erstellt wurde. was ist der Grund

hier ist der Code:

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

Sie gesagt sollte throw die Ausnahme. Also sollten Sie nicht fangen es innerhalb der Methode. Entfernen Sie die try {...} catch {..} um die wenn ‚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

Sie haben die Ausnahme zu werfen.

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

müssen Sie Ihren Code Handle aktualisieren außerhalb des Bereichs

Werte

zB:

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