Comment by antonvs

20 days ago

> scream for a case/switch-statement

Maybe I’m missing some context, but all that actually should be needed in the top-level else block is ‘gradient[idx]’. Pretty much anything else is going to be longer, harder to read, and less efficient.

7 comments

antonvs

Reply
e12e  20 days ago

True, with early return - there's no need to actually nest with else.

Logically this still would be a case/switch though...

  • vidarh  20 days ago

    The point was that logically it would be an array lookup by index.

    There's no need for any conditional construct here whatsoever.

    You'll note it has already constructed a string in the right order to do that, but then copped out with the if-else.

    • e12e  20 days ago

      True enough. On that note, I had a look at the language reference - there's arrays - but also this:

          (char_at s index)        # Get ASCII value at index (0-based)
    (string_from_char code)  # Create string from ASCII value

      So, you can pluck a character... From an UTF-8 string? What if the rendering used multibyte characters?

      4 replies →