17

Пример правки: https://ru.stackoverflow.com/review/suggested-edits/131976

Некто взялся и исправил форматирование кода:

введите сюда описание изображения

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

С другой стороны, как профессионал, я хорошо понимаю, что есть такая область, как стиль написания кода и лезть в чужой монастырь со своим уставом как-то непринято. Холивары годами идут о простых вопросах о отступах пробелами или табами! Тысячи копий сломано о вопросе открывать фигурную скобку на той же строки или открыть с новой! Вообще если задуматься, то новичку, который привык к какому-то стилю, наверное будет тяжело психологически вдруг увидеть, что его код заменили на какой-то чужой, который не факт, что работает, который нужно ещё проверить, что за ахалай махалай тут произошёл.

Вот как расценивать такие правки? Как улучшающие (акцептить) или ухудшающие (откатывать к исходной версии вопроса)?

Собственное мнение: правки улучшают вопрос, надо акцептить. Но есть сомнения, поэтому решил уточнить на мете.

По теме нашёл только крайне далёкий вопрос (тынц: правка 2 и фрагмент 16 (форматирование блока кода)), автор ответа тоже полагает правку улучшающей.

7
  • 6
    Конечно, улучшающие (если автор явно не возражает)
    – avp
    Commented 28 дек. 2016 в 22:12
  • 1
    @avp вот что-то я не вижу, чтобы в комментариях автора явно спросили. Утвердить правку и если не было возмущённых воплей, то значит "фух, прокатило без лишних уговоров"?
    – A K
    Commented 28 дек. 2016 в 22:14
  • Ну, под возражением я имел в виду, что автор должен явно (в тексте вопроса-ответа или в комментарии) заявить, что не дело менять его стиль.
    – avp
    Commented 28 дек. 2016 в 22:17
  • @avp а что Вы понимаете под стилем?
    – Bald
    Commented 29 дек. 2016 в 3:26
  • 5
    @avp я не совсем с вами согласен по поводу форматирования кода и разрешения автора, объясню почему: если автору удобно читать простыню кода то это его полное право, но если автор рассчитывает получить ответ то такие улучшающие правки я считаю необходимыми.
    – Bald
    Commented 29 дек. 2016 в 3:29
  • 2
    Не, некто взялся, а конкретно - я и взялся за правку этого примера кода )
    – Sublihim
    Commented 30 дек. 2016 в 9:41
  • @NickVolynkin так здесь же как раз стиль кода. То есть метка не к месту.
    – αλεχολυτ Мод
    Commented 26 апр. 2017 в 12:08

4 ответа 4

19

Надо понимать что код сюда выкладывается не для автора, а для остальных. Поэтому исправление оформления на общепринятое является однозначным благом. Автору неприятно когда его код правят? А должно быть неприятно такое выкладывать!

Но есть несколько исключений.

Во-первых, стилистическая правка не должна менять смысл кода. Особенно осторожным надо быть при редактировании вопросов вида "почему моя простыня кода без отступов не работает?!". Если риск что после простановки отступов простыня кода заработает - и никто не поймет в чем было дело.

Особая осторожность - в коде на Python, Haskell и Yaml, где отступы являются частью синтаксиса!

Во-вторых, стилистическая правка должна именно улучшать стиль - но не менять его под другой стайл-гайд. Не надо тут устраивать битв фанатов разных IDE.

В-третьих, никуда не девается принцип незначительных правок. Если во всем сообщении не хватает одного незначимого пробела - то вопрос о целесообразности его редактирования обсуждается в других местах.

12
  • Насчёт IDE: IntelliJ вроде бы уже умеет распознавать исходный стиль кода и подгоняет к нему изменения. Т.е. если скобки были египетскими, то такими и останутся. Аналогично таб, два или четыре пробела.
    – Nick Volynkin Мод
    Commented 29 дек. 2016 в 6:45
  • 2
    Если после изменения отступов всё заработало, то скорее всего вопрос закрывается с формулировкой "Ошибка была вызвана опечаткой" (исключая упомянутые языки с семантически значимыми отступами).
    – αλεχολυτ Мод
    Commented 29 дек. 2016 в 7:40
  • @alexolut это если заметят, что оно заработало. А если никто не поймет, что код стал рабочим? Commented 29 дек. 2016 в 8:57
  • Если кому-то вопрос будет интересен, он проверит его работоспособность в текущем состоянии. И когда всё заработает, напишет коммент вида "У меня всё работает", как-то так. Но вообще, я не помню случаев, чтобы подобные правки приводили к фиксу бага. У Вас есть такие на примете?
    – αλεχολυτ Мод
    Commented 29 дек. 2016 в 9:08
  • @alexolut вы часто проверяете работоспособность кусков кода с непонятным назначением?.. Commented 29 дек. 2016 в 9:22
  • @PavelMayorov если кусок кода нельзя проверить, я прошу ТС предоставить [mcve] или голосую за закрытие с аналогичной причиной.
    – αλεχολυτ Мод
    Commented 29 дек. 2016 в 9:30
  • 1
    @alexolut ну и какой смысл украшать кусок кода ежели вопрос - кандидат на закрытие? :) Commented 29 дек. 2016 в 9:40
  • 1
    @PavelMayorov так я и не говорил, что такие вопросы надо украшать. Мы обсуждаем последствия уже украшенного сообщения.
    – αλεχολυτ Мод
    Commented 29 дек. 2016 в 9:43
  • 1
    "Особая осторожность - в коде на Python и Haskell" Поэтому я не буду так форматировать код на этих языках )
    – Sublihim
    Commented 30 дек. 2016 в 9:40
  • Где можно почитать про принцип незначительных правок? Commented 4 янв. 2017 в 5:38
  • Ссылка на принцип незначительных правок нужна. ))
    – Nick Volynkin Мод
    Commented 26 апр. 2017 в 7:05
  • @NickVolynkin они слишком часто обсуждаются, а общего руководства нет. Добавил пока две ссылки с немного разными мнениями. Commented 26 апр. 2017 в 7:23
6

Раз уж существуют разные системы записи кода, то предпочитаю править код в сторону ближайшей системы. В коде на картинке автор оставил одну фигурную скобки в конце строки. Её перенёс бы в начало следующей, а отступы не трогал бы.

И в ответе стараюсь писать код в стиле автора, когда ответ — это небольшие дополнения к его коду. Если же мой код не похож на код автора или если проблема именно в стиле записи, то оформлю ответ по-своему, чтобы поделиться и оформлением кода тоже.

Правку стиля одобрю, если покажется, что:

  • у автора разнобой в стилях оформления;
  • автор скопировал чужой код ради решения задачи, а стиль ему не важен;
  • код переформатирован сниппетом, когда редактор менял что-то другое.

В правке по ссылке видно, что автор перемешал стили оформления.

1
  • @Bald Поправил текст, спасибо. Commented 29 дек. 2016 в 4:49
6

Очень похоже (и часто так бывает), что вместо кнопки форматирования кода, которая добавит по 4 пробела в начало каждой строки, просто руками сдвигают строки с меньшим отступом. В таком случае сбивается отступ одного уровня, а остальное вполне нормально отформатировано. В таком случае правки надо принимать.

Это похоже тот вариант, который на скриншоте, хотя там и другие правки есть. Но всё равно я бы утвердил.

1
  • 2
    Кстати, да, очень популярная ошибка.
    – Nick Volynkin Мод
    Commented 7 янв. 2017 в 5:26
4

Правки заключающиеся в расстановка отступов кода я считаю однозначным благом, так как читаемость кода только улучшается. В некоторых случаях(sql) я так же принудительно вставляю переносы строк, так как читать сложный sql запрос в одну строку мне кажется очень сложно.

Я с вами могу согласиться в плане того что не надо бездумно приводить код под себя: расстановка скобок, pascal/camel case - это то что может быть действительно непривычно автору, об этом лучше всего написать в комментарии с указанием ссылку на рекомендации, может быть автор просто не знает о принятых соглашения по написанию кода при использовании данного языка.

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

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