Что ты считаешь плохим в коде и что бы ты хотел сразу заменить?

Глобальные переменные (val, var на уровне объекта, к которым много прямого доступа).
Использование lateinit var без особой нужды (предпочтительнее nullable val или инициализация).
Длинные, сложные функции (более 10-15 строк бизнес-логики).
Отсутствие или недостаточность тестов (юнит-тесты, инструментальные).
Дублирование кода (copy-paste).
Жесткая связанность компонентов (Class A напрямую создает/использует Class B вместо использования DI или интерфейсов).
Недостаточные комментарии для неочевидных частей кода.
Отсутствие обработки ошибок или использование пустых catch блоков.
Зависимость от конкретных реализаций вместо абстракций (например, прямое использование Context в ViewModel).
Неоптимальное использование коллекций (например, frequent list manipulations instead of sets/maps).
Игнорирование рекомендаций линтера (Checkstyle, Detekt).

Я бы сразу заменил:

Использование findViewById на View Binding или Data Binding.
Прямое использование AsyncTask на Coroutines, RxJava или WorkManager.
Ручное управление жизненным циклом (например, добавление/удаление слушателей в onResume/onPause) на Ktx-extensions или Lifecycle-aware компоненты.
Использование старых подходов к навигации на Navigation Component.
Использование Bundle для передачи сложных объектов между активностями/фрагментами на Safe Args, Parcelize или сериализацию.

kotlin

Глобальные переменные (val, var на уровне объекта, к которым много прямого доступа).
Использование lateinit var без особой нужды (предпочтительнее nullable val или инициализация).
Длинные, сложные функции (более 10-15 строк бизнес-логики).
Отсутствие или недостаточность тестов (юнит-тесты, инструментальные).
Дублирование кода (copy-paste).
Жесткая связанность компонентов (Class A напрямую создает/использует Class B вместо использования DI или интерфейсов).
Недостаточные комментарии для неочевидных частей кода.
Отсутствие обработки ошибок или использование пустых catch блоков.
Зависимость от конкретных реализаций вместо абстракций (например, прямое использование Context в ViewModel).
Неоптимальное использование коллекций (например, frequent list manipulations instead of sets/maps).
Игнорирование рекомендаций линтера (Checkstyle, Detekt).

Я бы сразу заменил:

Использование findViewById на View Binding или Data Binding.
Прямое использование AsyncTask на Coroutines, RxJava или WorkManager.
Ручное управление жизненным циклом (например, добавление/удаление слушателей в onResume/onPause) на Ktx-extensions или Lifecycle-aware компоненты.
Использование старых подходов к навигации на Navigation Component.
Использование Bundle для передачи сложных объектов между активностями/фрагментами на Safe Args, Parcelize или сериализацию.

kotlin