Коллекции Java Commons удалитьВсе - PullRequest
Новая
       20

Коллекции Java Commons удалитьВсе

31 голосов
/ 04 декабря 2011

CollectionUtils :: removeAll () Commons Collections 3.2.1

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

Удаляет элементы из коллекции remove.То есть этот метод возвращает коллекцию, содержащую все элементы в c, которые не находятся в remove.

Этот небольшой тест JUnit 

@Test
public void testCommonsRemoveAll() throws Exception {
    String str1 = "foo";
    String str2 = "bar";
    String str3 = "qux";

    List<String> collection = Arrays.asList(str1, str2, str3);
    System.out.println("collection: " + collection);

    List<String> remove = Arrays.asList(str1);
    System.out.println("remove: " + remove);

    Collection result = CollectionUtils.removeAll(collection, remove);
    System.out.println("result: " + result);
    assertEquals(2, result.size());
}

не выполняется с

java.lang.AssertionError: ожидается: <2>, но было: <1>

и распечатки 

collection: [foo, bar, qux] 
remove: [foo] 
result: [foo]

От прочтения моей документации я должен ожидать [bar, qux].Что я пропустил?

1 Ответ

41 голосов
/ 04 декабря 2011

Изменить 1 января 2014 г. Apache Commons Collections 4.0 окончательно выпущен 21 ноября 2013 г. и содержит исправление для этой проблемы.

Ссылка на CollectionUtils.java

Строки, о которых идет речь (1688 - 1691), с подтверждением, что метод был ранее нарушен: 

/*
 ...
 * @since 4.0 (method existed in 3.2 but was completely broken)
 */
public static <E> Collection<E> removeAll(final Collection<E> collection, final Collection<?> remove) {
    return ListUtils.removeAll(collection, remove);
}

Оригинальный ответ

Нет, ты не сумасшедший. removeAll() на самом деле (неправильно) вызывает retainAll().

Это ошибка в CollectionUtils, влияющая на версию 3.2. Это исправлено, но только в ветке 4.0.

https://issues.apache.org/jira/browse/COLLECTIONS-349

И в качестве еще одного доказательства, вот ссылка на исходный код:

http://svn.apache.org/repos/asf/commons/proper/collections/tags/COLLECTIONS_3_2/src/java/org/apache/commons/collections/CollectionUtils.java

Проверьте эту строку: 

public static Collection removeAll(Collection collection, Collection remove) {
    return ListUtils.retainAll(collection, remove);
}

Да ... сломанный!

...