• qqq
    link
    fedilink
    arrow-up
    10
    ·
    edit-2
    2 months ago

    Not really, but I’d probably try to organize those into sub structures where it made sense. A data structure holding the UI state and FFT data all flat is kinda messy imo since it becomes unclear what is actually required where.

    • grue
      link
      fedilink
      arrow-up
      24
      ·
      2 months ago

      You can tell there ought to be substructures because he used comments to label the different groups. That’s a code smell right there.

    • wise_pancake@lemmy.ca
      link
      fedilink
      arrow-up
      2
      ·
      2 months ago

      Yeah that’s what I usually do, as long as it’s passed by reference there isn’t anything too bad here.

    • DrunkEngineer
      link
      fedilink
      arrow-up
      2
      ·
      2 months ago

      It is spite coding though.

      He probably has a bunch of gotos too.