Mentoring
January 5, 2022 ยท View on GitHub
Reasonable solutions
def isogram?(string)
letters = string.downcase.scan(/[a-z]/)
letters.uniq.length == letters.length
end
Common suggestions
- Suggest using
scanrather thangsubas it removes the need to break the string into chars.
Talking points
- Comparing lengths (
letters.uniq.length == letters.length) is marginally quicker than comparing letters (letters.uniq == letters) but its more code. Which is better?
Changelog
- 2021 Nov - Markdown adjustment to style guide (sentences and links)