New answers tagged csv
0
votes
Analyzing baby name data in CSV files
There is some overlap with the other reviews, I've voted accordingly, if you found the other answers useful consider voting for them to let them know.
Expressiveness
Code should be expressive and ...
- 11.5k
2
votes
CSV to TSV converter in C+yacc+lex and a Makefile
Makefile
The Makefile looks generally good.
We shouldn't need to write a .c.o rule - the built-in one is likely better.
I recommend adding some suitable warning/...
- 57.7k
5
votes
CSV to TSV converter in C+yacc+lex and a Makefile
Overview
Code Review
Starting at the bottom and working up:
Lexer
This looks like it should work.
The only thing I would do differently is the . match at the end.
...
- 91.5k
2
votes
Read a file format with mixed CSV and non-CSV content
While Reinderien posted an excellent solution to your problem. Let me take a closer look and give you some feedback on your code.
Let us just focus on the 3 biggest things which stands out to me and ...
- 5,493
1
vote
Analyzing baby name data in CSV files
For this assignment, you have been provided with a starter file which is:
...
- 693
1
vote
Analyzing baby name data in CSV files
This will be difficult to review without access to the edu.duke package so my review will be superficial.
parseBabies is a poor ...
- 47.9k
2
votes
Read a file format with mixed CSV and non-CSV content
You're working too hard. You concatenate all of the CSV rows anyway, so LINES= has no value. Just read the CSV, and choose an easy way to identify invalid rows - ...
- 47.9k
0
votes
CSV file parser with some checks for header fields and duplicate rows
Based on the feedback from 200_success, here is revised version of the code
...
- 161
5
votes
Accepted
CSV file parser with some checks for header fields and duplicate rows
Interface
The interface for this class is awkward, and it's not clear how it's meant to be used.
Is diff_headers() supposed to be part of the interface? How about ...
- 142k
Top 50 recent answers are included