Topic: describing a character's looks
I wonder what the group thinks about the following character description:
He also thought his cadet uniform made him look more attractive to girls. In reality, he didn’t need it. He was slim and handsome, with his mother’s auburn hair and emerald-green eyes.
Generally, the advice I've received in the past is not to "tell" a person's looks, but to weave it into a story, possibly even separating the attributes as you go along so as to roll it out slowly. Personally, if I'm going to read about a character's looks, I'd rather it be done in one place, so I can picture it in my mind and then move on.
What about the above? I think I've set it up well with the first two sentences before a simple "tell". Yes?