[jira] [Comment Edited] (TEXT-34) Add class to generate random strings

["Emmanuel Bourg (JIRA)" <ji...@apache.org>]

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

Emmanuel Bourg edited comment on TEXT-34 at 12/20/16 11:53 AM:
---------------------------------------------------------------

{quote}
Did you notice the fields I defined?
{quote}

I didn't, sorry. The extra methods would still be useful though. Most people discover the capabilities of a class by looking at the auto-completion suggestions from the IDE. In this case the {{withDigits()}} method would pop-up immediately and look as a more obvious choice than {{filteredBy()}}.

{quote}
Nothing prevents a custom filter from saving the characters seen so far
{quote}

Good point, the interface is good as-is then (well, until we use a parallel string generation method ;) )


was (Author: ebourg):
{quote}
Did you notice the fields I defined?
{quote}

I didn't, sorry. The extra methods would still be useful though. Most people discover the capabilities of a class by looking at the auto-completion suggestions from the IDE. In this case the {{withDigits()}} method would pop-up immediately and look as a more obvious choice than {{filteredBy()}}.

{quote}
{quote}

{quote}
{quote}

> Add class to generate random strings
> ------------------------------------
>
>                 Key: TEXT-34
>                 URL: https://issues.apache.org/jira/browse/TEXT-34
>             Project: Commons Text
>          Issue Type: New Feature
>            Reporter: Duncan Jones
>            Assignee: Duncan Jones
>
> The {{RandomStringUtils}} class will be deprecated in Commons Lang and reborn in Commons Text.
> Because the original class has some complicated parameters and potentially unnecessary capabilities, I'll simplify as I move it across. I'll also ensure the class plays nicely with Unicode characters.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)