• ThelieOP
      link
      fedilink
      arrow-up
      13
      arrow-down
      1
      ·
      6 months ago

      Wow you sound so smart!

    • ThelieOP
      link
      fedilink
      arrow-up
      10
      ·
      6 months ago

      Inside me are two wolves I guess…

      • gravitas_deficiency
        link
        fedilink
        English
        arrow-up
        5
        ·
        6 months ago

        Chief O’Brien:

        Sorry about that transporter malfunction, sir! Won’t happen again!

  • AggressivelyPassive@feddit.de
    link
    fedilink
    arrow-up
    29
    ·
    6 months ago

    I have to say, I’m getting more and more frustrated by the bad code I have to write due to bad business circumstances.

    I want clean, readable code with proper documentation and at least a bit of internal consistency and not the shoehorned mess of hacks, todos and weird corner cases.

        • magic_lobster_party@kbin.run
          link
          fedilink
          arrow-up
          3
          ·
          6 months ago

          It’s mostly a joke, but often when I find todos they’re so old they’re no longer relevant.

          Of course you shouldn’t blindly remove todos.

      • frezik@midwest.social
        link
        fedilink
        arrow-up
        1
        ·
        edit-2
        6 months ago

        Don’t just put “TODO”. If they’re in the final pull request, they need to mention a ticket that’s intended to fix that TODO. If you/your team decides it’s not important, then remove it and close out the ticket. Either way, you’re required to do something with it.

  • anton@lemmy.blahaj.zone
    link
    fedilink
    arrow-up
    6
    ·
    6 months ago

    Instead of

    if let Some(a_) = a{
        ()
    } else if let Some(b_)=b{
        ()
    } else {
        dostuff 
    }
    

    you could just use

    if a.isNone()&&b.isNone(){
        dostuff
    }
    

    Also if you don’t use the value in a match just use _

      • ThelieOP
        link
        fedilink
        arrow-up
        1
        ·
        6 months ago

        I’m not sure how I would go about this in an elegant way since I’m matching the string I get from a message…

        • jimitsoni18@lemmy.zip
          link
          fedilink
          arrow-up
          1
          ·
          6 months ago

          If the message used enums for actions/procedures like SPAM_MEMES, then using enums would be more performant

          • ThelieOP
            link
            fedilink
            arrow-up
            2
            ·
            5 months ago

            I think you’d be happy to know that I’ve gone for a bit of an overkill and used Pest to parse the commands, which automagically gets me an enum to match against in this position.

            The sad part is, I haven’t gotten the Media upload to work, so the project is on ice for a little while…

    • ThelieOP
      link
      fedilink
      arrow-up
      2
      ·
      6 months ago

      That’s a good point, thanks. Maybe I’ll go without the if entirely, the (janky) code is still very much in flux ;)

  • 6mementomori@lemmy.world
    link
    fedilink
    arrow-up
    1
    ·
    6 months ago

    I don’t nearly know enough to understand this but is anyone willing to help me get the thing on the top :>

    • ThelieOP
      link
      fedilink
      arrow-up
      2
      ·
      5 months ago

      To first give you some context, the thing on the top is from The “Representable Functors” chapter of Category Theory for Programmers. So technically, you only need to read 230 Pages of a maths textbook to get it ;)

      But this isn’t exactly what you asked for, so I’ll try to help you get it as best I can with my limited understanding of the subject. First of all it would be helpful to know what your prior knowledge in Maths, especially Set theory, is?