• anton@lemmy.blahaj.zone
    6·
    2 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 _

      • Thelie@sh.itjust.worksOP
        1·
        2 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
          1·
          2 months ago

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

          • Thelie@sh.itjust.worksOP
            2·
            1 month 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…

    • Thelie@sh.itjust.worksOP
      2·
      2 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 ;)