Hacker News Board Hacker News, browsed like a classic message board
Live data from Hacker News

World's Smallest CSV Parser (C#)

github.com

Post reply on HN
Print view View original

71–72 of 72 posts

Re: World's Smallest CSV Parser (C#)

#71

Post by vilark »

recursive wrote: post #21

`column.Substring(1, column.Length - 2)` could use the new-ish range indexing syntax. column[1..^1]

FYI I got rid of this line, now I just don't add the quotes in the first place, unless the caller requests it. Performance didn't actually change, but it looks smarter. Thanks again for the review though.

Re: World's Smallest CSV Parser (C#)

#72

Post by recursive »

ygra wrote: post #69
vilark wrote: post #64

Earlier quoted context omitted.

Thanks... I didn't think that would compile on .net 6, but it does!

You can probably shorten it to column[1..] and it compiles down to a Substring call, and ranges are part of C# 8, so they exist since .NET Core 3.1. But even if the syntax is newer (e.g. collection expressions in C# 12) you can often also use features on older target frameworks if they don't require additional runtime support (and even that can often be retrofitted internally).

That's different. It includes the last character.
Post reply on HN