Há algum tempo atrás falei sobre documentação de código. Acredito que devemos perseguir o objetivo de ter um código e um design tão bom e claro que qualquer documentação é desnecessária.
O Scott Bellaware fala um pouco sobre isto em seu post Soluble Code. Um ponto importante que ele lembra é que código "esperto" deve ser evitado em pró da legibilidade.
Gastar tempo descobrindo aquela maneira de transformar dez linhas de código em uma, praticamente inteligível, não é uma boa prática, pois deteriora a manutenibilidade do código.
Isto significa que devo abrir mão de performance em troca de legibilidade? Em muitos casos sim. Não que a performance deixe de ser uma preocupação. Muitas vezes em coisas mais simples é possível ganhar performance: chamando um comando somente quando necessário, analisando queries em banco, liberando recursos de memória o mais rápido possível.
A cada revisão do código pense em performance e em legibilidade, seu código ficar mais limpo, mais legível e menos defeituoso.