ooops wrong logic sign