i
а вот бизнес-логика часто не может быть очевидной
Size: a a a
2020 March 28
R
это не bad practies - это просто может быть признаком плохого кода: сам код должен быть понятен без комментариев.
Кокой бы код не был хороший без комментариев он плохой по умолчания ))
MB
Кокой бы код не был хороший без комментариев он плохой по умолчания ))
Неверно, спецификация кода по бизнесу может лежать в сценариях bdd-тестов, в этом случае в коде комментарий излишни
R
Неверно, спецификация кода по бизнесу может лежать в сценариях bdd-тестов, в этом случае в коде комментарий излишни
Без разницы даже следуя практикам тестирования TDD, BDD комментарии должны быть.
Тесты имеют обыкновение теряться со временем.
Особенно в legacy коде.
И потом это превращается в геморрой.
Частенько бывает что нужно вносить экстренные правки в рабочий проект.
Минуя подобные практики, а тесты пишутся уже позже.
Со временем подобные темы накапливаются и опять же создают проблемы.
Тесты имеют обыкновение теряться со временем.
Особенно в legacy коде.
И потом это превращается в геморрой.
Частенько бывает что нужно вносить экстренные правки в рабочий проект.
Минуя подобные практики, а тесты пишутся уже позже.
Со временем подобные темы накапливаются и опять же создают проблемы.
MB
Без разницы даже следуя практикам тестирования TDD, BDD комментарии должны быть.
Тесты имеют обыкновение теряться со временем.
Особенно в legacy коде.
И потом это превращается в геморрой.
Частенько бывает что нужно вносить экстренные правки в рабочий проект.
Минуя подобные практики, а тесты пишутся уже позже.
Со временем подобные темы накапливаются и опять же создают проблемы.
Тесты имеют обыкновение теряться со временем.
Особенно в legacy коде.
И потом это превращается в геморрой.
Частенько бывает что нужно вносить экстренные правки в рабочий проект.
Минуя подобные практики, а тесты пишутся уже позже.
Со временем подобные темы накапливаются и опять же создают проблемы.
Если теряются тесты - пропадут и комментарий. CI хотя бы прогонит тесты и проверит их актуальность, а комментарий могут не меняться годами. Здесь уже ничего не спасёт, разве что сама дисциплина и ответственность руководителя. Комментарий не несут полезной нагрузки к коду, тем более в Python, не можешь прочитать python код и понять что он делает - пишешь плохой код, он и сам как человекоподобный dsl для работы с разными ресурсами
v
Комменты как минимум нужны чтобы пояснить почему сделано вот так а не иначе
R
Если теряются тесты - пропадут и комментарий. CI хотя бы прогонит тесты и проверит их актуальность, а комментарий могут не меняться годами. Здесь уже ничего не спасёт, разве что сама дисциплина и ответственность руководителя. Комментарий не несут полезной нагрузки к коду, тем более в Python, не можешь прочитать python код и понять что он делает - пишешь плохой код, он и сам как человекоподобный dsl для работы с разными ресурсами
Полностью не согласен!
Первое правила комментирования заложены в стандартах языка не просто так.
Непрерывная интеграция используется далеко не всеми.
И она проверит запустит только существующие тесты.
Если к коду нет теста все это поедет дальше спокойно.
Читать код для понимания которого надо лезть в тесты это жесть.
Не знаю откуда у вас такие практики но я бы не хотел с вами работать.
Вот это простите вообще бред сивой кобылы.
Программисты бывают разные плохие и хорошие.
И в большом проекте будет коктейль из разработчиков разного уровня.
«Комментарий не несут полезной нагрузки к коду, тем более в Python, не можешь прочитать python код и понять что он делает - пишешь плохой код, он и сам как человекоподобный dsl для работы с разными ресурсами»
Первое правила комментирования заложены в стандартах языка не просто так.
Непрерывная интеграция используется далеко не всеми.
И она проверит запустит только существующие тесты.
Если к коду нет теста все это поедет дальше спокойно.
Читать код для понимания которого надо лезть в тесты это жесть.
Не знаю откуда у вас такие практики но я бы не хотел с вами работать.
Вот это простите вообще бред сивой кобылы.
Программисты бывают разные плохие и хорошие.
И в большом проекте будет коктейль из разработчиков разного уровня.
«Комментарий не несут полезной нагрузки к коду, тем более в Python, не можешь прочитать python код и понять что он делает - пишешь плохой код, он и сам как человекоподобный dsl для работы с разными ресурсами»
MB
Комменты как минимум нужны чтобы пояснить почему сделано вот так а не иначе
Так вспомните дзен python)
R
Так вспомните дзен python)
Видимо вы его неправильно трактуете!
MB
Полностью не согласен!
Первое правила комментирования заложены в стандартах языка не просто так.
Непрерывная интеграция используется далеко не всеми.
И она проверит запустит только существующие тесты.
Если к коду нет теста все это поедет дальше спокойно.
Читать код для понимания которого надо лезть в тесты это жесть.
Не знаю откуда у вас такие практики но я бы не хотел с вами работать.
Вот это простите вообще бред сивой кобылы.
Программисты бывают разные плохие и хорошие.
И в большом проекте будет коктейль из разработчиков разного уровня.
«Комментарий не несут полезной нагрузки к коду, тем более в Python, не можешь прочитать python код и понять что он делает - пишешь плохой код, он и сам как человекоподобный dsl для работы с разными ресурсами»
Первое правила комментирования заложены в стандартах языка не просто так.
Непрерывная интеграция используется далеко не всеми.
И она проверит запустит только существующие тесты.
Если к коду нет теста все это поедет дальше спокойно.
Читать код для понимания которого надо лезть в тесты это жесть.
Не знаю откуда у вас такие практики но я бы не хотел с вами работать.
Вот это простите вообще бред сивой кобылы.
Программисты бывают разные плохие и хорошие.
И в большом проекте будет коктейль из разработчиков разного уровня.
«Комментарий не несут полезной нагрузки к коду, тем более в Python, не можешь прочитать python код и понять что он делает - пишешь плохой код, он и сам как человекоподобный dsl для работы с разными ресурсами»
Настроили pytest cov 100%, куча плагинов flake8, mypy, CI на все проекты. Понятия не имею как вы в больших проектах удерживаете чистоту и качество кода без этих обязательных правил)
MB
Видимо вы его неправильно трактуете!
На проблему одно простое решение, если приходится комментить - значит неочевидно
VS
Настроили pytest cov 100%, куча плагинов flake8, mypy, CI на все проекты. Понятия не имею как вы в больших проектах удерживаете чистоту и качество кода без этих обязательных правил)
Прям 100%?
R
На проблему одно простое решение, если приходится комментить - значит неочевидно
MB
R
Давайте закончим обсуждение.
Я уже вынес для себя все нужное!
Я уже вынес для себя все нужное!
VS
На проблему одно простое решение, если приходится комментить - значит неочевидно
мм, если, например, какая-то заковыристая алгоримтическая задача - тоже без комментариев оставить?
VS
Проект небольшой?
VS
Выглядит круто, да)
v
Без коммента будешь читать код и не понимать "а нахуя написан вот этот кусок кода, вроде и без него всё будет работать"