Code smells e refatoração

Cheiros mais comuns

• Método longo e classe grande.
• Lista longa de parâmetros → agrupe num objeto.
• Obsessão por primitivos: CPF, dinheiro e e-mail como String/Double → crie tipos de valor.
• Inveja de funcionalidade: método que usa mais dados de outra classe que da própria.
• Cirurgia com espingarda: uma mudança exige editar dez arquivos.
• Switch repetido sobre o mesmo tipo → polimorfismo.

Refatorações de base

Do catálogo de Fowler: Extrair função, Renomear, Introduzir objeto de parâmetro, Substituir condicional por polimorfismo, Mover função. As IDEs (IntelliJ, Android Studio, VS Code) fazem várias com segurança.

Regra do escoteiro

Deixe o código um pouco melhor do que encontrou — em commits separados das mudanças de comportamento, para a revisão ficar fácil.

fun transferir(deCpf: String, paraCpf: String, valor: Double, moeda: String, agendada: Boolean, data: String?) {
  if (deCpf.length != 11) throw IllegalArgumentException()
  if (valor <= 0) throw IllegalArgumentException()
  // ...
}

@JvmInline value class Cpf(val digitos: String) {
  init { require(digitos.length == 11 && digitos.all(Char::isDigit)) { "CPF inválido" } }
}
data class Dinheiro(val centavos: Long, val moeda: Currency) {
  init { require(centavos > 0) { "valor deve ser positivo" } }
}
data class Transferencia(val de: Cpf, val para: Cpf, val valor: Dinheiro, val agendadaPara: LocalDate? = null)

fun transferir(t: Transferencia) { /* a validação já aconteceu nos tipos */ }