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

Split CSVRecord with a subclass for mapping ops.

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • Review, 1.x
    • Parser
    • None

    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.patch
          4 kB
          Adrian Crum
        2. CSV-104.diff
          49 kB
          Gary D. Gregory

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            ggregory Gary D. Gregory

            Dates

              Created:
              Updated:

              Slack

                Issue deployment