Kapselung getter & setter

[Käthe]

Hallo zusammen,

ich bin Anfänger und möchte zur Übung folgende Aufgabe lösen.

Aufgabenstellung: Berechnen Sie die Summe von 17 und 4 und geben Sie das Ergebnis aus.

Das ganze möchte ich mit gettern & settern lösen.

In der Console wird als Ergebnis immer 0 ausgegeben.

Kann mir jemand sagen, was ich da falsch mache? Irgendwie habe ich das mit der Objekterstellung und den gettern und settern doch wohl noch nicht richtig verstanden.

class Berechnen {
   private int zahl1;
   private int zahl2;
   private int ergebnis;
 
   public void setZahl1(int zahl1){
     this.zahl1 = zahl1;
   }
   public void setZahl2(int zahl2){
     this.zahl2 = zahl2;
   }
   public void setErgebnis(int ergebnis){
     this.ergebnis = this.zahl1 + this.zahl2;
   }
 
   public int getZahl1(){
     return zahl1;
   }
   public int getZahl2(){
     return zahl2;
   }
   public int getErgebnis(){
     return ergebnis;
   }
 
 
   public Berechnen(int zahl1, int zahl2){
     this.ergebnis = this.zahl1 + this.zahl2;
   
   }
}

public class Ergebnisberechnung{
   public static void main (String[]args){
  
     Berechnen ergebnis = new Berechnen(17,4);
      //ergebnis.getErgebnis();
    
     System.out.println(ergebnis.getErgebnis());
       
   }
}

Danke für jegliche Hilfe.
Käthe.

 

[InfectedBytes]

   public Berechnen(int zahl1, int zahl2){
     this.ergebnis=this.zahl1+this.zahl2;
   
   }

hier sagst du, dass die Instanzvariable ergebnis gleich der summe der beiden instanzvariablen zahl1 und zahl2 ist.
aber die Instanzvariablen zahl1 und zahl2 wurden noch nicht zugewiesen und sind somit 0.
die Parameter zahl1 und zahl2 beinhalten momentan nur deine Zahlen.
Was du machen kannst:

   public Berechnen(int zahl1, int zahl2){
     this.zahl1 = zahl1;
     this.zahl2 = zahl2;
     this.ergebnis=this.zahl1+this.zahl2;
   }

 

[lam_tr]

@InfectedBytes
Kann man in dem Fall nicht einfach this davor weglassen?

 

[InfectedBytes]

natürlich, aber dann sind die Instanzvariablen immernoch nicht gesetzt und somit würden zumindest die getter für Zahl 1 und 2 weiterhin 0 zurückgeben

 

[SeriousD0nkey]

public Berechnen(int paramZahl1, int paramZahl2){
    zahl1 = paramZahl1;
    zahl2 = paramZahl2;
    ergebnis = zahl1 + zahl2;
}

So würde es ohne this gehen und die Instanzvariablen werden trotzdem gesetzt. Macht aber IMHO so keinen Sinn und wenn du this verwendest, sieht du immerhin direkt, dass es sich auf eine Instanzvariable bezieht.

 

[lam_tr]

@InfectedBytes
Ja ok das macht Sinn. Sorry dass ich hier etwas ausschweife, aber Code-Stil hier betrachtet, welches soll man da eher nehmen?

   public Berechnen(int zahl1, int zahl2){
      this.zahl1 = zahl1;
      this.zahl2 = zahl2;
      this.ergebnis=this.zahl1+this.zahl2;
    }

oder

   public Berechnen(int zahl1, int zahl2){
      setZahl1(zahl1);
      setzahl2(zahl2);
      this.ergebnis=this.zahl1+this.zahl2;
    }

Viele Grüße
lam

 

[VfL_Freak]

Moin,

IMHO sind die Getter und Setter eigentlich für externe Aktionen mit Klassenobjekte gedacht und nicht um die Klassenvariablen zu belegen !!
Ergo: Version 1 und OHNE this (s. o.) !!

Gruß Klaus

 

[Dompteur]

Noch etwas anderes. Ich würde folgenden Code entfernen:

   public void setErgebnis(int ergebnis){
      this.ergebnis = this.zahl1 + this.zahl2;
    }

Erstens braucht man für einen berechneten Wert keinen Setter. Und zweitens ist es sehr un-intuitiv, wenn die Methode etwas anderes tut als der Name suggeriert.

 

[VfL_Freak]

Moin,

Noch etwas anderes. Ich würde folgenden Code entfernen:

   public void setErgebnis(int ergebnis){
      this.ergebnis = this.zahl1 + this.zahl2;
    }

Erstens braucht man für einen berechneten Wert keinen Setter. Und zweitens ist es sehr un-intuitiv, wenn die Methode etwas anderes tut als der Name suggeriert.

Richtig, die ist ja identisch zu "Berechnen" !!

Wie gesagt, die getter und setter brauchst Du nur, wenn Du extern auf ein Objekt der Klasse zugreifen willst!
Von daher reicht DeinEingangspost so völlig aus!

public class Ergebnisberechnung
{
    public static void main ( String[]args )
    {
        Berechnen ergebnis = new Berechnen(17,4);
        System.out.println( ergebnis.getErgebnis() );
    }
}

Wobei Du allerdings mal dringend über die Namensgebung der Klassen und Objekt nachdenken solltest!
Warum sollte ein Objekt der Klasse Berechnen "ergebnis" heissen ??

Gruß Klaus

 

[Käthe]

@alle,

Danke für die zahlreichen Antworten und für Eure Mühe.

Ich habe mich nun für diesen Code entschieden, habe alle möglichen von Euch vorgeschlagenene
Varianten ausprobiert, immer mit dem Ergebnis, dass 0 ausgegeben wurde (was auch imme ich
da falsch gemacht habe). Jetzt funktioniert es aber.
Da ich aber anscheindend immer noch einiges durcheinander bringe, werde ich mir eure Vorschläge
nochmal in Ruhe ansehen.

public Berechnen(int zahl1, int zahl2){
          setZahl1(zahl1);
          setZahl2(zahl2);
          this.ergebnis=this.zahl1+this.zahl2;
        }

@Klaus,
Dir auch nochmal Danke für die Tipps, wann man überhaubt getter und setter braucht, ich bringe da
wirklich noch viel durcheinander und über eine sinnvollere Namensgebung werden ich mir dann auch
mal Gedanken machen.

Ich danke euch und noch einen schönen Tag.

Grüße
Käthe

 

[SeriousD0nkey]

@Käthe
Warum vorher eine 0 rauskam kann ich dir erklären:
Dein erster Code war ja folgender:

public Berechnen(int zahl1, int zahl2){
     this.ergebnis = this.zahl1 + this.zahl2;
}

Warum hier Ergebnis 0 ist, liegt daran, dass du nicht die Parameter die du mitgibst (17 und 4) verwendest, sondern auf deine Instanzvariablen der Klasse Berechnen zugreifst (mit dem this beziehst du dich auf die Instanzvariablen der Klasse). Zu diesem Zeitpunkt hast du die Instanzvariablen aber noch nicht gesetzt, sondern lediglich die Parameter (17 und 4) mitgegeben. Und da diese Instanzvariablen vom Typ int sind, ist der default-Wert 0. Und 0 + 0 ergibt ja bekanntlicherweise 0. ;-)

public Berechnen(int zahl1, int zahl2){
     this.ergebnis = zahl1 + zahl2;
}

So würde es funktionieren.

Und deine finale Lösung funktioniert ebenfalls, weil du zwar beim Rechnen wieder die Instanzvariablen verwendest, aber du setzt diese vorher auch mit den Settern. Diese Lösung ist meiner Meinung nach auch die richtige, weil du so auch wirklich die Instanzvariablen verwendest und du später weiter mit diesen Arbeiten könntest.

public Berechnen(int zahl1, int zahl2){
    setZahl1(zahl1); // hier setzt du die Instanzvariable mit dem Parameter
    setZahl2(zahl2); // hier setzt du die Instanzvariable mit dem Parameter
    this.ergebnis=this.zahl1+this.zahl2;
}

Alternativ funktioniert auch folgende Lösung:

public Berechnen(int zahl1, int zahl2){
    this.zahl1 = zahl1;
    this.zahl2 = zahl2;
    this.ergebnis=this.zahl1+this.zahl2;
}

Hierbei musst du allerdings nicht extra den Setter aufrufen. Was nun wirklich besser ist, kann ich dir nicht sagen. Eclipse macht normalerweise automatisch die erste Lösung. Diese macht auch Sinn, sobald du irgendeine Validierung (Prüfung der Variable) benötigst. Denn diese kannst du sehr schön in einen Setter verpacke. So kannst du verhindern, dass irgendein anderer Programmierer (oder du selbst ) der Variable einen (logisch) ungültigen Wert zuweist.

Hoffe, du verstehst was ich meine.

 

[Käthe]

@SeriousD0nkey

Danke für Deine super Erklärung, ich glaube, jetzt habe ich`s begriffen.

Grüße
Käthe