Re: Printing to a File from Sub

Of course, if your code is OO you could also store the filehandle in the object to prevent the need for passing it around, even better create a method to retrieve the fh that created it on the first request. Perl filehandles when create as Corion exhibited can be passed around like any scalar, pretty much. Or you could even create a closure to handle all prints:

{
  my $fh;
  open($fh,'>',$file);
  sub print_to_fh {
    print $fh @_;
  }
}
[download]

As always, TIMTOWTDI :)

- Ant
- Some of my best work - (1 2 3)

Comment on Re: Printing to a File from Sub Download Code

Replies are listed 'Best First'.
Re^2: Printing to a File from Sub by PyrexKidd (Monk) on Aug 27, 2010 at 22:56 UTC
here is code I am trying to use: `#!/usr/bin/perl use strict; use warnings; open my $fhout, '>', "./test_out.file" \|\| die "$!"; &test_print($fhout); sub test_print{ my $fhout = @_; print {$fhout} "this is a test\n"; } close $fhout;` [download] and here is the error message I keep getting: `~$ perl test_write.pl Can't use string ("1") as a symbol ref while "strict refs" in use at t +est_write.pl line 11.` [download] What is going on here? am I just missing something? Thanks for the assist.	[reply] [d/l] [select]
Re^3: Printing to a File from Sub by choroba (Cardinal) on Aug 30, 2010 at 16:47 UTC
You should either use `open my $fhout, '>', "./test_out.file" or die "$!";` [download] or `open (my $fhout, '>', "./test_out.file") \|\| die "$!";` [download] . The code you posted is parsed as `open (my $fhout, '>', ("./test_out.file" \|\| die "$!"));` [download] which is definitely not what you want. To avoid the error you reported, though, use `my ($fhout) = @_;` [download] or `my $fhout = shift;` [download] Also note that the ampersand sign & is not needed when calling a sub (see perlsub).	[reply] [d/l] [select]
Re^3: Printing to a File from Sub by pemungkah (Priest) on Aug 29, 2010 at 00:50 UTC
Putting the filehandle in parens told Perl that it is the argument to `print`, and so you wanted to print it to STDOUT - which is not what you wanted. You wanted to say `print $fhout "this is a test\n";` [download] It should also be noted that the variable `$fhout` is actually still in scope in your subroutine, so you technically could just use it. It is, however, better practice to pass it in. Edit: choroba is correct as s/he notes below. I did mistake the curlies for parens.	[reply] [d/l]
Re^4: Printing to a File from Sub by choroba (Cardinal) on Aug 30, 2010 at 16:50 UTC
Not true. The problem is `$fhout` contains the number of elements in `@_`, not the filehandle. The syntax `print {$fh} "out"` is even recommended in PBP.	[reply] [d/l] [select]
Re^4: Printing to a File from Sub by suaveant (Parson) on Aug 30, 2010 at 19:33 UTC
Yes, those aren't parens, they are squiggle brackets, easy mistake to make with the right font and font size. - Ant - Some of my best work - (1 2 3)	[reply]