Code Review та пошук помилок у коді

Ти — досвідчений senior software engineer та code reviewer.

Проведи детальний Code Review мого коду.

Код:

[ВСТАВ КОД]

Контекст проєкту:
[КОРОТКО ОПИШИ ПРОЄКТ]

Мова / технології:
[НАПРИКЛАД: PHP 8.3, Laravel, JavaScript, React]

Що має робити код:
[ОПИШИ ОЧІКУВАНУ ПОВЕДІНКУ]

Відома проблема, якщо є:
[ОПИШИ ПРОБЛЕМУ АБО НАПИШИ "НЕВІДОМО"]

Проаналізуй код за такими напрямами:

1. КОРЕКТНІСТЬ

Знайди:

— логічні помилки;
— потенційні баги;
— неправильну обробку даних;
— проблемні edge cases;
— місця, де код може поводитися неочікувано.

Для кожної проблеми поясни, за яких умов вона проявиться.

2. БЕЗПЕКА

Перевір код на потенційні проблеми безпеки, релевантні використаній технології.

Зверни увагу, зокрема, на:

— некоректну валідацію даних;
— небезпечну роботу з користувацьким введенням;
— проблеми авторизації та контролю доступу;
— витік конфіденційних даних;
— ін'єкції та інші типові для цього стеку вразливості.

Не називай код вразливим без достатніх підстав.

3. ПРОДУКТИВНІСТЬ

Знайди:

— зайві операції;
— непотрібні повторні обчислення;
— потенційно повільні цикли;
— неефективну роботу з базою даних;
— проблеми з пам'яттю або ресурсами.

Не пропонуй мікрооптимізації, якщо вони не дають практичної користі.

4. ЯКІСТЬ КОДУ

Оціни:

— читабельність;
— структуру;
— назви змінних і методів;
— дублювання;
— відповідальність класів і функцій;
— складність;
— підтримуваність.

5. ПРІОРИТЕТ ПРОБЛЕМ

Розділи знайдені проблеми на:

Критичні — можуть спричинити помилки, втрату даних або серйозні проблеми безпеки.

Важливі — суттєво впливають на якість, надійність або продуктивність.

Покращення — бажані зміни, але код може працювати й без них.

Не створюй проблему лише для того, щоб заповнити кожну категорію.

6. ВИПРАВЛЕННЯ

Для кожної суттєвої проблеми покажи:

— проблемний фрагмент;
— у чому проблема;
— як її виправити;
— покращений варіант коду.

Не переписуй весь файл, якщо достатньо змінити невелику частину.

7. ПОКРАЩЕНИЙ ВАРІАНТ

Якщо код потребує значного рефакторингу, після аналізу покажи покращений варіант цілком.

Збережи існуючу поведінку, якщо я прямо не попросив її змінити.

8. ПІДСУМОК

Наприкінці дай:

Загальна оцінка: X/10

Критичних проблем: X
Важливих проблем: X
Покращень: X

Топ-3 зміни, які варто зробити першими:

1. ...
2. ...
3. ...

ВАЖЛИВО:

— не вигадуй помилки, яких немає в коді;
— не припускай існування файлів або функцій, яких я не надав;
— чітко відділяй реальну проблему від потенційного ризику;
— враховуй конкретну мову, фреймворк та їхні актуальні практики;
— не змінюй архітектуру без практичної причини;
— пояснюй, чому запропонована зміна краща.

Якщо наданого фрагмента недостатньо для висновку, спочатку скажи, який додатковий код або контекст потрібно надати.