Beefy Boxes and Bandwidth Generously Provided by pair Networks
Your skill will accomplish
what the force of many cannot
 
PerlMonks  

Re^2: Passing DATA filehandle to a module under strict

by blindluke (Monk)
on Jan 26, 2012 at 10:51 UTC ( #950070=note: print w/ replies, xml ) Need Help??


in reply to Re: Passing DATA filehandle to a module under strict
in thread Passing DATA filehandle to a module under strict

If the tool gets in your way of whatever you are doing, disable the tool for the time being.

I cannot agree with you on this one. I'd rather say, that if a tool, that is beneficial to the whole code, gets in the way of a small task, one should try to find out, whether there might be another way of doing this small task, without disabling the tool, and the benefits associated with its use.

regards,
Luke Jefferson


Comment on Re^2: Passing DATA filehandle to a module under strict
Re^3: Passing DATA filehandle to a module under strict
by ww (Bishop) on Jan 26, 2012 at 12:45 UTC
    and then, if you can't find "another way", disable the tool.
Re^3: Passing DATA filehandle to a module under strict
by JavaFan (Canon) on Jan 26, 2012 at 14:21 UTC
    I'd rather say, that if a tool, that is beneficial to the whole code, gets in the way of a small task, one should try to find out, whether there might be another way of doing this small task, without disabling the tool, and the benefits associated with its use.
    No, that doesn't make sense.

    You're going to spend more time (finding out whether there's a way of doing it while keeping your precious "strict" (or "warnings" or something else), for the sake of keeping your tool enabled, and likely to end up with pointless code -- after which chromatic starts nagging you about "synthetic code".

    If you're ever adding code just to silence a warning, you're doing it wrong. If you make code more complicated, just so you don't have to temporarely use "no strict", you're doing it wrong. If you're even going to spend time finding out whether you can do any of the above, you're not only doing it wrong, you're also wasting the money of whoever is paying you.

    # Bad code: $foo = ($bar // 0) + 4 if ($var1 // 0) == ($var2 // 0); # Better code: no warnings 'uninitialized'; $foo = $bar + 4 if $var1 == $var2;
    Feel free to draw some curly braces on your screen if one thinks this "better code" isn't part of some lexical block and is dumb enough to assume I advocate turning warnings off in the entire program.

      If you're even going to spend time finding out whether you can do any of the above, you're not only doing it wrong, you're also wasting the money of whoever is paying you.

      If you're going to be passing around bareword filehandles, you're supposed to use typeglobs or typeglob references -- its a lesson well worth the time learning, and if he just turned off strict, never would have learned it

        As I've told you (or one of your clones) before, in my first reply, I made it clear I was talking in general terms. But I guess actually reading and understanding a complete paragraph is a bit too difficult for some people.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://950070]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others avoiding work at the Monastery: (8)
As of 2014-10-22 10:53 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    For retirement, I am banking on:










    Results (115 votes), past polls