Можно ли в геттерах создавать новые сущности?

Question

[Evgeniy Rybalko]

Добрый день
Везде много раз читал, что каждый метод должен делать одно действие
Например, геттер - только чтобы достать значение поля

Но вот столкнулся с такой задачей
у меня есть сервис, который получает текущее состояние объекта, например
public State getState(Long chatId)
я бы хотел, чтобы в геттере, если состояние не было определено для этого сhatId, то оно создавалось
Правильный ли это подход? За такое отрывают руки сеньоры?

Answer 1

[Дмитрий]

Я бы так не делал, потому что если состояние не найдено для chatId, то это точно не обязанность класса это состояние создавать. Класс лишь возвращает состояние, если оно есть, а теперь получается. что класс еще и за создание этого состояния отвечает. А если объект состояния сложный, то в сущность придется передать целую фабрику, что как минимум принцип единой ответственности нарушает.

В таком случае, лучше вернуть null и делать проверку на null. Либо поступить правильнее и создать функцию:

public boolean checkState(Long chatId)
{
    return false;
}

и исходя из наличия состояния его либо вернуть, либо создать.

Comments

[Денис Загаевский]

откуда ты всё это знаешь? ТС не дал контекста для того, чтобы делать такие пространные рассуждения.

[Дмитрий]

Денис Загаевский, я лишь выражаю свое мнение на этот счет и прежний опыт. Не более. Опять же, я в самом начале написал, что я бы так не делал.

Answer 2

[Сергей П]

Вы собираетесь внедрить побочный эффект функции, которую можно было сделать "чистой" и идемпотентной.
Вообще предоставленных данных не достаточно, но если у вас есть концепция состояний, то архитектурно неправильно было бы делать состояния изменяемыми, а это значит что стартовое состояние вы можете делать либо при создании объекта, либо при каждом запросе этого стартового состояния, если конструктор состояния дешев а состояние не несет в себе зависимости от контекста (что тоже был бы ошибкой).

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

Короче, мало данных для конкретных рекомендаций. Однако менять внутреннее состояние объекта в читающем (по логике вещей) методе, а тем более в геттере свойства -- это плохая идея, она ухудшит код.

Comments

[Денис Загаевский]

Идемпоте́нтность (лат. idem — тот же самый + potens — способный) — свойство объекта или операции при повторном применении операции к объекту давать тот же результат, что и при первом.

И чему тут противоречит создании стейта в геттере?

[Сергей П]

Денис Загаевский, согласен, идемпотентность я зря помянул.

Answer 3

[Денис Загаевский]

Это у тебя что-то типа ленивой фабрики получается. Всё зависит от окружающего кода, имхо. Сам геттер, если он нужен только для того, чтобы доставать поля, вообще не нужен. Можно так-то и полем торчать.

Answer 4

[Dmitry Roo]

Лучше сделать два метода:
get
И
getOrCreate

Comments

[Dmitry Roo]

Возможно, окажется, что один из них не нужен и его можно выбросить

Answer 5

[Алексей Царапкин]

Чисто технически ничто не мешает так сделать, но для создания новой сущности я бы лично использовал другой метод. Если сущьность в гетере невозможно получить надо вернуть false например и проверять получение типа

if ($model->getState()) {

} else {

}

Comments

[Денис Загаевский]

в смысле в геттере вернуть false, это что за дичь такая?