29

Как сообщество относится к вопросам с предложением провести инспекцию (ревью) уже имеющегося кода?

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

3 ответа 3

38

Так как у нас нет аналога Code Review Stack Exchange, и вряд ли он нам светит (у нас даже Super User и Server Fault считаются слитыми со Stack Overflow), то предлагаю разрешить подобные вопросы.

Рекомендации для запросов ревью должны быть те же, что и на Code Review SE. В частности, там онтопиком считается запрос предложений по работающему коду:

  • Лучшие практики и использование паттернов
  • Проблемы с уязвимостями
  • Производительность
  • Корректная работа в непредвиденных случаях

Критериями хороших вопросов являются:

  • Наличие реального работающего кода (никаких гипотетических примеров)
  • Содержательный заголовок (никаких "как улучшить код")
  • Отсутствие избыточных слов в заголовке (никаких "не могли бы вы проверить")
  • Указание назначения кода (что код делает и зачем)
  • Описание, откуда этот код взялся (интервью, соревнование и др.)

Правда, учитывая, что сайт не специализирован на ревью, шансы, что пользователи предоставят вопросы в идеальном виде, практически нулевые. В связи с этим придётся вручную доводить их до нормального состояния.

7
  • 2
    Полностью за, только чрезмерные придирки к словам типа "как улучшить код" (конечно, я предполагаю, что кроме этой фразы в вопросе есть что-то содержательное, помимо самого кода) не нужны. В общем -- "зрим в корень".
    – avp
    Commented 14 нояб. 2015 в 13:57
  • 3
    Не боитесь вопросов "как мне улучшить по скорости printf("kekeke") ? PS: я больше за, чем против. Но нужно строго регламентировать и вынести на отдельную страницу в хэлпе.
    – Nofate Мод
    Commented 14 нояб. 2015 в 19:44
  • 1
    @Nofate да, отдельная страница нужна, чтобы можно было сразу отсылать к ней при необходимости. Commented 15 нояб. 2015 в 5:10
  • 1
    поддерживаю. Я бы добавил, что такие вопросы должны иметь обязательную метку (аналог [code-review]), чтобы было ясно какие ответы приветствуются. Полезное сообщение в тему: What distinguishes Code Review from Stack Overflow?.
    – jfs
    Commented 17 нояб. 2015 в 4:37
  • @jfs Есть инспекция-кода.
    – Kyubey
    Commented 17 нояб. 2015 в 9:42
  • 3
    В следствии большого положительного отклика среди участников сообщества, спешу предложить рассмотреть вопрос более конкретно. Для этого, было бы здорово, если бы вы, как автор инициативы, привели два изменения для справочного центра в базовом виде. 1) Изменения для страницы «На какие темы здесь можно задавать вопросы?». 2) Рыбу отдельной страницы, детально раскрывающей принципы вопросов такого рода. В будущем, возможно, понадобиться еще добавить дополнительную причину закрытия связанную с инспекцией кода.
    – Nicolas Chabanovsky СотрудникМод
    Commented 21 нояб. 2015 в 17:23
  • Этот вопрос проходит? ru.stackoverflow.com/questions/692902/…
    – FORTRAN
    Commented 18 июл. 2017 в 8:21
11

Сделал описание для метки . Предлагаю дополнять и улучшать

Запросы на инспекцию (ревью) кода. Предлагайте только работающий код. Обязательно опишите реализуемый функционал и назначение кода. Используйте стандартный для языка стиль кодирования — имена функций и переменных, отступы, пробелы и прочее. Подробная инструкция есть в полном описании.


Запросы на инспекцию (ревью) кода.

Подходящие темы для таких запросов:

  • Лучшие практики и использование паттернов
  • Проблемы с уязвимостями
  • Производительность
  • Корректная работа в непредвиденных случаях

Критериями хороших вопросов являются:

  • Наличие реального работающего кода
  • Содержательный заголовок
  • Отсутствие избыточных слов в заголовке
  • Указание назначения кода
  • Описание, откуда этот код взялся

Подробнее: Политика относительно вопросов с инспекцией кода (code review)

2
  • Нчитался этого описания создал вопрос и на те на прикрылы сам вопрос Этот вопрос проходит? ru.stackoverflow.com/questions/692902/…
    – FORTRAN
    Commented 18 июл. 2017 в 8:22
  • @FORTRAN у вас содержимое вопроса не в самом вопросе, а по ссылке на гитхабе. А должно быть в вопросе. И закрыто именно с такой формулировкой: «должны включать ... минимальный код ... прямо в вопросе»
    – Nick Volynkin Мод
    Commented 18 июл. 2017 в 8:43
-1

Проблема всех этих вопросов и просьб "посмотреть код", "улучшить код" в том, что:

  1. Они практически все opinion-based или too broad.
  2. Авторы вопросов особо не парятся по поводу локализации проблемы.
  3. Как правило, это портянки кода, из которых:
    1. Мало что понятно.
    2. Сложно ревьюить код, не представляй картины общей по проекту в целом.

Разрешить можно очень конкретные запросы. К примеру, по оптимизации алгоритма: ускорить, уменьшить потребление памяти и т.п. А вот вопросы вида "как мне улучшить архитектуру", "я начал изучать ООП, помогите улучшить иерархию классов" следуют закрывать.

1
  • Комментарии были перемещены в чат; пожалуйста, не продолжайте дискуссию здесь. Прежде чем разместить комментарий ниже этого, пожалуйста, ознакомьтесь с назначением комментариев. Комментарии, которые не запрашивают уточнения или не предлагают улучшения, скорее всего должны быть ответами или написаны в Чат Stack Overflow на русском. Комментарии, продолжающие дискуссию, могут быть удалены.
    – Grundy Мод
    Commented 4 дек. 2023 в 11:29

Войдите, чтобы ответить на этот вопрос.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками .