• Fades@lemmy.world
      link
      fedilink
      arrow-up
      15
      ·
      4 months ago

      Why are females typed differently than males instead of a base class human with a gender identity parameter? Why would human anything have a function called young?? What would that function even do???

      • purplemonkeymad@programming.dev
        link
        fedilink
        arrow-up
        4
        ·
        4 months ago

        HumanFemales and HumanM both inherit from the Ape base class, it’s from an older java code base. We tried to change it once but it turned out the person that had written had retired and any changes we made just broke stuff.

      • cheddar@programming.dev
        link
        fedilink
        arrow-up
        2
        ·
        4 months ago

        The young method returns a boolean parameter. Females have a different type for obvious biological discrepancies that require extra functionality.

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

          I can accept your second point, but in your PR I would absolutely request you to rename the method to isYoung, and then in making said comment I would then ask… what value isYoung providing, and where is the line between young and !young ultimately for trying to get the dev to reevaluate the design. It’s hyper specific in an obtuse manner and I think it hints at design flaws especially with the perspective of product evolution

      • qarbone@lemmy.world
        link
        fedilink
        English
        arrow-up
        2
        ·
        4 months ago

        Could be a subclass. However, it should just be an ‘is’ method which is passed the array of [young, pretty] as input