O que consideras uma falha no código que gostarias de substituir imediatamente?

A utilização de findViewById em vez de View Binding ou Data Binding. findViewById não é seguro em termos de tipo e leva a código redundante.

// Exemplo de uso de findViewById
val textView: TextView = findViewById(R.id.myTextView)
textView.text = "Hello"

Opções preferidas:

// Exemplo de uso de View Binding
// Ativado no build.gradle
// buildFeatures { viewBinding true }
val binding = ActivityMainBinding.inflate(layoutInflater)
setContentView(binding.root)
binding.myTextView.text = "Hello"

// Exemplo de uso de Data Binding
// Ativado no build.gradle
// buildFeatures { dataBinding true }
// O layout é envolvido na tag <layout>
val binding = DataBindingUtil.setContentView<ActivityMainBinding>(this, R.layout.activity_main)
binding.myTextView.text = "Hello"

Também é importante observar:

• A ausência de tratamento de valores null possíveis.
• O tamanho excessivo de uma classe ou método (violação do princípio da responsabilidade única).
• O uso de strings ou números "mágicos" em vez de constantes.
• A duplicação de código.
• A ausência de testes (unitários, de integração).
• O uso de APIs obsoletas (por exemplo, versões antigas das bibliotecas de suporte do Android).
• Código difícil de entender e manter sem comentários ou com comentários pouco informativos.
• Gestão incorreta do ciclo de vida dos componentes (por exemplo, vazamentos de memória).

Estes pontos frequentemente indicam problemas potenciais de confiabilidade, manutenção e desempenho do aplicativo.