this is weird. the else came as a surprise when reading the code
I always prefer to keep: 
if () {
} else {
}