Какво смяташ за недостатък в кода, който искаш да замениш веднага?

Използването на findViewById вместо View Binding или Data Binding. findViewById не е типобезопасен и води до излишен код.

// Пример за използване на findViewById
val textView: TextView = findViewById(R.id.myTextView)
textView.text = "Hello"

Предпочитани варианти:

// Пример за използване на View Binding
// Активира се в build.gradle
// buildFeatures { viewBinding true }
val binding = ActivityMainBinding.inflate(layoutInflater)
setContentView(binding.root)
binding.myTextView.text = "Hello"

// Пример за използване на Data Binding
// Активира се в build.gradle
// buildFeatures { dataBinding true }
// Разметката е обгърната в <layout> таг
val binding = DataBindingUtil.setContentView<ActivityMainBinding>(this, R.layout.activity_main)
binding.myTextView.text = "Hello"

Освен това, трябва да се има предвид:

• Липса на обработка на възможни null стойности.
• Твърде голям размер на клас или метод (нарушение на принципа за единствена отговорност).
• Използване на "магически" низове или числа вместо константи.
• Дублиране на код.
• Липса на тестове (юнит, интеграционни).
• Използване на остарели API (например, стари версии на библиотеките за поддръжка на Android).
• Труден за разбиране и поддръжка код без коментари или с неинформативни коментари.
• Неправилно управление на жизнения цикъл на компонентите (например, течове на памет).

Тези моменти често сочат към потенциални проблеми с надеждността, поддържането и производителността на приложението.