You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Henri Yandell (JIRA)" <ji...@apache.org> on 2008/03/01 09:27:51 UTC

[jira] Created: (SANDBOX-220) CSVParser.nextValue() seems pointless

CSVParser.nextValue() seems pointless
-------------------------------------

                 Key: SANDBOX-220
                 URL: https://issues.apache.org/jira/browse/SANDBOX-220
             Project: Commons Sandbox
          Issue Type: Bug
          Components: CSV
            Reporter: Henri Yandell


The nextValue method really doesn't seem very useful. There's no concept of end of line, so your csv file could be 1xn or nx1 and you wouldn't know.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SANDBOX-220) CSVParser.nextValue() seems pointless

Posted by "Bob Smith (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SANDBOX-220?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12672853#action_12672853 ] 

Bob Smith commented on SANDBOX-220:
-----------------------------------

It would be useful if the getLineNumber() method returned the current line in the output instead of the current line in the reader (since they would be different if there is a multi-line value).  Then you could use that method to tell which line the value is on. 

And if I'm looking at it correctly, the only special value it returns is null when it gets to the end of the file (which would be how you know that there are no more values).

> CSVParser.nextValue() seems pointless
> -------------------------------------
>
>                 Key: SANDBOX-220
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-220
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: CSV
>            Reporter: Henri Yandell
>             Fix For: CSV 1.0
>
>
> The nextValue method really doesn't seem very useful. There's no concept of end of line, so your csv file could be 1xn or nx1 and you wouldn't know.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SANDBOX-220) CSVParser.nextValue() seems pointless

Posted by "Henri Yandell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SANDBOX-220?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Henri Yandell updated SANDBOX-220:
----------------------------------

    Fix Version/s: CSV 1.0

Digging into the code - I think it returns a string containing a newline of some kind to indicate end of line. It is doing something for end of line, but I imagine it's not much fun to code against. Still thinking the method needs a big overhaul.

Maybe delete for 1.0 and make it a feature for 1.x.

> CSVParser.nextValue() seems pointless
> -------------------------------------
>
>                 Key: SANDBOX-220
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-220
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: CSV
>            Reporter: Henri Yandell
>             Fix For: CSV 1.0
>
>
> The nextValue method really doesn't seem very useful. There's no concept of end of line, so your csv file could be 1xn or nx1 and you wouldn't know.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.