Что хотят услышать интервьюеры:

Да, был опыт ревью и анализа чужого кода: поискал бы не только синтаксические ошибки, но и архитектурные проблемы, потенциальные баги, дублирование и нарушение стиля. Важно показать, что проверка кода помогает улучшить читаемость, поддержку и стабильность приложения. Хороший ответ — это не просто «смотрел код», а понимание, что именно проверялось и какой был результат.

Определение:

Анализ и проверка чужого кода — это ревью исходников другого разработчика с целью найти ошибки, улучшить качество реализации и убедиться, что код соответствует требованиям проекта. В Flutter это обычно включает проверку корректности работы с Widget-деревом, состоянием, навигацией, асинхронностью, производительностью и стилем оформления кода.

Пример использования:

Например, при ревью экрана на Flutter можно заметить, что в build() создаётся слишком много логики, setState() вызывается после await без проверки mounted, а список строится без const-оптимизаций. После замечаний код можно упростить, вынести бизнес-логику отдельно и снизить риск багов.

class ProfilePage extends StatefulWidget {
  const ProfilePage({super.key});

  @override
  State<ProfilePage> createState() => _ProfilePageState();
}

class _ProfilePageState extends State<ProfilePage> {
  bool _loading = false;

  Future<void> _loadData() async {
    setState(() => _loading = true);

    try {
      await Future.delayed(const Duration(seconds: 1));

      if (!mounted) return;
      setState(() => _loading = false);
    } catch (_) {
      if (!mounted) return;
      setState(() => _loading = false);
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: _loading
            ? const CircularProgressIndicator()
            : ElevatedButton(
                onPressed: _loadData,
                child: const Text('Load'),
              ),
      ),
    );
  }
}

Пояснение кода:

В этом примере при анализе чужого кода обращают внимание на несколько вещей.
Сначала проверяется, что состояние _loading изменяется корректно до и после асинхронной операции. Затем важно, что после await есть проверка mounted, чтобы не вызвать setState() у уже удалённого виджета. Также видно, что код интерфейса простой, но в более сложном случае из build() стоит выносить логику отдельно, чтобы не перегружать UI-слой.

Ключевые моменты:

Проверка чужого кода — это не только поиск ошибок, но и оценка читаемости, архитектуры и поддержки.
В Flutter особенно важно смотреть на работу со state, async/await, mounted и производительностью Widget-дерева.
Хороший ревьюер указывает не только на проблему, но и предлагает конкретное улучшение.
Полезно проверять соответствие кодстайлу, наличие дублирования и потенциальных утечек логики в UI.
На собеседовании важно показать, что ревью помогает снижать число багов и ускоряет развитие проекта.

Имели ли вы опыт проведения анализа и проверки чужого кода?