ifNull точно лишнее
Для деталей: /** * Такое */
чтобы было читабельнее там где используется, не пойдет?
Вообще, да. Но имхо у тебя какое-то несоответствие с названием. create подразумевает, что метод что-то вернёт, а у тебя он Unit Served имхо лишнее, потому что ты просто хочешь создать готовый объект, неважно каким образом Served/NotServed лучше разделить по типам. Либо наследование, либо sealed, либо маркерные интерфейсы IfNull - ну такое, если самому надо, то пиши
спасибо за развернутый фидбек
не, просто напиши createServedPlatformEntity() а в доке опиши в каком случае вызываешь
Пытайся избегать, чтобы методы влияли на объект внутри себя. Лучше что-то вернуть и присвоить снаружи, иначе получается нечитаемая дельфи каша heatWater() putOnion() putBeet() waitForReadiness()
Ну полную иммутабельность пропагандировать тоже не надо
++++
у меня через репозитории, где всё дерьмо делаю там
Я пропагандирую очевидность
В его случае, очевидно пропагандировать чистые функции
Обсуждают сегодня