Think back to the last time you looked at an unfamiliar block of code. Did you immediately understand what it was doing? If not, you’re not alone – many software developers, including myself, find it challenging to grasp unfamiliar code quickly…
  • traches@sh.itjust.worksEnglish
    0·
    28 days ago

    Agreed. OP was doing well until they replaced the if statements with ‚function call || throw error’. That’s still an if statement, but obfuscated.

    • BrianTheeBiscuiteer@lemmy.world
      0·
      28 days ago

      Don’t mind the || but I do agree if you’re validating an input you’d best find all issues at once instead of “first rule wins”.

      • rooster_butt@lemm.ee
        0·
        27 days ago

        Short circuiting conditions is important. Mainly for things such as:

        if(Object != Null && Object.HasThing) …

        Without short circuit evaluation you end up with a null pointer exception.