[CSV-104] Split CSVRecord with a subclass for mapping ops. - ASF JIRA

XML

Word

Printable

JSON

Type: Improvement
Status: Open
Priority: Major
Resolution: Unresolved
Affects Version/s: None
Fix Version/s: Review, 1.x
Component/s: Parser
Labels:
None
Environment:

Hide

Apache Maven 3.1.1 (0728685237757ffbf44136acec0402957f723d9a; 2013-09-17 11:22:22-0400)
Maven home: C:\Java\apache-maven-3.1.1\bin\..
Java version: 1.7.0_51, vendor: Oracle Corporation
Java home: C:\Program Files\Java\jdk1.7.0_51\jre
Default locale: en_US, platform encoding: Cp1252
OS name: "windows 7", version: "6.1", arch: "amd64", family: "windows"

Show
Apache Maven 3.1.1 (0728685237757ffbf44136acec0402957f723d9a; 2013-09-17 11:22:22-0400) Maven home: C:\Java\apache-maven-3.1.1\bin\.. Java version: 1.7.0_51, vendor: Oracle Corporation Java home: C:\Program Files\Java\jdk1.7.0_51\jre Default locale: en_US, platform encoding: Cp1252 OS name: "windows 7", version: "6.1", arch: "amd64", family: "windows"

Either a CSV file has headers or it does not.
Currently, the class CSVRecord is a catch all for both cases.

This patch refactors the code with a CSVRecord class CSVMappedRecord. The class could also be called CSVRecordMap.

The patch does not push the use of generics all the way. All tests compile.

Furthermore, the idea is that the CSVMappedRecord class will implement Map<String, String>, which is what I really want.

is related to

CSV-105 Add Map conversion API to CSVRecord