unnecessary else if