Использование findViewById вместо View Binding или Data Binding. findViewById не является типобезопасным и приводит к избыточному коду.
kotlin
Предпочтительные варианты:
kotlin
kotlin
Также обратил бы внимание на:
- Отсутствие обработки возможных
null значений.
- Слишком большой размер одного класса или метода (нарушение принципа единственной ответственности).
- Использование "магических" строк или чисел вместо констант.
- Дублирование кода.
- Отсутствие тестов (юнит, интеграционные).
- Использование устаревших API (например, старые версии библиотек поддержки Android).
- Сложный для понимания и поддержки код без комментариев или с неинформативными комментариями.
- Неправильное управление жизненным циклом компонентов (например, утечки памяти).
Эти моменты часто указывают на потенциальные проблемы с надежностью, поддерживаемостью и производительностью приложения.