Ao escrever o código, procure métodos que contenham a palavra e . Por exemplo:
public void doThisAndThat() {
....
}
Alguém no StackOverflow apontou isso para mim uma vez (eu gostaria de ainda ter um link para a pergunta para dar-lhes crédito). Isso é cheiro de código. Diz que seu método faz 2 coisas. Isso é ruim porque
- Não é um bom design de API. Não está exatamente claro o que você está fazendo com esse método ou por que isso e aquilo estão conectados.
- Provavelmente, você está acoplando fortemente a funcionalidade ao método.
- Deve haver algum motivo pelo qual você deseja que o código seja executado em uma ordem definida. Para mim, isso me faz sentir que você está preocupado com o estado de seus objetos (o que é bom), mas está fazendo errado. Reserve um momento para considerar o design de suas classes e seu ciclo de vida.