Программирование

Проведи code review

Проводит структурированное ревью кода: разделяет реальные дефекты, риски и улучшения, оценивает читаемость и поддерживаемость и предлагает точечные изменения вместо полного переписывания.

Автор: PromptingGo Средний Язык: ru Просмотры: 1 Копирования: 0 Есть настройка
Средняя оценка
/ 5
Голосов: 0
Войти, чтобы оценить

1 Готовый промпт

Выступи в роли опытного разработчика и проведи code review.

Язык или технология: C# / ASP.NET Core.
Контекст кода: MVC action, который открывает страницу промпта по id.

Код:
"""
public IActionResult Details(int id)
{
    var prompt = _context.Prompts.FirstOrDefault(x => x.Id == id);
    return View(prompt);
}
"""

Проведи ревью по следующим направлениям:

1. Корректность: найди потенциальные ошибки, неправильные предположения и крайние случаи.
2. Безопасность: отметь риски только там, где они действительно следуют из кода.
3. Производительность: укажи только практически значимые проблемы, а не микроптимизации без необходимости.
4. Читаемость и поддерживаемость: оцени имена, ответственность методов, повторения и сложность.
5. Соответствие типичным практикам указанной технологии.
6. Тестируемость: укажи сценарии, которые особенно важно покрыть тестами.

Раздели замечания на:
- Критично;
- Стоит исправить;
- Можно улучшить.

Для каждого существенного замечания объясни причину и предложи минимальное исправление. Не переписывай весь фрагмент только ради другого стиля. Если данных для уверенного вывода недостаточно, обозначь это как предположение.
Как использовать
Вставьте код и кратко опишите его назначение. Нейросеть разделит замечания по важности и предложит точечные исправления.

2 Настроить под себя

Заполните понятные поля ниже. Сервис автоматически соберёт новую версию промпта под вашу задачу.

Укажите стек, чтобы учитывать его практики.
Коротко объясните назначение фрагмента.
Для больших файлов лучше начать с наиболее важного фрагмента.

Поля со звёздочкой обязательны. Необязательные части исчезнут из результата, если оставить их пустыми.

3 Результат настройки

Заполните конструктор выше, и здесь появится готовый промпт.

4 Примеры результата

Текстовый пример

Критично:
Нет проверки результата `FirstOrDefault` на `null`. Для неизвестного `id` представление может получить отсутствующую модель.

Стоит исправить:
Если метод выполняет запрос к БД в асинхронном приложении, можно рассмотреть `FirstOrDefaultAsync`.

Можно улучшить:
Добавить тест на запрос несуществующего идентификатора и ожидаемый ответ 404.

5 Комментарии

0
Комментариев пока нет. Будьте первым, кто обсудит этот промпт.