Можно ли делать такое громоздкое условие в if?

Можно ссылочку на соответствующее постановление ВЦСПС?

на код ревью расскажут, если проект с налаженными процессами и команда с экспертизой

Вот задача https://javarush.com/tasks/com.javarush.task.task09.task0930, где мы сортируем по возрастанию и убыванию ввёденые строки и "числа" (тоже строки). Нужно было написать метод сортировки. Написала вот такой:

public static void sort(String[] array) {
        for (int i = 0; i < array.length - 1; i++) {
            for (int j = i + 1; j < array.length; j++) {
                if (!isNumber(array[i]) && !isNumber(array[j]) && isGreaterThan(array[i], array[j]) ||
                        isNumber(array[i]) && isNumber(array[j]) && Integer.parseInt(array[i]) < Integer.parseInt(array[j])) {
                        String temp = array[j];
                        array[j] = array[i];
                        array[i] = temp;
                }
            }
        }
    }

Метод isNumber(String s) определяет, является ли строка числом. У JR решение такое (четыре if-а):

public static void sort(String[] array) {
        for (int i = 0; i < array.length; i++) {
            for (int j = i + 1; j < array.length; j++) {
                if (isNumber(array[i]) && isNumber(array[j])) {
                    if (Integer.parseInt(array[i]) < Integer.parseInt(array[j])) {
                        String temp = array[i];
                        array[i] = array[j];
                        array[j] = temp;
                    }
                } else if (!isNumber(array[i]) && !isNumber(array[j])) {
                    if (isGreaterThan(array[i], array[j])) {
                        String temp = array[i];
                        array[i] = array[j];
                        array[j] = temp;
                    }
                }
            }
        }
    }

Ну вот теперь посмотри и реши сама, устраивает тебя внешний вид или не устраивает, можешь ли ты прочитать свой код и сходу понять, что там происходит, или для понимания, придется при прочтении делать в голове паузы и раскладывать по воображаемым полочкам? Если устраивает, можно оставить как есть и спокойно пойти решать дальше. Если возник вопрос, то вероятно, не устраивает и хочется сделать рефакторинг. В идеале каждый метод должен делать одно действие, отраженное в его названии. Метод сорт должен сортировать элементы. Если разбить сортировку на более мелкие действия - перебор элементов, сравнение и перемещение, то становится понятно, как произвести рефакторинг. Выносим каждое действие в отдельный метод:

public static void sort(String[] array) {
    for (int i = 0; i < array.length - 1; i++) {
        for (int j = i + 1; j < array.length; j++) {
            if (shouldSwap(array[i], array[j])) {
                swapElements(array, i, j);
            }
        }
    }
}

private static boolean shouldSwap(String first, String second) {
    if (isNumber(first) && isNumber(second))
        return Integer.parseInt(first) < Integer.parseInt(second);
    return !isNumber(first) && !isNumber(second) && isGreaterThan(first, second);
}

private static void swapElements(String[] array, int i, int j) {
    String temp = array[j];
    array[j] = array[i];
    array[i] = temp;
}

С одной стороны код стал наоборот, длиннее, но во время рефакторинга мы преследуем не цель сокращения числа строк, а цель сокращения времени, которое тратится на понимание кода. При прочтении метода sort, мы понимаем, что у нас попарно перебираются элементы, и если их нужно поменять местами (if should swap), мы их меняем местами (swap elements). А дальше уже только при необходимости можно углубиться в чтение и начать разбираться, при каком же условии они меняются местами и каким образом.

Конечно, выносить каждую строку кода в отдельный метод метод не следует, но рекомендуется соблюдать максимальную длину метода, не более 10-20-30-строк, одного экрана, где как принято. И не более одной логической операции в методе. Логической не в смысле boolean, а смысле того, какую функцию несет в себе этот код. Кстати, idea содержит множество удобных инструментов для рефакторинга. К примеру, можно выделить завершенный кусок кода, например все твое условие в ифе, нажать Ctrl+Alt+M (Method), и она предложит извлечь тебе выделенный код в отдельный метод, сохраняя логику. Аналогично можно объявлять переменные - Ctrl+Alt+V (Variable), поля класса - Ctrl+Alt+F (Field), параметры метода - Ctrl+Alt+P (Parameter), или наоборот заменять переменные/методы/поля/параметры их непосредственным содержанием при помощи Ctrl+Alt+N (iNline). Это операция, обратная предыдущим четырем)

Спасибо большое за комментарий, я всё беру на карандаш ) Про Идею и выделение логики в отдельный метод через сочетание клавиш - очень ценная информация, спасибо!

Я бы, наверное, и ходила этим путём, если бы не страшный валидатор ))

Их 4 if-а читабельнее одного вашего. Хотя их 2 аналогичных куска для swap - безусловно, не фонтан.