• @ThelieOP
      link
      1213 days ago

      Wow you sound so smart!

    • @ThelieOP
      link
      1013 days ago

      Inside me are two wolves I guess…

      • @gravitas_deficiency
        link
        English
        513 days ago

        Chief O’Brien:

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

  • AggressivelyPassive
    link
    fedilink
    2813 days 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
      link
      fedilink
      1313 days ago

      todos

      I found a simple trick against this: just remove them. Accept it ain’t gonna happen man.

      • AggressivelyPassive
        link
        fedilink
        513 days ago

        Well, yes, but the underlying issues still persist, so it’s not exactly a sustainable strategy.

        • magic_lobster_party
          link
          fedilink
          313 days 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.

      • @[email protected]
        link
        fedilink
        1
        edit-2
        12 days 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.

  • @[email protected]
    link
    fedilink
    512 days 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
        111 days 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…

        • @[email protected]
          link
          fedilink
          13 days ago

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

    • @ThelieOP
      link
      211 days ago

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

  • hswolf
    link
    fedilink
    213 days ago

    is this a section of a discord api implementation?

    • @ThelieOP
      link
      English
      2
      edit-2
      12 days ago

      Almost, but not quite. It’s built against Presage

  • @fsxylo
    link
    English
    113 days ago

    deleted by creator