getter, setter in abstrakter Klasse oder lieber Unterklassen

[steffi12]

Hallo zusammen,
angenommen ich habe eine abstrakte Klasse, die mehrere Unterklassen haben soll:

public abstract class Customer {

    protected String name, adress;
    protected boolean paid;
   
    public Customer(String name, String adress, boolean paid) {
        this.name = name;
        this.adress = adress;
        this.paid = paid;
    }

Meine Frage ist nun; sollte man die Getter-Methoden innerhalb dieser abstrakten Klasse definieren oder in den Unterklassen? Welche ist die "bessere" / "professionellere" Variante?

Also so:

public abstract class Customer {

   protected String name, adress;
   protected boolean paid;
   
   public Customer(String name, String adress, boolean paid) {
       this.name = name;
       this.adress = adress;
       this.paid = paid;
   }
   
   public void setPaid(boolean paid) {
       this.paid = paid;
   }
   
   public boolean hasPaid() {
       return this.paid;
   }
   
   public String getName() {
       return this.name;
   }
   
   public String getAdress() {
       return this.adress;
   }
}

oder so:

public abstract class Customer {

    protected String name, adress;
    protected boolean paid;
   
    public Customer(String name, String adress, boolean paid) {
        this.name = name;
        this.adress = adress;
        this.paid = paid;
    }

public class SubClass extends Customer{

    public SubClass(String name, String adress, boolean paid) {
        super(name, adress, paid);
    }
   
    public void setPaid(boolean paid) {
        this.paid = paid;
    }
   
    public boolean hasPaid() {
        return this.paid;
    }
   
    public String getName() {
        return this.name;
    }
   
    public String getAdress() {
        return this.adress;
    }

}

Übrigens die Klasse wird nicht SubClass heissen 🙂

 

[Tarrew]

Du solltest die getter/setter auf jeden Fall in der abstrakten Oberklasse definieren.

1. hättest du sonst in jeder Unterklasse die gleichen Methoden und somit ziemlich viel doppelten Code
und
2. würde sowas nicht klappen:

Customer c = new SubClass("a", "b", false);
String adresse = c.getAdress(); // <- Geht nicht

wenn die getter/setter nur in den Unterklassen definiert sind.

 

[steffi12]

Du solltest die getter/setter auf jeden Fall in der abstrakten Oberklasse definieren.

1. hättest du sonst in jeder Unterklasse die gleichen Methoden und somit ziemlich viel doppelten Code
und
2. würde sowas nicht klappen:

Customer c = new SubClass("a", "b", false);
String adresse = c.getAdress(); // <- Geht nicht

wenn die getter/setter nur in den Unterklassen definiert sind.

Vielen Dank für die Antwort. Also es ist immer sinnvoll solche Hilfsmethoden in der abstrakten Klasse zu definieren.

 

[mrBrown]

Also es ist immer sinnvoll solche Hilfsmethoden in der abstrakten Klasse zu definieren.

Streich das "Hilfs" und das "abstrakte" 😉
Methoden da deklarieren, wo die Dinge definiert sind, die sie brauchen - bei Gettern/Settern die Klasse, in der die Variablen deklariert sind (wenn man denn welche braucht)

 

[mihe7]

Also es ist immer sinnvoll solche Hilfsmethoden in der abstrakten Klasse zu definieren.

Faustregel: Attribute sind private. Damit hat sich die Frage dann auch schon erledigt 🙂