Le 08/08/2013 15:40, Gary Gregory a écrit :

> Sans type names:
> 
> parse(File, CSVFormat)
> parse(String, Charset, ClassLoader, CSVFormat)
> parse(String, Charset, CSVFormat)
> [parse(String)]
> parse(String, CSVFormat)
> parse(URL, Charset, CSVFormat)

That looks better. I would remove the methods for a classpath resource,
that's a less common case. That would make:

parse(File, CSVFormat)
parse(String, CSVFormat)
parse(URL, Charset, CSVFormat)

And you probably want a charset for the File too.


> [I'd probably remove parse(String) so that all APIs take a CSVFormat.]

+1.

And at this point you realize they could belong to CSVFormat, because
they all need one to operate.

    format.parse(file):

instead of:

    CSVParser.parse(file, format);


Emmanuel Bourg


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to