Доброго всем!
Случайно наткнулся на один бложик...точнее статью в нем. Статья называется "Программисты: русские, индусы, китайцы и канадцы" и лежит тут:

http://blog.sjinks.pro/humour/76-programmers-russian-indian-chinese-canadian/

П.С. написана она довольно давно, так что если кто читал - уж извините за баян (я увидел впервые).

Понравилась мне там вот эта фраза:

"Русский программист никогда не чинит чужого, бессмысленного, объектно-неориентированного, спагетти-кода"

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

А самое забавное наблюдать за собственной эволюцией....когда открываешь СОБСТВЕННУЮ разработку, проделываешь все тоже самое .что описано выше и опять-таки переписываешь все к чертям.

Вопрос (тут вроде как нужно вопрос задать)...бывает ли у вас подобное (в отношении чужик разработок и в отношении своих)?! Как филигранно удается уходить от желания "выкинуть все и переписать"?!

П.С. ведь не даром существет поговорка - "чтобы построить новый мир, сперва нужно уничтожить старый!"

Так и должно быть. Не кошерно править, если можно переписать.

русские фикси работают не на результат, а на процесс ))

А как же основной принцип - если работает, не трогай.

(0) Если есть возможность - переписываю. А так, в большинстве случаев руководствуешься первой заповедью "Не навреди!"

(1) встает вопрос оптимальности. Т.е. нужно ли бросаться в огонь и переписывать все тратя на это уйму времени...ведь до тебя его уже кто-то изрядно израсходовал.

Порой действительно оптимальней поставить "заплатку", исправить, внести корректировку....но естественно длиться это постоянно не может и набрав определенную "массу" все-таки нужно "сжечь" и построить заново.

Так вот как определить величину этой самой "критической массы"?!

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

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

Может быть, не совсем уместно. Напрашивается сравнение. Фирма 1С начинала с нуля уже неоднократно. Для встроенного языка мы за последние 20 лет видим уже третий созданный с нуля вариант. В фирме SAP сначала 70-х мало что изменилось. Что русскому хорошо, то немцу - смерть!

(0) Боюсь представить как ты код 1с переписываешь...

(7) я так не могу :(

Если время позволяет, то переделываю. Если не позволяет, то ставлю костыль, а потом переделываю.

(6) Это распространенное заблуждение, для франча главное чтобы БЫСТРО исправить и пох как оно работает, для фикси надеюсь не так иначе это разобьёт мою 1с-мечту и веру в идеалы профессии.

Ссылку не читал, но с выводом ТС согласен. Поддерживаю самописку на 7.7, в которую за 10 лет чего только не понапихали, во многих местах, усиленно мною правимых, весь код так до конца и не понимаю, правлю интуитивно:-) Осточертело уже настолько, что пишу на восьмерке всё заново, лишь бы не рыться в чужом коде:-)

(8) и в чем вывод?!

"В чем правда, брат?!" (с)

Это называется рефакторинг. Просто к нему нужно с умом подходить

(9) да не сцы, все тип-топ :)

(11) это не зависит от фра/фикси. Зависит от человека. На фикси гораздо чаше встречается г. код

(0) Помню как то открыл обработку, обматерил автора, а потом вспомнил что это я писал

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

(17) т.е. можно сделать выводы - что ты ростешь "профессионально"!

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

(19) Чем больше я расту тем больше желания у меня возникает материть чужой код.

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

(18) бывало и такое :)

Например документ "Установка типов цен по группам номенклатуры для покупателей" в УТ 10.3

Логику проведения переписал заново. Причем в результате "экспериментов" переписывал раза 3 (на протяжении полугода)

к (21) То есть наоборот тем меньше желания возникает.

(22) возможно! Разве я утверждал обратное?

Недавно дали задание - чуть поправить отчет на СКД, который писал какой-то украинский программист.
Открыл, посмотрел, задумался, понял что очень тяжело читается - проще переписать.
Переписал - потом открыл старый отчет и понял что на 90% получилось тоже самое, только поля по другому названы ))) Зато уже всё понятно, что к чему и откуда берется, а главное что надо сделать чтобы его поправить ))

(24) чем обосновано?

(16) "На фикси гораздо чаше встречается г. код" - а у меня обратные наблюдения, но полностью соглашусь с тем, что всё завит от человека. Видимо мне такие франчи попадались :)

Никогда не пишу код, если есть уже рабочий код. Проще поправить, чем писать самому

http://scaramouch.ru/content/view/6780
побаяню
http://habrahabr.ru/post/135090/

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

(13)Скорее, это ради прикола. Однако факты имеют место быть :-)

Самый страшный г(о)внокод пишут бывшие системные администраторы, переучившиеся на 1Сников. Это какой-то матерый звиздец.... Копирование процедуры, вместо того чтобы загнать все в СЗ или ТЗ, привязки к коду номенклатуры или товару - а работает же!!

(28) Внедрял одну из систем систему мобильной торговли осенью. Часть функционала была предоставлена "ихними" разработчиками (короче говоря обработки не которые). Открываю обработку очередную и вижу коменты примерно такого рода:

"Это гов***од! Работает очень медленно и по тупому, но довольно легок в понимании"

Как вы думаете, быстро возникло желание переписать код обработки?!

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

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

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

Старая школа постепенно забывается и уходит в историю... (имхо, благо возраст позволяет)

Ну а я, например, могу себе позволить вольность переписать заново только те алгоритмы, автор которых - я.

В чужом коде (типовые конфигурации - в том числе) стараюсь излишне не наследить. Но поставить его с головы на ноги короткими лаконичными вставками - доставляет эстетическое удовольствие :))

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

(37) Пффф... ну открой просто так и стучи по клавишам если кто за спиной стоит

(38) "Но поставить его с головы на ноги короткими лаконичными вставками - доставляет эстетическое удовольствие :))" - это таланат!

(0)Пишу в редких случаях. 70% потребностей при кодировании закрываются кусками из стандартных конфигураций. Главно уметь найти и правильно копипастить :))
Ну и допилить.

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

(37) ни разу, никто не подходил и не проверял, открыть ли у меня конфигуратор. Вообще наверное это не есть показатель работы.

сумхл=Окр(сумхл/1000,2,1);
    сумхла=Окр(сумхла/1000,2,1);
    сумбул=Окр(сумбул/1000,2,1);
    сумбулН=Окр(сумбул3/1000,2,1);
    сумкон=Окр(сумкон/1000,2,1);
    сумпроч=Окр(сумпроч/1000,2,1);
    _хла1Руб=Окр(_хла1*срхла1,2,1);
    _хл1Руб=Окр(_хл1*срхл1,2,1);
    _бул1Руб=Окр(_бул1*србул1,2,1);
    _булН1Руб=Окр(_бул_1*србулН1,2,1);
    _кон1Руб=Окр(_кон1*сркон1,2,1);
    _проч1Руб=Окр(_проч1*српроч1,2,1);
   _хлапр1РУб=Окр(сумхла/(товхла/1000)*100,2,1);

   _хлпр1РУб=?(товхл>0,Окр(сумхл/(товхл/1000)*100,2,1),0);
   _булпр1РУб=Окр(сумбул/(товбул/1000)*100,2,1);
   _булНпр1РУб=Окр(сумбулН/(товбулН/1000)*100,2,1);
   _Конпр1РУб=Окр(сумкон/(товкон/1000)*100,2,1);
   _Прочпр1РУб=Окр(сумпроч/(товпроч/1000)*100,2,1);
   Если _хлруб <> 0 Тогда
       хлпрРУб=Окр(_хл1руб/(_хлруб)*100,2,1);
   Иначе
       хлпрРУб = 0;
   КонецЕсли;
   //хлпрРУб=Окр(_хл1руб/(_хлруб)*100,2,1);
   хлапрРУб=Окр(_хла1руб/(_хларуб)*100,2,1);
   булпрРУб=Окр(_бул1руб/(_булруб)*100,2,1);
   булНпрРУб=Окр(_булН1руб/(_булНруб)*100,2,1);
   КонпрРУб=Окр(_кон1руб/(_конруб)*100,2,1);
   ПрочпрРУб=Окр(_проч1руб/(_прочруб)*100,2,1);

   _Товпр1=Окр(_тов2/(тов)*100,2,1);
   _хлпр1=?(хл>0,Окр(_хл2/(хл)*100,2,1),0);
   _хлапр1=Окр(_хла2/(хла)*100,2,1);
   _булпр1=Окр(_бул2/(бул)*100,2,1);
   _булНпр1=Окр(_бул3/(булН)*100,2,1);
   _Конпр1=Окр(_кон2/(кон)*100,2,1);
   _Прочпр1=Окр(_проч2/(проч)*100,2,1);



когда вижу нечто подобное, хочется встать и уйти, но лень и жадность побеждают

(45) офигеть и не лень же было .

(45) где взял....такое....

(46)а че, я только
_хлпр1РУб=?(товхл>0,Окр(сумхл/(товхл/1000)*100,2,1),0);
_хлпр1=?(хл>0,Окр(_хл2/(хл)*100,2,1),0);
поправил за 1500 рублей, переписывать оч. дорого будет =)

(43) Я бы еще твой трафик отслеживал

(48) по 750 руб за строчку...хорошая цена кода....

(0) Какой же ты программист, если тебе не лень переписывать, когда достаточно просто слегка поправить?

(14)это называется "переписка"

(51) думаешь ошибся с профессией?

(52) энгельса с каутским

http://video.yandex.ru/users/ser-joe/view/15/

(33) ты код бывших бухов не видел...

(0) матерю все, что вижу - и свое и чужое. переписываю по ситауции.

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

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

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

(0) в типовых нормальный код, особо не парюсь, в чужем не ковыряюсь

Вопрос переписывания собственных разработок решается просто и изящно- выкладываем на сайт новой версии (плюс один к старшему разряду) и одновременным увеличением текущей цены для новых лицензиатов.

(0) а что приходишь в новую компанию,и все нарботки переписываешь??!!

(61) +1 тогда я китаец.......

Ужас. никогда ниче не переписывал, ни своего, ни чужого. это ж скукота. Про таких как как (0):
Перфекционизм — в психологии, убеждение, что наилучшего результата можно (или нужно) достичь. В патологической форме — убеждение, что несовершенный результат работы неприемлем.

(56) Плюсуюсь.. бывшие бухгалтера, переквалифицировавшиеся в одинесников, пишут мегапортянки кода, состоящие из копипастов на 95%. Их код как правило легко можно ужать в 10 раз. А всё потому, что они пишут как дышат.. то есть, по-бухгалтерски, "открыживая" каждый вариант.

(64) брехня и безосновательный наезд )

(65) Неа, чистая правда. А еще они принципиально не комментируют код)

(66) ну тут согласен. И то пока молодые да неопытные) а на счет портянок брехня )

(66) когда то бывший бухгалтер, переписал за бывшими программистами, которые взялись за 1с модуль в 16 000 строчек.   Осталось 3000 и то, потому что бухгалтер устал и хотел спатки). Так вот, результат что при работе 3000 строчек что при 14 000 был одинаковый)

(0) [:|||||||||:]
И даже [:|||||||||||||||||||||||||||||||||||||||||||||||||:]
А по смыслу - источник холиваров (либо приятное чтение после работы). Больше ни на что этот текст не годен.

(63) может тебе пока просто повезло?

(68) А результат разве оценивается по количеству строк кода? А я-то думал, что по функциональности и производительности системы.

(70)
видел отчет: надо исправить ошибку.

Открыл - а там пзец. Даже отладка логику не говорит.
Уточнил постановку у заказчика и переписал за 2 часа.

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

(72) такие примерчики у меня есть. Когда не уверен в чужом коде(а надо быть 100 процентов уверенным и основания есть), это конечно прикольно... Хочется конечно переписать все.

(71) Это ты оцениваешь по функциональности.
А "эффективным менеджерам" - нужно количество строк :-)

(71) ну так в том и вопрос, что результат при различном количестве кода был одинаковый. Но имея 14 000 строчек приходилось тратить на добавление одной колонки в отчете 4 часа времени просто тупого лазания по коду, который был даже не отформатированный и контр с конр в добавляя. После "рефракторинга", та же операция в дальнейшем занимала 15 минут

(63) Не надо кидаться в крайности.
"Искусство ради искусства" - тут да, надо бы и поостеречься.
Но исправить заведомый "ляп" - что тут не так?

(76) греческое правило золотой середины никто еще не отменял)

(0) Татарские программисты такие же. Отвечаю.

(3) Если потребовались доработки, значит "не работает" :)

(0) все зависит от того, что проще по трудозатратам. Если проще все переписать - то переписывай.

92) ты еще скажи что фрилансеры работают на результат - япоржу...

Многие русские люди любят ругать "на кухне" власть, начальство, свой город, свою страну. А русских прогер 1С любит ругать других прогеров, начальство 1С, все платформы в целом.

(82) отсюда и желание всё полностью переписать после "дурака" и ещё понтануться этим где-нить, чтоб умнее себя ощутить.

Соловушка поёт, а на душе так грустно,
Скажи мне, кто не знает, кто такие Русские
Тебя спиной закрою, если ты мне брат или друуг,
Русские идут!

Вот еще древняя статейка в тему

РУССКИЙ КОД, БЕССМЫСЛЕННЫЙ И БЕСПОЩАДНЫЙ

http://relyef.livejournal.com/132521.html

(3) Это плохой принцип, когда работает 3 часа, а должен уже через 20 сек выдавать результат :)

(0) Это болезнь, срочно лечись.

http://habrahabr.ru/post/120000/

Синдром рефакторинга

(31) А франчу тоже не надо, чтобы клиент его потом гонял забесплатно все исправлять. Ему надо взять деньгу, сделать работу уложившись в бюджет, но так чтобы его не дергали потом по гарантийным обязательствам.

Если есть возможность обойтись добавкой одной-двух строчек в чужом коде так и делаю, если лезть надо глубоко, то действительно, лично мне иногда гораздо легче написать свое, чем разбираться в многострочье чужого кода (чужую логику иногда очень сложно понять). Что касается своих старых работ, то тут не все так однозначно, у меня бывает два варианта, один из них описан (т.е. надо срочно сесть и переписать все с начала), а вот второй противоположный - "Ахренеть!!! Неужто я умел когда-то ТАК круто писать!"

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

Господа, чё тут расмуссоливать-то
Обычно поддерживать хреновый код гораздо затратнее для бюджета и нервов, чем допилить и превратить в хороший
Мне один раз передали проектик по 1С, так там пипец что творилось. Полдня иногда уходило чтобы разобраться в очередном баге
Легче было переписать заново по ТЗ

(90) (91) вот поэтому го*вно*д не страшен, страшно отсутствие документации)

(0) ну я хз, от задачи зависит, не дале как вчера исправил чужое творение, поскольку это целесообразнее

(0) В последнее время стал открывать свои старые обработки и удивлять - а что это за функция - ух ты надо запомнить )))

(0) Надоело переписывать чужой г-код. Люди деньги/репутацию получили, а тут их попу прикрываешь да еще и забесплатно. По возможности оставляю как есть дописывая свое. Хороший принцип - "не влезай - убъет". А уж карму испортить предыдущему "работничку" самое оно.

раньше я тоже плевался на чужой "быдлокод", но со временем - это прошло. быдоколд ну и былокод (а кто без греха?). работает и ладно. хреново работает - перепишем. Гораздо хуже не то что по коду видно что это быдлокод, а то что по коду видно что кодеру наплевать на работу/участок. который он программит. вот это и есть настоящий быдлокод(ер).
.
у мну - количество быдокода наверное аж борзометр зашкаливает ;-) (правда стараюсь поменьше его плодить). Однако, до тех пор, пока не найдется кто-то, кто сможет сделать лучше (а таковых в ближайших окрестностях и перспективе не видно) - ну и фиг с ним ;-)
.
Проблема не столько в быдлокоде, сколько в быдляцком построении системы. так как четкого понимания чтои как должно быть и почему и в какие сроки - нет, система напоминает ухоженный шанхай.. ;-)

Если при знании предметной области, свой код непонятен через полгода- значит или пишешь редко или кончай писать г.овнокод.
Кстати, г.овнокод - слово не совсем внятное.

Да это вечный цикл: проектирование -> разработка -> эксплуатация -> проектирование.

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

100

далматинец

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

Вы уж извините но это не профессионально, в смысле это до того непрофессионально, что даже стыдно должно быть за это...

Вес 1Сники ВСЕГДА переписывают ЧУЖОЙ код!!!!
ВСЕГДА

(103) только глупые 1С-ники

(104) глупые вообще 1С не знают.

(105) тогда это не 1Сники а обновляторы...

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

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

(108) с учетом приобретенного опыта понимаешь что некоторые задачи вообще не надо решать, в принципе.

(0)"Русский программист никогда не чинит чужого, бессмысленного, объектно-неориентированного, спагетти-кода"
Вообще то это основное правило программиста. Но 1с за счет своей узконаправленности, шаблонности и многоконструктности позволяет нормальному прогу за нормальным смело программить. Но конечно попадаються и такие за которыми лучше переписать заново.

(110) Ну это не только в 1с, как и (0)

В смысле:
- Сергей, подправь тут немного за Раджой. За сутки сделаешь?
- Да все на мази к обеду будет
.......Через две недели....

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

(113) +100500

Как правило есть методология написания кода. И у 1с она тоже есть, но тупые одинесники ее не читают. Куле тут удивляться?

(68) Вероятно просто выкинул результат причудливых отражений прикладной области в головах кодеров.

(115) Её похоже и некоторые разработчики типовых конфиграций не читают..

(117) В большей семье не без урода

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