Форум программистов, компьютерный форум, киберфорум
Java EE (J2EE)
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.50/4: Рейтинг темы: голосов - 4, средняя оценка - 4.50
1 / 1 / 4
Регистрация: 05.05.2017
Сообщений: 521

Optional orElse некорректно работает

01.04.2020, 17:00. Показов 872. Ответов 10
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
контроллер получает json с помощью jackson замапленный в Section. Пытаюсь сохранить его в сервисе. Section имеет связанную таблицу GeologicalClasses. Соответственно я сначала проверяю по code(он уникален) есть ли уже такая запись в бд и если нет, то создает ее. Сделал с помощью JpaRepository и Optional. Почему то даже если findByCodeEquals находит запись в бд, то orElse все равно вызывается и выбрасывает exception о нарушении уникальности поля code. "a" ввел специально для проверки возвращает ли репозиторий значение.
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    @Transactional
    public Optional<Section> save(Section section){
        Optional<Section> new_section = Optional.of(repository.saveAndFlush(section));
        if(new_section.isPresent()) {
            section.getGeologicalClasses().forEach(g -> {
                g.setSection(new_section.get());
                Optional<GeologicalClass> a = geologicalClassRepository
                        .findByCodeEquals(g.getCode());
                geologicalClassRepository
                        .findByCodeEquals(g.getCode())
                        .orElse(geologicalClassRepository.saveAndFlush(g));
            });
        }
        return new_section;
    }
Java
1
2
3
4
@Repository
public interface GeologicalClassRepository extends JpaRepository<GeologicalClass, Long> {
    Optional<GeologicalClass> findByCodeEquals(String code);
}
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
@Entity
public class GeologicalClass {
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long id;
 
    private String name;
 
    @Column(unique = true)
    private String code;
 
    @JsonIgnore
    @ManyToOne
    @JoinColumn(name="section_id", nullable=false)
    private Section section;
 
    public Long getId() {
        return id;
    }
 
    public void setId(Long id) {
        this.id = id;
    }
 
    public String getName() {
        return name;
    }
 
    public void setName(String name) {
        this.name = name;
    }
 
    public String getCode() {
        return code;
    }
 
    public void setCode(String code) {
        this.code = code;
    }
 
    public Section getSection() {
        return section;
    }
 
    public void setSection(Section section) {
        this.section = section;
    }
}
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
01.04.2020, 17:00
Ответы с готовыми решениями:

Stream API - FindFirst orElse работает не верно
Здравствуйте, не могу понять, почему stream работает не так, как задумано. Это я чего-то не понимаю, или действительно проблема в API? Я...

PUP.Optional.22ChromeEXT, PUP.Optional.Banggood, PUP.Optional.Legacy, PUP.Optional.MySearch
Случилась такая беда: В Хроме автоматически грузятся левые страницы вместо уже открытых (одна из...

В Google Chrome завелись PUP.Optional.22ChromeEXT, PUP.Optional.Legacy,PUP.Optional.Mail.Ru
Рабочий комп достался с такими проблемами : PUP.Optional.22ChromeEXT, PUP.Optional.Legacy,PUP.Optional.Mail.Ru ( 3 вируса) При запуске...

10
Эксперт функциональных языков программированияЭксперт Java
 Аватар для korvin_
4576 / 2775 / 491
Регистрация: 28.04.2012
Сообщений: 8,780
01.04.2020, 17:43
Цитата Сообщение от Pantheonptz Посмотреть сообщение
Почему то даже если findByCodeEquals находит запись в бд, то orElse все равно вызывается и выбрасывает exception о нарушении уникальности поля code.
Во-первых, Optional.orElse ничего не знает ни о каких БД и никакого такого exception'а не может выбрасывать в принципе. Читай стэктрейс внимательней.
Во-вторых, подобное исключение возникает, когда в БД больше чем одна запись с искомым значением, а ты пытаешься найти одну и только одну с помощью find.
0
1 / 1 / 4
Регистрация: 05.05.2017
Сообщений: 521
01.04.2020, 17:56  [ТС]
Метод сгенерированный JpaRepository возвращает Optional, значение не пустое, но orElse все равно выполнятся. Поле уникальное, поэтому запись найдется только одна. Исключение вызывает метод вызываемый orElse, которое не должно вызываться по описанной причине.
0
Эксперт Java
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
01.04.2020, 18:56
Цитата Сообщение от Pantheonptz Посмотреть сообщение
которое не должно вызываться по описанной причине.
с чего ты это взял?

Добавлено через 1 минуту
inb4 - сравни с orElseGet
0
1 / 1 / 4
Регистрация: 05.05.2017
Сообщений: 521
01.04.2020, 20:06  [ТС]
Если метод возвращает Optional у которого value не пустое разве orElse должно срабатывать?
0
Эксперт Java
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
01.04.2020, 20:19
так ты сравнил?
0
1 / 1 / 4
Регистрация: 05.05.2017
Сообщений: 521
01.04.2020, 20:53  [ТС]
Цитата Сообщение от Pantheonptz Посмотреть сообщение
geologicalClassRepository
                        .findByCodeEquals(g.getCode())
                        .orElse(geologicalClassRepository.saveAn dFlush(g));
я думал, что в подобной конструкции orElse вызовет метод только если findByCodeEquals вернет Optional c value = null
0
 Аватар для Goongala
1022 / 562 / 185
Регистрация: 18.08.2013
Сообщений: 2,027
Записей в блоге: 2
01.04.2020, 21:07
Цитата Сообщение от Pantheonptz Посмотреть сообщение
orElse вызовет метод
orElse ничего не вызывает — он получает значение. Значение, возвращаемое geologicalClassRepository.saveAndFlush(g), вычисляется перед вызовом orElse. Тебе непрозрачно намекнули, что нужно посмотреть в сторону метода orElseGet, который получает в качестве параметра не значение, а функцию, это самое значение вычисляющую. В такой конструкции значение будет вычисляться только тогда, когда Optional содержит в себе null
0
1 / 1 / 4
Регистрация: 05.05.2017
Сообщений: 521
02.04.2020, 01:09  [ТС]
В данном случае мне не нужно вычислять значение. мне нужно создать запись, если её нет в бд. т.е. я хотел
Java
1
.findByCodeEquals(g.getCode())
- найти запись по коду и если её нет то создать
Java
1
.orElse(geologicalClassRepository.saveAndFlush(g));
0
02.04.2020, 08:22

Не по теме:

да уж, жоска

0
Эксперт функциональных языков программированияЭксперт Java
 Аватар для korvin_
4576 / 2775 / 491
Регистрация: 28.04.2012
Сообщений: 8,780
02.04.2020, 09:41
Цитата Сообщение от Pantheonptz Посмотреть сообщение
мне нужно создать запись, если её нет в бд.
Зачем ты пытаешь делать это в коде, если в БД уже есть constraint на уникальность поля code? Просто перехватывай исключение.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
02.04.2020, 09:41
Помогаю со студенческими работами здесь

PUP.Optional.Legacy, Adware.RuKometa, PUP.Optional.FakeYandex, PUP.Optional.RussAd
Доброго времени суток! Прошу помощи в удалении вирусов из заголовка. adwcleaner_7.0.8.0 определяет, но не удаляет. Запустить...

PUP.Optional.Legacy, Adware.RuKometa, PUP.Optional.FakeYandex, PUP.Optional.RussAd
Доброго времени суток! Прошу помощи в удалении вирусов из заголовка. adwcleaner_7.0.8.0 определяет, но не удаляет. Запустить...

Самопальный optional, темплейты, попытка в реализацию fmap для optional
Вкратце, есть у меня мой собственный optional. То, что это optional, не суть важно, дело не в нем, все упоминания о нем можно спокойно...

OrElse вместо Or
в .NET можно вообще забыть за OR и всегда юзать OrElse это продуктивней ведь если первое условие совпадет, то все остальные вообще...

Объясните как работает &optional на примере
Объясните пожалуйста как работает здесь опшионал: у него здесь при каждом рекурсивном обращении в f присваивается хвост списка?...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
11
Ответ Создать тему
Новые блоги и статьи
Хочу заставить корпорации вкладываться в здоровье сотрудников: делаю мат модель здравосохранения
anaschu 22.03.2026
e7EYtONaj8Y Z4Tv2zpXVVo https:/ / github. com/ shumilovas/ med2. git
1С: Программный отбор элементов справочника по группе
Maks 22.03.2026
Установка программного отбора элементов справочника "Номенклатура" из модуля формы документа. В качестве фильтра для отбора справочника служит группа номенклатуры. Отбор по наименованию группы. . .
Как я обхитрил таблицу Word
Alexander-7 21.03.2026
Когда мигает курсор у внешнего края таблицы, и нам надо перейти на новую строку, а при нажатии Enter создается новый ряд таблицы с ячейками, то мы вместо нервных нажатий Энтеров мы пишем любые буквы. . .
Krabik - рыболовный бот для WoW 3.3.5a
AmbA 21.03.2026
без регистрации и смс. Это не торговля, приложение не содержит рекламы. Выполняет свою непосредственную задачу - автоматизацию рыбалки в WoW - и ничего более. Однако если админы будут против -. . .
1С: Программный отбор элементов справочника по значению перечисления
Maks 21.03.2026
Установка программного отбора элементов справочника "Сотрудники" из модуля формы документа. В качестве фильтра для отбора служит значение перечислений. / / Событие "НачалоВыбора" реквизита на форме. . .
Переходник USB-CAN-GPIO
Eddy_Em 20.03.2026
Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .
Оттенки серого
Argus19 18.03.2026
Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .
SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++
8Observer8 17.03.2026
Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru