Exception unreachable

[jimbo1000]

Hallo,

ich verstehe den fehler einer Testklasse nicht ganz Hier mal den Ausschnitt um den es geht meiner Testklasse.

public void testZuJungExceptionGeworfenStudent(){
                        
        try {
            Student stud1 = new StudentAgeChecked("Martin", 2008);
            assertTrue("Falsch, stud1 darf nicht angelegt werden", false);
        } catch (ZuJungZumStudierenException zje) {
            assertTrue("Korrekt, stud1 darf nicht angelegt werden und es wird stattdessen die ZuJungZumStudierenException geworfen", true);
            assertEquals("Die Fehlermeldung stimmt nicht", "Die Person ist zu jung zum Studieren.", zje.getMessage());
        } catch (ZuJungException e) {
            assertTrue("Falsch, es wird zwar die ZuJungException geworfen, aber es sollte die ZuJungZumStudierenException kommen", false);
        } catch (Exception e) {
            assertTrue("Falsch, es wird zwar eine Exception geworfen, aber es sollte die ZuJungZumStudierenException kommen", false);
        }        
        
    }
    
    public void testZuJungExceptionGeworfenStudentMitMatrnr(){
        
        try {
            Student stud1 = new StudentAgeChecked("Martin", 2008, "111111");
            assertTrue("Falsch, stud1 darf nicht angelegt werden", false);
        } catch (ZuJungZumStudierenException zje) {
            assertTrue("Korrekt, stud1 darf nicht angelegt werden und es wird stattdessen die ZuJungZumStudierenException geworfen", true);
            assertEquals("Die Fehlermeldung stimmt nicht", "Die Person ist zu jung zum Studieren.", zje.getMessage());
        } catch (ZuJungException zje) {
            assertTrue("Falsch, es wird zwar die ZuJungException geworfen, aber es sollte die ZuJungZumStudierenException kommen", false);
        } catch (Exception e) {
            assertTrue("Falsch, es wird zwar eine Exception geworfen, aber es sollte die ZuJungException kommen", false);
        }

hier mein code von der zu testenden Klasse

public class StudentAgeChecked extends Student {
    
    public StudentAgeChecked(String name, int geburtsjahr)throws ZuJungZumStudierenException {
        
        super(name,geburtsjahr);
        
        if(this.getGeburtsjahr() >= 2001) {
            
        }else throw new ZuJungZumStudierenException();
        
        
    }
    
    public StudentAgeChecked(String name, int geburtsjahr, String matrnr) throws ZuJungZumStudierenException {
        this(name,geburtsjahr);
        matrnr = this.getMatrNr();
        
    }

}

an der Testklasse kann ich nichts ändern ich versuche im konstruktor von der klasse Student die Exception die ich selbst definiert habe abzufangen. Allerdings bekomme ich immer den Fehler " Unreachable catch block for ZuJungException. This exception is never thrown from the try statement body ".

 

[httpdigest]

Dadurch, dass am Konstruktor von StudentAgeChecked nur angegeben ist, dass potenziell eine `ZuJungZumStudierenException` geworfen wird (und keine `ZuJungException`), weiß der Compiler, dass der Konstruktor auch wirklich NIEMALS `ZuJungException` werfen wird, weil das sonst vom Konstruktor auch angegeben werden müsste. Somit wird niemals der `catch (ZuJungException)` betreten.

 

[jimbo1000]

ok danke habe meinen Fehler gefunden habe die ZuJungZumStudieren einfach von der zuJung exception erben lassen. Allerding sehe ich jetzt den Fahler das in meiner StudentAgeChecked Klasse das Objekt angelegt wird und danach die prüfung stattfindet. Ich habe aber absolut keine idee wie ich die Püfuung sonst in einem konstruktor machen soll. Ich muss ja vorher mit super aufrufen und if(super(konstruktor von oben) geht ja nicht. Hast du vllt noch einen Denkanstoß?

 

[Xyz1]

Bitte den Code nicht fett machen. Innerhalb von Code-Tags kann ich ihn dann nicht mehr lesen.
( Auch nicht kursiv )

 

[jimbo1000]

tut mir leid ich merke es mir

 

[jimbo1000]

hat niemand einen denkanstoß oder tipp für mich? Ich will in dem Konstruktor

public StudentAgeChecked(String name, int geburtsjahr)throws ZuJungZumStudierenException {

die prüfung machen und die exception auslösen

 

[Xyz1]

Ein normaler Programmfluss sollte nicht über Exceptions gesteuert werden.

 

[Xyz1]

AAAAber, wo wäre die super class Student?

 if(this.getGeburtsjahr() >= 2001) {
            
        }else throw new ZuJungZumStudierenException();

Das müsstest Du swappen:

 if(this.getGeburtsjahr() >= 2001) {
            throw new ZuJungZumStudierenException();
        }

 

[jimbo1000]

Ok danke für den Hinweis. Mich deine Lösung scheint zu klappen. Ich hatte vorher meine if nicht ganz richtig formuliert. Hier dein code vorschlag.

public class StudentAgeChecked extends Student {
   
    public StudentAgeChecked(String name, int geburtsjahr)throws ZuJungZumStudierenException {
       
        super(name,geburtsjahr);
       
        if(this.getGeburtsjahr() >= 2001) {
            throw new ZuJungZumStudierenException();
        }//else throw new ZuJungZumStudierenException();
       
       
    }
   
    public StudentAgeChecked(String name, int geburtsjahr, String matrnr) throws ZuJungZumStudierenException {
        this(name,geburtsjahr);
        matrnr = this.getMatrNr();
       
    }
   
   
   
   

}

 

[jimbo1000]

hier mein Konstrukt mit deiner Korrektion:

public class StudentAgeChecked extends Student {
    
    public StudentAgeChecked(String name, int geburtsjahr)throws ZuJungZumStudierenException {
        
        super(name,geburtsjahr);
        
        if(this.getGeburtsjahr() <= 2001) {
            //throw new ZuJungZumStudierenException();
        }else throw new ZuJungZumStudierenException();
        
        
    }
    
    public StudentAgeChecked(String name, int geburtsjahr, String matrnr) throws ZuJungZumStudierenException {
        this(name,geburtsjahr);
        matrnr = this.getMatrNr();
        
    }
    
    
    
    

}

 

[jimbo1000]

könnt ihr mir sagen ob das so Konvention ist das dass hier mit den reihenfolge klappt im Konstruktor? Normalerweise würde ich im ersten Moment behaupten das erst das Objekt erzeugt wird und er dann an die if geht. Hier scheint das ja wirklich als "packet" behandelt zu werden.

 

[jimbo1000]

danke für eure bisherige hilfe

 

[mihe7]

könnt ihr mir sagen ob das so Konvention ist das dass hier mit den reihenfolge klappt im Konstruktor? Normalerweise würde ich im ersten Moment behaupten das erst das Objekt erzeugt wird und er dann an die if geht. Hier scheint das ja wirklich als "packet" behandelt zu werden.

Wovon schreibst Du? Ein Konstruktor wird während der Objekterzeugung aufgerufen. Dabei werden erst der/die Konstruktor(en) der super-Klasse aufgerufen, bevor der jeweilige Konstruktor abgearbeitet wird.