top 50 comments

sorted by: hot top controversial new old
[–] 75 points 2 months ago* (4 children)

It's not a glass, it's the

hydrationContainerManagerObjectFactory

Not to be confused with the

hydrationContainerManagerTypeFactory

Which is the template

  • source
  • hideshow 4 child comments
  • load more comments (1 reply)
    [–] 41 points 2 months ago* (3 children)

    That's not a good comment. What it is should be clear from the naming and context. A good comment explains the why when that is not obvious:

    // This glass is at 50% capacity because we don't want it to overflow if it rains
    
  • source
  • hideshow 3 child comments
  • [–] 38 points 2 months ago* (10 children)

    Unless it is in a vacuum, the glass is full.

    50% water, 50% gas mixture.

  • source
  • hideshow 10 child comments
  • [–] 26 points 2 months ago (1 child)

    Opportunist... "It's an empty glass. I drank it while you were arguing."

  • source
  • hideshow 1 child comment
  • [–] 23 points 2 months ago (4 children)

    //don't remove this comment or else

  • source
  • hideshow 4 child comments
  • [–] 22 points 2 months ago (3 children)

    Engineer: this glass is twice as big as it needs to be.

  • source
  • hideshow 3 child comments
  • load more comments (2 replies)
    [+] 20 points 2 months ago* (last edited 1 month ago) (2 children)
  • [–] 18 points 2 months ago

    Senior engineer: the glass is self documenting.

  • source
  • [–] 11 points 2 months ago* (2 children)
    // This is a Glass class
    class Glass {
         filled(halfFull,  halfEmpty) {
             this.halfFull;
             this.halfEmpty;
             }
    }
    
  • source
  • hideshow 2 child comments
  • [–] 11 points 2 months ago (10 children)

    The current legacy code I'm dealing with is full of these:

    // TODO: Class description here to satisfy linter

  • source
  • hideshow 10 child comments
  • [–] 7 points 2 months ago (8 children)
  • [–] 4 points 2 months ago (5 children)
    """
    only one line for your comment?
    fucking amateur
    this comment uses five lines
    """
    
  • source
  • parent
  • hideshow 5 child comments
  • [–] 5 points 2 months ago (4 children)
    /*
     * this five line comment
     * is formatted C plus plus
     * and is a haiku
     */
    
  • source
  • parent
  • hideshow 4 child comments
  • [–] 3 points 2 months ago (3 children)

    A haiku has to use elements of the natural world in a symbolic manner.

  • source
  • parent
  • hideshow 3 child comments
  • load more comments (1 reply)
  • load more comments (1 reply)
    [–] 10 points 2 months ago
    /**
     * @brief This is the coffee cup class
     * 
     * This class describes a cup or mug that neatly integrates a
     * container for vectors of cCoffee instances with a fully
     * functional handle to carry the cup or mug around. Used for 
     * pretty much everything.
     */
    class cTallGlass : public cGlass {
    
  • source
  • [–] 8 points 2 months ago* (1 child)

    AI loves inserting comments like this.

    Either that or 20-line comments documenting the bugs in the old code, what this code does differently, and the exact implementation.

    Comments describing the exact implementation become stale very quickly. Comments shouldn't explain what the code does (eg "add two to rowCount"); they should explain why it does it (eg "include header and footer row in count too"). Extra details are what the commit message is for, and history is what source control is for.

    I call it out in a lot of my code reviews ("code looks good but clean up the AI slop comments before landing") but a lot of other developers don't care as much about code and documentation quality.

  • source
  • hideshow 1 child comment
  • load more comments (1 reply)
    [–] 6 points 2 months ago

    IT support chatbot when the client asks how to get an empty/full glass to display: You're absolutely right! I couldn't have said it better myself: This is a glass. 😊🥛

  • source
  • [–] 6 points 2 months ago

    Engineer: The glass is too big!

  • source
  • [–] 5 points 2 months ago*
    Glass _glass = new Glass(); //make variable glass of type glass and make a new glass
    ///<captainusefuldoc>Gets the glass</captainusefuldoc>
    Glass getGlass() => _glass; //get the glass in _glass
    
  • source
  • [–] 5 points 2 months ago* (last edited 2 months ago)

    Commented out glass is certainly a problem (if it's not only a comment) as the instance of object is created elsewhere but there's some more problems in the code:

    -Software tester removes the glass but the water still stands there in the shape of the glass.

    -QA tries to pour the water in to the glass but water doesn't register and the glass doesn't fill.

    -Performance testing notes that rendering the glass needs some optimization since the glass doesn't fill over 50% before being out of memory.

    -Junior programmer can't get the glass to run since the liquids library wasn't locked down and got an incompatible update.

    -Automation testing results in to a crash with a screenshot and logs that point out that glass frontend use the old version of the glass backend API leading the glass to freeze.

    -External testing points out that there's a bug in the glass but they can't identify it.

  • source
  • [–] 5 points 2 months ago
    // This might be a glass
    Glass* glass = dynamic_cast<Glass*>(container);
    
  • source
  • [–] 5 points 2 months ago (3 children)

    //This a glass // To get the quantity and type of content use
    // glass->qty : float
    // glass->content : glass_content

  • source
  • hideshow 3 child comments
  • [–] 5 points 2 months ago

    You guys are getting code comments???

  • source
  • [–] 3 points 2 months ago

    Marketing: the glass is perfectly dimensioned to transport that amount of water without spilling anything! Buy yours now for a discount! Terms and conditions may apply.

  • source
  • load more comments
    view more: next ›