Rückgabe oder Parameter?

[Marsman]

Hallo Ihr!

Leider fällt mir oft die Entscheidung schwer, ob ich ein Objekt durch eine Methode zurückgeben oder es als Parameter übergeben und von der Methode ändern lassen soll. Zum Beispiel bei einem Datensatz, der durch eine SQL-Anweisung ermittelt wird.

Beispiel 1:

Datensatz ds = getDatensatz();

Datensatz getDatensatz() {
    Datensatz ds = new Datensatz();
    // Code zum holen der Daten
    ds.setField1 = resultSet.getInt(1);
    return ds;
}

Beispiel 2:

Datensatz ds = new Datensatz();

void getDatensatz(Datensatz ds) {
    // Code zum holen der Daten
    ds.setField1 = resultSet.getInt(1);
}

Mir ist nicht ganz klar, wo die Vor- und Nachteile liegen. Oder ist es einfach nur Geschmackssache?

Danke schonmal,

Titus

 

[dotlens]

mit der 2ten Variante kannst du diesen Datensatz setzen, den du möchtest, beim ersten ist er fix. ausserdem verfehlt das erste beispieil irgendwie denn sinn der sache.
kannst du ja gerade so gut in deinem code schreiben, da diese methode kein bischen wiederverwendbar ist...

 

[Marsman]

mit der 2ten Variante kannst du diesen Datensatz setzen, den du möchtest, beim ersten ist er fix. ausserdem verfehlt das erste beispieil irgendwie denn sinn der sache.
kannst du ja gerade so gut in deinem code schreiben, da diese methode kein bischen wiederverwendbar ist...

Ich verstehe deine Formulierung nicht ganz...

Warum verfehlt das erste Beispiel den Sinn? Es gibt den Datensatz als Objekt zurück. Es gibt doch viele Methoden, die Objekte zurückgeben, statt sie zu ändern.

Und was meinst du mit wiederverwendbar? Vererbbar? Warum sollte sie das nicht sein?

Gruß, Titus

 

[Sky]

Ich glaube bei deinen Beispielen kommt es einfach darauf an, was Ziel der Methode sein soll:

In der 1.) Variante ist das Ziel eher "ermittele mir die Daten und gib sie mir"; in der 2.) Variante geht es darum "Hier ist ein Datensatz und mach was damit".

Ansonsten gibt es natürlich noch andere Möglichkeiten; ein Beispiel:

public void doSomething() {
  Datensatz ds = new Datensatz();
  if ( getDatensatz(ds) ) {
    // Mache irgendwas mit 'ds'
  }
}

...
 
public boolean getDatensatz( Datensatz ds ) {
  // 1. ) Versuch den Datensatz zu ermitteln
  // 2.) boolean zurückgeben: true, wenn erfolgreich geladen / false im Fehlerfall
}

 

[Guest]

Ansonsten gibt es natürlich noch andere Möglichkeiten; ein Beispiel:

(...)

Deine Variante gefällt mir inzwischen auch am besten. Denn unabhängig von den Daten selbst kann die Methode den Aufrufer noch über den Erfolg der Verarbeitung informieren. Wenngleich sie das auch über eine Exception könnte.

Ich habe nur einfach immer noch ein komisches Gefühl beim Ändern von Eingabeparametern in einer Methode. Bisher habe ich Funktionen (um es mal allgemein zu benennen) immer nach dem Prinzip 0-n Eingabeparameter und maximal einen Ausgabeparameter programmiert. Dieser musste dann eine Struktur sein, wenn eine Gruppe von Werten zurückgegeben werden soll.

Titus

 

[Sky]

Denn unabhängig von den Daten selbst kann die Methode den Aufrufer noch über den Erfolg der Verarbeitung informieren. Wenngleich sie das auch über eine Exception könnte.

Na Klar könnte Sie das. Aber: Exceptions sind -wie der Name schon sagt- Ausnahmen und deshalb sollte man m.E. sparsam damit umgehen. Denn: Exceptions sind langsamer als o.g. Konstrukt. Wenn also davon auszugehen ist, dass nur wenige Ausführungen schief gehen, so sind Exceptions der richtige Ansatz. Ist die Wahrscheinlichkeit hoch, dass z.B. das Laden der Datensätze mal nicht geht, so sollte man dann dies lieber über einen Statuswert zurückgeben (i.d.R. boolean; aber ein eigenes Error-Objekt oder auch ein int kann manchmal hilfreich sein)

Bisher habe ich Funktionen (um es mal allgemein zu benennen) immer nach dem Prinzip 0-n Eingabeparameter und maximal einen Ausgabeparameter programmiert. Dieser musste dann eine Struktur sein, wenn eine Gruppe von Werten zurückgegeben werden soll.

Das mit der Rückgabe von Strukturen ist auch kein Problem:

class Struktur {
  private boolean m_bStatus;
  private int m_iErrorCode;
  private String m_sErrorText;
  ...
  // getter  und setter dazu.
}

...

public Struktur doSomething() {
Struktur oStruktur = new Struktur();
// Mache irgendwas
oStruktur.setStatus( false );
oStruktur.setErrorCode( 99 );
oStruktur.setErrorText( "Leider ist das Laden nicht möglich" );
return oStruktur;
}