It's not a deal-breaker, but I normally try to avoid multiple exits from a function. (e.g. It makes it easier to top'n'tail.)