Essa discussão é antiga. No dojo-sea nós sempre a temos e ninguém sabe ao certo quando isso deve ocorrer, no dojo-brasilia também tivemos uma discussão mas ninguém quis apontar uma solução ou fixar uma opinião. Então farei isso.

Primeiro vamos entender de fato o que significa refatorar, a Wikipédia diz: “é o processo de modificar um sistema de software para melhorar a estrutura interna do código sem alterar seu comportamento externo.”
De posse desse conceito eu digo que um código “só” deve ser refatorado quando está em seu estado final. Considerando que um código em “estado final” possui vários testes unitários, nós podemos verificar o impacto da mudança facilmente, pois pequenos erros podem ser inseridos durante a refatoração. Denomino esse caso de “polimento de código”.

Agora vamos desconstruir o raciocínio. Suponha que uma classe X está em desenvolvimento, você está escrevendo os testes antes da implementação, quando eles quebram você implementa e roda de novo, e por ai vai… Porém a cada dois minutos você tem que escrever vários comentários sobre o que aquele trecho de código esta fazendo, percebeu? O.o? O funcionamento do código não está claro o suficiente, devemos refatorá-lo! Ou quando estamos escrevendo uma classe Y que tem trechos idênticos as classes X e Z, ora por que não separamos esses trechos? Costumo chamar esses casos de “necessidade de refactoring”.

Seguindo o raciocínio proposto chegamos a dois conceitos criados por mim, não riam, que são refatorações para polimento e refatorações por necessidade. Nós ainda temos refatorações que visam desempenho, testabilidade, etc. Essas ficam para uma próxima oportunidade.

Este post está longe de definir todos os casos em que uma refatoração é necessária, mas abre a discussão, e você o que acha sobre o assunto?