Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Date: Fri, 30 Jun 2017 17:36:00 +0000 (UTC)
From: "Amey Jadiye (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Message-ID: <JIRA.13083445.1498738169000.147834.1498844160506@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: Fri, 30 Jun 2017 17:36:08 -0000


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

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

*Yes*, For scope of this JIRA I strongly believe that. what Mr. [~greenth] want can be accommodated easily with flexibility of the API.

Ex. for
*.randomNumeric()*
{code}
char [][]  ranges = {{'0','9'}};
RandomStringGenerator generator = new RandomStringGenerator.Builder().withinRange(ranges).build();
{code}

*.randomAlphabetic()*
{code}
char [][]  ranges = {{'A','Z'}}; // or {{'A','Z'},{'a','z'}}
RandomStringGenerator generator = new RandomStringGenerator.Builder().withinRange(ranges).build();
{code}

*.randomAlphanumeric()*
{code}
char [][]  ranges = {{'0','9'},{'A','Z'},{'a','b'}};
RandomStringGenerator generator = new RandomStringGenerator.Builder().withinRange(ranges).build();
{code}


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