Предоставленный фрагмент кода демонстрирует неправильное использование типов Optional, даже для частого отправителя. Он проверяет, не является ли строка пустой, а затем оборачивает не-null dto в Optional.ofNullable. Эта практика сводит на нет предполагаемую выгоду от Optional, которая заключается в корректной обработке потенциально null значений. По сути, это позволяет избежать использования удобного синтаксического сахара, который Optional предоставляет для упаковки типов. Хотя сама по себе эта практика не является проблематичной, она широко распространена по всей кодовой базе. Автор отмечает, что типы Optional разбросаны повсеместно, даже в функциях, которые не возвращают nullable типы. Такое широкое и некорректное применение Optional не улучшает качество кода и не уменьшает количество ошибок. Вместо этого оно способствует появлению ошибочного и подверженного ошибкам кода. Автор выражает скептицизм относительно вероятности того, что эта проблема когда-либо будет исправлена. Это подчеркивает распространенную проблему, когда мощные языковые средства неправильно понимаются и применяются. Чрезмерная упаковка и распаковка добавляет ненужную сложность. В конечном итоге это сводит на нет цель использования Optional для предотвращения исключений нулевых ссылок. Этот пример служит предостережением о неправильном внедрении современных конструкций программирования.