Found use of die. Use an exception instead at line 38, column 8. Exce +ption objects should be used instead of the standard Perl error mechanism. (Severit +y: 4)
The 'offending' code is:
$hECO = retrieve($ECO_path) or die "Couldn't open $ECO_path : $!"; $hNIC = retrieve($NIC_path) or die "Couldn't open $NIC_path : $!"; $hOpening = retrieve($Opening_path) or die "Couldn't open $Opening_path : $!";
I'm wondering if the following is equivalent?
eval { $hECO = retrieve($ECO_path); 1; } or do { print "Couldn't open $ECO_path : $@"; exit; } eval { $hNIC = retrieve($NIC_path); 1; } or do { print "Couldn't open $NIC_path : $@"; exit; } eval { $hOpening = retrieve($Opening_path); 1; } or do { print "Couldn't open $Opening_path : $@"; exit; }
I'm also wondering if it is safe? What alternatives are there that don't drag in a mountain of dependencies? I've looked at Try::Tiny and TryCatch. Error is pretty much deprecated. If I have to I'll leave the code as is— but as a sort of intellectual challenge, I'd like to see what can be done.
--hsm
"Never try to teach a pig to sing...it wastes your time and it annoys the pig."In reply to eval to replace die? by hsmyers
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |