• @[email protected]
    link
    fedilink
    10
    edit-2
    2 months ago

    Hard disagree - that’s very helpful:

    // Calculates Personal Income Tax by formula from section 1.2.3 of tax code. Other taxes like VAT are not calculated.
    function calculateTax() { }
    
    • @Aux
      link
      -42 months ago

      If it calculates personal income tax, just call calculatePersonalIncomeTax.

        • @[email protected]
          link
          fedilink
          32 months ago

          I’m a new developer. Is that referring to page 123 of the in-house documentation? Version 12.3 of the code? I have no clue.

          You’d have to call it something like calculatePersonalIncomeTaxPerTaxCodeSection1_2_3, but I get exhausted just looking at that. There comes a point where the cognitive work of reading crazy long camel case names is more trouble than it’s worth.

          An explanation of what specification a function was written to implement is a perfectly appropriate comment. Could be improved by a direct link where possible. But it’s worth noting what that comment isn’t doing - specifying any implementation details. For that, I really can just read the code.

        • @Aux
          link
          12 months ago

          Yeah, why not?

        • @Aux
          link
          02 months ago

          Add a flag.