Как сообщество относится к вопросам с предложением провести инспекцию (ревью) уже имеющегося кода?
На сайте имеется несколько подобных вопросов, и отношение к ним неоднозначное. Предлагаю определиться.
Так как у нас нет аналога Code Review Stack Exchange, и вряд ли он нам светит (у нас даже Super User и Server Fault считаются слитыми со Stack Overflow), то предлагаю разрешить подобные вопросы.
Рекомендации для запросов ревью должны быть те же, что и на Code Review SE. В частности, там онтопиком считается запрос предложений по работающему коду:
Критериями хороших вопросов являются:
Правда, учитывая, что сайт не специализирован на ревью, шансы, что пользователи предоставят вопросы в идеальном виде, практически нулевые. В связи с этим придётся вручную доводить их до нормального состояния.
printf("kekeke")
? PS: я больше за, чем против. Но нужно строго регламентировать и вынести на отдельную страницу в хэлпе.
[code-review]
), чтобы было ясно какие ответы приветствуются. Полезное сообщение в тему: What distinguishes Code Review from Stack Overflow?.
Сделал описание для метки инспекция-кода. Предлагаю дополнять и улучшать
Запросы на инспекцию (ревью) кода. Предлагайте только работающий код. Обязательно опишите реализуемый функционал и назначение кода. Используйте стандартный для языка стиль кодирования — имена функций и переменных, отступы, пробелы и прочее. Подробная инструкция есть в полном описании.
Запросы на инспекцию (ревью) кода.
Подходящие темы для таких запросов:
- Лучшие практики и использование паттернов
- Проблемы с уязвимостями
- Производительность
- Корректная работа в непредвиденных случаях
Критериями хороших вопросов являются:
- Наличие реального работающего кода
- Содержательный заголовок
- Отсутствие избыточных слов в заголовке
- Указание назначения кода
- Описание, откуда этот код взялся
Подробнее: Политика относительно вопросов с инспекцией кода (code review)
Проблема всех этих вопросов и просьб "посмотреть код", "улучшить код" в том, что:
Разрешить можно очень конкретные запросы. К примеру, по оптимизации алгоритма: ускорить, уменьшить потребление памяти и т.п. А вот вопросы вида "как мне улучшить архитектуру", "я начал изучать ООП, помогите улучшить иерархию классов" следуют закрывать.