When good developers are afraid of refactoring and adding new classes is something to be feared, you need to rethink your architecture.

In fact, if there's ever that "dark corner" of code that no-one likes to work with, you've got to fix it.

It's like continuous deployment. We do it often because it's hard and having to deploy regularly forces us to make it easier.

When your senior says he may as well stops working as I'm always refactoring his code...

Same sentence says I copy what you've done in other places so I don't see why it isn't good enough. By copy he leaves redundant code in there too.

Am I a being a douche is he just being over the top?

- He writes code and expects it to live for a long time.
- I write code and will go home and refactor my own code.