Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Date: Sat, 1 Jul 2017 15:15:00 +0000 (UTC)
From: "Amey Jadiye (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Message-ID: <JIRA.13083445.1498738169000.153096.1498922100057@Atlassian.JIRA>
In-Reply-To: <JIRA.13083445.1498738169000@Atlassian.JIRA>
References: <JIRA.13083445.1498738169000@Atlassian.JIRA> <JIRA.13083445.1498738169498@jira-lw-us.apache.org>
Subject: [jira] [Commented] (TEXT-96) Convenience methods needed for
 RandomStringGenerator
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Sat, 01 Jul 2017 15:15:04 -0000


    [ 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)