удалил код Егора и написал как надо

да статики еще ладно

while с break'ами куда страшнее выглядит

Че - все никак не отойдете? Раунд третий?)

?

Давайте мой код тогда пообсираем. Так я применяю EO вместе с SOLID в небольшой тулзинке: https://github.com/skapral/puzzlerbot. Мне насрать что местами процедуры и статики. Мне насрать что Егор критикует extends и RuntimeExtension. Но вся HL архитектура моего пока маленького приложения наглядно видна сразу, невооруженным взглядом, как и завещал Мартин в Clean Architecture. Просто достаточно заглянуть в core. Каждый класс я могу зареюзать где угодно без модификации. Каждый интерфейс отражает миссию и цель моего тула. Каждый класс подчиняется LSP и имеет единственную причину поменяться. Вы не найдете там ни одного класса именованного именем паттерна. И я отлично живу без DI и DTO. Я не заморачиваюсь кактусами и тейксами и юзаю что есть в джаве, но не более того что мне реально нужно.

Сыграем с заказчика и разраба? Сделайте мне любой пропозал по энхансменту и я почти сразу скажу как я его буду делать и сколько примерно это займет и почему.

Давайте мой код тогда пообсираем. Так я применяю EO вместе с SOLID в небольшой тулзинке: https://github.com/skapral/puzzlerbot. Мне насрать что местами процедуры и статики. Мне насрать что Егор критикует extends и RuntimeExtension. Но вся HL архитектура моего пока маленького приложения наглядно видна сразу, невооруженным взглядом, как и завещал Мартин в Clean Architecture. Просто достаточно заглянуть в core. Каждый класс я могу зареюзать где угодно без модификации. Каждый интерфейс отражает миссию и цель моего тула. Каждый класс подчиняется LSP и имеет единственную причину поменяться. Вы не найдете там ни одного класса именованного именем паттерна. И я отлично живу без DI и DTO. Я не заморачиваюсь кактусами и тейксами и юзаю что есть в джаве, но не более того что мне реально нужно.

вы наверно и кончаете от отражения в зеркале?

Дим, привет

добавь меня, пожалуйста, в чат с интервью

добавь меня, пожалуйста, в чат с интервью

его не существует

вы наверно и кончаете от отражения в зеркале?

Если по существу возразить нечего, то нечего и обсирать то что не понимаете.

Ну я же свой парень. на спринге пишу, все дела

Если по существу возразить нечего, то нечего и обсирать то что не понимаете.

я не обсирал ваш код

просто ваша подача - агрессивная

не я один это говорил вам

посмотрел юз-кейсы в тестах, lisp-style )))) https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/test/java/com/github/skapral/puzzler/core/text/threepars/AssertTextInstanceProducesCertainParagraphsTest.java

я не обсирал ваш код

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

если конструкторы заменить на методы-фабрики и из заимпортить через import static то вообще можно от new избавиться

тогда каждый класс будет описывать действие

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

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

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

Я никому ничего не навязываю, я аргументирую свою позицию

>Вы не найдете там ни одного класса именованного именем паттерна. увольнять сразу

?

>Вы не найдете там ни одного класса именованного именем паттерна. увольнять сразу

Да? AbstractComponentBeanFactoryProvider типа лучше? Сразу понятно его бизнес-назначение, ага

хм, тогда EO - это попытка писать на java, как на lisp или haskell (списки операций, отложенные вычисления)

Да? AbstractComponentBeanFactoryProvider типа лучше? Сразу понятно его бизнес-назначение, ага

великий класс великого фреймворка

хм, тогда EO - это попытка писать на java, как на lisp или haskell (списки операций, отложенные вычисления)

Справеливо но не совсем. LISP это LISP, Java это Java. Java все таки попопулярнее будет, хоть и незаслуженно, возможно.

public class Cp_PORT

великий класс великого фреймворка

Я не стану конечно утверждать что у меня длинных имен нет. Но мои длинные имена хотя бы с целью приложения связать можно.

что за нейминг

ЗАТО НЕ ДЛИННЫЙ

что за нейминг

Субъективная привычка. Критикуемо. Cp - ConfigProperty.

ЗАТО НЕ ДЛИННЫЙ

Про длинные имена - выше. Проблема не в том что они длинные

Субъективная привычка. Критикуемо. Cp - ConfigProperty.

зачем нижнее подчеркивание?

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

зачем нижнее подчеркивание?

Просто так. Почему нет. Хотелось вот мне чтобы имя ENV переменной выглядело как имя переменной и было отделено подчеркиванием

причем то, как внедряются компоненты - очень похоже на спринг

вот прикол

только Сергей делает это руками

его право

а тут есть те, кто прочитал книгу (книги) "Elegant objects" и после этого у него сложилось мнение, что он "втирает какую-то дичь"? (но только условие с тем, что надо прочитать книгу пред этим - обязательное)))

простите, походу это я виноват. Пришла пора вскрыть карты: на самом деле я думал как и бОльшая часть из нас по поводу ЕО, мол "@yegor256 , ты втерашешь мне какую-то дичь". Такое мнение сложилось после просмотра пары спитчей с конференций и прочтения пары статей Егора. Потом мне стало интересно, а что он там такого в книгах своих написал, и я даже прочитал... Стало уже более понятнее, но все же меня не покидала мысль, что на бумаге это звучит хорошо, a на практике - нереально так писать. Потом я переписал один из своих проектов на ЕО стиль. Теперь этот проект живет в два бранча "процедурный" и "ЕО". И вот смотрю я на это все, читаю код и думаю: а ведь Егор в чем-то прав. Нет, я не адепт, а скорее прагматик который решил проверить теорию на практике. И меня забавляет, как люди, которые не читая начинают осуждать. Наверное поэтому я и задал такой вопрос, чтобы получить мнения именно с прочитавших. Спасибо :-)

Давайте мой код тогда пообсираем. Так я применяю EO вместе с SOLID в небольшой тулзинке: https://github.com/skapral/puzzlerbot. Мне насрать что местами процедуры и статики. Мне насрать что Егор критикует extends и RuntimeExtension. Но вся HL архитектура моего пока маленького приложения наглядно видна сразу, невооруженным взглядом, как и завещал Мартин в Clean Architecture. Просто достаточно заглянуть в core. Каждый класс я могу зареюзать где угодно без модификации. Каждый интерфейс отражает миссию и цель моего тула. Каждый класс подчиняется LSP и имеет единственную причину поменяться. Вы не найдете там ни одного класса именованного именем паттерна. И я отлично живу без DI и DTO. Я не заморачиваюсь кактусами и тейксами и юзаю что есть в джаве, но не более того что мне реально нужно.

я, конечно, не эксперт, но там пара десятков джава классов по 50+ строк кода, в которых только одна строка что-то делает, а остальные - синтаксический мусор наследования, оверрайдинга методов и тд. зачем?

до сих пор собираю мнения с людей прочитавших, и если есть кому что-то добавить из тех кто прочитал - закиньте мне фитбэк в личку плиз. Не хочу здесь разжигать.

я, конечно, не эксперт, но там пара десятков джава классов по 50+ строк кода, в которых только одна строка что-то делает, а остальные - синтаксический мусор наследования, оверрайдинга методов и тд. зачем?

вот такое же мнение сложилось, пытаюсь уже 10 минут код найти

обычный бойлер

я все нашел

понятно как работает

я, конечно, не эксперт, но там пара десятков джава классов по 50+ строк кода, в которых только одна строка что-то делает, а остальные - синтаксический мусор наследования, оверрайдинга методов и тд. зачем?

Ну что есть то есть. Не я же джаву проектировал - меня то что винить что там синтаксический мусор. Идея не в этом. Идея в том что классы - это не какие то сервисы и DTOшки, которые бизнесу непонятны, а компоненты решающие определенные *бизнес* задачи.

Никогда не пользовался EO, но теперь буду. SOLID от народа.

Ну что есть то есть. Не я же джаву проектировал - меня то что винить что там синтаксический мусор. Идея не в этом. Идея в том что классы - это не какие то сервисы и DTOшки, которые бизнесу непонятны, а компоненты решающие определенные *бизнес* задачи.

сервисы решают бизнес задачи

как и дто

я все нашел

где в каком классе код

где в каком классе код

https://github.com/skapral/puzzlerbot/blob/master/puzzler-app/src/main/java/com/github/skapral/puzzler/app/Bootstrap.java

норм, это все?

сервисы решают бизнес задачи

Чьи зачади решает тот Abstractблаблабла бин? Программерские разве что - бизнес такими словами не говорит.

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/operation/AssertOperationSuccessful.java

Чьи зачади решает тот Abstractблаблабла бин? Программерские разве что - бизнес такими словами не говорит.

это же кор либа

это ваще тесты походу

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/operation/AssertOperationSuccessful.java

ну вот файл на 55 строк ради одной строки Assertions.assertThatCode(() -> { operation.execute(); }).doesNotThrowAnyException();

а вот MailService - решает задачи

и при этом все это обмазано throws Exception

ну вот файл на 55 строк ради одной строки Assertions.assertThatCode(() -> { operation.execute(); }).doesNotThrowAnyException();

самое главное - классы ппц как на спринг компоненты похоже

ERROR: S client not available

с той лишь разницей, что Сергей все руками собирает

а может Сергей просто поклонник рич модели

человек соблюдает SOLID, но не соблюдает KIS

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

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/operation/AssertOperationSuccessful.java

хм, я вижу вызов метода, надо переписать на new Start( new SrvGrizzlyWithJersey( new Cp_PORT(), new PuzzlerAPI() ) );

ну вот файл на 55 строк ради одной строки Assertions.assertThatCode(() -> { operation.execute(); }).doesNotThrowAnyException();

Не ради одной строки, а ради всех будущих имплементаций Operation которые я напишу. Этот ассершн подойдет для тестирования всех них.

Не ради одной строки, а ради всех будущих имплементаций Operation которые я напишу. Этот ассершн подойдет для тестирования всех них.

ну то есть ты дал нам посмотреть проект, в котором кода так и нет

есть бойлерплейт, который не делает ничего

элегентно, че

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/operation/OpDoNothing.java#L38

ну то есть ты дал нам посмотреть проект, в котором кода так и нет

Что значит - нет. Работающий проект - хоть щас можете развернуть

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/operation/OpDoNothing.java#L38

Это стаб для тестов

https://github.com/skapral/puzzlerbot/blob/master/puzzler-core/src/main/java/com/github/skapral/puzzler/core/operation/OpDoNothing.java#L38

круто

есть бойлерплейт, который не делает ничего

Что такое бойлерплейт?

Это стаб для тестов

.../src/main/java

.../src/main/java

И что? Это реюзабл стаб для тестов, который могут использовать компоненты вне этого джарника.

Null Object Pattern ещё его называют

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

И что? Это реюзабл стаб для тестов, который могут использовать компоненты вне этого джарника.

ну обычно все что относится к тестам лежит в директории с тестами

вопрос

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

И что? Это реюзабл стаб для тестов, который могут использовать компоненты вне этого джарника.

эээ, такое обычно увозят в отдельные подмодули, чтобы их по testCompile включать можно было

а потом вызову persistPuzzle

что будет?

ну обычно все что относится к тестам лежит в директории с тестами

Ну а я вот ставлю этот довод под вопрос

Ну а я вот ставлю этот довод под вопрос

гошника ответ

Ну а я вот ставлю этот довод под вопрос

это не под вопрос

разберитесь куда тесты складывают

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

что будет?

LSP - это не про java.util.List. Такова жизнь.

разберитесь куда тесты складывают

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