X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=lib%2FMojoFacets%2FImport%2FCSV.pm;h=c94ab5af0b885e56dfc1f05dc42c5c438828dc38;hb=7814084ef7332a6288f2db0374eb7b9b4933fe92;hp=46ef1f5b380d7cb8ea8408fd44447066ccc2e31a;hpb=795070127e336b91bf709c4bf75ce53ed57463fc;p=MojoFacets.git

diff --git a/lib/MojoFacets/Import/CSV.pm b/lib/MojoFacets/Import/CSV.pm
index 46ef1f5..c94ab5a 100644
--- a/lib/MojoFacets/Import/CSV.pm
+++ b/lib/MojoFacets/Import/CSV.pm
@@ -5,80 +5,60 @@ use strict;
 
 use base 'Mojo::Base';
 
-use File::Slurp;
+use Text::CSV;
 use Data::Dump qw(dump);
-use Encode;
-
-__PACKAGE__->attr('path');
-__PACKAGE__->attr('full_path'); # FIXME remove full_path
-
-my $null = ''; # FIXME undef?
-
-sub _split_line {
-	my ( $delimiter, $line ) = @_;
-	my @v;
-	while ( $line ) {
-		my $v;
-		if ( $line =~ s/^"// ) {
-			$line =~ s/""/_qq_/gc;
-			$line =~ s/^\s*([^"]+)\s*"\Q$delimiter\E?// || die "can't parse $line";
-			$v = $1;
-		} elsif ( $line =~ s/^\s*([^\Q$delimiter\E]+)\s*\Q$delimiter\E?// ) {
-			$v = $1;
-		} elsif ( $line =~ s/^\s*\Q$delimiter\E// ) {
-			$v = $null;
-		} else {
-			die "can't parse [$line]\n";
-		}
 
-		$v =~ s/^\s*(.+?)\s*$/$1/;
-		$v =~ s/_qq_/"/g;
-		push @v, $v;
-	}
-
-	return @v;
-}
+__PACKAGE__->attr('full_path');
 
 sub data {
 	my $self = shift;
 
-	my $path = $self->full_path || $self->path;
+	my $path = $self->full_path;
 
-	my $data = read_file $path, { binmode => ':raw' }; # FIXME configurable!
 	my $encoding = 'utf-8';
 	if ( $path =~ m/\.(\w+).csv/i ) {
 		$encoding = $1;
 	}
-	warn "decoding ", length($data), " bytes using $encoding\n";
-	$data = decode($encoding, $data);
-
-	my @lines = split(/\r?\n/, $data);
-	$data = { items => [] };
 
-	my $delimiter = ',';
+	my $data = { items => [] };
+	my @header;
 
-	if ( $lines[0] !~ /;/ && $lines[1] =~ /;/ ) {
-		shift @lines; # FIXME ship non-header line
-		$delimiter = ';';
+	open my $fh, "<:encoding($encoding)", $path or die "$path: $!";
+	my $first = <$fh>;
+	my $possible_delimiters;
+	while ( $first =~ s/(\W)// ) {
+		$possible_delimiters->{$1}++;
+	}
+	warn "# possible_delimiters = ",dump($possible_delimiters);
+	seek $fh,0,0; # rewind for Text::CSV
+
+	my @sep_by_usage = sort { $possible_delimiters->{$b} <=> $possible_delimiters->{$a} } keys %$possible_delimiters;
+	my $sep_char = shift @sep_by_usage;
+	while ( $sep_char =~ m/^\s$/ ) {
+		warn "## skip whitespace separator ",dump($sep_char);
+		$sep_char = shift @sep_by_usage;
 	}
 
-	warn "$path ", $#lines + 1, " lines encoding: $encoding delimiter:",dump($delimiter);
-
-	my $header_line = shift @lines;
+	warn "sep_char = [$sep_char] for $path\n";
 
-	my @header = _split_line( $delimiter, $header_line );
-	warn "# header ",dump( @header );
+	my $csv = Text::CSV->new ( { binary => 1, eol => $/, sep_char => $sep_char } )
+		or die "Cannot use CSV: ".Text::CSV->error_diag ();
 
-	while ( my $line = shift @lines ) {
-		chomp $line;
-		my @v = _split_line($delimiter, $line);
+	while ( my $row = $csv->getline( $fh ) ) {
+		if ( ! @header ) {
+			@header = @$row;
+			next;
+		}
 		my $item;
-		foreach my $i ( 0 .. $#v ) {
-			$item->{ $header[$i] || "f_$i" } = [ $v[$i] ];
+		foreach my $i ( 0 .. $#{$row} ) {
+			$item->{ $header[$i] || "f_$i" } = [ $row->[$i] ];
 		}
 		push @{ $data->{items} }, $item;
 	}
 
+	$csv->eof or $csv->error_diag();
+	close $fh;
+
 	$data->{header} = [ @header ];
 	
 	return $data;