compiling and running but not correct