-
Verifique novamente sua motivação. Se você acha que o código deve ser alterado, você deve ser capaz de articular alguma razão porque você acha que deve ser alterado. E essa razão deveria ser mais concreta do que "eu teria feito diferente" ou "é feio". Se você não pode apontar para algum benefício que vem de sua mudança proposta, então não há muito sentido em gastar tempo (dinheiro também) em mudá-lo.
-
Cada linha de código no projeto é uma linha que deve ser mantida. O código deve ser o tempo necessário para realizar o trabalho e ser facilmente compreendido, e não mais. Se você pode encurtar o código sem sacrificar a clareza, é bom. Se você puder fazer isso enquanto aumenta a clareza, isso é muito melhor.
-
O código é como o concreto: é mais difícil mudar depois de ficar parado por um tempo. Sugira suas alterações mais cedo, se puder, para que o custo e o risco de alterações sejam minimizados.
-
Toda mudança custa dinheiro. Reescrever o código que funciona e provavelmente não precisa ser alterado pode ser um esforço desperdiçado. Concentre sua atenção nas seções que estão mais sujeitas a alterações ou que são mais importantes para o projeto.
-
A forma segue a função e, às vezes, vice-versa. Se o código estiver confuso, há uma strong probabilidade de que ele também contenha erros. Procure esses erros e critique a funcionalidade defeituosa, em vez do recurso estético do código. Sugira melhorias que tornem o código mais eficiente e facilite a verificação do código.
-
Diferencie design e implementação. Uma classe importante com uma interface de baixa qualidade pode se espalhar através de um projeto como o câncer. Isso não apenas diminuirá a qualidade do restante do projeto, mas também aumentará a dificuldade de reparar o dano. Por outro lado, uma classe com uma interface bem projetada, mas com uma implementação ruim, não deve ser um grande problema. Você sempre pode reimplementar a classe para melhor desempenho ou confiabilidade. Ou, se funcionar corretamente e for rápido o suficiente, você pode deixá-lo em paz e se sentir seguro sabendo que seu lixo está bem encapsulado.
Para resumir todos os pontos acima: Certifique-se de que as alterações propostas adicionam valor.