Update isogram blurb#2659
Conversation
| @@ -1,4 +1,4 @@ | |||
| title = "Isogram" | |||
| blurb = "Determine if a word or phrase is an isogram." | |||
| blurb = "Determine whether a phrase has any repeated letters." | |||
There was a problem hiding this comment.
Isn't isogram more focused on not having any repeated letters? It's a subtle difference.
There was a problem hiding this comment.
It is. If you determine there aren’t repeated letters, it’s an isogram so a negative means an affirmative in practice. It’s a little clumsy so we can change it so an affirmative to the question here means it is an isogram.
There was a problem hiding this comment.
Negative logic is harder to process. Affirmative logic is simpler. While the function's output is the inverse of this, the same logic applies to either phrasing (modulo a not operator). The affirmative wording is simpler and blurbs are meant to be short and simple.
This blurb is the inverse of the exercise, but it's simpler and better conveys what this exercise is about. Is the improved simplicity worth the technical inaccuracy?
There was a problem hiding this comment.
"Determine whether a phrase is an isogram, a word with no repeated letters." is a bit longer but it lines up the affirmative response with what we're expecting later in the instructions and test suite.
There was a problem hiding this comment.
I like that as well, and would happily approve if it was changed to that.
4 participants
See https://forum.exercism.org/t/tweak-metadata-blurb-for-isogram/54253. The current blurb assumes the reader knows what an isogram is.