Unbenutztes Element

🚧

Diese Seite befindet sich in Bearbeitung

🚧

🤓

Diese Seite ist eine Bewertungsrichtlinie, die ab Blatt 1 annotiert und ab Blatt 2 abgezogen wird.

🤓

Beschreibung

Unbenutzte Elemente wie Klassen, Methoden und Zeilen sollten aus dem finalen Produkt entfernt werden. Sie haben keinen weiteren Nutzen und stören lediglich den Lesefluss.

Auch Elemente, die einen vermeintlichen zukünftigen Nutzen haben könnten, sollten nicht in der finalen Version vorhanden sein. Das sind zum Beispiel Getter- und Setter–Methoden für Attribute, die zurzeit nicht genutzt werden.

Sollte eine Person diese Attribute in einer zukünftigen Version nutzen wollen, kann sie die nachträglich hinzufügen.

Des Weiteren wird das Programm durch zu viele unbenutzte Elemente unnötig groß, was außerdem das Debugging erschwert. Auch wenn der Code im aktuellen Zustand einen überschaubaren Umfang hat, sollte von vorne herein darauf geachtet werden, dass im finalen Release keine unbenutzten Elemente zu finden sind.

Moderne IDEs weisen auf unbenutzte Elemente hin.

Negativbeispiel:

public static void main(String[] args) {
    Vector vector = new Vector(1, 2, 3);
    System.out.println(vector.getLength());
}

...

public class Vector {
    private final int x;
    private final int y;
    private final int z;
    
    public Vector(int x, int y, int z) {
        this.x = x;
        this.y = y;
        this.z = z;
    }

    public int getX() {
        return this.x;
    }

    public int getLength() {
        return Math.sqrt(x*x + y*y + z*z);
    }
}

Auch wenn es in Zukunft vielleicht gut wäre, die Getter-Methode für die x-Koordinate des Vektors zu behalten, ist sie in unserem Programm gerade unbenutzt und kann und soll demnach entfernt werden.

Positivbeispiel:

public static void main(String[] args) {
    Vector vector = new Vector(1, 2, 3);
    System.out.println(vector.getLength());
}

...

public class Vector {
    private final int x;
    private final int y;
    private final int z;

    public Vector(int x, int y, int z) {
        this.x = x;
        this.y = y;
        this.z = z;
    }

    public int getLength() {
        return Math.sqrt(x*x + y*y + z*z);
    }
}