• lukstru@feddit.de
    link
    fedilink
    arrow-up
    13
    arrow-down
    6
    ·
    1 year ago

    If your function is longer than 10 statements, parts can almost always be extracted into smaller parts. If named correctly, this improves readability significantly

    • words_number@programming.dev
      link
      fedilink
      arrow-up
      10
      arrow-down
      2
      ·
      1 year ago

      HELL NO! If you split that function into three, but these always have to be called in succession, you win nothing but make your code WAY harder to read/follow.

      • MindlessZ@lemm.ee
        link
        fedilink
        arrow-up
        2
        arrow-down
        1
        ·
        1 year ago

        Well named functions, called in succession increase readability, not decrease.

        • r1veRRR@feddit.de
          link
          fedilink
          arrow-up
          3
          arrow-down
          1
          ·
          1 year ago

          What do you gain from that approach, compared to comments, and appropriate whitespace? If you spread out your function over three, you now potentially have triple the moving parts. You have to manage in- and output, and you have to hope noone coming after you sees your subfunction, and assumes it’s there for using.

          • MindlessZ@lemm.ee
            link
            fedilink
            arrow-up
            1
            ·
            1 year ago

            Testability for one, but I would also argue that those functions are there for using. If some block of logic is sufficient to stand on its own, it should. I’m not saying do it arbitrarily, but it’s been my experience that small functions lead to more readable code and better testing. Most people write a 15 line function treating it as if it does a single thing when in reality it’s doing two or three discrete operations