undefined | Better HN

0 pointsnmz2y ago0 comments

oh boy. here's where it breaks

- supporting "" (single)

- Supporting newlines in "", oops, now you can't getline() and instead need to getdelim()

- Supporting comments # (why is this even a thing)

- Supporting multiple "" in a field

- Escaping " with "" or \"

- length based csv, so all fields are seekable.

It's a mess, which one's your csv?

0 comments

outop2y ago

The vast majority of CSVs do not have strings which include either quotes or newlines.

No CSV I have ever encountered has comments.

nmzOP2y ago

Google contacts does.

outop1y ago

It seems that Google generally try and find ways of making exporting their data to use in other services as painful as possible.

1 more reply

eviks2y ago

So you're fine with a lot of bugs in the case of a vast minority?

outop1y ago

Well, most code that loads CSVs is intended to work with certain files from certain sources, and not with all the CSVs that have ever existed.

So yes, I am happy with code that works for a subset of files. There are thousands of applications which work with CSVs and they all do exactly this.

1 more reply

j / k navigate · click thread line to collapse

0 pointsnmz2y ago0 comments

oh boy. here's where it breaks

- supporting "" (single)

- Supporting newlines in "", oops, now you can't getline() and instead need to getdelim()

- Supporting comments # (why is this even a thing)

- Supporting multiple "" in a field

- Escaping " with "" or \"

- length based csv, so all fields are seekable.

It's a mess, which one's your csv?

0 comments

outop2y ago

The vast majority of CSVs do not have strings which include either quotes or newlines.

No CSV I have ever encountered has comments.

nmzOP2y ago

Google contacts does.

outop1y ago

It seems that Google generally try and find ways of making exporting their data to use in other services as painful as possible.

1 more reply

eviks2y ago

So you're fine with a lot of bugs in the case of a vast minority?

outop1y ago

Well, most code that loads CSVs is intended to work with certain files from certain sources, and not with all the CSVs that have ever existed.

So yes, I am happy with code that works for a subset of files. There are thousands of applications which work with CSVs and they all do exactly this.

1 more reply

j / k navigate · click thread line to collapse