• AP (unregistered)

    Frist ?? Frist

  • yeah (unregistered)

    trwtf seems a parent

  • my name is missing (unregistered)

    Most code reviews devolve into a null process and become more of a checkmark for management,

  • RLB (unregistered)

    Evidently, this manglement has successfully internalised the ISO-9001 message.

  • (nodebb)

    If null coalesce null to null enough times, it becomes a value. They should've added more coalescing.

  • Zach (unregistered)

    They were really desperate to get the parent ID

  • (nodebb)

    “Realistically, the root cause was a process which was more focused on having a process than doing a useful code review.”.

    Well, duh. That's what all the quality management of all the last decade is all about.

  • King (unregistered)

    What if ParentID is a property that executes code?

  • Mason Wheeler (unregistered)

    int ParentID { get { return Random.Next(1000); } }

  • Paul Neumann (unregistered)

    In the words of Maury Povich: You are NOT the father!

  • Rob M (unregistered) in reply to King

    Believe it or not, I've seen way too many examples of code that does just that at this place. I did check, and it is a property, but it only uses the auto getters and setters. For a moment, I thought that the getter was going to do some more coalescing into null or something.

  • Appalled (unregistered) in reply to akaSomeoneElse

    Yeah. It's called CMM, Capability Maturity Model. These guys are probably at CMM level 3 or so. They have IT processes in place. At levels 4 and 5 they'll actually start making them work correctly.

  • (nodebb) in reply to akaSomeoneElse

    Someone has been missing the point for the last decade. Cynicism is not the same as intelligence.

  • Ulysses (unregistered)

    I submitted a good WTF article ages ago, but flavorless crud like this continues to get posted. Sigh.

  • Jim Did It (unregistered)

    I used to work for a small aerospace company. My job was to write code for microcontrollers that were used in various instruments on aircraft, and then take that code through the FAA approval process. We spent months and a bucket full of money prepping everything for the FAA review. The FAA representatives were only intent on reviewing our PROCESS, not our code or verification results. The entire time we spent with their team, the most often asked question was about how we followed their guidelines, never once about whether our code actually worked or not.

  • (>'-')> (unregistered) in reply to King

    At that point, throw the whole language away.

  • Supersonic Tumbleweed (unregistered)

    TRWTF is the name: sortedParentChildItems

  • Supersonic Tumbleweed (unregistered)

    But down to bits this code was probably from before code review was in place, or from some troubled times. Hard to compare current state, with results of past state in an apples to apples way

  • RLB (unregistered) in reply to mozzis

    Neither is documentation and a full schedule of meetings.

  • (nodebb) in reply to RLB

    LOL. I work with a clown now that thinks going to meetings is all he has to do and the low quality software with no processes in sight will magically fix itself..

  • GLEN (unregistered)

    Shows the overall value of code review, the review of the review(this) i believe is incorrect.

  • (nodebb) in reply to Ulysses

    Why don't you post it in the Side Bar (i.e. what.thedailywtf.com )?

Leave a comment on “Without a Parent”

Log In or post as a guest

Replying to comment #:

Log Out

« Return to Article