• beardedrhino
    link
    fedilink
    arrow-up
    8
    ·
    1 year ago

    What am I looking at here? It just always goes to the first case right? 🤔

    • fsniper@kbin.socialOP
      link
      fedilink
      arrow-up
      2
      arrow-down
      2
      ·
      edit-2
      1 year ago

      It’s not the real issue. I introduced that while anonymizing the data. It’s that a 3-5 liner code became a huge switch case by just incrementing the code and never thinking how it should be done. This was caused by like 15 engineers in time :)

      Something like below would be huge improvement:

      subs1 = ["cluster1", "cluster2"]
      if subs1.contains(clusterName) return subs1