Hartgecodete Daten auslagern

[aCleanCoder]

Hallo,

folgender Fall: Eine von mir vor einem Jahr erstelle Anwendung möchte ich ein wenig überarbeitet. In der Anwendung wird mit Schriftstücken gearbeitet, welches u. A. aus einer ID und einer Bezeichnung bestehen. Dieser Daten werden an verschiedensten Stellen an irgendwelche Services übergeben. Aktuell werden die Informationen hardgecodet übergeben, wodurch die Wartung natürlich erschwert wird. Die Informationen habe ich nun ausgelagert, bin mir aktuell aber noch unsicher, ob dies so Designtechnisch "in Ordnung" ist, oder ob ich es evtl. anders lösen sollte. Ich habe mal einen vereinfachten Fall konstruiert.

public class Schriftstueck {
    public static final Schriftstueck DOK_1 = new Schriftstueck("id_1","bez_1");
    public static final Schriftstueck DOK_2 = new Schriftstueck("id_2","bez_2");
    public static final Schriftstueck DOK_3 = new Schriftstueck("id_3","bez_3");
    public static final Schriftstueck DOK_4 = new Schriftstueck("id_4","bez_4");
    public static final Schriftstueck DOK_5 = new Schriftstueck("id_5","bez_5");
    
    String id;
    String bezeichnung;



    private Schriftstueck(String id, String bezeichnung) {
        this.id = id;
        this.bezeichnung = bezeichnung;
    }


    public String getId() {
        return id;
    }


    public void setId(String id) {
        this.id = id;
    }


    public String getBezeichnung() {
        return bezeichnung;
    }


    public void setBezeichnung(String bezeichnung) {
        this.bezeichnung = bezeichnung;
    }
    
}

public class Main {

    public static void main(String[] args) {   
     serviceaufruf(Schriftstueck.DOK_2.getId(), Schriftstueck.DOK_2.getBezeichnung());
     serviceaufruf(Schriftstueck.DOK_4.getId(), Schriftstueck.DOK_4.getBezeichnung());       
    }
    
    static void serviceaufruf(String id, String bezeichnung) {
        System.out.println("id: " + id);
        System.out.println("bezeichnung: " + bezeichnung);
    }

}

Passt das soweit?

 

[httpdigest]

Die zwei Setter in Schriftstueck solltest zu entfernen (sonst kann man ja aus Versehen die Felder in den vordeklarierten static Instanzen verändern) und die Felder private final deklarieren. Ansonsten, klar, passt.
Wenn du willst, kannst du aus Schriftstueck auch ein enum machen, denn du hast ja hier eine endliche Anzahl an möglichen Werten/Instanzen.

 

[aCleanCoder]

Die zwei Setter in Schriftstueck solltest zu entfernen (sonst kann man ja aus Versehen die Felder in den vordeklarierten static Instanzen verändern) und die Felder private final deklarieren. Ansonsten, klar, passt.
Wenn du willst, kannst du aus Schriftstueck auch ein enum machen, denn du hast ja hier eine endliche Anzahl an möglichen Werten/Instanzen.

Stimmt, Setter brauche ich ja hier nicht. Ist irgendwie die Gewohnheit, dass ich mir für alle Klassenvariablen Getter/Setter generieren lasse. 🙄 Danke!

 

[httpdigest]

Ist irgendwie die Gewohnheit, dass ich mir für alle Klassenvariablen Getter/Setter generieren lasse.

Hier kann man auch einfach Lombok verwenden. Dann einfach mit @Value (für immutable Dinge) oder @Data annotieren und du hast keine solchen Probleme mehr.

 

[yfons123]

es kommt drauf an wie und was gebraucht wird..

so wie du es jetzt im moment hast kannst du es auch in eine Enum umwandeln .. da du wahrscheinlich nur die 5 schriftstücke hast und die Fix sind
ODER
du hast ganz viele und die sind nur "der standard" dann willst du natürlcih nicht beim hinzufügen von einem schriftstück dein programm neu kompilieren.. dann kannst du zb die Datenformate nutzen und diese laden anstatt sie hard zu coden

xml
json
datenbank ( vllt auch sqlite lokal und gar keinen Server )
CSV
....

somit kannst du das verhalten deines programmes ändern ohne den code anzufassen