Re: Issue with Spreadsheet::WriteExcel

My guess is that at least one of the 3 variables passed to the write function is undefined. Check each with print or defined:

print "$row\n";
print "$col\n";
print "$temp\n";

if (defined $col) {
    print "col is defined: $col\n";
}
else {
    print "col is not defined\n";
}
[download]

That is a strange usage of my, by the way.

Use code tags: Writeup Formatting Tips

Comment on Re: Issue with Spreadsheet::WriteExcel Download Code

Replies are listed 'Best First'.
Re^2: Issue with Spreadsheet::WriteExcel by Crackers2 (Parson) on Aug 24, 2010 at 17:29 UTC
I think the message means `$worksheet` is undefined, not one of the parameters. And I think the real problem is because of all the extra `my`'s in there. my $workbook; # No need for the = () my $worksheet; # No need for the = () # Create output file with user filename: if ($outfile ne "d") { $workbook = Spreadsheet::WriteExcel->new($outfile); # No my, or $wo +rkbook will go out of scope at the end of the if block $worksheet = $workbook->add_worksheet(); # same here print "What should we label the column containing the data?\n"; my $colname = <STDIN>; chomp($colname); $worksheet->write(0,"A",$colname); } # Other code here I assume # Write 00 temperature data into file: $read = 4+(($day-1)*58); $temp = $xls->1{("B", $read)}; $row = $day + 1; $worksheet->write($row,$col,$temp); # No my $day = $day + 1; [download] Now `$worksheet` won't go out of scope after the if block, and the later code can still use it.	[reply] [d/l] [select]
Re^3: Issue with Spreadsheet::WriteExcel by mprogan365 (Initiate) on Aug 24, 2010 at 17:52 UTC
Problem solved! thanks a lot folks	[reply]
Re^2: Issue with Spreadsheet::WriteExcel by mprogan365 (Initiate) on Aug 24, 2010 at 17:23 UTC
Thanks for the advice. A couple of my variables were undefined because I declared them like so " my $row = ( );" which I guess PERL doesn't like. However, I still get the same error at the same line. So I tried to redefine "my $workbook = 0;" and "my $worksheet = 0;" since they were both set " = ( ); ". and it still doesn't work. Is there another way I should define "my $worksheet" (other than = 0) if it is doing this: ? < my $worksheet = $workbook->add_worksheet("data"); > and < $worksheet->write($row,$col,$temp); >	[reply]