I might be missing something stupidly as this simple code doesn't work as expected. What is wrong