commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Amey Jadiye (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TEXT-96) Convenience methods needed for RandomStringGenerator
Date Sat, 01 Jul 2017 15:15:00 GMT

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

Amey Jadiye commented on TEXT-96:
---------------------------------

I have submitted PR for accepting multiple pairs, now the questions is can we make it more
simpler and strait forward like giving direct methods like _randomNumeric(), randomAlphabetic(),
randomAlphanumeric()_ ?

TEXT-97 will  able to do the task but not so simpler as Mr [~greenth] expect, i'm +0 on this
front. that is I'm Ok to add those method but Ok as well if we don't add as TEXT-97 can do
that.

> Convenience methods needed for RandomStringGenerator
> ----------------------------------------------------
>
>                 Key: TEXT-96
>                 URL: https://issues.apache.org/jira/browse/TEXT-96
>             Project: Commons Text
>          Issue Type: Improvement
>    Affects Versions: 1.1
>            Reporter: Peter Phillips
>            Priority: Minor
>
> {{RandomStringGenerator}} is extremely verbose compared to the deprecated commons.lang3
{{RandomStringUtils}}.
> Previously we could write:
> {code:java}
> RandomStringUtils.randomNumeric(10)
> {code}
> to generate a numeric string whereas this now has become:
> {code:java}
> new RandomStringGenerator.Builder().withinRange('0', '9').build().generate(10)
> {code}
> although in practice we would then also use static imports too.
> The {{randomAlphabetic}} conversion is even more verbose:
> {code:java}
> new RandomStringGenerator.Builder().withinRange('A', 'z').filteredBy(new CharacterPredicate()
{
>                     @Override
>                     public boolean test(int codePoint) {
>                         return codePoint >= 'a' || codePoint <= 'Z';
>                     }
>                 }).build().generate(10))
> {code} and at that point I lost enthusiam with trying to replicate {{randomAlphanumeric}}.
> I don't think the average java developer would understand what a code point is in the
first place so then trying to get our automation testers to use the new API to implement random
alphanumeric character generation would be difficult.
> I therefore suggest that commons-text should have a copy of {{RandomStringUtils}} which
can even delegate to {{RandomStringGenerator}} or alternatively convenience static methods
for the common use cases.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message