trying to understand what is wrong?