Последнее время делаю довольно много code review и замечаю такую вещь - пошло прямо таки какое то повальное увлечение переусложнением кода. Что это - не понимание сути и попытка впихнуть везде, где только можно свежеполученные знания или реальная необходимость, которой я никак не могу проникнуться?
Как пример:
Есть некая функция. которая что то там делает, но основная логика скрыта в вызываемой библиотечной функции. Эта библиотечная функция синхронизированна. И возвращает список. причём эта библиотечная функция в классе-синглтоне.
По логике программы эта функция должна вызываться в цикле несколько раз с разными параметрами. Программист оборачивает функцию в Future, внутри результаты работы библиотечной функции пишет в потокобезопасную коллекцию(причём над коллекцией он тоже сделал обёртку-синглтон), и все эти фьчерсы засылает в CompletableFuture.allOf. А по whenComplete выбирает данные из потокобезопасной коллекции и что то с ними делает. Вопрос - а зачем так сложно? всё равно же всё споткнётся на синхронизированном методе ( а там кроме него по сути и нет ничего) и будет выполняться фактически последовательно, без всякого парралелилизма?
Или ещё - функция возвращает строку. Эта строка передаётся в другую функцию. Другая функция этой строке что то там ищет. Всё. Зачем первая функция строку оборачивает в Option ? Не, я всё понимаю - функциональщина, защита от null и всё такое.. Но как бы любые лишние телодижения - это лишние тормоза и каждая лишняя строчка - это потенциальный источник проблем (все мы человеки - чем больше напишешь, тем больше вероятость ошибки) зачем это всё? особенно если первая функция гарантированно возвращает не null
Или ещё - есть некие действия, которые нужно делать в отдельном потоке. Данные для них приходят периодичеки. Для каждой приходящей порции данных создаётся свой поток, в нём по каким то условиям стартуют ещё потоки, там создаются какие то фьючерсы, а потом это всё долго и мучительно синхронизируется.... Но там всё что надо - это чтобы данные обрабатывались НЕ в основном потоке. И абсолютно наплевать, в одном потоке они обработаются или в 10.. После того, как все эти сложности повыкидывал - необходимость в синхронизации вообще отпала. и работать всё стало в разы стабильнее. Опять таки - что это было ? непонимание потоков автором или я опять что то не понимаю в современном стиле программирования на java ? А то я когда то на плюсах начинал - может такое отношение ко всем этим наворотам это просто издержки моего прошлого ?

Андрей Панфилов,

Да, реализацию найти тоже проблематично. Обёртка над обёрткой, а там функциональщина..
Просто ыглядит красиво, солидно, модно, стильно, молодёжно. Я бы тупо написал вызов функции в цикле с накоплением результата. А там накручено - выглядит уж больно солидно
Я писал на js - стиль может и похож, но не помню бесмысленного на мой взгляд переусложнения.
Я пытаюсь понять - оно бессмысленно только на мой взгляд или одно из двух...