commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Duncan Jones (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LANG-1030) StringUtils.same opposite method to StringUtils.differ
Date Thu, 24 Jul 2014 06:58:39 GMT

    [ https://issues.apache.org/jira/browse/LANG-1030?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14072903#comment-14072903
] 

Duncan Jones commented on LANG-1030:
------------------------------------

I'm not convinced this is a method we should be adding to the already bloated {{StringUtils}}
class; it doesn't immediately strike me as a commonly required code fragment. Is there a common
use case I've not considered?

Side note: no need to have {{return new String(str1);}}, since strings are immutable. Just
having {{return str1;}} will suffice.



> StringUtils.same opposite method to StringUtils.differ
> ------------------------------------------------------
>
>                 Key: LANG-1030
>                 URL: https://issues.apache.org/jira/browse/LANG-1030
>             Project: Commons Lang
>          Issue Type: Improvement
>    Affects Versions: 3.3.2
>            Reporter: Michael Bazos
>            Priority: Trivial
>              Labels: StringUtils
>             Fix For: Review Patch, Discussion
>
>
> This is pretty easy to do but I have seen this come up and was surprised to find that
StringUtils didn't have a utility method.
> The difference() method is described as doing the following:
> Compares two Strings, and returns the portion where they differ.
> What I am proposing is essentially the opposite of the difference method called same().
 This method would compare two String and return a new String when they are the same.
> This is essentially the implementation:
> {code:java}
>     public static String same(String str1, String str2) {
>         if (str1 != null && StringUtils.equalsIgnoreCase(str1, str2)) {
>             return new String(str1);
>         }
>         return null;
>     }
> {code}
> {code:java}
> StringUtils.same(null, null) = null
> StringUtils.same("", "") = "";
> StringUtils.same("123", "") = null
> StringUtils.same("123", "123") = "123";
> {code}
> If there is already a way to do this using the apache lang library please point me in
the right direction.  Otherwise I would be more than happy to do the testing, coding and documentation
for this.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message