Что вернуть если не null в JdbcTemplate?

Question

[Евгений Краснов]

Что лучше вернуть, как правильно, если не null?

public User getEmailByName(String name) {

        try {
            String sql = "SELECT email FROM user WHERE name=?";
            return jdbcTemplate.queryForObject(sql, new EmailByNameMapper(), name);

        } catch (EmptyResultDataAccessException e) {
            return null;
        }
    }

public class EmailByNameMapper implements RowMapper<User> {

    public User mapRow(ResultSet resultSet, int i) throws SQLException {

        User user = new User();
        user.setEmail(resultSet.getString("email"));
        return user;
    }

}

Answer 1

[Dmitry Roo]

Optional

Comments

[Евгений Краснов]

Я пытался, не получается с этим мапером

[Dmitry Roo]

Евгений Краснов, рекомендую ознакомиться с библиотекой vavr (https://www.vavr.io/).
С ней можно создавать такие элегантные конструкции:

return Try.of(() -> jdbcTemplate.queryForObject(sql, new EmailByNameMapper(), name))
                .toOption();

[Dmitry Roo]

Евгений Краснов, ну или без vavr:

public User getEmailByName(String name) {

        try {
            String sql = "SELECT email FROM user WHERE name=?";
            return Optional.of(jdbcTemplate.queryForObject(sql, new EmailByNameMapper(), name));
        } catch (EmptyResultDataAccessException e) {
            return Optional.empty();
        }
    }

[Евгений Краснов]

Ок. Посмотрю.

[Евгений Краснов]

Пока я изучаю функциональное программирование и Optional, у меня ещё есть небольшой в довесок вопрос - Как мне теперь вытащить User из Optional?

[Dmitry Roo]

Евгений Краснов, https://m.habr.com/ru/post/346782/

[Евгений Краснов]

Dmitry Roo, Интересно, спасибо огромное. Но оно не решает на данный момент моей задачи. Мне просто нужно ещё Mapper обернуть как то в этот Optional

[Евгений Краснов]

Dmitry Roo, Или я вот так делаю

User user = userServiceJdbc.EmailByNameMapper(name).get();

и оно не выходит

[Dmitry Roo]

Евгений Краснов,

Mapper обернуть как то в этот Optional

сомневаюсь, что вам это нужно.
userServiceJdbc - в своем шаре предсказаний я не вижу этой переменной и не понимаю почему вы у нее хотите вызвать класс EmailByNameMapper (sic!)

[Евгений Краснов]

Dmitry Roo, Всё, сработало. Это просто моя невнимательность. У меня последнее к теме которую вы мне дали https://m.habr.com/ru/post/346782/ Там написано - Будет получен объект User, запакованный в Optional. Такая конструкция крайне опасна, поскольку минует проверку на null и лишает смысла само использование Optional и вот конструкция которая у меня сработала

User user = userServiceJdbc.EmailByNameMapper(name).get();

и собственно сам вопрос. Это нормально так? И ещё вот то что в IntelliJ IDEA подсвечивает жёлтым цветом но не исправляет, меня это настораживает просто.

[Dmitry Roo]

Евгений Краснов, idea вам подсвечивает, что вы вызываете get(), без проверки наличия значения. Посмотрите, у optional есть несколько решений: проверить значение isPresent() или сразу предусмотреть действие, в случае отсутствия значения orElseGet(). Обо всем этом можно почитать в указанной статье, например, или в документации.

[Евгений Краснов]

Dmitry Roo, Спасибо. Ещё вернусь сюда, но надеюсь что нет. Пошёл я изучать функциональщину и опциональщину ))) Так как пришло это самое время когда подошёл момент полностью погрузится в изучение этого, иначе удачи не видать )))