in reply to open failure code

What all these wise monks forgot to tell you is how they arrived at the answer :)
perl -MO=Deparse,-p
$down="false";
open (FH, $log) || $down = "true";
__END__
Can't modify logical or (||) in scalar assignment at - line 2, near ""
+true";"
- had compilation errors.
($down = 'false');
((open(FH, $log) || $down) = 'true');

[download]
B::Deparse

Replies are listed 'Best First'.
Re: Re: open failure code
by Grygonos (Chaplain) on Nov 27, 2003 at 00:09 UTC

    You could also use this type of contruct 

    open(FH,"<".$log) ? print "opened\n" : print "unable to open $!\n";

    [download]
    it's not any better at all just making you aware of another option.

    Grygonos
[reply]
[d/l]
      Though this works, the ternary operator, like the grep and map operators, should not really be use in void context. 
      if (open(FH,"<".$log) {
    print "opened\n";
}
else {
print "unable to open $!\n";
}

      [download]
      is preferred.


      Who is Kayser Söze?
[reply]
[d/l]
        Of course the same could be done just with proper logical short circuit operators. Prefer the low-precedence ones for this sort of thing: 

        open FH, "<", $log and print "Success\n" or print "Failure\n";

        [download]

        That avoids trinaries and if/else's.


        Dave


        "If I had my life to live over again, I'd be a plumber." -- Albert Einstein
[reply]
[d/l]

In Section Seekers of Perl Wisdom