Actually that "if" should not even be there ;)

Gary

On Fri, Sep 21, 2012 at 2:50 PM, Bruno P. Kinoshita <
brunodepau...@yahoo.com.br> wrote:

> +1 as well Gary, and if you want to test the "if (objs == null) {", then
> you just have to add something similar to:
>
> sb.append("Hi", (Object[])null);
>
> And then check the result.
>
> Cheers
>
>
> Bruno P. Kinoshita
> http://kinoshita.eti.br
> http://tupilabs.com
>
>
> >________________________________
> > From: Simone Tripodi <simonetrip...@apache.org>
> >To: Commons Developers List <dev@commons.apache.org>
> >Sent: Friday, 21 September 2012 4:54 AM
> >Subject: Re: [lang] StrBuilder append(String, Object...)
> >
> >+1 very well thought, Gary!
> >
> >even if StrBuilder would look like java.util.Formatter, I think that
> >this is a nice addition.
> >
> >best,
> >-Simo
> >
> >http://people.apache.org/~simonetripodi/
> >http://simonetripodi.livejournal.com/
> >http://twitter.com/simonetripodi
> >http://www.99soft.org/
> >
> >
> >On Thu, Sep 20, 2012 at 11:12 PM, Gary Gregory <garydgreg...@gmail.com>
> wrote:
> >> I just wrote this up for a different project and it seems like a nice
> add
> >> for StrBuilder:
> >>
> >>     @Test
> >>     public void testAppendFormatted_Object() {
> >>         StrBuilder sb;
> >>
> >>         sb = new StrBuilder();
> >>         sb.append("Hi", "Alice");
> >>         assertEquals("Hi", sb.toString());
> >>
> >>         sb = new StrBuilder();
> >>         sb.append("Hi %s", "Alice");
> >>         assertEquals("Hi Alice", sb.toString());
> >>
> >>         sb = new StrBuilder();
> >>         sb.append("Hi %s %,d", "Alice", 5000);
> >>         assertEquals("Hi Alice 5,000", sb.toString());
> >>     }
> >>
> >>     /**
> >>      * Appends an formatted string to this string builder.
> >>      * Appending null will call {@link #appendNull()}.
> >>      *
> >>      * @param format  the format string, see {@link
> String#format(String,
> >> Object...)}
> >>      * @param objs  the objects to use in the format string
> >>      * @return this, to enable chaining
> >>      */
> >>     public StrBuilder append(String format, Object... objs) {
> >>         if (objs == null) {
> >>             return appendNull();
> >>         }
> >>         return append(String.format(format, objs));
> >>     }
> >>
> >> Thoughts?
> >>
> >>
> >> --
> >> E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> >> JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
> >> Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
> >> Blog: http://garygregory.wordpress.com
> >> Home: http://garygregory.com/
> >> Tweet! http://twitter.com/GaryGregory
> >
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> >For additional commands, e-mail: dev-h...@commons.apache.org
> >
> >
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>


-- 
E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Reply via email to