Как можно оптимизировать данный код сортировки в Java?

Question

[Борис Хрипко]

Всем привет. Помогите если не трудно.
Выполнил задание и теперь пытаюсь понять как его оптимизировать чтобы выглядело нормально. Потому что то что приходит в голову выглядит отнюдь не нормально.
Есть две сортировки одного массива объектов одного и того же класса.
Нужно чтобы они отсортировались от меньшего к большему учитывая определенные свойства, но с учетом что ячейки содержащие null должны оказаться в самом конце массива, а перед ними все ячейки у которых определенные поле равно 0 или null (в зависимости от того по какому полу делается сортировка).
Вот код:

Arrays.sort(devices, new Comparator<Device>() {
                @Override
                public int compare(Device o1, Device o2) {
                    if (o1 == null && o2 == null) {
                        return 0;
                    }
                    if (o1 == null) {
                        return 1;
                    }
                    if (o2 == null) {
                        return -1;
                    }
                    if (o1.getIn() == 0) {
                        return 1;
                    }
                    if (o2.getIn() == 0) {
                        return -1;
                    }
                    return Integer.compare(o1.getIn(), o2.getIn());
                }
            });

Arrays.sort(devices, new Comparator<Device>() {
                @Override
                public int compare(Device o1, Device o2) {
                    if (o1 == null && o2 == null) {
                        return 0;
                    }
                    if (o1 == null) {
                        return 1;
                    }
                    if (o2 == null) {
                        return -1;
                    }
                    if (o1.getProductionDate() == null) {
                        return 1;
                    }
                    if (o2.getProductionDate() == null) {
                        return -1;
                    }
                    return o1.getProductionDate().compareTo(o2.getProductionDate());
                }
            });

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

Comments

[Денис Загаевский]

А в чём проблема, зачем это оптимизировать? Максимум вынести компараторы в отдельные классы, чтобы не захламлять код.

[Dmitry]

1. можно использовать switch ( правда не знаю, что быстрей, if по порядку или switch)
2. у вас есть условия с одинаковыми результатами, их можно объединить, например:

if (o1.getProductionDate() == null) {
                        return 1; }
                    if (o1 == null) {
                        return 1;
                    }

if ((o1 == null) || (o1.getProductionDate() == null)) return 1;

[Борис Хрипко]

Janus74: объединять их нельзя, так как именно в таком порядке они дают верную сортировку. Уже проверял.

[Борис Хрипко]

Денис Загаевский: После проверки задания мне сказали что его можно и нужно оптимизировать.

[Денис Загаевский]

Борис Хрипко: ну тогда сделай как там ниже сказали. Оптимизация ради оптимизации.

Answer 1

[Сергей]

У Вас есть в Компараторах две части
1. сравнение с нул
2. сравнение неких полей.
сравнение с нул дублируется. это уже плохо. нужно выносить за пределы этих двух классов.
сравнение двух полей однотипно. т.е. взяли некое поле и сравнили его компареТу. тоже похоже дублируются.

Можно сделать свой компаратор, но с параметром -
будет примерно так
sort(new MyDeviceComparatorOn(Device::getIn))

Comments

[Денис Загаевский]

Это дублирование настолько ничтожно, что почти не имеет смысла его куда-то выносить.

[Сергей]

Денис Загаевский: конечно дело вкуса. но я дал идею как сделать компаратор универсальным. передавая геттер как параметр.

[Борис Хрипко]

Сергей: Мы на данном этапе к изучению восьмерки ещё не дошли. Вы могли бы пояснить что обозначает двойное двоеточие?

[Сергей]

Борис Хрипко: да, это действительно java1.8
лямбда выражения
если по простому нужно определить интерфейс который дает значения определенного поля Device .
например ExtractField{
T get(Devive);
}.

[Сергей]

и делать что-то вроде IdExtractField implementation ExtractField ....
метод get(device) {return device.getId())
тогда

MyDeviceComparatorOn ,будет получать idExtractField как параметр и делать сравнение для пустых значений, а затем для поля id.
idExstractField.get(device1).compareTo(idextractField.get(device2))
для сравнения по другому полю, нужно переопреденить только реализацию ExtractField