Beefy Boxes and Bandwidth Generously Provided by pair Networks
"be consistent"

Re: CSV File Parsing

by Tux (Abbot)
on Feb 28, 2013 at 20:55 UTC ( #1021114=note: print w/replies, xml ) Need Help??

in reply to CSV File Parsing

  • Do not use parse (), use getline ()
  • Use a lexical file handle
  • Never use an else branch if the if branch ends the enclosing scope (exit, return, die, last, )
  • Use auto_diag
  • Use positive logic for validation
# Use auto_diag to see all parsing errors immediately my $csv = Text::CSV_XS->new ({ binary => 1, auto_diag => 1, allow_whit +espace => 1 }); open my $fh, "<", $file_to_parse or die "Could not open '$file' $!"; while (my $row = $csv->getline ($fh)) { # Check first and third field to be numeric only (and not empty) $row->[0] =~ m/^[0-9]+$/ && $row->[2] =~ m/^[0-9]+$/ or exit; # wrong format # THERE CANNOT BE AN ELSE AFTER AN EXIT ! }

update: your sample data contains spaces. You will have to deal with those, as the current tests will fail for leading and training spaces. Adding allow_whitespace option to the constructor will help you here.

Enjoy, Have FUN! H.Merijn

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1021114]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others musing on the Monastery: (4)
As of 2016-10-22 00:59 GMT
Find Nodes?
    Voting Booth?
    How many different varieties (color, size, etc) of socks do you have in your sock drawer?

    Results (291 votes). Check out past polls.