Встречается на собеседованиях • сегодня

Что бы не хотел видеть на код ревью

  1. Неочевидные имена переменных/функций
    Избегаем a, temp, handleClick. Лучше: userList, calculateTotal, onButtonClick.

  2. Дублирование кода
    Повторяющиеся блоки — повод для рефакторинга. Пример:

    javascript
    // Плохо
if (user.age > 18) validateUser();
if (product.price > 100) validateProduct();

// Лучше
const validate = (condition, validator) => condition && validator();
validate(user.age > 18, validateUser);

text

3. **Магические числа/строки**  
Вместо `if (status === 3)` используем константы: `STATUS_ACTIVE = 3`.

4. **Избыточные комментарии**  
Комментарии типа `// увеличиваем счетчик` — признак плохо читаемого кода. Лучше писать самодокументируемый код.

5. **Грязные побочные эффекты**  
Функции, которые меняют внешние переменные или делают неочевидные запросы. Чистые функции предпочтительнее.

6. **Игнорирование ошибок**  
Пустые `catch`-блоки или `console.error` без обработки. Ошибки должны быть обработаны или явно проброшены.

7. **Слишком большие компоненты/функции**  
Если функция/компонент делает больше одной вещи — пора разбивать.
Sophi
Софи собрала все вопросы. Тренируйся и получай
офферы быстрее!
Попробовать бесплатноArrow

Следующий вопрос

Это единственный вопрос по вашему фильтру

как отвечать на вопрос
пример собеседования
фреймворки на собеседовании
типичные вопросы junior
интервью вопросы и ответы