commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henri Yandell (JIRA)" <j...@apache.org>
Subject [jira] Updated: (COLLECTIONS-256) ListUtils.transformedList(list, transformer) doesn't transform
Date Wed, 05 Sep 2007 13:39:33 GMT

     [ https://issues.apache.org/jira/browse/COLLECTIONS-256?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Henri Yandell updated COLLECTIONS-256:
--------------------------------------

    Attachment: COLLECTIONS-256.patch

Patch with tests and new methods for:

Buffer
Set
SortedSet
SortedBag
Bag
Collection
List

> ListUtils.transformedList(list, transformer) doesn't transform
> --------------------------------------------------------------
>
>                 Key: COLLECTIONS-256
>                 URL: https://issues.apache.org/jira/browse/COLLECTIONS-256
>             Project: Commons Collections
>          Issue Type: Bug
>          Components: List
>    Affects Versions: 3.2
>         Environment: Java 1.5
>            Reporter: Kurtis Williams
>            Assignee: Henri Yandell
>             Fix For: 3.3
>
>         Attachments: COLLECTIONS-256.patch
>
>
> The static method ListUtils.transformedList(List list, Transformer trans) does not apply
the transformer before returning the list.  I've traced into the code and it falls all the
way through to the TransformedCollection constructor which has documentation that states that
items existing in the list are NOT transformed.  This yields bizarre unexpected behavior for
this method.
> Conversely, the ListUtils.predicatedList(List list, Predicate pred) DOES apply the predicate.
 These methods are very similar but behave inconsistently.
> If the transformer is not applied to the list, the method's usefulness is somewhat limited
and causes the user to create an extra unwanted list in some circumstances:
> // To transform an existing list
> List myList = ... some list to be transformed ...
> List transformed = ListUtils.transformedList(new ArrayList(), myTransformer);
> transformed.addAll(myList);
> I would expect a much shorter implementation such as:
> List transformed = ListUtils.transformedList(myList, myTransformer)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message