Способ вернуть несколько возвращаемых значений из метода: поместить метод в класс, представляющий возвращаемое значение. Это хороший дизайн?

Мне нужно вернуть 2 значения из метода. Мой подход заключается в следующем:

1. создайте внутренний класс с двумя полями, которые будут использоваться для хранения этих двух значений
2. положить метод внутри этого класса
3. создать экземпляр класса и вызвать метод.

Единственное, что будет изменено в методе, это то, что в конце он назначит эти 2 значения полям экземпляра. Затем я могу обратиться к этим значениям, ссылаясь на поля этого объекта.

Это хороший дизайн и почему?

Я бы поспорил по этому поводу:

• Почему именно ваш метод возвращает несколько значений? О каком единстве мы говорим - должны ли эти значения быть полями в одном классе, или они просто по совпадению возвращаются одним и тем же методом, но никак не связаны? Если это последнее, вы можете рассмотреть возможность разделения метода на два метода. Изменить: используйте свое мнение здесь; иногда тип «случайной» сплоченности может быть лучшим вариантом. Другой вариант - использовать пару или кортеж, хотя в ООП они обычно не видны в общедоступных API-интерфейсах (некоторые заметные исключения - это стандартные коллекции и т. Д.).
• Если значения заслуживают формирования класса, я бы, вероятно, не советовал использовать внутренний класс. Внутренние классы обычно используются как внутренние детали реализации, которые скрыты снаружи. Есть ли какая-то причина, по которой этот результат сам по себе не должен быть «полноценным» классом?
• Какие операции применимы к этому новому классу, кроме хранения данных? В объектно-ориентированном проектировании вы хотите, чтобы связанное поведение было близко к соответствующим данным (что, похоже, также является вашим намерением). Должен ли метод, на который вы ссылаетесь, жить не в этом классе?

Подводя итог, я хотел бы посмотреть, смогу ли я превратить этот «объект данных» в полноценный класс с данными и поведением. В качестве дополнительного комментария вы можете сделать класс неизменным, поскольку его состояние устанавливается один раз. Если сделать его неизменным, это поможет предотвратить его неправильную настройку или изменение позже (скажем, кто-то установит одно из полей в null и передаст его).

Редактировать: Как правильно отмечает Паткос Чаба, применяемый здесь принцип - это принцип единой ответственности ( SRP ) - класс, который вы пытаетесь создать, должен действительно иметь одну ответственность (определяемую как причина для изменения ). Это руководство по проектированию должно помочь вам определить, принадлежат ли ваши два поля одному классу или нет. Если придерживаться примера Википедии, ваш класс может рассматриваться как тип отчета, и в этом случае он соответствует SRP, но его сложно комментировать без дополнительной информации.

Есть концепция Tuple, которая представлена ​​в других языках, таких как Python.

Можно вернуть экземпляр этого обобщенного класса, который легко использовать повторно:

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;
  }
}

Кажется, что этот класс берет на себя ответственность от другого класса, и это заставляет меня думать, что этот дизайн не велик.

Для метода, который возвращает многозначные значения, я бы предпочел

• вернуть общий контейнер (например, список или карту), содержащий возвращаемые значения

или

• создать класс для возвращаемого значения, который содержит только необходимые поля + геттеры + конструктор со всеми полями

Пример для второго варианта:

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;
   }
}

Помещение нескольких возвращаемых значений из метода в его собственный класс / структуру часто используется в системах , основанных на сообщениях, которые имеют один класс для запроса и ответа . Примером этого является простой протокол доступа к объектам (SOAP) .

Это хороший дизайн и почему?

По крайней мере, это довольно часто. Хорошо это или плохо, зависит от вашего особого варианта использования.

Краткий ответ: вы можете вернуть массив или список с двумя значениями.

Я лично написал бы два разных метода, как

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