Доброе программирование, или сказки для программистов

03.03.18

Разработка - Рефакторинг и качество кода

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

"Добрым быть совсем, совсем не просто…"

 

Однажды в сети я встретил статистику (где именно не помню), что программисты примерно 80% времени читают код и 20% времени код пишут.

Сложно сказать, насколько это правда, но если говорить про 1С - то можно с уверенностью сказать, что читать приходится много и в основном чужого кода.

Через год порой уже и на свой код смотришь как в первый раз.

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

Можно сказать, что Доброе программирование - это написание кода с заботой о том, кто его будет читать.

Обычно читающий знает меньше автора, может быть, он хуже знает предметную область, а может быть, просто новый человек в проекте.

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

 

  1. Вовка в тридевятом царстве или двое из ларца (и так сойдет).

Как правило, именно тогда, когда код написан и заработал, программист лучше всего понимает, каким должен быть код, вот только на самом деле он совсем не такой. Иногда, чтобы код стал красивым, надо его переписать почти полностью, иногда достаточно сделать рефакторинг того, что есть. В любом случае именно сейчас есть понимание того, как надо, и надо именно так и сделать. Оставить "как есть" - это зло).

 

  1. Жили были КИТ и КОТ - было все наоборот.

Теперь самое время прочитать код и проверить, действительно ли этот код понятен читающему.

Все ли имена переменных и функций означают именно то, что имелось в виду.

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

 

  1. Как зовут трех поросят - знает каждый.

Чтобы читать много - надо читать быстро, а чтобы читать быстро, надо читать нужное и пропускать остальное.

Для этого надо, чтобы код был разделен на участки, назначение которых можно понять при беглом прочтении, а количество таких участков, отображаемых за один раз, должно быть "обозримым".

Тут нам помогут числа из сказок, 3 и 7.

3 - это минимальное количество элементов, которые воспринимаются как группа.

7 - это максимальное количество элементов, которое воспринимается при  беглом прочтении. Все, что больше - трудно.

 

  1. "Казнить нельзя помиловать" или полтора землекопа.

Остались приятные мелочи - описать ограничения, неочевидные соглашения и ошибки.

Тут надо подумать о том, что может сделать читающий, чтобы "улучшить" ваш код.

Зачем он стал его читать? Может, нет сообщения об ошибке,  а может, пользователю непонятно, как работает программа.

Может быть, код применяют не по назначению, а может, его решили "улучшить", но вы уже это пробовали, и получилось плохо.

Одним словом, следует добавить комментарии, написать справку, добавить сообщения об ошибках, которые позволят избежать лишнего копания кода.

Самое быстрое чтение - это когда читать не надо.

 

Ну вот, теперь код стал "добрее" и тот, кто станет его читать, скажет больше добрых слов)))

См. также

Результаты ревью кода 1500+ решений каталога Инфостарт: наиболее частые ошибки разработчиков в коде

Рефакторинг и качество кода Платформа 1С v8.3 Бесплатно (free)

Поделюсь своим опытом аудита кода авторских продуктов с Infostart.ru как одним из элементов применения DevOps-практик внутри Инфостарт. Будет настоящий код, боевые скриншоты, внутренние мемы от команды ИТ-лаборатории Инфостарт и прочее мясо – все, что любят разработчики.

10.04.2024    6150    artbear    80    

77

Ниндзя-код

Рефакторинг и качество кода Платформа 1С v8.3 Россия Бесплатно (free)

Предлагаю вашему вниманию советы мастеров древности. Программисты прошлого использовали их, чтобы заострить разум тех, кто после них будет поддерживать код. Гуру разработки при найме старательно ищут их применение в тестовых заданиях. Новички иногда используют их ещё лучше, чем матёрые ниндзя. Прочитайте их и решите, кто вы: ниндзя, новичок или, может быть, гуру? (Адаптация статьи "Ниндзя-код" из учебника JavaScript)

01.04.2024    2366    DrAku1a    15    

33

Практическое программирование: когда скорость важнее совершенства

Рефакторинг и качество кода Бесплатно (free)

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

01.04.2024    623    Prepod2003    6    

2

Когда понадобился новый оператор

Рефакторинг и качество кода Платформа 1С v8.3 Конфигурации 1cv8 Бесплатно (free)

Когда понадобился новый оператор, но его нет в синтакс-помощнике, что делать?

18.03.2024    1365    ZhokhovM    4    

4

Когда разработчик платформы не добавил проверку препроцессоров

Рефакторинг и качество кода Платформа 1С v8.3 Конфигурации 1cv8 Бесплатно (free)

Когда разработчик платформы решил пойти на кухню за кофе, а проверку препроцессоров не добавил, и вот тут-то и началось: "Что, опять все сломалось? Ну и кофе же я забыл сделать!".😅

18.03.2024    3035    ZhokhovM    4    

9

Реструктуризация - бесконечная история

Рефакторинг и качество кода Платформа 1С v8.3 Бесплатно (free)

При разработке программ требуемый функционал ставят на первое место, но есть еще и архитектура программы. На горизонте 5-10 лет она становится важнее функционала, который должен работать при масштабировании и росте данных. Реструктуризация 5 терабайтной базы 1С 8.2 в формат 1С 8.3, складывает весь пазл архитектурных просчетов, которые сделали ради функционала. Как это исправить? - для разработки правильной архитектуры, нужно всего лишь сместить фокус с функционала и подумать о «вечном».

29.09.2023    2098    1CUnlimited    15    

23
Комментарии
Подписаться на ответы Инфостарт бот Сортировка: Древо развёрнутое
Свернуть все
1. unichkin 1564 04.03.18 16:55 Сейчас в теме
"Делай добро и беги..."
корум; creatermc; tindir; brr; +4 Ответить
2. HAMMER_59 245 05.03.18 07:17 Сейчас в теме
Вспомнилась, серия из Масяни "В небе ангелочки летают по цветочкам ... а на земле полная ОПА, всё плохо сделано..."
Читаешь Инфостарт, прямо все за документирование, правильный код (перед работой, обычно, читаю статьи на Инфостарт), а потом начинается непосредственно работа, эти бесконечные вопросы:
"А почему здесь так, а тут этак?"
"А действительно почему?" а потому, что документация конфигураций у 1С-ников хромает на обе ноги, а документации кода, в принципе нет. В итоге сидишь в отладчике, и волосы где только не шевелятся, когда пытаешься разобраться в их волшебном коде.
Так что 1С пока злое зло :)

А как вспомнишь про партнерские конфигурации :), так уже думаешь, что типовые конфигурации не такое уж и зло.
nagaitseff; корум; Award; Tangram; Бубузяка; Nelli_A86; cassper; +7 Ответить
5. brr 183 05.03.18 09:39 Сейчас в теме
(2) Ахах, как это мне близко - 90% времени разбираешься как работает, 10% вставляешь правки.
3. dmt 66 05.03.18 08:25 Сейчас в теме
Пункт 1 радует. Особенно тогда, когда не очень-то и понятно насколько данная фича будет востребована.
4. spectre1978 60 05.03.18 08:42 Сейчас в теме
Гаишники в день экзамена: сегодня ездите по правилам. А завтра будете ездить, как все :)
creatermc; JohnConnor; brr; +3 Ответить
6. KAV2 156 07.03.18 09:56 Сейчас в теме
На самом деле очень редко когда код в мире 1С: Предприятие живет долго, его и так приходится постоянно переписывать ибо весь функционал так или иначе завязан на изменения законодательства. Таже история с покрытием тестами.
Да можно было бы делать по другому: весь функционал завязанный на законодательство выносить в отдельное, типовое решение и отдать его актуализацию всецело на откуп самой компании 1С, но это же не наш метод, мы будем все делать в одном месте, так чтобы постоянно нужно было все актуализировать.
7. DonAlPatino 176 07.03.18 10:59 Сейчас в теме
(6) "Да можно было бы делать по другому: весь функционал завязанный на законодательство выносить в отдельное, типовое решение и отдать его актуализацию всецело на откуп самой компании 1С". Уже в этом месте утопия - мне вот еще не попадался ни один работодатель, который был бы согласен хотя бы процентов на 60% с 1С на тему соответствия законодательству. А уж после перехода на ЗУП 3.1.4 меня вообще терзают смутные сомнения, что расхождение мнения 1С с мнением моей группы расчета зарплаты (поддерживаемое финдиром и главбухом) достигло всех 100 :-(
user811769; romankoav; +2 Ответить
8. Samarkan63 07.03.18 11:45 Сейчас в теме
Полезно, почему бы и нет
9. herfis 499 07.03.18 11:57 Сейчас в теме
Нужная штука. Побольше таких.
Неплохо бы еще че-нить в духе "Как хитрый фрилансер злобного франча перехитрил" и колыбельную для самых маленьких одинэсников.
user811769; nagaitseff; корум; +3 Ответить
Оставьте свое сообщение