I know it should no longer come as any surprise to me, but I just found this method (and several others like it) lurking in a Data Access Object. Grrr…

Much of software design / architecture isn't technically difficult, it just requires a little thought - "where does this functionality belong?" - rather than forcing code into the first class you come across.

Did I mention how useful NDepend is in exposing idiocy like this? I think I did. And I really should get round to blogging about how handy CodeRush and Refactor Pro are for cleaning up such smells.