CsvFileReader#readAllAsSequence fails on non-equal sized rows

[doyaaaaaken]

CsvFileReader#readAllAsSequence fails on non-equal sized rows

[doyaaaaaken]

The "CSV" definition is defined on RFC4180.
It says here.

Within the header and each record, there may be one or more fields, separated by commas. Each line should contain the same number of fields throughout the file.

So, your sample code doesn't write "CSV" file, so it's not unnatural that kotlin-csv cannot read your writing file.

[koral--]

OK, thanks. Now I understand the explanation.
One little question then: is it intended that library allows writing a file which does not conform to CSV format?
In this particular case the 2nd #writeRow() call may throw an exception (or it can be thrown when closing file, technical details are not important now).

[doyaaaaaken]

Yeah, right. Thank you.
Maybe, it'd be better to create another issue, relate this issue on that, and close this issue.

[doyaaaaaken]

@koral--
Thanks for reporting!

Throwing exception is expected behavior.
If you change behavior, the skipEmptyLine option on CsvReader is available.
See https://github.com/doyaaaaaken/kotlin-csv#customize

[doyaaaaaken]

Any other question on this topic? Feel free to ask.
If not, I'll close this issue.

[koral--]

I don't longer need this library, but can help resolve this issue if needed.

I don't understand your explanation. Could you elaborate?
Is it really intended that CSV written by this library is not readable by itself?

[doyaaaaaken]

One little question then: is it intended that library allows writing a file which does not conform to CSV format?

Yes, it's intended.

In this particular case the 2nd #writeRow() call may throw an exception (or it can be thrown when closing file, technical details are not important now).

Yes, it should be. To throw exception on the 2nd writeRow() call is useful.
At this point, I don't plan to implement this validation by myself, but pull requests are welcomed. 👍

[koral--]

OK so I guess that this issue can be rephrased to something like "Fail fast on writing malformed CSV" and tagged as enhancement., right?