DoNothingOperation - это не тест. И AssertOperationSuccessful - еще не тест. Что вас вводит в диссонанс?

отсутствие пометки - что это стаба под тест

пакетик бы завели .test для стаб

DoNothingOperation - это не тест. И AssertOperationSuccessful - еще не тест. Что вас вводит в диссонанс?

то что есть проект в котором под 50 классов который при этом ничего не елает, я такое же могу собрать в одном файле на строчек 100-200

отсутствие пометки - что это стаба под тест

Ну мне не критично пока. Хотя да, признаю. доля смысла в этом есть

Ну мне не критично пока. Хотя да, признаю. доля смысла в этом есть

выше на вопрос про лист ответьте

у вас же классы ниибаца реюзабельные

оно еще и читаемее будет

выше на вопрос про лист ответьте

Я избегаю там использовать джавовые листы под композицию объектов. Композиция обязана оставаться иммутабельной. Вместо этого юзаю vavr

Я избегаю там использовать джавовые листы под композицию объектов. Композиция обязана оставаться иммутабельной. Вместо этого юзаю vavr

у вас эксепшен упадет

вы понимаете?

Я избегаю там использовать джавовые листы под композицию объектов. Композиция обязана оставаться иммутабельной. Вместо этого юзаю vavr

зочем?

вы понимаете?

Щас, перечитаю, погодьте. Чето я вас не понимаю

интерфейс позволяет передавать туда asList, singletonList, emptyList

я делаю add и получаю UnsupportedOperationException

если я передам Arrays.asList в конструктор

Давайте так - в чей конструктор?

Давайте так - в чей конструктор?

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/itracker/ItToList.java#L40

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/itracker/ItToList.java#L40

О! А это кстати косяк...

Там должен был быть вавровый лист

ну или просто мутабельный лист задайте у поля

если предполагается его мутация через метод персист паззл

в джаве иммутабельность на эксепшенах

?

ну или просто мутабельный лист задайте у поля

Это опасно. Композиция должна оставаться немутабельной, иначе все напрасно.

Это опасно. Композиция должна оставаться немутабельной, иначе все напрасно.

вооот

что мы видим - вы заложник книг

или чьих то методов/доводов

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

>я делаю add и получаю UnsupportedOperationException

>я делаю add и получаю UnsupportedOperationException

не делай add

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

Ну вы и со спрингом в ногу се выстрелить можете. Волков бояться - на джаве не кодить.

все-таки в реальный мир неидеальной джавы надо иногда возвращаться и соломки подкладывать?

не делай add

может мне еще базу снести и удалить код из мастера>

я ведь просто инициализирую список и передаю его в конструктор, а дальше по коду еще что-то добавляю

ведь у класса есть такой метод

все-таки в реальный мир неидеальной джавы надо иногда возвращаться и соломки подкладывать?

А я и не стремлюсь сделать все по элегантному. Я считаю это ошибкой Егора - что он лоулевел тулы перепиисывает.

Я использую элегантность только там, где бизнес и цели

А внизу у меня все равно Jersey

И кстати мог бы быть даже spring mvc

Сергей, вы согласны, что делать в контракте метод добавить паззл и дать возможность передавать иммутабельные коллекции джавовые - несколько опрометчиво?

Сергей, вы согласны, что делать в контракте метод добавить паззл и дать возможность передавать иммутабельные коллекции джавовые - несколько опрометчиво?

Так. Я думаю надо кое что прояснить про persistPuzzle

А! Блин, я все попутал! Правильно там мутабельный лист стоит.

Заболтался уже

Щас

Я его дропнуть просто забыл, он левый

ItToList всмысле

Тут же JVM чятик? Что за lazy инициализация в Kotlin?

Тьфу, lazy property

Тут же JVM чятик? Что за lazy инициализация в Kotlin?

по запросу через делегат

инстансится

по запросу через делегат

Ага, спасибо. Извиняюсь за тупой вопрос.

Сергей, вы согласны, что делать в контракте метод добавить паззл и дать возможность передавать иммутабельные коллекции джавовые - несколько опрометчиво?

Просто контрактом IssueTracker предполагалось что persistPuzzle будет создавать паззлы в средах где их создание возможно (в ишшью трекерах в основном). И я хотел создать тоже такой стабик который паззлы просто будет складывать в память. А потом понял что это не очень удачная идея и отказался от нее, а про класс забыл. Так что да, косяк.

Просто контрактом IssueTracker предполагалось что persistPuzzle будет создавать паззлы в средах где их создание возможно (в ишшью трекерах в основном). И я хотел создать тоже такой стабик который паззлы просто будет складывать в память. А потом понял что это не очень удачная идея и отказался от нее, а про класс забыл. Так что да, косяк.

ну в целом код аккуратный и чистый

ну в целом код аккуратный и чистый

Ну хоть теперь нормально поговорили. А то срались только.

по запросу через делегат

Внезапно, не совсем: инициализация поля откладывается до первого использования объекта

ну да

Ну хоть теперь нормально поговорили. А то срались только.

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

а я чего сказал

что под капотом "первого использования" ?)

а я чего сказал

По запросу через делегат. Я распарсил это как: при обращении к объекту...

Блджад

)))

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

Справедливости ради, подход работает в одном и только одном случае - если в команде есть человек, знающий предметную область приложения, цели и миссию, и обладающий полномочиями как минимум архитектора.

Если понимания нет, то все будет как всегда

+ код ревью

значит знание надо передавать другим

Справедливости ради, подход работает в одном и только одном случае - если в команде есть человек, знающий предметную область приложения, цели и миссию, и обладающий полномочиями как минимум архитектора.

я бы сказал компетенциями, а не полномочиями

ERROR: S client not available

я бы сказал компетенциями, а не полномочиями

И компетенциями, и полномочиями

Я видал архитекторов с компетенциями но без полномочий

Они больше на маркетологов смахивают

а я наоборот видел - полномочия есть, компетенций ноль

а я наоборот видел - полномочия есть, компетенций ноль

Ну тут все понятно)

да)

а я наоборот видел - полномочия есть, компетенций ноль

боль...

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

Вместо этого гоняем ДТО по слоям сервисов)

и ведь не докажешь ничего таким

Вместо этого гоняем ДТО по слоям сервисов)

спринг рекомендует анемичную модель

и ведь не докажешь ничего таким

ты абсолютно прав

Вместо этого гоняем ДТО по слоям сервисов)

Все, реально, давайте пока на этом и остановимся. Свои доводы защитили, срач теряет смысл.

я понял Сергея и его доводы, мне кажется ( ну по крайней мере по тому, что я от него слышал) ему стоит делать упор на чистые java core проекты

а может даже и свое что-то опенсорсить

угу, потому что в аутсорсе не поймут

ну и мешать будут

+

те вот какой-то продуктовый отдел компании

идеально

Или менять язык

тоже неплохая идея

Или менять язык

Ну нее. А вместе с языком учить заново всю нужную экосистему? Похерив весь многолетний стаж по джаве? Ну нафиг

Ну нее. А вместе с языком учить заново всю нужную экосистему? Похерив весь многолетний стаж по джаве? Ну нафиг

скала или котлин

ваш опыт пригодится там

особенно в котлине я думаю

скала или котлин

Может быть. Котлин заинтересовал впринципе.

Можно ещё вырасти в архитектора и навязывать свои правила. Хз правда насколько жизнеспособно это будет)

Можно ещё вырасти в архитектора и навязывать свои правила. Хз правда насколько жизнеспособно это будет)

Я не верю в это. Насмотрелся на других архитекторов

Это лет десять назад архитектор был архитектором а я был джуном и хотел быть архитектором

? можно стать архитектором, писать на сложуре, а всем пропаривать что это плагин такой к спрингу

Это лет десять назад архитектор был архитектором а я был джуном и хотел быть архитектором

А щас я не-джун, вижу архитекторов (или не вижу), и уже не хочу им быть(

Мне с ними везло на старте карьеры. Очень много чего дали.