• NotSteve_@lemmy.ca
      link
      fedilink
      arrow-up
      15
      ·
      2 hours ago

      That exact chain of events happened to me at my last job and I audibly laughed realising it was my own code. To my own credit though, it was a file I had written four years prior which at that point was more than half my whole career in the past

    • dan@upvote.au
      link
      fedilink
      arrow-up
      7
      ·
      2 hours ago

      These days I see so much AI slop that my reaction when I see code I hand-wrote myself is “hey, that’s pretty good”.

      My team’s code is great, but we use a lot of shared code written by other teams, with varying levels of quality.

    • hdsrob@lemmy.world
      link
      fedilink
      English
      arrow-up
      57
      ·
      4 hours ago

      Been writing the same software for 20+ years now, don’t even need git blame to figure out what asshole wrote this shit.

    • JATothrim_v2@programming.dev
      link
      fedilink
      arrow-up
      4
      ·
      2 hours ago

      I have lately jokingly guessed when I see the particular style and confusion: it’s you isn’t it? And so far I have been right. The particular author’s magic as expired, and I see the same fault replicated everywhere he has been.

      • Feathercrown@lemmy.world
        link
        fedilink
        English
        arrow-up
        3
        ·
        2 hours ago

        One of my coworkers is fond of using ternary expressions instead of “if” blocks. Even ones without an “else”. So I see things like:

        condition ? someVar = "blah" : null;

        or

        condition ? doSomething() : null

        Which should both just use “if” statements. Or my favorite:

        condition ? someVar = "foo" : someVar = "bar"

        which should really be

        someVar = condition ? "foo" : "bar"

        • UUUuuuuuh, I am not a programmer (you’re going to say “thank god”), but…

          I sometimes even chain them. You can put yet another ternary operator in the else and keep going. You know, else-if.
          So anyway, I can get ternary operators spanning 2 - 3 lines.
          Oh, I also often have issues thinking of proper loops, so you’d see a few terribly used goto statements.

          Although I do remove ones that are obvious brain fart.
          For example, quite obvious

          void example(bool true_or_false){
              if(true_or_false){
                  //code if true
              }
              else{
                  //code if false
              }
              //other code
          }
          

          Well, I’ve already had my brain goof up even that once or twice. “How the fuck”, you’re asking?

          void example(bool true_or_false){
              if(true_or_false){
                  goto if_true;
              }
              //code if false
              goto end_false_if;
          if_true:
              //code if true
          end_false_if:
              //other code
          }
          

          The brain-fart if-else.

          • Feathercrown@lemmy.world
            link
            fedilink
            English
            arrow-up
            1
            ·
            2 hours ago

            I haven’t seen him do it lately in any code I’ve reviewed but I do change it whenever I see it if I’m doing work nearby lol