Java Commons Colecciones removeAll

Question

CollectionUtils::removeAll() Commons Collections 3.2.1

Me estoy volviendo loco, becuase parece que este método está haciendo lo contrario de lo que el estado docs:

Removes the elements in remove from collection. That is, this method returns a collection containing all the elements in c that are not in remove.

Este pequeño JUnit prueba

@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()); 
} 

Está fallando con

java.lang.AssertionError: expected:<2> but was:<1>

y grabados

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

De mi lectura de los documentos debo esperar [bar, qux]. ¿Qué me he perdido?

Answer 1

Edit 1 de enero de 2014 Apache Commons Collections 4.0 fue finalmente lanzado el 21 de noviembre de 2013, y contiene una solución para este problema.

Link to CollectionUtils.java

líneas en cuestión (1688-1691), con acuse estaba roto previamente el método:

/* 
... 
* @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); 
} 

---

respuesta original

No, no estás loca. removeAll() está (incorrectamente) llamando al retainAll().

Esto es un error en CollectionUtils, que afecta a la versión 3.2. Se ha solucionado, pero solo en la rama 4.0.

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

Y como una prueba más, aquí hay un enlace al código fuente:

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

Salida esta línea:

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

Sí ... roto!