> - I would prefer to see the methods named "validateXXX()" or "verifyXXX()". >
+1 - I like these names better. > > - I would prefer to see them defined within the classes that call them, > rather than delegating to a separate CollectionArgChecks class. Even if it > introduces a bit of redundancy, I think the encapsulation offered by this > approach is preferable. > I personally don't actually mind the separate class in this case, but I think it should be package private.
