• Admiral Patrick@dubvee.org
    link
    fedilink
    English
    arrow-up
    24
    arrow-down
    1
    ·
    1 month ago

    Just going to leave this horror here. It’s the post feed logic from Tesseract that determines what posts should be displayed or hidden.

  • Odinkirk@lemmygrad.ml
    link
    fedilink
    arrow-up
    5
    ·
    1 month ago

    That’s why you set the alternate/exit cases as individual if statements before whatever was going to be inside the original if block.

    To me too long to learn that.

    • Traister101
      link
      fedilink
      arrow-up
      6
      ·
      1 month ago

      My code got much more readable when I learned about early returns lol

      • lad@programming.dev
        link
        fedilink
        English
        arrow-up
        2
        ·
        1 month ago

        Early returns improve readability in that they make it simpler to read, but I also find them decreasing readability in that you may miss an early return and wonder why is execution not hitting the line you expect it to

        • Traister101
          link
          fedilink
          arrow-up
          3
          ·
          1 month ago

          I’d say in most cases that’s a sign something needs to be extracted into a separate function. Course sometimes code is just complicated and extracting only makes things harder to follow. Even then I’d much rather use early return than nested ifs as those are significantly harder for me to follow.