commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <>
Subject [jira] [Commented] (LANG-1236) Add StopWatch#getTime with String Format Argument
Date Tue, 24 May 2016 15:05:13 GMT


Sebb commented on LANG-1236:


I'm not sure it makes sense to add formatting to the StopWatch class.

There are lots of other LANG methods that return numeric values which need to be formatted
for output.

It would not make sense to add formatting to each and every one.
Better to create generic formatters for the various different numeric types.
This would also be much easier to test.

> Add StopWatch#getTime with String Format Argument
> -------------------------------------------------
>                 Key: LANG-1236
>                 URL:
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.time.*
>            Reporter: Nick Manley
>            Priority: Minor
> Add a version of the StopWatch#getTime method which takes a String to format the time
in the desired format.
> {code}
> StopWatch watch= StopWatch.createStarted();
> // sleep for 1 hour, 19 minutes, 45.554 seconds
> try {Thread.sleep(4785554);} catch (final InterruptedException ex) {}
> watch.stop();
> System.out.println(watch.getTime("HH:mm:ss.SSS");
> // prints: 1:19:45.554
> {code}
> I have a patch ready, but I wasn't sure if this kind of thing should be a part of the
getTime method or if toString should be overloaded to take a format argument. Either way seems
okay to me. I put it on getTime, but if there are disagreements about that, I would be okay
with putting it in toString instead.

This message was sent by Atlassian JIRA

View raw message