Как лучше? (все именования для примера) Имеется билдер вида Builder::main1(first: 'x', second:

Question

Как лучше? (все именования для примера) Имеется билдер вида Builder::main1(first: 'x', second:

'y')->a()->b()->build(): C, создаёт инстанс класса C с некоторыми свойствами. Встала необходимость добавить в C ещё одно обязательное свойство, например, foo.

Варианты:
— Builder::main1(first: 'x', second: 'y', fooValue: 'abc')->a()->b()->build()

— Builder::main1(first: 'x', second: 'y')->a()->b()->withFoo('abc')->build()

Кроме main1() есть ещё три аналогичных метода-конструктора (но аргументы разные), всем придётся добавить ещё один аргумент и отрефакторить вызовы (в гитлоге, да и вообще, правка строки выглядит и читается хуже, чем добавление/удаление строки).

Вроде, уже два минуса есть и второй вариант выигрывает?

#laravel #programming #russian

0

16.05.2023

22 ответов

42 просмотра

Adel

ну он же обязательный!

0

16.05.2023

Артём Автор вопроса

E V.
Визуально второй красивее, но если первый вариант ...

Второй работать не будет, потому как результат не сбилдится, аргумент-то обязательный

0

16.05.2023

Артём Автор вопроса

Adel
ну он же обязательный!

Первый вариант строже, факт. Считать это более приоритетным?

0

16.05.2023

E V.

Артём
Второй работать не будет, потому как результат не ...

Ну ->b() же будет отдавать какой то экземпляр

0

16.05.2023

Артём Автор вопроса

E V.
Ну ->b() же будет отдавать какой то экземпляр

Билдер

0

16.05.2023

Adel

Артём
Второй работать не будет, потому как результат не ...

сбилдится) но не выполнится

0

16.05.2023

Артём Автор вопроса

Adel
сбилдится) но не выполнится

Нет, упадёт, когда в конструктор C придёт нулл вместо значения для этого аргумента

0

16.05.2023

Adel

Артём
Нет, упадёт, когда в конструктор C придёт нулл вме...

да я понятиями компилируемого языка говорю)

0

16.05.2023

E V.

Артём
Билдер

Вот ) то есть до этого этапа код дойдёт) и если я на этом прекращу, то у меня будет какой то билжер болтаться в коде ошибочный с самого начала )) Вообщем я за первый варик ))

0

16.05.2023

Артём Автор вопроса

E V.
Вот ) то есть до этого этапа код дойдёт) и если я ...

Ничего болтаться не будет

0

16.05.2023

E V.

Adel
да я понятиями компилируемого языка говорю)

А я думал чем отличается компилируемый от интерпретируемого)) Неплохой пример )

0

16.05.2023

Артём Автор вопроса

Adel
да я понятиями компилируемого языка говорю)

Не везде

0

16.05.2023

E V.

Артём
Ничего болтаться не будет

Ну а кто мне запретит создать такой экземпляр билдера? Я же не знаю когда создаю объект что ещё 1 обязательный метод нужно применить ) и ваще не знаю обязательный он или нет ))

0

16.05.2023

Артём Автор вопроса

E V.
Ну а кто мне запретит создать такой экземпляр билд...

Ну создал ты его — и что?

0

16.05.2023

Adel

и вообще. что ты не как ларавельщик. где модели и контроллеры? Нафиг тебе эти билдеры!?

0

16.05.2023

E V.

Adel
и вообще. что ты не как ларавельщик. где модели и ...

А что плохого в таких билдерах?

0

16.05.2023

Adel

E V.
А что плохого в таких билдерах?

они не благословлены Тейлором!

0

16.05.2023

Артём Автор вопроса

Adel
и вообще. что ты не как ларавельщик. где модели и ...

Душа-то просит — займи и выпе.. ой, не то

0

16.05.2023

E V.

Adel
они не благословлены Тейлором!

Я только увидел что статический метод не замокать в тестах

0

16.05.2023

Adel

E V.
Я только увидел что статический метод не замокать ...

билдеры не мокают

0

16.05.2023

Артём Автор вопроса

E V.
Я только увидел что статический метод не замокать ...

А его не надо мокать

0

16.05.2023

E V. · Accepted Answer

E V.

Визуально второй красивее, но если первый вариант явно упадёт без нового аргумента, то второй спокойно будет работать и его хер отдебажишь и найдёшь проблему) Я всегда за явное падение кода в нестандартный ситуации

0

16.05.2023

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

Как лучше? (все именования для примера) Имеется билдер вида Builder::main1(first: 'x', second:

22 ответов

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