Ce defect în cod consideri că ai dori să îl înlocuiești imediat?

Utilizarea findViewById în loc de View Binding sau Data Binding. findViewById nu este sigur din punct de vedere al tipului și duce la cod redundant.

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

Opțiuni preferate:

// Exemplu de utilizare View Binding
// Activat în build.gradle
// buildFeatures { viewBinding true }
val binding = ActivityMainBinding.inflate(layoutInflater)
setContentView(binding.root)
binding.myTextView.text = "Hello"

// Exemplu de utilizare Data Binding
// Activat în build.gradle
// buildFeatures { dataBinding true }
// Layout-ul este încapsulat în tag-ul <layout>
val binding = DataBindingUtil.setContentView<ActivityMainBinding>(this, R.layout.activity_main)
binding.myTextView.text = "Hello"

De asemenea, trebuie avut în vedere:

• Lipsa gestionării valorilor null posibile.
• Dimensiunea prea mare a unei clase sau metode (încălcarea principiului responsabilității unice).
• Utilizarea de stringuri sau numere "magice" în loc de constante.
• Duplicarea codului.
• Lipsa testelor (unitare, de integrare).
• Utilizarea API-urilor învechite (de exemplu, versiuni mai vechi ale bibliotecilor de suport Android).
• Cod dificil de înțeles și întreținut fără comentarii sau cu comentarii neinformative.
• Gestionarea incorectă a ciclului de viață al componentelor (de exemplu, scurgeri de memorie).

Acești factori indică adesea probleme potențiale de fiabilitate, întreținere și performanță ale aplicației.