is this else statement needed ?