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

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

19.11.2023, 17:22. Показов 1253. Ответов 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
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D
8Observer8 04.03.2026
Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий
8Observer8 02.03.2026
Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&amp;d=1772460536 Одним из. . .
Реалии
Hrethgir 01.03.2026
Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru