Re^6: Reading Excel file in perl

[Pauler]

This node falls below the community's threshold of quality. You may see it by logging in.

[marto]

You claim to know about Spreadsheet::ParseExcel, have you read the documentation? Lots of code as well as links to examples. Super Search for more threads on this subject.

[Pauler]

I have the below code which reads the column value. But the prb in the code is it prints each column value 3times. Could someone please help in finding the bug. The excel looks like below:

 
      perl flavors    modules     class     Candidate Name
       vanilla          excel     first       Nancy
       strawberry      dumper     second       jimmy
       lettuce           array     third       roma

The perl code is as below:

 
#use strict;
use Spreadsheet::ParseExcel;

my $FileName = "/home/dujnne/praice/excel/test.xls";
my $excel = Spreadsheet::ParseExcel::Workbook->Parse($FileName) or die
+ "Unable to open $FileName\n";

#locate columns in the spreadsheet from which we want to extract data

my $found = 0;
my ($rowCnt, $colCnt) = (0,0);

foreach my $sheet (@{$excel->{Worksheet}}) {
    printf("Sheet: %s\n", $sheet->{Name});

    $sheet->{MaxRow} ||= $sheet->{MinRow};

    $rowCnt = $sheet->{MinRow};

    foreach my $row ($sheet->{MinRow} .. $sheet->{MaxRow}) {
        $sheet->{MaxCol} ||= $sheet->{MinCol};
        $colCnt = $sheet->{MinCol};

        foreach my $col ($sheet->{MinCol} ..  $sheet->{MaxCol}) {
                     my $cell = $sheet->{Cells}[$row][$colCnt];
                                  if ( lc $cell->{Val} eq "class" ) {
                $found = 1;
                print "OKAY: found the column as "class"\n";
                print "INFO: searching for all classes \n";
                $colCnt++;
            }
            if ( $found ) {
                print $sheet->{Cells}[$row][$colCnt]->{Val}, "\n";
                next;
            }
        }
    }
    $found = 0;


}

[Athanasius]

Hello Pauler,

I don’t really understand what this code is trying to do, so I’ll just offer some observations:

1. The code as given does not compile. This line:

   print "OKAY: found the column as "class"\n";
   

   needs to be:

   print "OKAY: found the column as \"class\"\n";
   

   with the inner double-quote characters backslashed.

2. You should always — yes, always! — begin each script with:

   use strict;
   use warnings;
   

   — and never comment them out again!

3. I suspect the script’s main problem is with the next statement on the 6th-last line. This ends the current iteration of the innermost loop, but as it’s the last statement within that loop, it currently does nothing. Perhaps you meant last? Or next LABEL; where LABEL refers back to a previous (i.e., outer) loop? See next and last in Perl documentation.

Hope that helps,