Проверьте, все ли значения на карте равны нулю

Question

Проверьте, все ли значения на карте равны нулю

1

Мне нужно проверить, являются ли все значения на карте нулевыми, у меня есть этот метод, который я хочу заменить встроенным, если это возможно. Ограничения: Java 5 и доступ к библиотекам Apache Commons.

/* Checks if all values are null */
public static boolean isEmpty(Map<Dboid,?> aMap){

    boolean isEmpty = true;

    Iterator<?> it = aMap.entrySet().iterator();

    while(it.hasNext() && isEmpty){
        Object value = it.next();
        if(value != null) {
            isEmpty = false;
        }
    }

    return isEmpty;
}

Lluis Martinez 14 март 2014, в 18:58

Источник

Теги:

java

map

5 ответов

4

Другое решение без использования сторонних библиотек.

Collections.frequency(aMap.values(), null) == aMap.size()

kapex 14 март 2014, в 15:31

2

как насчет

return CollectionUtils.find(aMap.values(),NotNullPredicate.INSTANCE).isEmpty();

Abhijeet Kushe 14 март 2014, в 16:53

0

Выглядит эквивалентно ответу Рохита.
Lluis Martinez 14 март 2014, в 18:00
0

Да, но это лучше вернуть! CollectionUtils.exists (map.values (), NotNullPredicate.INSTANCE);
Abhijeet Kushe 14 март 2014, в 18:08

2

Для этого нет встроенного метода. В частности, нет ничего, что обеспечивало бы средство "найти элемент, который не равен чему-то".

Однако, если карта, содержащая только null значения, определяется вашими бизнес-правилами как "пустые", это означает, что null значения означают "нет", и в этом случае вы можете построить код таким образом, чтобы null значения никогда не добавляли в первую очередь. Тогда вы можете просто использовать встроенный isEmpty().

Jason C 14 март 2014, в 16:29

1

Нет API, который даст вам это, однако вы можете немного оптимизировать этот метод.
Нет необходимости проверять переменную isEmpty на каждой итерации. Это небольшая оптимизация.

/* Checks if all values are null */
public static <K,V> boolean isMapEmpty(Map<K,V> aMap){
    for (V v: aMap.values()) {
      if (v != null) { return false; }
    }
    return true;
}

Deian 14 март 2014, в 16:17

0

@Jason typpo - должен быть aMap.values (); Оптимизация заключается в том, что он не нуждается в дополнительной переменной и дополнительной проверке на каждой итерации ... что складывается.
Deian 14 март 2014, в 17:53
0

эй, для чего понижатель? просто хочу помочь парню
Deian 14 март 2014, в 17:55
0

Скорее всего, это не складывается, и имеет нулевой эффект. Пожалуйста, предоставьте контрольные показатели и данные профиля для подтверждения заявки, а также сравнение с фактическими требованиями к производительности ОП, прежде чем предлагать преждевременную микрооптимизацию в качестве допустимого подхода. Поощрение того, что мыслительный процесс причиняет больше вреда, чем помогает, и именно поэтому у нас есть так много вопросов, связанных с плохой «производительностью», которые нужно отразить здесь.
Jason C 14 март 2014, в 17:56
0

Это более компактная версия моего текущего кода.
Lluis Martinez 14 март 2014, в 17:58
0

Я согласен - это микро-оптимизация и очистка кода. И это все, что я предложил ... Спасибо за отзыв.
Deian 14 март 2014, в 18:05
0

@Deian проверяет флаг isEmpty в цикле while.
Lluis Martinez 14 март 2014, в 18:06
0

@LluisMartinez - да, в твоем коде это есть, но оно тебе на самом деле не нужно. Как только вы видите ненулевое значение, вы можете оставить на месте.
Deian 14 март 2014, в 18:10
0

Да, но иногда я следую за единственным возвратом в конце метода метода. Хотя не всегда :-)
Lluis Martinez 14 март 2014, в 22:00

Показать ещё 6 комментариев

Ещё вопросы

Выглядит эквивалентно ответу Рохита.
Да, но это лучше вернуть! CollectionUtils.exists (map.values (), NotNullPredicate.INSTANCE);
@Jason typpo - должен быть aMap.values (); Оптимизация заключается в том, что он не нуждается в дополнительной переменной и дополнительной проверке на каждой итерации ... что складывается.
эй, для чего понижатель? просто хочу помочь парню
Скорее всего, это не складывается, и имеет нулевой эффект. Пожалуйста, предоставьте контрольные показатели и данные профиля для подтверждения заявки, а также сравнение с фактическими требованиями к производительности ОП, прежде чем предлагать преждевременную микрооптимизацию в качестве допустимого подхода. Поощрение того, что мыслительный процесс причиняет больше вреда, чем помогает, и именно поэтому у нас есть так много вопросов, связанных с плохой «производительностью», которые нужно отразить здесь.
Это более компактная версия моего текущего кода.
Я согласен - это микро-оптимизация и очистка кода. И это все, что я предложил ... Спасибо за отзыв.
@Deian проверяет флаг isEmpty в цикле while.
@LluisMartinez - да, в твоем коде это есть, но оно тебе на самом деле не нужно. Как только вы видите ненулевое значение, вы можете оставить на месте.
Да, но иногда я следую за единственным возвратом в конце метода метода. Хотя не всегда :-)

Rohit Jain · Accepted Answer · 2014-03-14T16-22-00.000Z

Как таковой нет прямого метода для этого, но вы можете использовать метод Apache Commons CollectionUtils.countMatches() и передать NullPredicate экземпляр NullPredicate. Конечно, вы должны передать значения на карте с помощью метода Map#values():

public static <K, V> boolean hasAllNullValues(Map<K, V> map) {
    int size = map.size();
    return CollectionUtils.countMatches(map.values(), NullPredicate.INSTANCE) == size;
}

или даже лучше, используйте метод CollectionUtils.exists(), чтобы проверить, есть ли хотя бы один элемент, который удовлетворяет NotNullPredicate переданному как второй аргумент:

public static <K, V> boolean hasAllNullValues(Map<K, V> map) {
    return !CollectionUtils.exists(map.values(), NotNullPredicate.INSTANCE);
}