Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl Monk, Perl Meditation
 
PerlMonks  

Re^8: Convert undef to empty string in a hash

by Anonymous Monk
on Jan 16, 2015 at 11:23 UTC ( #1113490=note: print w/replies, xml ) Need Help??


in reply to Re^7: Convert undef to empty string in a hash
in thread Convert undef to empty string in a hash

Why must he ask that? Because it looks wrong. And is not accompanied by a comment warning that, though it looks wrong, it is intended.

Why does it look wrong? How does it look wrong?

Since you ask that question, I assume you haven't any experience maintaining large legacy code bases, with minimal test coverage, where the original authors have long since left the company.

Ugh, don't take this the wrong way, but that sounds like pure sundialsvc4ism

If code is that bad, how can you trust when they wrote  $_ //= '' for values %number_of_immortal_perl_monks they really meant to write that? After all it has no comment explaining even though it looks correct, we didn't make a mistake, we understood it for real to be correct, for really real this time

How can I learn when code "looks wrong"? I want to learn from you , can you teach me?

All three are equally clear at a glance (I glanced at them, equally clear, no headache)
Yet you did not notice the difference between "immoral" and "immortal"?

I did notice it, your first example was also a typo, and like I already said, perl will notice, perl is very good at catching typos

Replies are listed 'Best First'.
Re^9: Convert undef to empty string in a hash
by eyepopslikeamosquito (Bishop) on Jan 16, 2015 at 12:12 UTC

    Ugh, don't take this the wrong way, but that sounds like pure sundialsvc4ism
    Well–played(!?!) and good”—day to “you” SIR !¡!

    You have won “our→‘little’battle” by playing–the sundialsvc4–♠‘card’♠  a-K-a  “Reductio ad sundialsvc4um”™.   Frankly, my dear, I don’t give a damn!©     upvoted.  Fav'd.    “++”

      Ah, aaww...

      I feared that would backfire

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others cooling their heels in the Monastery: (9)
As of 2021-05-06 12:40 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Perl 7 will be out ...





    Results (74 votes). Check out past polls.

    Notices?