[ http://issues.apache.org/jira/browse/LANG-266?page=all ]
Stepan Koltsov updated LANG-266: -------------------------------- Attachment: commons-lang-join-collection-stepancheg.diff The patch. Also hacks join(Iterator, *) to aviod creation of StringBuffer if iterator gives zero or one elements. > Wish for StringUtils.join(Collection, *) > ---------------------------------------- > > Key: LANG-266 > URL: http://issues.apache.org/jira/browse/LANG-266 > Project: Commons Lang > Type: Wish > Versions: Nightly Builds > Reporter: Stepan Koltsov > Priority: Trivial > Attachments: commons-lang-join-collection-stepancheg.diff > > There is a lack of StringUtils.join(Collection, String) method in > commons-lang. Could you please add it? > Commons-lang has method join(Iterator, String), however join(Collection, > String) is still useful. Join is used very often, and in most situations it > is used with Collection, not with Iterator, because almost all methods return > Collections, not Iterators and a lot of data is stored as Collections, not as > Iterators. The string ".iterator()" has 11 characters while line for code is > only 72 chars. I've found that in my code there are several methods that > could be one line shorter if it was StringUtils.join(Collection, String). > Compare: > // with StringUtils.join(Collection, String) > return "UNIQUE (" + StringUtils.join(unique.getColumnNames(), ", ") + ")"; > against: > // without > Iterator columnNames = unique.getColumnNames().iterator(); > return "UNIQUE (" + StringUtils.join(columnNames, ", ") + ")"; > Motivation is almost same as "Why we have StringUtils.join(Object[], String) > while we could write StringUtils.join(Arrays.asList(array).iterator(), > separator)?" -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]