Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of michele.rossi@gmail.com
 designates 209.85.214.179 as permitted sender)
MIME-Version: 1.0
From: Michele Rossi <michele.rossi@gmail.com>
Date: Sun, 15 Sep 2013 21:05:54 +0200
Message-ID: 
 <CAEPNGTJnJxU5How-7_q-RSvq0cFjVWsN+O8A5wd_GVxLdpbf7A@mail.gmail.com>
Subject: [csv] should CSVRecord's values() method be public?
To: dev@commons.apache.org
Content-Type: multipart/alternative; boundary=089e0149d0ae1acf1d04e670c826

--089e0149d0ae1acf1d04e670c826
Content-Type: text/plain; charset=ISO-8859-1

hi,
I am trying to use commons-csv for a simple CSV transformation tool and I
have encountered a couple of issues that I would like to discuss.

The CSVPrinter takes object arrays as input but the "values" method of
CSVRecord is not public.

In the code I am writing I check a condition to determine whether I should
transform the CSVRecord or not.
But the if that condition returns "false" I am forced to make an akward
copy of the
CSVRecord in an object array as
- CSVRecord objects can not be fed to CSVPrinter
- CSVRecord's values() method is not public

If "values()" was public I could use the data read from one CSV to directly
feed another CSV.

Seems a reasonable use case.

Also, shouldn't CSVRecord implement Map<String, String> ?
The key could be the column header.
It would make sense right?
Or am I missing something?


thanks,
Michele

--089e0149d0ae1acf1d04e670c826--