• Fades@lemmy.world
    link
    fedilink
    arrow-up
    179
    ·
    edit-2
    5 months ago

    WHY IS THE HEALTH INPUT PARAMETER A GODDAMN STRING???

    Why are you passing ‘%’ inside said goddamn string?!?! Not to mention the static reference instead of the actual instance.

    Shame on you

    • katy ✨
      link
      fedilink
      arrow-up
      35
      ·
      5 months ago

      OldMan.setHealth(“Robert’); DROP TABLE Students;–”)

    • blind3rdeye@lemm.ee
      link
      fedilink
      arrow-up
      29
      ·
      edit-2
      5 months ago

      I guess its just a reminder that getting a PhD is often more about dedication than it is about practical knowledge.

    • lud@lemm.ee
      link
      fedilink
      arrow-up
      18
      ·
      5 months ago

      It’s not his fault the world is made this way.

      He just has to follow it or else that man dies.

    • VeganPizza69 Ⓥ@lemmy.world
      link
      fedilink
      arrow-up
      11
      ·
      5 months ago

      The high level setter function should be made to handle both string and numeric values.

      If it contains “%” it’s a percentage value.

      If it’s a string without a “%” it’s an absolute value and needs to be normalized.

      If it’s a numeric value, it’s an absolute value.

      If it’s a numeric 100, it’s 100%.

      If it’s a subunitary numeric value, it’s a percentage.

    • Fat Tony@lemmy.world
      link
      fedilink
      arrow-up
      9
      ·
      edit-2
      5 months ago

      Honestly, if someone were to try to safe my life. And I find out he uses a string as a parameter to do so. Just let me die right there.

      • Fades@lemmy.world
        link
        fedilink
        arrow-up
        3
        ·
        5 months ago

        Yes absolutely, the parameter even if not in a strongly typed language should be a specific number and the unit should be implied. Overload the method to support different units if necessary or provide a unit as an additional parameter instead of forcing the method to parse the string for any unit type hints that may or may not be there