I've never heard of this convention of putting non-degenerate case first before. It's up to you, but it always makes the code more readable when the if clause is a positive condition.