toString überschreiben + Exception werfen

[Dag B.]

Hallo,

wie kann ich eine Funktion so überschreiben, dass ich eine Exception werfen kann?
Ich möchte toString überschreiben, aber bei falscher Stringlänge soll das Prog eben abbrechen und eine Exception werfen.

triplet ist ein char[3].

Wenn ich die Kommentarzeichen vor dem Exceptionkrams wegnehme, krieg ich logischerweise nen Fehler, da toString ja normal keine Exceptions wirft...
Aber ich find auch nirgends was, wie das sonst funktioniert, sobald ich sage "throws Exception" mag ers nich compilieren.
Und einfach catchen geht eigentlich auch nicht, denn das Programm soll ja abbrechen...

@Override
        public String toString()
        //throws Exception
       {
           if(triplet.length == 3)
           {
               String triplet_string = String.valueOf(triplet[0]) + String.valueOf(triplet[1])
                + String.valueOf(triplet[2]);
                if(triplet_string.length() == 3)
                    return triplet_string;
                else
                    return "III";// throw new Exception("Triplet(string) length too long/short"); //TODO: ERROR: Triplet(string) length
           }    
           else
               return "III";// throw new Exception("Triplet(string) length too long/short"); //TODO: ERROR: Triplet(string) length
       }

 

[Marco13]

Mit [c]throw new RuntimeException(...)[/c] geht das theoretisch, aber in der toString eine Exception zu werfen ist IMHO gröbster Unfug. Es sollte einfach immer sichergestellt sein, dass triplet.length == 3 gilt, und wenn das nicht geht, sollte toString vielleicht sowas wie "[INVALID TRIPLET]" zurückgeben (was auch noch unschön wäre, aber besser, als eine Exception zu werfen...)

 

[musiKk]

Was auch immer Du damit vor hast: Mach es anders. Warum sollte der Aufruf von toString() eine Exception werfen? Irgendwo wird die Variable triplet ja gesetzt (Konstruktor oder Setter). Dort sollte es z. B. eine IllegalArgumentException geben, wenn ein falsches übergeben wurde.
toString() ist nur für eine informative Ausgabe für den Entwickler da und selten auch für Ausgaben für den Nutzer (das sollte aber entsprechend dokumentiert sein; eine eigene Methode ist imho immer besser).

 

[Dag B.]

Hm, ok, danke.

Die toString benötige ich für equals und hashCode, sonst verwende ich sie nicht. Die beiden Methoden benötige ich wiederrum, weil ich eine HashMap verwende.

Sind auch die einzigen Stellen wo es überhaupt benötigt wird.
Eigentlich kann triplet auch nie != 3 werden, da es eine private Variable ist "private char[] triplet = new char[3];" auf welche zugegriffen wird. Oben, über den Methoden, ich vergesse immer wie das heißt. Ich dachte global, aber mein Kumpel meinte wens private ist, isses nich global. :bahnhof:
Kann man also vermutlich eh einsparen. Kenne mich da nicht so aus, mir wurde nur gesagt, ich solle alle möglichen Fehler abfangen/verarbeiten und das fiel mir damals eben mit ein. ^^

private char[] triplet = new char[3];

(...Constructor und co)

    @Override
    public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (getClass() != obj.getClass())
            return false;
        Triplet other = (Triplet) obj;
        if (!toString().equals(other.toString ()))
            return false;
        return true;
    }

    @Override
    public int hashCode() {
        int hash = 5;
        hash = toString().hashCode();
        return hash;
    }