Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trial
Daniel Tkach
7,608 PointsNo \r, only \n
Hello folks, I only have a \n, and well I can use the Split method with '\n\' as a parameter and it works just fine. I was wondering why I don't have this \r and if there are any implications. I guess before splitting a csv file I should add some code to check if the \r is present? What do you think?
4 Answers
Steven Parker
231,248 PointsIt wouldn't hurt to play it safe.
Depending on the OS, the standard for line ending might be just "\n" (Linux or Mac), or "\r\n" (Windows).
Just to be safe, you might want to do a Replace("\r\n", "\n") before the split.
Ole Vølund Skov Mortensen
27,842 PointsOK, I got Your answers. If You get the above issue, select the csv file. click text, ctrl+A, and look for the LF in the lower right. click and select CRLF(Caridge return, line feed). and save. run the program again and you have it fixed.
Manuel wandeto
Courses Plus Student 10,504 PointsFor the purpose of splitting the csv file into an array of substrings i do agree that having one character to use for delimiting makes it easier and hence replacing \r\n with just \n would serve perfect for the task, all i am saying is that for the purpose of those who would wish to follow closely along with the video, the csv file provided seems to only have \n as the newline character, and hence using \n as the only delimiting character does not bring about the problem of empty strings in the resulting string array and so if people would so wish to see this problem first hand they would rather replace the \n with \r\n to get a clear understanding of why we are using the StringSplitOptions.RemoveEmptyEntries at the end to solve this problem.
Also, if in a case of generally creating text files to be read across many windows systems, \r\n would be better as a go to newline character to be safe, as it is backwards compatible with older windows systems that only recognize the combination of both characters as a new line character.
Steven Parker
231,248 PointsWe must be looking at different videos. The one I saw definitely does have an issue with two delimiters and a problem with empty strings, as demonstrated around time index 3:50. It gets handled (around 5:00) by the addition of StringSplitOptions.RemoveEmptyEntries as an extra argument to "Split". The use of "Replace" originally suggested here would be an alternative fix.
And for creating output files (not part of the video exercise), I agree with your suggestion if the file is likely to be used on multiple platforms. But in the more likely case where the output will remain on the same system as where it is created, I recommend sticking with the convention used on that system.
Manuel wandeto
Courses Plus Student 10,504 Pointsi believe having both "\r\n" is a good thing for backwards compatibility with older windows systems that require both the carriage return and line feed characters, in the case of a text file only having "\n", calling the Replace("\n" , "\r\n") would replace all instances of the escape sequence "\n" in the text file with "\r\n" thereby making it backwards compatible with older systems. but if it's only for the cause of the tutorial then only "\n" would do just fine.
Steven Parker
231,248 PointsWe must be talking about different purposes for the code. In the video, the string containing multiple lines is being split into separate strings for each line. So converting to just a single line terminating character makes this job easier to do, no matter what kind of system the code is being used on. The resulting strings have no terminating characters so the concern about "backwards compatibility" would not apply here.
Manuel wandeto
Courses Plus Student 10,504 PointsManuel wandeto
Courses Plus Student 10,504 PointsActually since Replace(string OldValue, string NewValue), it should be the other way round, Replace("\n", "\r\n").
Steven Parker
231,248 PointsSteven Parker
231,248 PointsI think the point was that only "\n" is needed here so the replace suggestion removes the "\r", and only if it exists.
If you reverse it, then if you start with "\r\n" you will end up with "\r\r\n".