29

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

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

3 ответа 3

38

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

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

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

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

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

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

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

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

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


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

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

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

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

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

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

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

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

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

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

20
  • Закрывать, поскольку эти темы (в широком виде) лично Вам не интересны?
    – avp
    21 ноя 2015 в 18:46
  • @avp вы мой ответ читали? Потому что на них нельзя дать однозначного ответа, и они зависят от мнения. 21 ноя 2015 в 18:47
  • 1
    Но это же на самом деле нормально (а то, что это тут теперь не приветствуется -- не нормально и д.б. изменено (поначалу хотя бы в рамках code-review))
    – avp
    21 ноя 2015 в 18:50
  • @avp что значит не приветствуется? Решили стать частью SO, надо соблюдать правила. 21 ноя 2015 в 18:52
  • Даже если они ошибочные?
    – avp
    21 ноя 2015 в 19:29
  • @avp вы меня троллите или как? К законам и конституции вы так же относитесь? Если вы хотите принимать участие в жизни сообщества, вы обязаны соблюдать правила. 21 ноя 2015 в 19:30
  • 1
    При всём уважении к @avp, поддерживаю автора ответа. Мне кажется, что на укрепление Ru SO нужно ещё пару лет. Дело в том, что до этого в рунете не было сообщества уровня SO. И русскоязычные пользователи не привыкли ещё к таким правилам. Если сейчас добавить сюда Code Review, то будет полный бардак и если среди этого хлама я лично не смогу быстро выбирать хорошие вопросы, на которые можно чётко ответить, то скоро всё это надоест. Таково моё субъективное мнение. Через пару лет можно будет и Ru Code Review открыть. Через пару лет, когда сообщество будет к этому готово.
    – LEQADA
    21 ноя 2015 в 19:54
  • 1
    @LEQADA, через пару лет такого порядка здесь останутся законопослушные, но вряд ли действительно творческие личности. Понимаете, ограничение свободы обычно выдавливает таких людей. Сразу скажу, наличие свободы обсуждения вовсе не означает, что я приветствую всякий хлам в виде и вопросов и ответов. / Что же касается демагогии о законах и конституции, то пожалуй не менее демагогически заявлю, что реально всегда следует действовать по ситуации и в соответствии со здравым смыслом.
    – avp
    21 ноя 2015 в 21:30
  • @avp проблема именно в том, что нельзя действовать по ситуации. Должны быть нормы. Мне самому интересны вопросы про оптимизацию алгоритмов и по архитектуре, но, не знаю. Сложно однозначно вынести вердикт =/ 21 ноя 2015 в 21:33
  • @Suvitruf: соблюдать правила — одно дело, но вы же не будете запрещать их обсуждать? Правила гибкие, а ru.SO находится в стадии Беты, и это означает что сейчас самое время их обсуждать и корректировать.
    – Nick Volynkin Мод
    21 ноя 2015 в 21:47
  • 1
    @avp: поддерживаю про главенство здравого смысла над правилами.
    – Nick Volynkin Мод
    21 ноя 2015 в 21:48
  • @avp тогда может обсудить это и переписать правила? 21 ноя 2015 в 21:49
  • Вот, Андрей (посмотрел Ваш профиль...), это здравая идея. Вам же самому, как разработчику будет значительно интересней обсудить свои мысли и сомнения с другими разработчиками, а не только отвечать на вопросы типа "почему у меня System.out.println не работает" -)
    – avp
    21 ноя 2015 в 21:54
  • @avp я уже давно на англоязычном SO сижу. Там подобные обсуждение как-то не особо приветствуются =/ 21 ноя 2015 в 21:56
  • А что нам мешает вести себя несколько иначе? Они на 2 порядка больше, да и в сети StackExchange есть сайт для разработчиков (а на русском ничего кроме ru.SO не предвидится). А так, я вижу мы могли бы начинать в вопросе, допустим, с тегом "инспекция-кода", а потом уходить в автоматом создаваемый чат и продолжать там (но, технология обсуждений, конечно, еще не выработана). Главное, чтобы такие вопросы не объявлялись сходу "требующими правки" по формальным признакам однозначности, субъективных мнений и т.п.
    – avp
    21 ноя 2015 в 22:07

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

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