Generally prefer to move the 'IsNullOrEmpty' check to the top as it's essentially an invariant for the method. It's good practice to have the invariants block as the first thing that appears in a method