Пусть программирует Серёжа
https://habr.com/ru/post/670700/

У каждого клиента – свои предпочтения. Не только в выборе автомобиля, блюда на обед или корпоративной информационной системы. Клиенты любят выбирать программистов.

Ну, что программисты разные – ежу понятно. Считается, что клиенты предпочитают профессионалов. Мы тоже так думали, и искренне стремились сделать каждого своего программиста этим самым профессионалом.

Однако, несколько клиентов, ставя нам задачи, упорно твердили: пусть программирует Серёжа. Хотя Серёжа – лютейший говнокодер, объект всеобщей жалости и главный поставщик материалов для конференций на тему «Как не надо программировать».

(482) "Заставить код работать недостаточно. Работоспособный код часто неряшлив. Программисты, которые заставляют свой код работать и на этом считают свою задачу выполненной, ведут себя непрофессионально" (с) Роберт Мартин

(483) Заказчику нужен работспособный код и больше ничего лишнего. А заказчик всегда прав.

(484) да ладно? Заказчику также нужно, чтобы этот работоспособный код в будущем можно было сопровождать, развивать

(485) Это тебе кажется что его можно развивать, а как реальная задача придет, то окажется то, что ты наворотил никому не нужно и них... не развиваетмо

нельзя просто взять и написать хороший код.
хороший код - это результат десятков и сотен часов рефакторинга

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

обывателю (мариванне) пох, на запросах или в цЫкле или еще какая магия внутри


вот, например, хороший код (вычисления там всяких мат.функций)

Книга знаний: Математические вычисления в 1С

мариванне (зокащеку) то что с этого? Сирожа пусть страдает за денежку, развивает

(488) обывателю становится не пофиг, когда Сережа сваливает и не берет трубки, а новые Вася, Петя, Коля говорят, что все фигня, только заново переписывать)

(489) а новые Вася, Петя, Коля говорят, что все фигня, только заново переписывать)


А потом приходят еще новые проги - говорят , что все говно - и переписывают уже за Васей Колей Петей

(489) не братие трубок - это всегда риск работы с фришниками. но зато цена меньше чем у франча и качество всегда одинаковое

(489) ты не поверишь, дописали коллективом погромистов УТ10.3 за 5 лет хороший код

а тут бац, и от УТ10.3 стало плохо пахнуть. похоже сдохла

надо теперь хороший код, только на УТ11.4 или на ERP2.5.8

(490) обычно никто не переписывает, а делают залепуху ибо на переписывание денег нет

(489) >новые Вася, Петя, Коля говорят, что все фигня

патамушта Вася, Петя, Коля не читатели, а писатели

нет времени думать, работу работать нужно

и чтобы мариванна не офигивала от ошибок - в Попытку Исключение обернуть и норм

(494) "патамушта Вася, Петя, Коля не читатели, а писатели " - потому, что никто не читатели, так как код написан пяткой и его разбор и рефакторинг будет стоить еще дороже, чем переписать с чистого листа.

(494) (495) Да я про то и говорю, что все буду кричать , что у других говнокод, у них только хороший)))) А заказчик выберет того кто делает , быстро и работоспособно, то что нужно бизнесу. Заказчику весь этот ор про говнокод не интересен. Главное чтобы работало, быстро и без ошибок

(494) жиза...

(486) >>окажется то, что ты наворотил никому не нужно и них... не развиваетмо

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

(496) "все буду кричать , что у других говнокод" - не правда, когда код написан более менее вменяемо, а не портянкой на 100500 страниц в одной процедуре, то такой код можно и быстро доработать, особенно, когда название функций/процедур и переменных сами себя документируют.

(496) >>Заказчику весь этот ор про говнокод не интересен. Главное чтобы работало, быстро и без ошибок

А платить тройную стоимость владения за говнокод ему интересно?

(498) так нелья просто взять и написать нормальный код.
Только в N-ой итерации

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

(501) можно, если писать с умом, соблюдая ряд принципов

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

(500) А кто сказал что он платит тройную стоимость? Может в результате твоей переписке кода, еще больше будет говнокод. И будет шестерная стоимость ? он готов за это платить ?

только не говори что такого не бывает ))

(505) как правило платят, ибо как хрюши не понимают в апельсинах. а аудит кода со стороны денег тоже наверн стоит. и кто бодаться будет с исполнителем - "всё херня. переписывайте нафиг"?

(504) Именно про это я и писал

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

(505) платит. Может и тройную стоимость заплатить, и даже десятерную. Говнокод, он замедляет работу каждого следующего разработчика, да и автора говнокода тоже. Вместо того, чтобы потратить 3 часа на задачу, с говнокодом потратишь все 10 часов. Ибо дофига времени уйдёт просто чтобы разобраться, как же эта наркомания работает. Когда разберешься, будешь долго и мучительно вносить изменения

(494) А что, правда попытка/исключение универсальный быстрый способ убрать ошибку. Часто практикую)

(511) это признак г@внокода :)

в данной транзакции уже происходили ошибки :)

(512) +100500 :))))

(512) Помогает реально запустить систему, а разбор причин уже на досуге или никогда)))

Главное чтобы в архитектуре все было продумано - справочники/документы/регистры. А всякие отчетики и формочки для юзеров вполне могут содержать (и чаще всего содержат) говнокод, ибо они быстрее всего меняются и быстрее устаревают. А для бизнеса важно решить проблему сегодня, так как сегодняшние деньги дороже завтрашних

(510) Говнокод, он замедляет работу каждого следующего разработчика, да и автора говнокода тоже. Вместо того, чтобы потратить 3 часа на задачу, с говнокодом потратишь все 10 часов. Ибо дофига времени уйдёт просто чтобы разобраться, как же эта наркомания работает. Когда разберешься, будешь долго и мучительно вносить изменения


то , что ты сейчас описал очень напоминает типовые конфы от 1с последние особенно ЗУП и ERP

И почему 1с переписывает ERP, от релиза к релизу кардинально, вплоть до архитектуры ? видимо в 1с тоже сплошные говнокодеры, которые правят код друг друга

(516) ЗУП - прекрасный образец узаконенной наркомании.

(518) полностью согласен))

(514) помогает предусмотреть не предусмотренные ошибки, и по возможности отработать их не стопорнув отгрузку или еще чего-нибудь. Например, автогенерируемый документ просто сохранить, вместо сохранить и провести. Конечно если знаешь, что в 1с транзакции самые транзакнутые.

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

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

(522) Конечно, пишешь так, как будто бы без попытки и она не нужна совсем. Но потом все равно оборачиваешь в попытку. Хорошо если сразу сообразишь, а не по факту, когда отгрузка все же стопорнулась.

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

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

(523) попытка никак не поможет продолжить отгрузку.
Если например документ не проведется, то его и распечатать нельзя, а следовательно и отгрузить

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

(524) не поможет, если не умеешь.

Либо при загрузке прайса у часть товара вообще не прочиталось и по ним новые цены не встали и менеджеры торгуют по старым низким ценам, вот красота, зато никакой ошибки)

(527) не загрузились курсы валют на дату, выдать ошибку деления на ноль или обернуть в попытку и посчитать по неправильному курсу :) бухи потом с веркой и с водкой разберутся

(525) (527) ну что тут скажешь, кроме повторить. Нужно правильно их делать, а не лепить. Так же как и весь остальной код.

(529) ну так можно и не делать, когда это важно. В общем данную конструкцию нужно использовать с умом, а не при любом косяке

(529) приведи пример, когда ошибка не стопорит отгрузку
Те ошибка есть, а отгрузка проходит

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

(531) легко))

Загружаем заказ с сайта:

Попытка
Цена=ДанныеСайта.Цена;
Исключение
Цена=ПолучитьЦенуПродажную(); // Ну или Цена=1; ))
КонецПопытки;

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

*ой в попытке
Цена=Число(ДанныеСайта.Цена)

(533) цену с сайта вообще опасно брать. а то можно и за дешево продать.
Ведь передать могут любую цену

(534) записать "бланк" заказа - это нормально. где тут попытка?

(536) да я как пример проверки на число данных

(537) имеется в виду, когда тебе могут вернуть на запрос неликвидные данные, а ты пытаешься их впихнуть в поля определенного типа, если же такое происходит, то обмен с сайтом стопорится и новые заказы не падают. А пока это заметят и поправят уже можно с отгрузкой пролететь. Поэтому в (534) он загружает все что может, если что-то в исключение попало, то просто уведомляет юзверов. Только вот как вопрос решать, когда нужно зарезервировать заказ сразу?) А для этого нужно провести документ

(539) ну это как раз пример из (532) единственно применимый

(531) когда не делаешь запрет на печать не проведенного документа. Сохранили, распечатали, отдали водителю. Все подписались, что отгрузка соответсвует. А что там в 1с не получилось, это можно позже разбираться.

Другой пример, понашлепанность попыток. Здесь отдельные попытки и на запись, и на проведение, и сообщения всем и вся если что-то не сработало.

Процедура ПровестиЗаказ(конт, идЗаказа, зак)
    
    Попытка
        зак.Записать(РежимЗаписиДокумента.Проведение, РежимПроведенияДокумента.Неоперативный);
        возврат;
    Исключение
        тхт = "Не удалось провести заказ: "+зак+", "+ОписаниеОшибки();
    КонецПопытки;
    
    стрЗак = ""+СокрЛП(зак.Номер)+" от "+зак.Дата;
    мсж(конт, 3, стрЗак+": "+тхт+" ("+зак.Контрагент+", "+зак.СуммаДокумента+")");
    
    мсжДляСайта = "1С: ОШИБКА: "+тхт;
    ЗагрузкаЗаказов_ДобавитьСообщениеНаСайт(конт, идЗаказа, мсжДляСайта);
КонецПроцедуры

Процедура ЗагрузитьЗаказ(конт, идЗаказа, Автозагрузка)
    
    струЗаказ = Неопределено;
    заказСохранен = Ложь;
    
    КоличествоПопыток = 0;
    Пока КоличествоПопыток < 3 Цикл
        сообщениеОбОшибке = "";
    
        Попытка
            НачатьТранзакцию();
            струЗаказ = ЗагрузитьСоздатьОбъектЗаказа(конт, идЗаказа, Автозагрузка);
            ЗафиксироватьТранзакцию();
            заказСохранен = Истина;
            Прервать;
        Исключение
            сообщениеОбОшибке = "1С:ОШИБКА: "+ОписаниеОшибки();
        КонецПопытки;
        
        Попытка
            ОтменитьТранзакцию();
        Исключение
            сообщениеОбОшибке = сообщениеОбОшибке + ", +ОШИБКА отмены транзакции";
            прервать;
        КонецПопытки;

        Если СтрНайти(сообщениеОбОшибке, "Не удалось заблокировать таблицу") = 0 Тогда
            прервать;
        КонецЕсли;
        
        КоличествоПопыток = КоличествоПопыток + 1;
    КонецЦикла;
    
    Если НЕ заказСохранен Тогда
        ЗагрузкаЗаказов_ДобавитьСообщениеНаСайт(конт, идЗаказа, сообщениеОбОшибке);
    ИначеЕсли НЕ струЗаказ.ПометкаУдаления Тогда
        ПровестиЗаказ(конт, идЗаказа, струЗаказ.Заказ);
    КонецЕсли;
    
КонецПроцедуры

И если один из заказов все же даже не сохранится, то остальные все же продолжат загружаться.

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

(541) Не провелось, потому что нет номера и даты фиксации в ЕГАИС. Машина уехала и ее стопорнули по этому поводу.

(544) Говнокод это как херовый непонятный почерк.
Да писать от руки быстро, но через некоторое время уже сам с трудом читаешь.
А кто то другой с иным почерком хрена с два разберет и будет писать заново.

(544) >>Если последующий прогер не может разобрать с наскока логику (а все наши программы по логике работают), может ему просто время нужно или мозгов?

Ну-ка, занырни вот в этот говнокодец и разбери логику. Чего тебе тут не хватит, времени или мозгов?

    Для уу = 0 по ИндексПЭ-1 цикл
        // ** вычисление продаж в розницу**
        Если Отчет.НетСобственныхТорговыхСетей Тогда
            ПроданоВРозницу   = Окр(ОН[уу], 4) + п4_1[уу] + п4_2[уу] + п4_3[уу] - п5_2[уу] - п5_3[уу] - п5_4[уу] - п5_5[уу] - п5_5[уу] - Окр(ОК[уу],4);
            //ПроданоВРозницу = Окр(ОН[уу], 6) + п4_1[уу] + п4_2[уу] + п4_3[уу] - п5_2[уу] - п5_3[уу] - п5_4[уу] - п5_5[уу] - п5_5[уу] - Окр(ОК[уу], 6);
            //п5_7[уу]=п5_7[уу]+ПроданоВРозницу;
            п5_7[уу]=ПроданоВРозницу;
        Иначе
            ПроданоВРозницу =   Окр(ОН[уу], 4) + п4_1[уу] + п4_2[уу] + п4_3[уу] - п5_2[уу] - п5_3[уу] - п5_4[уу] - п5_5[уу] - п5_7[уу] - Окр(ОК[уу], 4);
            //ПроданоВРозницу = Окр(ОН[уу], 6) + п4_1[уу] + п4_2[уу] + п4_3[уу] - п5_2[уу] - п5_3[уу] - п5_4[уу] - п5_5[уу] - п5_7[уу] - Окр(ОК[уу], 6);
            п5_5[уу] = п5_5[уу] + ПроданоВРозницу;
        КонецЕсли;
        // ** вычисление колонки Итого ***
        
        Если Отчет.ТабачныеИзделия Тогда
            Если уу = 8 Тогда    //  уу=6 или  уу=5
                //колонку "Тонны" в колонку итог "млн. штук" не смешиваем....
                Продолжить;
            КонецЕсли;         
        КонецЕсли;

        п4_1[ИндексПЭ]        = п4_1[ИндексПЭ]    + п4_1[уу];
        п4_1_1[ИндексПЭ]    = п4_1_1[ИндексПЭ]    + п4_1_1[уу];
        п4_1_2[ИндексПЭ]    = п4_1_2[ИндексПЭ]    + п4_1_2[уу];
        п4_1_3[ИндексПЭ]    = п4_1_3[ИндексПЭ]    + п4_1_3[уу];
        п4_2[ИндексПЭ]        = п4_2[ИндексПЭ]    + п4_2[уу];
        п4_3[ИндексПЭ]        = п4_3[ИндексПЭ]    + п4_3[уу];
        п5_2[ИндексПЭ]        = п5_2[ИндексПЭ]    + п5_2[уу];
        п5_3[ИндексПЭ]        = п5_3[ИндексПЭ]    + п5_3[уу];
        п5_4[ИндексПЭ]        = п5_4[ИндексПЭ]    + п5_4[уу];
        п5_5[ИндексПЭ]        = п5_5[ИндексПЭ]    + п5_5[уу];
        п5_7[ИндексПЭ]        = п5_7[ИндексПЭ]    + п5_7[уу];  
        ОН[ИндексПЭ]        = ОН[ИндексПЭ]        + ОН[уу];
        ОК[ИндексПЭ]        = ОК[ИндексПЭ]        + ОК[уу];
        
    Конеццикла;    
    
    Для уу = 0 по ИндексПЭ цикл
        п5[уу] =п5_2[уу] + п5_3[уу] + п5_4[уу] + п5_5[уу] + п5_7[уу];
        п4[уу] =п4_1[уу] + п4_2[уу] + п4_3[уу];
    Конеццикла;

(548) о! А это вообще вершина олимпа говнокодинга!

(548) это минификация, а не говнокодинг.

(551) это уже другой вопрос.
Но это как бы библиотечная функция, ее не нужно доработывать

(552) Для минификации-обфускации надо иметь исходный не минифицированный вариант, для доработок

(550) там даже комментарий к функции истинно-говнокодерский: бесполезный и замусоренный

(552) как и положено истинному говну, оно изобилует ошибками. Но исправить эти ошибки, как ты догадываешься, невозможно

Небольшая функция и сразу страшно?
А в функциях ЗУП не то же самое? А запросы полотнянные - тоже страшно?
Хз, кто-то разбивает на полсотни маленьких модулей, кто-то "простыню" пишет - все читаемо.
Я вот в муз.школе ноты тоже читать не смог. Научили. Слух правда не появился)

(557) да кто же спорит! Всё читаемо. Только вопрос в трудозатратах на прочтение. ЗУП - отдельный вид наркомании. Прям пособие, как писать НЕ надо. За каждым чихом ЗУП заставляет исследовать тонны его подванивающего кода

(545) не дрейфь, это то же уже давно предусмотрели.

(559) денег не хватает
слишком мало платят за ковыряние в этом

(548) это "падеж Крохотулька" со времен 7.7
в ряде типовых пошли тупо. справочник склонений фамилий по падежам (для подписи документов, например).
на "говнокодинг" явно ума не хватило ;-)

(562) такую штуку можно считать демо сценой от 1с. Не все же время в мире ынтерпрайза жить

(563) это есть мастера.. типа Ёпрсат или NS с его шахматами

(447) не-а, у каждого инструмента есть "область применения"... И у 1с в т.ч.

(564) это те самый шахматы для 77? Я так и не понял, то ли человек действительно так код пишет, то ли забрутфорсил модуль...

(546) не совсем, запрос в цикле легко читаемо но не эффективно.

но шахматы были зачетные.

(514) Одного врага вычислили. А ветка полезная. Хотя выглядела как Серосрач.

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

(566) Он говорил что логику с паскаля утащил, со своих ранних разработок. Это перевод с китайского на монгольский. На паскале правда программа обсчитывала около 100 000 позиций в секунду, а на 7.7 стала считать порядка 120 позиций. И на турнире с чессмастером NS проиграл по цейтноту.

(571) я сомневаюсь что на турнир NS выходил с написанной на 77

(572) :)
...иначе бы не было цейтнота

(572) Вроде на 8 он позже гораздо переписывал.

Его даже уговаривали слегка нетленку переписать на восьмерку.

(547) тут не хватает комментариев

(548) а где
// !55! ПадежЫ - для оглы и кызы
? :-)
зы. до сих пор работает.

(577) когда я писал свое склонение (на инфостарте есть) я тоже давал возможность ведения исключений

(580) > свое склонение

кого и к чему склонял? "Новые киски Фиксина"

(581) "Карл Клару склонял к аморалу..."©