oops logic error