commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedikt Ritter (JIRA)" <>
Subject [jira] [Commented] (CSV-147) Better error handling in CSV, see included code
Date Tue, 13 Jan 2015 07:10:34 GMT


Benedikt Ritter commented on CSV-147:

I think it would be better to wrap {{CSVRecord nextRecord()}} into a try catch block and throw
a custom exception that contains the desired information in case of an error.

> Better error handling in CSV, see included code
> -----------------------------------------------
>                 Key: CSV-147
>                 URL:
>             Project: Commons CSV
>          Issue Type: Improvement
>          Components: Parser
>    Affects Versions: 1.1
>            Reporter: Steven Peterson
>              Labels: easyfix
>             Fix For: Discussion
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
> When parsing long files, and there is problem with parsing the data, it would be very
useful to know the exact data that is bad when an exception is thrown.  As CSVParser maintains
a portion of the current record that was read when an exception is thrown (in the 'record'
property), can we make this information public through something like this:
> {code:java}
> public String[] getLastRecordData() {
>   return this.record.toArray(new String[this.record.size()]);
> }
> {code}
> With a method like this, it would be easy to pull in part of the data that was in the
faulty record after an exception was thrown, aiding in cleaning up the data.

This message was sent by Atlassian JIRA

View raw message