Работаем с коллегой в отделе занимающимся 1с.Оба пишем код, я хорошо понимаю администрирование 1с, а колгега хорошо понимает алгоритмы и пишет качественный код. С недавнего времени стал прикалывать что я пишу быдло код =(. Конечно когда нужно сделать еще вчера то нет времени писать качественно, делаешь просто что бы работало. Но сама мысль не дает теперь покая, как улучшить качество кода ? И есть оправдание быдло коду ?

(56) мне - нормально, недавно полез исправлять свою поделку на дельфях десятилетней давности, мне все понятно было...

(56) Не, десятилетный код - это уже не ужас, а комедия.

кстати, да
найди и попроси коллег твоего тру-кодера выслать его код, который он на старте писал...
поржёте вместе

заодно будут живые примеры гамнакода, с пояснениями автора

(57) Ну дело тут не в понятности. Просто студенческие поделки кишмя кишат велосипедами, проистекающими из незнания темы. После того, как становишься в теме, свои старые проекты прямо руки чешутся переписать правильно. Только лень мешает.

Ни с одним из вариантов не согласен

(3) Можно пользоваться деревенским уличным туалетом, биде и делать всё в резиновых одноразовых перчатках. И при этом писать красивый код.

Красивый код - это не только читаемый, но и эффективный.
А еще он должен выполнять свою функцию, а не гонять данные ради красоты.

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

Плюс красиво работу кода подавать, рисовать интерфейс, строки состояния, вопросы при нажатии на кнопки.

Ну и конечно не писать
"Если Квартира.Наименование = "23" тогда"
и прочие запросы в циклах.

(64)>>ограничивать ошибки пользователя.
верно. Программа должна быть ДУРАКОУСТОЙЧЕВАЯ ))

(63) ссать на пол и при этом писать красивый код - это оксюморон. разруха в голове не бывает частичной.

(0) Код не бывает красивый или некрасивый. С объективной точки зрения, всякий код - убожество (субъективно - код может казаться красивым его автору). Код бывает работающим и не работающим.

ТС про красоту ни слова не писал

(68) Можешь заменить "красивый" на "качественный" без потери смысла.

(67) есть (два стула, на одном ...) два куска кода.
делают одно и то же, делают это успешно.
первый написан одной процедурой в 1000 строк без комментариев с цикличными неявными чтениями бд по десятку раз за круг, другой оформлен модулем с несколькими точками входа в экспортных процедурах, параметризуются по потребности, повторяемые части кода вынесены в функции, сперва происходит сбор всех необходимых данных, отдельно  цикличная математика и блоки условий, постобработка.
какой сам напишешь, какой у коллеги прочитаешь?

Нет единого рецепта.

Меня, например дико раздражают чьи-то переменные названные примерно так :
Параметр
Параметр
Параметр
Видимо, на сях сидел человек когда-то.
Вроде ж ничего такого, но не нравится))

Еще переменные типо
КакаяТоНеведомаяХерня = ..

ну как так можно вообще..

упс..
форматировался текст в (71)
Имелось в виду параметр, начинающийся с одного, двух , трех или более символов нижнего подчеркивания.

еще из "не люблю".
Не люблю когда пишут, так, будто экономят "бумагу"
Если ... Тогда .. Иначе .. КонецЕсли  и это все на одной строке.

(66) Не на пол, а в специально проделанное для этого отверстие в полу.

(0) "Как писать качественный код?" - Качественный код писать очень просто: надо писать его качественно! :)

(70) И то и другое - убожество.

(73) так код быстрее выполняется. серьезно.
выполните код и сравните результаты
Для Сч=1 по 10000000 цикл
    Если 2=1 Тогда а=1 иначе а=2 КонецЕсли;
КонецЦикла;
Для Сч=1 по 10000000 цикл
    Если 2=1
        Тогда а=1
    Иначе
        а=2
    КонецЕсли;
КонецЦикла;

(77) толсто

(75) Ну или так: Чтобы писать качественный код, не нужно писать некачественный.

(79)"Делай хорошо - плохо само получится. " (цы)

(0)https://habrahabr.ru/post/328864/
Завтра тяпница, почитаешь

(5) +1
Еще попросите коллегу устраивать вам "ревизию кода" - это перед тем как пустить в продакшн - показываете ему код, он говорит, что надо переделать, как делать нельзя, что надо исправить и на что.
Я так своих коллег ростил, передавал им свой опыт, что-то узнавал от них.

Вредные советы: http://catalog.mista.ru/public/391560/

(77) а так еще быстрее будет:
Для Сч=1 по 10000000 цикл Если 2=1 Тогда а=1 иначе а=2 КонецЕсли;  КонецЦикла;

(78) а ты возьми и замерь. толсто ему...

Тема ни о чём

из того, что в этой теме прочитал, на вид правильнее всего (12)
Начни с малого, тяжело написать качественно большой объём кода. Напиши качественно маленькую функцию и качественный её вызов.

Этой фразе лет 40:
Каждый раз читая свой год 5-летней давности, понимаешь,  что его писал какой-то дебил.

(84) угу, а если пробелы почистить , то еще быстрее..
----
и что, часто надо организовывать цикл типо
Для Сч=1 по 10000000 цикл  ... ?

ИМХО, экономия на спичках. Не в том месте надо время экономить

(0) если фикси - забей... для фикси нет понятия быдло-код..
если франь - напрягись...

(0) Плохой код пишется долго.

Если ты пишешь код долго - ты написал его плохим, убей его.

(0) Хороший код - мертвый код

(90) Зря ты так. Плохой код портит карму. Оставишь его потомкам - они проклянут автора кода и нашлют на него проклятия "бедности и нищеты"...

(93) ты уверен что потомки поймут что такое 1с и раcшифруют код?

Давайте решать проблемы по мере их возникновения

+94 Наши потомки будут так же как мы )))
http://d.radikal.ru/d20/1712/cb/9ee1839d547e.jpg

Хочу задать вопрос по теме: как лучше называть куски кода, когда надо в свою обработку вставить некий кусок (например функцию из общего модуля), и внести в неё изменения? Единственное что приходит в голову: писать в комментарии "Заимствованный код", но уж больно сложное слово, хотелось бы попроще (без потери смысла).

(97) И кому будет полезен этот комментарий? Всем пофиг откуда ты её скомуниздил.

(81) почитал и поборол в себе Эффект Даннинга — Крюгера

(90) Имхо как раз наоборот, у фикси есть достаточно свободного времени, чтобы вылизать код, а франи оставляют первый работающий вариант, нет времени для этого. Мы вот юзаем франей, и от многих их спецов отказались по причине обнаруженного г-кода, и сейчас всё очень даже сносно.

(98) Хочу писать подобное для себя, чтобы через год самому было проще вспомнить. Или при обновлении базы - не бегать по всем обработкам вручную, а искать по тегу в комментарии.

(101) В данном случае достаточно коммента "Добавил Вася Пупкин, 22 декабря 2017г".

(101) Даже не знаю, что сложнее: соблюдать всю эту бюрократию или раз в год разобраться что отвалилось и почему. Причем с бюрократией есть один нюанс...
Все эти строительные леса в виде якорей в коде, 100% покрытия тестами и прочие перделки так и не образовали устойчивые стандарты, ибо снижают количество ошибок не до нуля, а на процент. И вопрос о том, что важнее - протерянные ресурсы на поддержку всего этого или гипотетические ошибки в продукте очень сильно зависит от области применения. Не все космические корабли программируют.

Ща набегут упоротые TDDшники.

(0) "что я пишу быдло код"

Забей. Придирки к качеству кода - это задротство, а твой коллега - задрот. "Качество кода" - понятие сугубо индивидуальное, у каждого оно свое. Если задача решена в срок, все работает правильно и не тормозит - значит все ок и код качественный. Ориентируйся на результат, а не на процесс.

(102) как найти все вытащенные из общих модулей процедуры и функции, если у них нет общего тега?

(103) Конечно это всё бюрократия, вот только когда переходишь с одной подверсии конфигурации на другую, вся эта бюрократия так выручает! Из моего недавнего прошлого УТ - 11.1->11.2-11.3. И из предстоящего - 11.4.

(19) Убогий.

(105) Общий тег "Вася Пупкин"

(104) Есть такое понятие как "сложность поддержки информационной системы". Чем хуже код - тем сложнее его поддерживать. Нужны дополнительные ресурсы. А это деньги. На плохом коде организация будет терять деньги.

(108) А по каким критериям отделять "плохой код" от "хорошего"?

(106) покажешь свой "гениальный" код?

(107) тег "Вася Пупкин" - общий для любого добавления, поэтому для моего случая не подходит.

(110) Нет :)

(109) Давай я покажу тебе код, а ты реши.  Итак, процедура формирования ТТН на вывоз (часть кода):

ОбластьСтрока.Параметры.ТоварНаименование            = "Боковина без лестницы";
                ОбластьСтрока.Параметры.КодПродукции                 = "";
                ОбластьСтрока.Параметры.НомерПрейскуранта            = СтрСчет.СчетНаОплату.ДоговорКонтрагента.Номер;
                ОбластьСтрока.Параметры.Артикул                      = "";
                ОбластьСтрока.Параметры.Количество                      = 2 * Коэффициент;
                ОбластьСтрока.Параметры.КоличествоМест                  = "";
                ОбластьСтрока.Параметры.БазоваяЕдиницаНаименование = Справочники.КлассификаторЕдиницИзмерения.НайтиПоНаименованию("шт");
                ОбластьСтрока.Параметры.ВидУпаковки                    = Справочники.КлассификаторЕдиницИзмерения.НайтиПоНаименованию("шт");
                ТабДокумент.Вывести(ОбластьСтрока);
                
                ОбластьСтрока.Параметры.ТоварНаименование            = "Боковина с лестницей";
                ОбластьСтрока.Параметры.КодПродукции                 = "";
                ОбластьСтрока.Параметры.НомерПрейскуранта            = СтрСчет.СчетНаОплату.ДоговорКонтрагента.Номер;
                ОбластьСтрока.Параметры.Артикул                      = "";
                ОбластьСтрока.Параметры.Количество                      = 2 * Коэффициент;
                ОбластьСтрока.Параметры.КоличествоМест                  = "";
                ОбластьСтрока.Параметры.БазоваяЕдиницаНаименование = Справочники.КлассификаторЕдиницИзмерения.НайтиПоНаименованию("шт");
                ОбластьСтрока.Параметры.ВидУпаковки                    = Справочники.КлассификаторЕдиницИзмерения.НайтиПоНаименованию("шт");
                ТабДокумент.Вывести(ОбластьСтрока);
                
                ОбластьСтрока.Параметры.ТоварНаименование            = "Перекладина";
                ОбластьСтрока.Параметры.КодПродукции                 = "";
                ОбластьСтрока.Параметры.НомерПрейскуранта            = СтрСчет.СчетНаОплату.ДоговорКонтрагента.Номер;
                ОбластьСтрока.Параметры.Артикул                      = "";
                ОбластьСтрока.Параметры.Количество                      = 2 * Коэффициент;
                ОбластьСтрока.Параметры.КоличествоМест                  = "";
                ОбластьСтрока.Параметры.БазоваяЕдиницаНаименование = Справочники.КлассификаторЕдиницИзмерения.НайтиПоНаименованию("шт");
                ОбластьСтрока.Параметры.ВидУпаковки                    = Справочники.КлассификаторЕдиницИзмерения.НайтиПоНаименованию("шт");
                ТабДокумент.Вывести(ОбластьСтрока);

еще пример:

Процедура МояМегаПроцедура()

Если УсловиеВыполяется Тогда
// тут 100500 строк кода

КонецЕсли;

КонецПроцедуры

(90) новый анекдот
"Франч без быдлокода" ...
в довесок к "Франч покрыл всю систему тестами"

(113) (114) И что я должен решить?

Для Сч=1 по 10000000 цикл
    Если 2=1
        Тогда а=1
    Иначе
        а=2
    КонецЕсли;
КонецЦикла;


так еще быстрее:

а=2;
Сч = 10000000;

(115)>>"Франч без быдлокода" ...
ты еще скажи франч- без студентов

(118) без дошика

(119) Про Дошик не надо, Дошик - это святое :)

(113) когда забанили в конструкторе запросов...

(40) Странный программист. Не настроены приоритеты по прерываниям. Более важные дела должны были прервать выполнение и выйти из цикла.

(122) у него обработчик прерывания был кривой
Звонок с телефона не предавался как параметр, потому что в другой комнате был )

(85) еще толще

троллей развелось тут я смотрю

В итоге решил скопированный и измененный код помечать вот так:

//Вытащено из ЧекККМ.НастроитьПраваДляПользователя() для того и сего

(124) http://forum.infostart.ru/forum34/topic33723/message372696/#message372696

(125) Я типовой вообще никогда не трогаю, делаю вот так:

//Nyarlathotep 22.12.2017 мояпочта@почтовыйсервер.ru {
//По задаче "Тут название задачи", автор задачи - "Тут автор задачи"
//Тут описание функционала добавляемого/изменяемого кода
//Закомментировал {
Тут закомментированный типовой код без изменений
// } Закомментировал
Тут мой добавленный/измененный код
// } Nyarlathotep 22.12.2017 мояпочта@почтовыйсервер.ru

(127) Таким образом откат к типовому функционалу решается комментированием/раскомментированием пары находящихся рядом секций кода, в случае чего.

Срача по поводу того, чей же код правильнее, я так понимаю, не будет? :)

(129) коментов в коде будет больше чем самого кода )

(129) тут даже срача франч/фикси нет... все выдохлись на Мисс Бюст

// { Автор Задача
//Закоментированный
//типовой
//код
//// Коммент из типового кода

Новый код
// коммент мой

// }

Смотря где..
В 1С половина написания кода это найти место внутри 10 вложенного проваливания в функции, далее там твой "качественный" код будет базироваться на уже написанном "высококачаственном" и понятном коде от 1С.

(113) недавно пришлось в ЗиКе семёрошном переделывать какое-то расчет, а там на 90% код копипастой написан, ну и как в (114) тоже. Сломал весь моск, пытаясь найти, куда бы впилится. Плюнул, занялся рефакторингом... ну небыстро да... Через какое-то время код сократился в 10 (десять) раз, и наконец-то стало всё понятно. Заодно работать стало быстрее раза в три.

Вот вам и "лишь бы работало".

(132) в комменте только номер задачи из системы регистрации задач/проблем/инцидентов

короче, вот так получается ))
https://www.youtube.com/watch?v=JsG94qngMNU

(126) И?
ты хочешь доказать присутствия мозга у людей, которые закладываются на особенности компилятора при разработке не embedded-систем?
особенно учитывая, что их нормо-час стоит часто несколько новых процессоров.
или нынче в технических вузах перестали преподавать экономику и заставлять обсчитывать экономическую эффективность своей работы?

(0) Стать профессионалом https://spark.ru/startup/makeright/blog/34614/prevraschenie-v-professionala-kratkij-obzor-knigi-stivena-pressfilda

(134) мне интересно, когда-нибудь разработчики БСП почувствуют, что ушли немного не туда и займутся наведением порядка?

(139) с чего вдруг? у них стандарты такие, всё абсолютно нормально.

(97) Пишу перед функцией комментарий:
// Сперто из общего модуля...

С красивым кодом все непросто. Заказчик просит сделать быстро и еще вчера, как обычно. Ты либо делаешь как быстрее, либо получаешь под зад. Как быстрее, это как привычнее, даже если не красиво. Кто заморачивается на красоту под зад получает чаще при равноценной конкуренции. А кто пишет некрасиво получает под зад, если наталкивается на любителя красоты со стороны заказчика, но это реже.

(141) Хорошо, но немного грубовато. Я плохой, на быструю руку накаляканный код помечаю как //Костыль и молюсь, чтобы это место никто не увидел. Пока красивую замену слову "Костыль" не нашёл.

(143) Можно писать :
// Взять из Документ.СчетФактураВыданный.МодульМенеджера

Читать ИТС. По быдлокоду просто конкретно спросить где сделал ошибку, и дальше много кода писать.

(0)А можно примеры?
Обычно быстрее написать сразу нормально.

Возврат;    
КонецПроцедуры

Первая мысль от бога(как бы быстро), вторая лукавого(быдлкод) ))

(134) "код сократился в 10 (десять) раз, и наконец-то стало всё понятно. Заодно работать стало быстрее раза в три."

Вот бы с типовыми также.

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

подпишусь

(128) такой коммент возможен, только если одна вставочка. А обычно как бывает

//Nyarlathotep 22.12.2017 мояпочта@почтовыйсервер.ru {

//По задаче "Тут название задачи", автор задачи - "Тут автор задачи"
//Тут описание функционала добавляемого/изменяемого кода

//Закомментировал {

Тут закомментированный типовой код без изменений

в нем вася Пупкин такой-то год

тут же здесь быол Гриша.

тут же Nyarlathotep 2013

опять типовой

опять Вася Пупкин.

и опять продолжение //Nyarlathotep 22.12.2017 мояпочта@почтовыйсервер.ru


И в итоге такой бред получается. Особенно если одну строчку комментировали сразу несколько человек. Ну или один человек, но с интервалом в 2 года.

(108) и что, много, например 1сников модифицирует исходный код? Именно модифицирует, а не лепит нашлепки?

Спроси любого "программиста" - адынесники "не программисты". Отсюда следует: пиши любой код - всем пофер.

У нас для типового кода используется безымянная вставка

//--> PRS

Тут изменения

//--< PRS

Остальная информация по изменениям - в шапке молуля по датам: кто, что, когда, зачем (по какой задаче).

Ну и плюс - хранилище, в котором более подробно все в комментариях расписано

Для пиления старой УПП - хватает за глаза

В (15) собственно все написано, но подобного варианта в голосовашке нет.