While that might behave properly, it doesn't express the intention of the code and would lead to bugs down the road. The only block of code precluded by the isCancelled check is here, but if someone were to write a block of code below the outer if-statements, those blocks would be skipped as well. As you write more and more code, you learn that its often better to write correct code than it is to write short code.