• Ensign_Crab@lemmy.world
    link
    fedilink
    English
    arrow-up
    52
    ·
    edit-2
    4 hours ago

    especially your own code.

    “This is obvious” I said. “Surely I won’t need to comment this,” I said.

    • NotSteve_@lemmy.ca
      link
      fedilink
      arrow-up
      19
      ·
      4 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

    • hdsrob@lemmy.world
      link
      fedilink
      English
      arrow-up
      73
      ·
      6 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.

    • dan@upvote.au
      link
      fedilink
      arrow-up
      10
      ·
      4 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.

    • JATothrim_v2@programming.dev
      link
      fedilink
      arrow-up
      5
      ·
      edit-2
      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 has expired, and I see the same fault replicated everywhere he has been.

      • Feathercrown@lemmy.world
        link
        fedilink
        English
        arrow-up
        5
        ·
        4 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
            2
            ·
            4 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

  • WesternInfidels@feddit.online
    link
    fedilink
    English
    arrow-up
    42
    ·
    edit-2
    6 hours ago

    I go back and look at my old code and find it clear and beautiful, easy to understand, a pleasure to read. “Ah yes,” I’ll say to myself, “that approach was clever and elegant. Gosh, past me was pretty smart!”

    I like to appreciate it in this manner. Because that way, for a moment at least, I can forget about how it doesn’t actually work.

  • AeronMelon@lemmy.world
    link
    fedilink
    arrow-up
    19
    ·
    5 hours ago

    Electrical engineer: “what was that other guy thinking?”

    Software engineer: “What was I thinking?” (It’s code from last night)

    • rumba@lemmy.zip
      link
      fedilink
      English
      arrow-up
      5
      ·
      5 hours ago

      We went out for drinks one night after work. Upon stumbling back to the office, I remembered I had forgotten that I signed up to make a tool page to mange some data ingest. It was due first AM. I was three sheets to the wind. Fired up LAMP stack, took the samples and made an ingest function. Wired up a textbox, tested it and went home.

      Next AM, I turned it in, there was a minor bug. No problem, I’ll just find the issue and they’ll be good to go.

      cracked the scripts open…

      I could read it. I could see what a lot of it did. I could NOT figure out what some of it was there for. I spent 30m trying to figure out what I was doing. it was only a couple hundred lines. It wasn’t even a copy/paste job. Eventually I ran out of time and just leaned into phpdump, and breakpoints to find the exact error. One function hit a bonefied php bug that caused the debug to go silent. large swaths of the code were unreachable due to essentially a couple of typos. The only reason it worked as well as it did was because their sample data was as simple as imaginable. I put on some Nine Inch Nails and just remade it in about 30 minutes (10m before it was absolutely needed)

    • zqwzzle@lemmy.ca
      link
      fedilink
      English
      arrow-up
      2
      ·
      4 hours ago

      What was I thinking?-after every interruption that could have been an email

  • bedwyr@piefed.ca
    link
    fedilink
    English
    arrow-up
    5
    ·
    4 hours ago

    Electricians are a rather self-impressed bunch in my experience, like I would rather drink with a couple plumbers than electricians.

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

      Electricians think their way is the only way. Get three electricians together and you’ll get four ways of running the conduit, and a six hour argument. Electricians are constantly upset because “those bastards in HVAC put the ductwork in my way”. There are three types of screw an electrician will run into in the field, which is why the average electrician owns forty-seven screwdrivers.

      Plumbers only need to know three things:

      1. Poop rolls downhill
      2. Payday is Friday
      3. Don’t chew your fingernails

      Oh, and if you want to piss off a pipe fitter, call them a plumber

  • rumba@lemmy.zip
    link
    fedilink
    English
    arrow-up
    10
    ·
    5 hours ago

    I watched a team invent a new language to get around updating some eccentric code.

    They could have sat down and commented it and made their changes

    They could have refactored what was there.

    They could have scrapped it and wrote fresh

    Instead, they designed an entire natural language system so that non-programmers who were writing in XML could just write in English.

    They ended up making so many required keywords as helpers that the non-programmers kept using the old system because the XML was easier for them work with.

    Note: wasn’t my code, wasn’t my dept, when I heard the plan I went to check it out, the old system was functional but like C- work at best. At some point, they wrote a compiler for the new system.

    • applebusch@lemmy.blahaj.zone
      link
      fedilink
      English
      arrow-up
      4
      ·
      3 hours ago

      inventing a new language is almost never the right solution. there was a guy at my last job who tried to do this pretty much every time he ran into a problem with some shitty legacy software he had to work with. rather than take the time to fix it to do what he needed, he took ten times longer to slap another layer of custom bullshit on top of it. ultimately it came down to him being a really shitty engineer too afraid to change existing code, too lazy to do his due diligence, just clever enough to implement a shitty workaround, but not clever enough to realize how shitty it was. everything he made barely worked, was way overcomplicated, and no one else even wanted to try to learn his arcane bullshit syntax.

      • rumba@lemmy.zip
        link
        fedilink
        English
        arrow-up
        1
        ·
        1 hour ago

        We do all look at code, get immediately annoyed that it doesn’t just make sense. most of us at least have the wherewithal to stick with it and work on the engine as it sits :)

      • marcos@lemmy.world
        link
        fedilink
        arrow-up
        3
        ·
        4 hours ago

        In the 80s. If you look at what your examples have in common, you’ll notice you’ve heard it in the 80s.

        Too bad 80s nostalgia is all the hype nowadays, but at least this seems to be passing in other subcultures. I really hope it passes in software development too.

        • lime!@feddit.nu
          link
          fedilink
          arrow-up
          2
          ·
          2 hours ago

          only one of the ones i linked is from the 80s. i think you’ll find the majority are from the 50s.

  • lime!@feddit.nu
    link
    fedilink
    arrow-up
    10
    ·
    6 hours ago

    i did some fun metaprogramming today. i can practically hear my future self screaming.

  • CompactFlax@discuss.tchncs.de
    link
    fedilink
    English
    arrow-up
    6
    ·
    5 hours ago

    Two red flags for trades and swe and other work:

    1 - never saying that they would have done it differently if not outright roasting the work 2 - letting someone else take over their project, willingly, and not because they’re busy. Assuming it’s not a pile of shit they don’t want to do.

    • slazer2au@lemmy.worldOP
      link
      fedilink
      English
      arrow-up
      7
      ·
      5 hours ago

      Fun trick with Markdown. Put a double space at the end of a sentence so it formats the lines correctly.

      1. Double spaces
      2. allow for lists.
  • anon_8675309@lemmy.world
    link
    fedilink
    arrow-up
    4
    ·
    edit-2
    5 hours ago

    That’s the part I hate most about being a SWE. Just shut up and do your job. I’ve seen your code. You code like you read blogs for a living.

  • A_Union_of_Kobolds@lemmy.world
    link
    fedilink
    arrow-up
    4
    ·
    edit-2
    5 hours ago

    Am electrician can confirm

    If I dont do it the homeowner tries to and I have to awkwardly nod instead of fixing the problem and that just takes so much longer

    • DarkSirrush@piefed.ca
      link
      fedilink
      English
      arrow-up
      2
      ·
      1 hour ago

      I mean, I once had to explain to my electrician that the reason the dryer plug was screwed to the side of my panel box (6’ off the ground) was because the previous homeowner was the son of an electrician.