Правильно ли использовать несколько условий OR в одном операторе if, каковы альтернативы?

Question

Правильно ли использовать несколько условий OR в одном операторе if, каковы альтернативы?

1

Я хотел бы знать, какой способ более подходит и эффективен в достижении чего-то вроде: проверки наличия текста в поле

if(text1.equals("") || text2.equals("") || text3.equals("") || text4.equals("")) {
  //Do Stuff
}

Я вижу, как это быстро становится беспорядочным с более чем двумя полями, и хотелось бы знать, правильно ли это это сделать или существуют альтернативы?

Ilja 17 март 2014, в 18:07

Источник

Теги:

java

if-statement

conditional

2 ответа

2

Решение fge, безусловно, умное и сокращает строки кода. Тем не менее, я чувствую, что некоторые значения теряются при взгляде на этот фрагмент кода.

Я бы посоветовал вам использовать читаемый код, даже если он немного более подробный. Мое решение состояло бы в том, чтобы проверить эти предварительные условия перед началом метода. Если предварительное условие терпит неудачу, выбросьте исключение.

Либо напишите свои собственные методы проверки, либо используйте что-то вроде Apache Commons Lang Validate. Тогда ваш код будет выглядеть примерно так:

Validate.notEmpty(text1, "error message");
Validate.notEmpty(text2, "error message");
Validate.notEmpty(text3, "error message");
Validate.notEmpty(text4, "error message");

// This line is only reached if all fields are non-empty

Я думаю, что это легко читать, что составляет половину битвы.

Duncan Jones 17 март 2014, в 14:43

Ещё вопросы

fge · Accepted Answer · 2014-03-17T14-14-00.000Z

Вы можете сделать что-то вроде этого:

if (Arrays.asList(text1, text2, etc).contains(""))

Но лучшее решение будет зависеть от контекста.

Обратите внимание, что это немного медленнее, чем ваш текущий код. Если вы не выполняете этот тест на bazillion раз в секунду, тем не менее, это не имеет значения.

EDIT, поскольку в комментариях было сказано, что строки с пробелами также должны удовлетворять условию... Вот так:

private static final Pattern PATTERN = Pattern.compile("\\s*");

private static boolean emptyOrSpacesOnly(final String... strings)
{
    for (final String s: strings)
        if (PATTERN.matcher(s).matches())
            return true;
    return false;
}

Затем в коде:

if (emptyOrSpacesOnly(text1, text2, etc))

Чистое решение. Перед OP паникой, возможно , я укажу , что это очень маловероятно , что будет никаких проблем в связи с кодом будучи незначительно медленнее. Не хочу поощрять преждевременную оптимизацию здесь :-)
Я проверил код и смог обойти проверку, введя пробелы в полях, я знаю, что это не совсем то, что указано в моем вопросе, но есть ли способ проверить, что введенный текст был не просто пустым или пробелом?
Или используя JDK 8: Stream.of(text1,text2,text3).anyMath(text -> text.isEmpty()) .
@ Илья, ты не упомянул об этом в вопросе!
@ EdwinDalorzo Мне не нравится, как примеры Java 8 заставляют меня чувствовать, что мне нужно снова изучать язык :-D
@ EdwinDalorzo - это что- anyMath правильное или должно быть anyMatch ?
@ gla3dr Это опечатка, она должна быть anyMatch .