• Owen (unregistered)

    it would've been kinda usefull would there be breakpoints on the //shenanigans

  • Jay (unregistered) in reply to T.C
    T.C:
    The TRWTF is likely to be that Lucy is seen by management and other staff as being 'too slow and pedantic' while the other developer 'quickly gets the job done'. By the time it comes round to figuring out why the exception handling mechanism isn't working as it should Lucy will probably be handed the job of fixing it and told she needs to work faster this time cause there are deadlines to meet!

    It reminds of a company I use to work for that practiced 'Agile' development. Basically what it meant was they did everything as quickly as possible. Design? No time for that! We have a (impossible) deadline to meet! Want to test that important piece of functionality? Testing takes to long! We have a (now even more impossible) deadline to meet!

    As one of those developers that actually cares about those pesky things such as bugs and maintainability I was soon approached by the boss and told I needed to 'learn how to code better'. To add insult to injury, the boss got a 'senior' developer to 'mentor' me in the error of my ways.

    Thankfully I resigned as soon as I could - the project in question was delivered 3 months late and had a huge bug count.

    Testing? It's much more cost-effective to just let your users do that after you deploy. If there are bugs in the code, the users will find them and let you know.

  • vereor (unregistered) in reply to OldHand
    OldHand:
    TK:
    ThePants999:
    it does leave me as the odd man out
    I think you're guaranteed to be the odd man out if your name is Lucy.
    Except that, in this context, "man" can refer to a male or female.

    Or a grammar nazi?

    (Really annoying that I can't cut&paste the CAPTCHA, I have to type the damn thing)

    Don't worry, there are only like 3 different captchas, pretty soon auto-complete will take good care of you.

  • Andrea Giammarchi (unregistered)

    damn it, I have to report these guys they forgot to write

    /* shenanigans */

    in every single empty catch!!!

    http://gist.github.com/200075

  • Kirby L. Wallace (unregistered) in reply to Crash Magnet

    Something horrible WILL happen if Foo = Null! ;-)

    Let that be a lesson to you. Never let your foo be null!

    ;-)

  • (cs)
    I learned how unimportant robust error handling was when a "senior developer" corrected code like that by simply removing the unhandled paths of execution!

    Then you learned wrong! Much better not to handle an exception at all than to silently suppress it, because then it will propagate upwards and be caught by an error-handling mechanism elsewhere instead of resulting an untraceable bug.

    The only mitigating circumstance for this is that we are told the code is for a GUI. You don't really want all your errors reaching the user. But still, best design would be to catch anyt hat haven't been caught elsewhere at the last possible moment, log the error, and present the user with a generic error message and some sensible choices.

  • Anonymous coward (unregistered)

    WTF², that doest not look like an error or exception condition.

  • a paratus (unregistered) in reply to Sellars
    Sellars:
    Wolfraider:
    ThePants999:
    it does leave me as the odd man out
    I think you're guaranteed to be the odd man out if your name is Lucy.

    It's because he is related to a boy named Sue

    Either that or a man named Jayne.

    (/me notices the Whedonites lurking)

    Shirley you must be joking.

  • Crono (unregistered)
    (...) it appears to only be a comment, but hopefully, one day I will gain that understanding... or find another job!

    Hi, welcome to Walmart!

  • chrisb (unregistered) in reply to campkev
    campkev:
    Zylon:
    You don't even need to read that code to tell it's going to be hideous-- Just looking at the brace style is evidence enough.
    Die. Or at least go find another line of work. In my experience, the only people who care about bracing style that much are shitty programmers.

    Oh snap!

    Tis true.

    Captcha = genitus /giggle

  • (cs)

    PrgmSHNANGNS:ClrHome:Disp "ERR:SHENANIGANS","1:QUIT","2:Goto ":Return:Disp "SECRET MESSAGE!","TI-BASIC 84+SE"

  • Grandpa's Long Drawers (unregistered)

    This just in:this is actually Lucy.

Leave a comment on “The Shenanigans Handler”

Log In or post as a guest

Replying to comment #:

Log Out

« Return to Article