New answers tagged

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 ...
user avatar
  • 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/...
user avatar
  • 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. ...
user avatar
  • 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 ...
user avatar
1 vote

Analyzing baby name data in CSV files

For this assignment, you have been provided with a starter file which is: ...
user avatar
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 ...
user avatar
  • 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 - ...
user avatar
  • 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 ...
user avatar
  • 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 ...
user avatar

Top 50 recent answers are included