Я работаю над составлением жалобы на кодовую базу со списком правил PMD. К ним относятся, что вам не разрешено бросать или улавливать NullPointerException
s. Я нашел этот метод, для которого для запуска требуется 2 экземпляра переменных. В противном случае это NullPointerException
:
public String build() {
if (apiKey == null || url == null) {
throw new NullPointerException(
"Cannot build: API key and URL must be set");
}
Что он должен бросать? мне кажется, что NullPointerException
будет вызовом bast. Единственное, что приходит на ум, это IllegalArgumentException
, но это не аргументы, поэтому я не думаю, что это действительно подходит.
IllegalStateException
приходит на ум.
Здесь следует рассмотреть вопрос о том, должно ли это быть проверенное или неконтролируемое исключение. Вы также можете подумать о том, чтобы сделать свой собственный явный Исключение для этого случая, если его можно с пользой использовать иначе, чем общее IllegalStateException
.
IllegalStateException
выглядит вполне уместным:
Сигналы о том, что метод был вызван в незаконное или ненадлежащее время.
Это достаточно ясно описывает ситуацию, не так ли?