I just completed the challenge and made a pull request. Looking at other pull requests, though, it looks like I was the only one who used one class, instead of splitting it up to multiple classes.
I thought the task was simple enough to write a simple class: 36 lines with four methods and each taking less than 5 lines. I initially had two classes, but it felt like making simple things more complicated.
Was my approach completely wrong? Can someone please review my code and leave some comments? What would be the sure sign (or smell) that the class should be broken up?
My pull request: https://github.com/thoughtbot/shakespeare_analyzer/pull/19/files
Thank you in advance!