My rule of thumb is always "which version is easier to understand and verify for a code reviewer?" Sometimes doing DRY means you've nicely separated out and en

My rule of thumb is always "which version is easier to understand and verify for a code reviewer?" Sometimes doing DRY means you've nicely separated out and encapsulated the common code that can be checked just once in isolation. Sometimes doing DRY means you end up with a complex abstraction that is hard to understand on its own.