Uploaded image for project: 'Commons CSV'
  1. Commons CSV
  2. CSV-104

Split CSVRecord with a subclass for mapping ops.

    XMLWordPrintableJSON

    Details

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

      Description

      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.

        Attachments

        1. CSV-104.diff
          49 kB
          Gary D. Gregory
        2. CSV-104.patch
          4 kB
          Adrian Crum

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                ggregory Gary D. Gregory
              • Votes:
                1 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated: