Eine Möglichkeit, mehrere Rückgabewerte von einer Methode zurückzugeben: Setzen Sie die Methode in die Klasse, die den Rückgabewert darstellt. Ist es ein gutes Design?

Ich muss 2 Werte von einer Methode zurückgeben. Mein Ansatz ist wie folgt:

1. Erstellen Sie eine innere Klasse mit 2 Feldern, die verwendet werden, um diese 2 Werte beizubehalten
2. Fügen Sie die Methode in diese Klasse ein
3. Instanziieren Sie die Klasse und rufen Sie die Methode auf.

Das einzige, was an der Methode geändert wird, ist, dass diese beiden Werte am Ende den Feldern der Instanz zugewiesen werden. Dann kann ich diese Werte ansprechen, indem ich mich auf die Felder dieses Objekts beziehe.

Ist es ein gutes Design und warum?

Ich würde dies folgendermaßen argumentieren:

• Warum gibt Ihre Methode genau mehrere Werte zurück? Um welche Art von Zusammenhalt handelt es sich? Sollten diese Werte tatsächlich Felder in einer einzelnen Klasse sein, oder werden sie nur zufällig mit derselben Methode zurückgegeben, aber ansonsten ohne Bezug? In letzterem Fall können Sie die Methode in zwei Methoden aufteilen. Bearbeiten: Verwenden Sie Ihr Urteil hier; manchmal kann eine Art "zufälliger" Zusammenhalt die beste Option sein. Eine andere Option ist die Verwendung eines Paar- oder Tupelkonstrukts, obwohl diese in OOP normalerweise nicht in öffentlichen APIs angezeigt werden (einige bemerkenswerte Ausnahmen sind Standardauflistungen usw.).
• Wenn die Werte es verdienen, eine Klasse zu bilden, würde ich wahrscheinlich davon abraten, eine innere Klasse zu verwenden. Innere Klassen werden normalerweise als interne Implementierungsdetails verwendet, die von außen verborgen sind. Gibt es einen Grund, warum dieses Ergebnis nicht für sich genommen eine "ausgewachsene" Klasse sein sollte?
• Welche Operationen gelten für diese neue Klasse außer dem Halten von Daten? Im objektorientierten Design möchten Sie das zugehörige Verhalten in der Nähe der relevanten Daten haben (was auch Ihre Absichten zu sein scheinen). Sollte die Methode, auf die Sie sich beziehen, nicht eher von dieser Klasse leben?

Zusammenfassend würde ich sehen, ob ich dieses "Datenobjekt" in eine vollwertige Klasse mit Daten und Verhalten umwandeln kann. Als zusätzlichen Kommentar möchten Sie die Klasse möglicherweise unveränderlich machen, da ihr Status einmal festgelegt wird. Wenn Sie es unveränderlich machen, können Sie verhindern, dass es falsch eingestellt oder später geändert wird (z. B. wenn jemand eines der Felder auf null setzt und es weitergibt).

Bearbeiten: Wie Patkos Csaba richtig hervorhebt, wird hier das Prinzip der einmaligen Verantwortung ( Single Responsibility Principle, SRP ) angewendet. Die Klasse, die Sie erstellen möchten , sollte wirklich eine Verantwortung haben (als Grund für eine Änderung definiert ). Diese Gestaltungsrichtlinie soll Ihnen dabei helfen, herauszufinden, ob Ihre beiden Felder zu einer Klasse gehören oder nicht. Um beim Wikipedia-Beispiel zu bleiben, könnte Ihre Klasse als eine Art Bericht angesehen werden. In diesem Fall entspricht sie der SRP, ist jedoch ohne weitere Informationen schwer zu kommentieren.

Es gibt das Konzept eines Tupels, das in anderen Sprachen wie Python vorkommt.

Man könnte eine Instanz dieser generierten Klasse zurückgeben, die leicht wiederverwendbar ist:

public class TypedTuple<L, R> implements Serializable {
private static final long serialVersionUID = 1L;

  protected L left;
  protected R right;

  protected TypedTuple() {
    // Default constructor for serialization
  }

  public TypedTuple(L inLeft, R inRight) {
    left = inLeft;
    right = inRight;
  }

  public L getLeft() {
    return left;
  }

  public R getRight() {
    return right;
  }
}

Es scheint, dass diese Klasse einer anderen Klasse die Verantwortung abnimmt, und das lässt mich denken, dass dieser Entwurf nicht großartig ist.

Für eine Methode, die mehrere Werte zurückgibt, würde ich eher

• gib einen generischen Container zurück (zB eine Liste oder Map), der die Rückgabewerte enthält

oder

• Erstellen Sie eine Klasse für den Rückgabewert, die nur die erforderlichen Felder + Getter + einen Konstruktor mit allen Feldern enthält

Beispiel für die zweite Option:

public Class FooBarRetval {
   private String foo;
   private int bar;

   public FooBarRetval (String foo, int bar) {
      this.foo = foo;
      this.bar = bar;
   }

   public String getFoo() {
      return foo;
   }

   public int getBar() {
      return bar;
   }
}

Das Einfügen mehrerer Rückgabewerte von einer Methode in eine eigene Klasse / Struktur wird häufig in nachrichtenbasierten Systemen verwendet , die eine Klasse für Anforderung und Antwort haben . Ein Beispiel hierfür ist das Simple Object Access Protocol (SOAP) .

Ist es ein gutes Design und warum?

Zumindest ist es durchaus üblich. Ob dies gut oder schlecht ist, hängt von Ihrem speziellen Verwendungszweck ab.

Kurze Antwort: Sie können ein Array oder eine Liste mit den beiden Werten zurückgeben.

Ich persönlich würde gerne zwei verschiedene Methoden schreiben

int x = obj.getX();
int y = obj.getY();