Форум программистов, компьютерный форум, киберфорум
Java
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/3: Рейтинг темы: голосов - 3, средняя оценка - 4.67
7 / 0 / 0
Регистрация: 18.05.2018
Сообщений: 158

Как отрефакторить этот код с проверкой на null?

19.11.2023, 17:22. Показов 1201. Ответов 1
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Всем привет! На работе дали задачу с рефакторингом такого кода:
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
49
50
51
52
53
54
55
56
57
Predicate predicate = builder.conjunction();
 
        if (paymentFilter.getCounterpartName() != null)
            predicate = builder.and(predicate,
                    builder.like(builder.lower(root.get(COUNTERPART_NAME)),
                            PayOrderServiceConst.PERCENT + paymentFilter.getCounterpartName().toLowerCase() + PayOrderServiceConst.PERCENT));
 
        if (paymentFilter.getCounterpartInn() != null)
            predicate = builder.and(predicate,
                    builder.like(root.get(COUNTERPART_INN),
                            PayOrderServiceConst.PERCENT + paymentFilter.getCounterpartInn() + PayOrderServiceConst.PERCENT));
 
        if (paymentFilter.getDescription() != null)
            predicate = builder.and(predicate,
                    builder.like(builder.lower(root.get(DESCRIPTION)),
                            PayOrderServiceConst.PERCENT + paymentFilter.getDescription().toLowerCase() + PayOrderServiceConst.PERCENT));
 
        if (paymentFilter.getSourceAccountNumber() != null)
            predicate = builder.and(predicate,
                    builder.like(root.get(ACCOUNT).get(NUMBER),
                            PayOrderServiceConst.PERCENT + paymentFilter.getSourceAccountNumber() + PayOrderServiceConst.PERCENT));
 
        if (paymentFilter.getNumber() != null)
            predicate = builder.and(predicate,
                    builder.like(root.get(NUMBER).as(String.class),
                            PayOrderServiceConst.PERCENT + paymentFilter.getNumber() + PayOrderServiceConst.PERCENT));
 
        if (paymentFilter.getClientId() != null)
            predicate = builder.and(predicate,
                    builder.equal(root.get(ACCOUNT).get(BANK_CLIENT).get(CLIENT).get(ID).as(String.class),
                            paymentFilter.getClientId()));
 
        if (paymentFilter.getClientInn() != null)
            predicate = builder.and(predicate,
                    builder.like(root.get(ACCOUNT).get(BANK_CLIENT).get(CLIENT).get(INN).as(String.class),
                            PayOrderServiceConst.PERCENT + paymentFilter.getClientInn() + PayOrderServiceConst.PERCENT));
 
        if (paymentFilter.getSourceAccountId() != null)
            predicate = builder.and(predicate,
                    builder.equal(root.get(ACCOUNT).get(ID).as(String.class),
                            paymentFilter.getSourceAccountId()));
 
        if (paymentFilter.getStartDate() != null)
            predicate = builder.and(predicate,
                    builder.greaterThanOrEqualTo(root.get(CREATED_DATE),
                            LocalDateTime.ofInstant(paymentFilter.getStartDate(), ZoneOffset.UTC)));
 
        if (paymentFilter.getEndDate() != null) {
            LocalDateTime endDate = LocalDateTime.ofInstant(paymentFilter.getEndDate(), ZoneOffset.UTC);
            endDate = endDate.plusDays(1).toLocalDate().atStartOfDay();
            predicate = builder.and(
                    predicate,
                    builder.lessThan(
                            root.get(CREATED_DATE),
                            endDate
                    )
            );
Погуглив в инете не нашел ничего похожего с моей проблемой. Можете подсказать в каком напрвлении двигаться для рефакторинга такого кода?
Заранее спасибо!
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
19.11.2023, 17:22
Ответы с готовыми решениями:

SQL запрос с проверкой на null
Есть база Access в ней форма с 4-мя выпадающими списками. При выборе любого значения из любого списка + выбора дополнительных значений из...

Запрос к базе данных с проверкой на NULL
Здравствуйте, подскажите пожалуйста с запросом имею базу данных USB с таблицами ID, FID, VID, PID NAME поле NAME имеет значение NULL ...

Построить запрос с проверкой поля на NULL
-> If Advisor ID is set, link to Advisor Profile. If Advisor ID is set to NULL - check . if . = . then the Value should be a link...

1
 Аватар для Coffeini
753 / 370 / 133
Регистрация: 01.02.2020
Сообщений: 1,096
Записей в блоге: 1
19.11.2023, 22:32
Что-то типо того, наверное:
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
public static Predicate merge(Builder builder, Optional<Predicate>... predicates) {
        Predicate predicate = builder.conjunction();
        for (var optPredicate : predicates) {
            final var finalPredicate = predicate;
            predicate = optPredicate
                    .map(p -> builder.and(finalPredicate, p))
                    .orElse(finalPredicate);
        }
        return predicate;
    }
 
    public static <T, U> Optional<U> toOpt(T t, Function<T, U> function) {
        return Optional.ofNullable(t).map(function);
    }
 
    public static void main(String[] args) {
        final Predicate predicate = merge(builder,
                toOpt(paymentFilter.getCounterpartName(), counterpartName ->
                        builder.like(builder.lower(root.get(COUNTERPART_NAME)),
                                PayOrderServiceConst.PERCENT + counterpartName.toLowerCase() + PayOrderServiceConst.PERCENT)),
                toOpt(paymentFilter.getCounterpartInn(), counterpartInn ->
                        builder.like(root.get(COUNTERPART_INN),
                                PayOrderServiceConst.PERCENT + counterpartInn + PayOrderServiceConst.PERCENT)),
                toOpt(paymentFilter.getDescription(), description ->
                                builder.lower(root.get(DESCRIPTION)),
                        PayOrderServiceConst.PERCENT + description.toLowerCase() + PayOrderServiceConst.PERCENT)),
        ...
                );
    }
Хотя в идеале это все в билдер завернуть, но я не знаю его контракта.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
19.11.2023, 22:32
Помогаю со студенческими работами здесь

Соеденение двух таблиц с результирующим полем с проверкой на Null
Нужно получить расчетное поле из двух таблиц в запросе, при левом соединении. фактически заменить отсуствующие значения ВЫБОР ...

Сравнение скорости вызова делегата без проверки на null и с проверкой
Рекомендуется всегда использовать вызов делегата с проверкой на null del?.Invoke(); (del - экземпляр делегата) Допустим, нужно сделать...

Не работает код: m_web1.Navigate ('gidro.htm',0,0,NULL,NULL);
Друзья помогите начинаюшему значит делаю m_web1.Navigate ('с:\gidro.htm',0,0,NULL,NULL);работает делаю m_web1.Navigate...

Не выполняется код после вызова функции ClientSocket = accept(ListenSocket, NULL, NULL);
Это стандартный пример сокетов для windows. Но суть вопроса заключается не в win API - вообщем после функции accept(ListenSocket, NULL,...

Очередной код с возможным вирусом. Как думаете этот код опасен?
&lt;?php // no direct access defined('_JEXEC') or die('Restricted access'); ?&gt; &lt;ul class=&quot;sections&lt;?php echo...


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

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
Новые блоги и статьи
Access
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
Фото: Daniel Greenwood
kumehtar 13.11.2025
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru