Yeah, I'd prefer the second one.
In that case, there's no actual reason to have an else statement. Sure it makes more logical sense to have an else included, but if you don't need it, why generate the extra code?
I don't know if you guys had thought about it but mutually exclusive events may be obvious to some, but the more complex the logic the harder it may be to see if a certain set of statesments are or are not mutually exclusive. So i was wondering if there is a guiding principle when it comes to writing readable code()?
Agreed with Galik.
Make your variable names clear and detailed as possible. Don't call a vector of strings holding student names 'x', refrain from unmanaged infinite loops, yada-yada. And comment the crap out of everything. That's what i do.