This is bad because its in a getter for a property, I think even if you call it once you should have the logic outside getter.