Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl-Sensitive Sunglasses
 
PerlMonks  

Comment on

( #3333=superdoc: print w/ replies, xml ) Need Help??
I think you already had plenty of good advices on this thread,
however I'd like to add a question and a comment :

Assuming that your goal was to make this guy code better :
Why didn't you try to talk to the coder first ?
You could have made him explain why he choosed to code this way.
You could even have made him understand why it's bad!

Don't get me wrong I'm not talking about morale/ethics/good...
I'm talking about efficiency.

When communicating with people you'll have to take as the fact that people listen/understand what you say
depending on the way/the moment/the context you say it.

Keeping that in mind, I'm pretty sure that he won't integrate your (good) advices during the planned meeting
which will now probably sound more like a court for him...
(He'll be the central point of a discussion about what he did wrong in front of judges(managers))
Usually you can't have efficient discussion with people on defensive...

It wouldn't have been the same if you had explained it to him, man to man, in front of a beer...
Just a "I've seen some problems but I talked to the coder and he will correct them soon" would have been fine with your manager, IMHO.

I'm not saying you did wrong ! I just show you another way to try, next time.

"Only Bad Coders Code Badly In Perl" (OBC2BIP)

In reply to Re: How do you critique another person's code? by arhuman
in thread How do you critique another person's code? by Rhose

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post; it's "PerlMonks-approved HTML":



  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • Outside of code tags, you may need to use entities for some characters:
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.
  • Log In?
    Username:
    Password:

    What's my password?
    Create A New User
    Chatterbox?
    and the web crawler heard nothing...

    How do I use this? | Other CB clients
    Other Users?
    Others chilling in the Monastery: (5)
    As of 2014-08-01 07:44 GMT
    Sections?
    Information?
    Find Nodes?
    Leftovers?
      Voting Booth?

      My favorite superfluous repetitious redundant duplicative phrase is:









      Results (257 votes), past polls