Early returns are generally a bad idea.  Can you make this more explicit that this is an if/else?