possible wrong if insetead of else