Множественный оператор ||

273
14 января 2018, 04:53

Интересует, как лучше отрефакторить такой фрагмент кода :

public boolean isDeletable(ConfigDto config){
StatusCode code = configDto.getStatusCode();
return (code == ACTIVE || code == APPROVED || code == REJECTED|| code == NEW || 
   code == AWAITING_APPROVAL || code == AWAITING_CORRECTION )
}

Стоит ли ввести сет и проверять содержится ли там элемент ? (в моем случае enum)

public boolean isDeletable(ConfigDto config) {
Set <StatusCode> set = EnumSet.of(ACTIVE, 
APPROVED, REJECTED, NEW, AWAITING_APPROVAL, AWAITING_CORRECTION);
return set.contains(config.getStatusCode());
}

Какие еще могут быть варианты, чтобы код выглядил чище ?

READ ALSO
Авторизация на сайте без апи

Авторизация на сайте без апи

Подскажите, как авторизоватся без апи на каком-либо сайте имея лишь логин и пароль? Если не трудно дайте кусок кода на java, либо же линк, где...

222
Ошибка Exception in thread &ldquo;main&rdquo; java.lang.StackOverflowError

Ошибка Exception in thread “main” java.lang.StackOverflowError

При запуске программы появляется ошибка:

267