тестовое задание для новичков

@Alexandra Shpa · Регистрация: 02.05.2023

Студворк — интернет-сервис помощи студентам

Что не так с этим кодом (предположим, что все используемые методы и классы реализованы)?

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
public int getCustomerType(int customerId) {
  Connection con = getDatabaseConnection();
  Customer customer = Utils.getCustomerById(con, customerId);
  con.close();
  return customer.getType();
}    
  
public Color getCustomerColor(int customerId) {
  if (getCustomerType(customerId)==Customer.VIP) 
    return Color.RED;
  else
  if (getCustomerType(customerId)==Customer.BLOCKED) 
    return Color.GREY;
  else
    return Color.WHITE;
}

спасибо за помощь

@IOException · 05.05.2023, 17:38

Предположим клиента с неким id не существует в базе. В таком случае Utils.getCustomerById вероятней всего кинет исключение, которое не обрабатывается. В таком случае программа упадет, при чем connection не будет закрыта. Возможно также что Utils.getCustomerById вернет null (но это менее вероятно, зависит от реализации). Тогда при выполнении customer.getType() будет вызвано исключение NullPointerException и вся программа опять же упадет.
Думаю задача об отсутствии обработки вероятных исключений.

@Arsegg · 05.05.2023, 19:28

Alexandra Shpa,
1) Утечка ресурсов: что если в методе getCustomerType произойдет ошибка при создании коннекта к БД и/или вызове метода getCustomerById?
2) Множество запросов к БД: getCustomerColor, в худшем случае, вызовет 2 раза getCustomerType. Коннекты к БД не бесконечный ресурс, да и стучаться по одному запросу к БД такое себе. Мб, какая-нибудь блокирующая очередь ок тут (эксперты поправьте, плз).

Новые блоги и статьи Все статьи Все блоги /
Установка Qt Creator для C и C++: ставим среду, CMake и MinGW без фреймворка Qt 8Observer8 05.04.2026 Среду разработки Qt Creator можно установить без фреймворка Qt. Есть отдельный репозиторий для этой среды: https:/ / github. com/ qt-creator/ qt-creator, где можно скачать установщик, на вкладке Releases:. . .	AkelPad-скрипты, структуры, и немного лирики.. testuser2 05.04.2026 Такая программа, как AkelPad существует уже давно, и также давно существуют скрипты под нее. Тем не менее, прога живет, периодически что-то не спеша дополняется, улучшается. Что меня в первую очередь. . .	Отображение реквизитов в документе по условию и контроль их заполнения Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .	Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .
Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизитов табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: 1. Реализовать контроль заполнения реквизита. . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .

@IOException 98 / 75 / 25 Регистрация: 17.03.2023 Сообщений: 161
	05.05.2023, 17:38
	Предположим клиента с неким id не существует в базе. В таком случае Utils.getCustomerById вероятней всего кинет исключение, которое не обрабатывается. В таком случае программа упадет, при чем connection не будет закрыта. Возможно также что Utils.getCustomerById вернет null (но это менее вероятно, зависит от реализации). Тогда при выполнении customer.getType() будет вызвано исключение NullPointerException и вся программа опять же упадет. Думаю задача об отсутствии обработки вероятных исключений. 1

@Arsegg 3582 / 2182 / 571 Регистрация: 02.09.2015 Сообщений: 5,510
	05.05.2023, 19:28
	Сообщение было отмечено Alexandra Shpa как решение Решение Alexandra Shpa, 1) Утечка ресурсов: что если в методе getCustomerType произойдет ошибка при создании коннекта к БД и/или вызове метода getCustomerById? 2) Множество запросов к БД: getCustomerColor, в худшем случае, вызовет 2 раза getCustomerType. Коннекты к БД не бесконечный ресурс, да и стучаться по одному запросу к БД такое себе. Мб, какая-нибудь блокирующая очередь ок тут (эксперты поправьте, плз). 2

тестовое задание для новичков

Решение