What's so horrible about it? Seems a sensible way to clarify what variable mean and what expected values are...
Well, it's not
terrible. I retract that word, but it's unnecessary, and it pollutes the code. Clean code should read just as clear as prose. For example, if a variable measures how much Holly wants to be like Lena, then why not name that variable something like holly_imitates_lena? And have the variable name express what it means.
Anyways this just a little nitpick from my part. Miss Eva is doing a great job with the story so far, so she gets a free pass. And besides, ian_chad is a hilarious variable name, conventions be damned.