Программирование, т. 8

но всё равно, когда пишу такой код даже с условием, что я его больше не увижу, мне становится хуёво 

в томже питоне можно сделать любую сортировку в одну строку с кучей скобочек [], отрицательный индексинг этож фича

но онож раскладывается до техже циклов

хочешь больше "высокоуровневости" использую scala или javascript

где накодили еще миллион встроенных методов, к базовым джавовским

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

джава боги, есть ебанутые таски:

1) отфильтровать (удалить или оставить) с массива объекты по типу. Тип фильтрации может быть null, те оставить все у которых obj.type == null, остальные удалить с массива), у объекта тип тоже может быть null, сам объект может быть null).

Есть ли вообще ауты сделать это все "красиво" через stream.filter без ебанутых проверок внутри? Пока такая штука сделана обычным итерированием массива с ифами внутри, выглядит очень мерзко.

 

2) отсортировать массив по критерию у объекта, объекты в массиве, как и поля объекта нужные для сортировки, могут быть null. сортировать должно по условию: null в конец, объекты у которых поле null перед нуллами, остальное по возрастанию.

Это тоже сделал через компаратор, но опять же, выглядит как хуйня... Мб есть какие-то варианты это сделать более красивым?

 

Хелпаните плез, просто заебался от мерзких костылей...

А итеративный процесс не получится использовать в твоей сортировке?

это ты про foreach или другой итератор, то нет, ибо там могут быть null.

Кидай код, может ты говна въебал

допустим фильтрация по type 

        for (int i = 0; i < devices.length; i++) {
            if (devices[i] != null) {
                if ((type != null && devices[i].getType() == null)
                        || (devices[i].getType() != null && (type == null || !devices[i].getType().equals(type)))) {
                    devices[i] = null;
                }
            }
        }

Ебать ну ты и ублюдок

Кидай код, может ты говна въебал

 
  for (int i = 0; i < devices.length; i++) {
            if (devices[i] == null) continue;

            boolean first = type != null && devices[i].getType() == null;
            boolean second = devices[i].getType() != null
            boolean third = type == null || !devices[i].getType().equals(type);

            if (first || (second && third)) devices[i] = null;
        }

не благодари

for (int i = 0; i < devices.length; i++) {

  Type device = devices[i];


  if (device == null) {

    continue;

  }


  Type deviceType = device.getType();

  boolean first = type != null && deviceType == null;

  boolean second = deviceType != null;

  boolean third = type == null || !deviceType.equals(type);


  if (first || (second && third)) {

    devices[i] = null;

  }

}

ну я канеш и далбаеб, так хоть сразу понятно что происходит...

а всего-то переменные объявили

с нуллами надо optional использовать

их быть не должно

джава боги, есть ебанутые таски:

1) отфильтровать (удалить или оставить) с массива объекты по типу. Тип фильтрации может быть null, те оставить все у которых obj.type == null, остальные удалить с массива), у объекта тип тоже может быть null, сам объект может быть null).

Есть ли вообще ауты сделать это все "красиво" через stream.filter без ебанутых проверок внутри? Пока такая штука сделана обычным итерированием массива с ифами внутри, выглядит очень мерзко.

        Object[] array = new Object[]{555, 5454, "pidor", null, null, null};
        System.out.println("Before: " + Arrays.toString(array));
        array = Arrays.stream(array).filter(o -> {
            if (o == null) {
                return false;
            } else if (o instanceof Integer) {
                return false;
            } //todo else if
            return true;
        }).toArray();
        System.out.println("After: " + Arrays.toString(array));

отфильтровать (удалить или оставить) с массива объекты

Есть ли вообще ауты сделать это все "красиво" через stream.filter без ебанутых проверок внутри?

тебе надо понимать, что стримы не поменяют твой массив, а дадут новый, если с етим нету проблемы то юзай пример индекса

Это тоже сделал через компаратор, но опять же, выглядит как хуйня...

типа сам компаратор имплементил?

Arrays.sort(huici, nullsLast(comparing(Hui::getDlina, nullsLast(naturalOrder()))))

        devices = Arrays.stream(devices).filter(device -> {
            if (device == null) {
                return false;
            }
            Type deviceType = device.getType();
            boolean first = type != null && deviceType == null;
            boolean second = deviceType != null;
            boolean third = type == null || !deviceType.equals(type);
            return !(first || (second && third));
        }).toArray();  

Интегрировал с кодом выше

типа сам компаратор имплементил?

Arrays.sort(huici, nullsLast(comparing(Hui::getDlina, nullsLast(naturalOrder()))))

да, ну вот я так и сделал через Arrays.sort и мой компаратор, правда без nullsLast

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

бтв там у меня НПЕ вылазит в тесте с вашими проверками, но я уже поправил (это был скорее всего мой косяк с изначальными цепочками условий, которые можно было оптимизировать)

всем спасибо)

PS: сам кейс учебный, не думаю, что с такими хуйнями сталкиваются часто в ИРЛ

        Arrays.sort(array, (o1, o2) -> {
            Integer o1priority = o1 == null ? Integer.MAX_VALUE : o1.getType() == null ? Integer.MAX_VALUE - 1 : o1.getDlina();
            Integer o2priority = o2 == null ? Integer.MAX_VALUE : o2.getType() == null ? Integer.MAX_VALUE - 1 : o2.getDlina();
            return Integer.compare(o1priority, o2priority);
        });

Как бы я сортировку сделал.

ну это выглядит не лучше чем мой компаратор на пару условий :)
я просто думал мб есть варики без особого описания своей логики сортировки (допустим до этого я не знал про nullFirst\nullLast)

я просто думал мб есть варики без особого описания своей логики сортировки (допустим до этого я не знал про nullFirst\nullLast)

Есть варики. Винду переустановить например

кажется пора смахнуть пыль с правила о винампе