My version is the whole method - no need for early returns in this case. I realise they're useful in some cases, but should be avoided as they can cause unexpected behaviour (e.g. closing file handles, releasing resources, etc.)