Спосіб повернення декількох повернених значень методу: ввести метод всередині класу, що представляє повернене значення. Це гарний дизайн?

Мені потрібно повернути 2 значення з методу. Мій підхід такий:

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

Єдине, що буде змінено в методі, це те, що врешті-решт він призначить ці 2 значення полям екземпляра. Тоді я можу звернутися до цих значень, посилаючись на поля цього об’єкта.

Це хороший дизайн і чому?

Я б заперечував це наступним чином:

• Чому саме ваш метод повертає кілька значень? Про яку згуртованість ми говоримо - чи повинні ці значення насправді бути полями для одного класу, чи вони просто випадково повертаються тим самим методом, але інакше не пов'язані між собою? Якщо це останнє, ви, можливо, захочете розглянути розділення методу на два методи. Редагувати: використовуйте судження тут; іноді тип "випадкової" згуртованості може бути найкращим варіантом. Іншим варіантом є використання парних або кортежних конструкцій, хоча в OOP вони зазвичай не спостерігаються в загальнодоступних API (деякі помітні винятки - це стандартні колекції тощо).
• Якщо ці цінності заслуговують на формування класу, я, мабуть, радить не використовувати внутрішній клас. Внутрішні класи зазвичай використовуються як внутрішні деталі реалізації, які приховані зовні. Чи є якась причина, чому саме цей результат не повинен бути «повноцінним» класом?
• Які операції застосовні до цього нового класу, крім зберігання даних? В об'єктно-орієнтованому дизайні ви хочете, щоб відповідна поведінка була близькою до відповідних даних (які, здається, теж ваші наміри). Чи повинен метод, який ви посилаєтесь, не жити на цьому класі?

Підводячи підсумок, я побачив би, чи зможу я перетворити цей "об'єкт даних" у розроблений клас із даними та поведінкою. В якості додаткового коментаря ви можете зробити клас незмінним, оскільки його стан встановлюється один раз. Зробити його незмінним допоможе запобігти його неправильному встановленню чи зміні пізніше (скажімо, хтось встановить одне з полів на нуль і передасть його уздовж).

Редагувати: Як правильно вказує Паткос Цсаба, принцип, який тут застосовується, - це Принцип єдиної відповідальності ( 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;
  }
}

Здається, що цей клас знімає відповідальність з іншого класу, і це змушує мене думати, що цей дизайн не чудовий.

Для методу, який повертає значення, що перемножуються, я вважаю за краще

• повернути загальний контейнер (наприклад, Список або Карта), що містить значення, що повертаються

або

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

Приклад другого варіанту:

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();