170 похожих чатов

Всех приветствую! Архитектурный вопрос. На сколько хороша такая конструкция: @Component @RequiredArgsConstructor public class ConfirmationTokenFactory

{
private final ConfirmationTokenProperties tokenProperties;

/**
* Create new confirmation token for person.
* @param person The target person
* @return The new confirmation token for saving into repository
*/
public ConfirmationToken createConfirmationToken(Person person) {
LocalDateTime createdAt = LocalDateTime.now();
long milliseconds = tokenProperties.getMillis();
LocalDateTime expiresAt = createdAt.plus(milliseconds, MILLIS);
String token = UUID.randomUUID().toString();
ConfirmationToken confirmationToken = ConfirmationToken.builder()
.createdAt(createdAt).expiresAt(expiresAt)
.person(person).token(token).build();
return confirmationToken;
}
}
Сейчас рефакторю код, и хочу вынести создание объектов в отдельные классы (раньше token создавался прямо в коде, по среди метода).
И вопрос: правильно ли называть такой класс - фабрикой (она же вроде строиться на полиморфизме, создании разных типов), или лучше придумать какое-нибудь другое название? Или вообще - есть более элегантный способ в создании "базовых" объектов для сохранения?

10 ответов

25 просмотров

как измеряется элегантность?

Andrzej-Jankowski Автор вопроса
Akim Glushkov
как измеряется элегантность?

Наверное, в структурированности и читаемости

Andrzej Jankowski
Наверное, в структурированности и читаемости

Как оцениваешь текущий метод создания обьекта по этим критериям?

Конечно есть, прям in-place, а это извращение. Тогда сразу ListFactory делать, а потому ListFactoryFactory

У тебя это не репозиторий

Почему это архитектурный вопрос) Делай как проще Зачем тебе фабрика? Ты ещё собираешься где-то юзать? Где и как часто? Сколько имплементаций? Одна?) нагородить несколько классов чтобы что? Читаемость якобы? С чего повысится читаемость вместо условного метода в несколько строчек придётся прыгать по фабрике, интерфейсу и имплементации? Чем проще тем лучше. Но это сложнее сделать обычно

Andrzej-Jankowski Автор вопроса
Evg😎
Почему это архитектурный вопрос) Делай как проще ...

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

Andrzej Jankowski
Мне удобнее создавать в отдельном классе, т.к. я б...

Не спорю Вполне может быть это оправдано

Andrzej Jankowski
Мне удобнее создавать в отдельном классе, т.к. я б...

На вид казалось, что это просто фабрика для дтошки. Тогда лучше сделать интерфейс TokenGenerator<T>, где T - тип токена

Похожие вопросы

Обсуждают сегодня

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта