fixed redundant if / else if