Собственно сабж. Меня ужасно бесит. Обычно пишешь так, чтобы потом прочитать можно было, стараешься соблюдать правила хорошего тона. А кто-нибудь залезет и исправит так, что элементарно отступы не соблюдет. На фразе "я поправил/поправила твой код" разве что глаз не дергается

вот

замуж пора

Судя по голосовалке сомневаюсь, что ТС пишет код с соблюдением правил хорошего тона :))

замуж пора.

мне пох.
код, написанный мной на работе - собственность работодателя.
код, написанный мной дома - мой код.

(0) Покажи, что ты написала и как это исправили.

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

(2) спасибо, была, с тех пор еще 2й глаз, который за замуж ответственный, не перестал дергаться

Если там нет багов исправителя, то

(8) он тоже правил твой код?

пофик, пишу так чтоб потом не правили

(5) ну хочется же чтобы и то что ты сделал на работе было приличным

(11) а если слегка изменилась задача? и исправили под измененную задачу

:)...проблема только в том как это обосновать.....трудно...доводы бурлят...но их слишком много

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

(12)раз исправили значит там косяк, а красиво оформленный косяк все равно не приличный

полезли - сами виноваты

Это проблемы того, КТО правит.

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

Если я это вижу, значит - я есть по близости. А значит, что могу и сам исправить, если есть в этом потребность.

Проголосовать забыл.

(13)и в чем собственно тогда проблема?

(13)все равно косяк! если не работает то косяк, какие бы аргументы не были

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

(22) в том что исправить-то человек исправил, вроде работает, но написано так что блевать хочется: ни отступов, слепые имена, ну и так далее

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

(24) а ларьки-то тут причем?

психоз

замуж пора!

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

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

1. Да

(31) ну я собственно и исправляю

Было дело - написал конфигурацию для одной компании.
Через полгода попросили прописать еще один блок - дык там местный бух наковырял.
Обиделся и отказался

(29) в следующий раз прикручу и такой вариант в голосовалке

фотку давай

(0) а фотко где?

был один типус. понапихал в ГМ своих каких-то процедур целую тучу. Которые потом нигде не использовались. Удалил и забыл

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

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

(36) (37) совсем оборзели 1с-ники, их вроде и помочь ничего не просишь, а они уже "фотку-фотку"

(39) взаимно :)

(42)Не обращай на меня внимания, а то второй глаз трестись начнет, извени.

Смотря, кто правит...
Было: более опытный коллега. Но всё делает на бегу, быстро, уталкивает, комменты не пишет, отступы не делает. Бесило, но не сильно. Решилось, когда ему пришлось в моем коде, потом им поправленном, снова ковыряться. Очень плевался :)
Было: неопытный коллега. Ни фига не знает, ни фига не умеет, но понтов - МАССА. такого наисправлял.... Мама - не горюй. Сказать, что бесило - ничего не сказать.
А по уму, кто поправил, тот и ковыряется потом.

(41) как это ничего не просишь?
Ты просишь наше мнение, наше участие в твоей теме... а фотку отказываешься давать...

пофиг

(41) почему оборзели, сиське для мужиков есть естественная потребность

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

*будете

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

Меня бесят люди, которые писали код до меня.

не трать нервы на такие пустяки

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

что значит приличное оформление? есть стандарты... пришел в новый проект - соблюдай принятые стандарты оформления

чтоб меньше править, делай меньше ошибок)

Не бесит, но правильно оформленный код (хотя сам по молодости грешил) сильно экономит время

(53) Расслабься. Все хорошо. Из-за того, что кто-то не похож на тебя, не стоит его убивать. Каждый пишет как умеет.

Мечтаю, чтоб кто-нибудь хотя бы раз поправил мой код!!!

Некому :)

(50) а че надо в каждой поправленной строке писать, здесь был фиксин?

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

(0) В основном нет.

(50) Ни фига! В институте с 1 курса твердили: "Пишите комменты. Хотя бы краткие." На 3 курсе дошло зачем - через пару месяцев и не вспомнишь, что в данной процедуре-функции происходит. Комменты, хотя краткие, нужны.
Отступы - бывает время поджимает, быстренько накидываешь ручками. И тут не отступов. Но потом (после отладки) найти пару минут, что форматнуть можно.
Опытный коллега исправился :)))

(59) нет, хотя бы соблюдать отступы, не создавать переменных с именами типа Таблица1, д2

(60) так научи коллегу делать Ctrl+A и Alt+Shift+F

А, в принципе, подними руку повыше :)))))
Не парься:)))))))
Как правило, жизнь все расставляет на свои места.

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

че вы тут жуете? Есть "Система стандартов и методик разработки конфигураций для платформы 1С:Предприятие 8" от фирмы 1С
если другое не принято на проекте, придерживаться её

+(51) Пример го_внокода:

ВыбратьСтроки();    
Пока ПолучитьСтроку() = 1 Цикл
   Если ДатаДок > Организация.ДатаУстановкиКонтроля Тогда  
   Если (СокрЛП(КорСчетПредставление)=СокрЛП(СчетПоКоду("8111"))) или (СокрЛП(КорСчетПредставление)=СокрЛП(СчетПоКоду("8211"))) или (СокрЛП(КорСчетПредставление)=СокрЛП(СчетПоКоду("8311"))) или (СокрЛП(КорСчетПредставление)=СокрЛП(СчетПоКоду("8411"))) Тогда    
       Предупреждение("ВНИМАНИЕ счета затрат 8111, 8211, 8311, 8411 нельзя выбирать !!!!");
       НеПроводитьДокумент();
       СтатусВозврата(0);
       Возврат;
   КонецЕсли;
   Если (СокрЛП(Субконто2) = "") и (СокрЛП(Субконто3) = "") Тогда
       Продолжить;    
   КонецЕсли;    
   Спр=СоздатьОбъект("Справочник.ВидыЗатрат");
   Спр.ВыбратьЭлементы();
   Если Субконто2.Вид() = Спр.Вид() Тогда
       Пока Спр.ПолучитьЭлемент() > 0 Цикл  
           Если Спр.ЭтоГруппа() = 1 Тогда
               Продолжить;
           КонецЕсли;  
           Если СокрЛП(Субконто2) = Спр.Наименование Тогда
               СчетЗатрат = Спр.СчетЗатрат;
               Если (СокрЛП(Спр.СчетЗатрат)=СокрЛП(СчетПоКоду("293"))) Тогда
                   СчетЗатрат = "2930";
               КонецЕсли;
               Если СокрЛП(КорСчетПредставление) = СокрЛП(СчетЗатрат) Тогда
                   Прервать;
               КонецЕсли;
               Предупреждение("ВНИМАНИЕ выбран не тот счет !!!!");
               НеПроводитьДокумент();
               СтатусВозврата(0);
               Возврат;
           КонецЕсли;
       КонецЦикла;
   КонецЕсли;  
   Если СокрЛП(Субконто3) = "" Тогда
       Продолжить;    
   КонецЕсли;    
   Если Субконто3.Вид() = Спр.Вид() Тогда
       Пока Спр.ПолучитьЭлемент() > 0 Цикл  
           Если Спр.ЭтоГруппа() = 1 Тогда
               Продолжить;
           КонецЕсли;  
           Если СокрЛП(Субконто3) = Спр.Наименование Тогда  
               СчетЗатрат = Спр.СчетЗатрат;
               Если (СокрЛП(Спр.СчетЗатрат)=СокрЛП(СчетПоКоду("711"))) Тогда
                   СчетЗатрат = "7110";
               КонецЕсли;
               Если (СокрЛП(Спр.СчетЗатрат)=СокрЛП(СчетПоКоду("731"))) Тогда
                   СчетЗатрат = "7310";
               КонецЕсли;    
               Если СокрЛП(КорСчетПредставление) = СокрЛП(СчетЗатрат) Тогда
                   Прервать;
               КонецЕсли;
               Предупреждение("ВНИМАНИЕ выбран не тот счет !!!!");
               НеПроводитьДокумент();
               СтатусВозврата(0);
               Возврат;
           КонецЕсли;
       КонецЦикла;
   КонецЕсли;
   КонецЕсли;    
КонецЦикла;

По разному.
Когда грамотно сделают - нормально.
За быдлокод - убивал бы

от индуса приятно слышать, что за быдлокод он убивал бы =)

(68) Вот только gовнокода тут  не хватало, можно подумать его никто не видел:D

вот если бы все были такими умными, сколько бы R/3 весил за шиисят лет к каждой строчке по тыщ так цать камметов

(68) симорочка :)

(68) так это еще не худший вариант

(5) Скорее всего код принадлежит тому, кому ты его передал - если писал дома, а потом внедрил на работе - то работодателю.

(71) Под спойлер его

(68) Фу, семерка, там другого и не бывает

(73)+ а ваще, щидевр, да :)

(70)быдлокод и индуский - разные вещи. если чо

(77) не надо

(68)Блин, похоже на мой кусок кода!

Пофиг. Правленный код слетает с гарантии :)

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

(83) по твоему стилю написания постов уже можно судить о твоём ховнокоде :)

(79) индусский - пряностями пахнет?

Замечу - любой код будет го.внокодом, если написан не тобой

(0) Не пиши ничё... 4444 сообщений набрала..

(86) вот с этим не соглашусь

Если правка по делу и только улучшит - то нет, не раздражает.

(88) Взяла и испортила всё.. ппц..

А если правят и спускают пустые строки или пробелы вместо табуляции или пробелы/табуляции в конце строки - то да.

(90) Ты об двух цыхрах 8?

Что вы с ней разговариваете? У нее и фотки нету в профиле

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

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

Меня бесит, когда код безграмотный.

+(68) Добавлю: этот код в процедуре проведения.

(94) То есть если была бухшей, то станет красивой, а если СИшницей - то не судьба?...

Если поправлено с умом

(100) нет. бывают и страшные бухши.
Просто в программистки идут в основном умные девушки.
Умная девушка --> Хорошо занимающаяся и изучающая предмет.
Девушка хорошо изучающая предмет и занятия --> Много времени тратит.
Тратит много времени на изучение --> Мало гуляет.
мало гуляет --> нет парня.
Нет парня --> Страшная.

Как то так

(94) не правда, знаю двух очень симпатичных джависток

Запрос

- скажите, а вы правильный писатель?
...
- не пишите больше
(с) булгаков

надо писать так, чтобы не понимали как написано...

Ктитику не любите?

+ (112) сори, кРитику - очепятка по Фрейду )))

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

(112) к критике отношусь нормально, вот бардак не люблю

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

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

(117)+1

считать отступы - это уже маразм

(116) ну у вас и конторка...

(118) Тем не менее, стиль говорит о многом. И помогает, кстати.

мельком прочла тему как "Раздражает ли вас, когда лезут в ваши штаны"))))наверное пора к последователям дядюшки Фрейда))))

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

(118) это структура текста, проще выделить логические блоки при чтении

(116) бугого)

всем срочно замуж

(120) О чем говорит стиль? Я иногда продумываю алгоритм, но в процессе написания понимаю, что не то получается, переписываю на ходу. В итоге текст получается не отформатированным) Пару раз носом ткнули в это, стало автоформатирование делать после написания.

(122) Потому что код типовой написан с соблюдением методологии самой 1С.
Вы ее читали? Вы ее соблюдаете? Вот я честно могу признаться, что не читала. И чего потом обижаться, если правят)
И кстати, стиль типовых меня иногда просто убивает

(118) Alt-Shift-F же!

 




ищу жену




 

(128) давно пропала?

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

(126) ну там по крайней мере нет переменных типа тз1 и тз2 (а если есть, то очень редко)
Я тоже, если и читала методологию, то давно и по диагонали. Но не суть, привычка читабельно оформлять код еще в инсте на сишнике или паскале начинается, это не что-то уникальное для 1с. Тем более что не читая методологию, нам же понятен текст типовой?

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

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

(103) Мда, даже не знаю как в эту логику себя впихнуть. Парня нет, есть муж и двое детей. Не катит?

мне тут недавно сказали что мой код на клипер 5 досих пор люди юзают - лет 30-ть уже. Хорошо видать с бодуна получился
ЗЫ славатехоподи что хоть камменты некогда было писать

(128)проси возврат хреноветок.. найдешь жену себе :)

(133) фигня в том что мой код читабелен не только для меня))) а вот код коллеги...

(136) в хреноветках жену не найдешь.

(137) ну тогда вынеси эту тему на обсуждение в конторе. Или тебе просто пожаловаться некому? :)

(138)ну как-же.. нашел ведь один мембер :)

(138) это мужа там не найдёшь :)

(139) да это так крик души, в общем такая ситуация почти в любой конторе встречается. В общем-то понятно что г.внокод не от большой грамотности получается

(103)
мало гуляет --> нет парня.
Нет парня --> Страшная.

Чувак, твоя логика сломала мой женский мозг

(141) а я и не ищу

(143) гормоны иптить же :)

(145) У кого гормоны?

(137) горемыкою, лет шестнадцати, на кирпичный завод подалась?

Гретта появилась, фото тоже нет. Вот к чему ведет попустительство!

Морковка Ты код (и исправления) покажешь или нет?

А я не люблю править чужой код особенно если он дебильный)

(147) г.внокодер? мне меряться нечем, я ж девушка

пох, пусть лезут.
но и понимают, что за критику по башке можно получить.

(151) Не верим! Фото!

Вот как правильнее?

   Форма.ФлагНовый.Видимость(?(МоиПрава="ADM",1,0));
   Форма.Заказ.Видимость(?(МоиПрава="ADM",1,0));
   Форма.ПриоритетПроцентаОтдела.Доступность(?(МоиПрава="ADM",1,0));

ИЛИ

   Если (МоиПрава="ADM") Тогда
       Форма.ФлагНовый.Видимость(1);
       Форма.Заказ.Видимость(1);
       Форма.ПриоритетПроцентаОтдела.Доступность(1);
   Иначе
       Форма.ФлагНовый.Видимость(0);
       Форма.Заказ.Видимость(0);
       Форма.ПриоритетПроцентаОтдела.Доступность(0);
   КонецЕсли;

(149) да там сущая мелочь, просто последняя капля :)

да за ради всех святых. Они же потом все шишки и получат.

(126) А че и методолгоия есть?

(155) но показывать таки не хочешь)

(154) я бы предпочла 2: и читабельней, и по идее производительней (хотя там такие крохи). А можно еще в слой засунуть или в процедуру с параметром

(154)
Видимость=?(МоиПрава="ADM",1,0);
Форма.ФлагНовый.Видимость(Видимость);
Форма.Заказ.Видимость(Видимость);
Форма.ПриоритетПроцентаОтдела.Доступность(Видимость);

(154) второй вариант как-то больше нравится ))

(154) Ни так и ни эдак

+(162) А вообще по.уй - работать будет

(159) тебе платят за количество строк?

(154) оба варианта гамно)

(160) Во, я также бы наколбасил.

(158) ну что то вроде

Пока...
    //тут вот цикл
КонецЦикла; //это было
      Записать(); //а вот это вставили

реально фигня, но что сложно сделать нормально? прост окогда на такие вещи из раза в раз натыкаешься

(155) на //суки, хватит уж ломать мой код, потому что уже зае...
- обиделась что ли?

(159) второй вариант как раз плох для редактирования. надо будет дублировать чтобы изменить что-то

Это нормальная ситуация при групповой разработке , главное что бы правильно правили , и не забывали комментировать

(167) Может там документы в цикле создавались и ты забыла их записать :) а человек поправил и вместо благодарности...)

(167) Это реальный код или ты сейчас из головы придумала? Т.е. вы реально такие комментарии оставляете в коде?

(164) а дело не в количестве строк. Вопрос читабельности: явное условие всегда проще читается чем оператор выбора. + еще в профильных инстах на первых курсах вообще-то расказывают, что меньше строк и красиво - это не значит производительнее

(172) ну коментарии приписала, а код вполне реальный

В общем, не впадайте в маразм)

(173) Работает и ладно.
То что работает производительнее - часто читаеться вообще никак...

пофиг.
На работе написанный код может править кто угодно.
Дома написанный код никто не правит. Либо правит,но меня не ставит уже в известность))

(171) не там в приОткрытии закрыть() вставили, хотя вообще там статусВозврата(0) уже был
Просто не люблю когда делают тяп-ляп

(176) согласна

(177) ты дома для себя пишешь? или для работы?

Обычный рабочий процесс.

Женский код всегда надо поправлять.

(182) А код ещё по полу различается?

(178) чего-то все таки ты не договариваешь... Зачем СтатусВозврата(0) надо было менять на Закрыть() ? просто так от балды никто ничего менять не будет.

(182) вообще чаще мужчины лишние буковки ленятся писать, все под девизом Краткость -СТ

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

(183) В этом можешь даже не сумневаться.

+(182) Го_внокод, китайский го_внокод, индусский го_внокод, индусский го_внокод женский.

(184) да просто открыли обработку криво, без параметра, вот и решили что это обработка не правильная, а не вызов

(189) Если открыли обработку без параметров то следуя твоей логике она просто не должна была открыться так как там в ПриОткрытии стоит СтатусВозврата(0) так? какой смысл было вставлять туда еще и закрыть() ? ну явно же с тобой рядом не дебилы работают.

Посмотрела сейчас на свой код... Нифига б никто не понял наверное) Но начальство одобрило))

можно ещё советскую энцивплопедию, всю, в отчет вкопипастить - читабельный будет пипец

(180) подработки

(190) ну вот видишь, вставляют же, и код некрасиво офрмляют. А у меня потом глаз дергается :(

(68)Нормальный код, кстати. Быстро и работает.

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

Морковка возьмешь в ученики? (поучится правильно\красиво писАть код никогда не помешает)

(126) Путешествия по просторам паутины общих модулей БСП?)))

(195) Где-же он нормальный? Ты в своём уме?

(196) да я особо и не трачу, вот тут на мисте слила негатив и никаких нервов, а потом не все же автоформатированием исправишь, сам код-то тем же останется

(197) учиться у девушки...фиии..

как-то задался целью xls в ТЧ дока подгружать как есть, хотелось что б покороче
ЗЫ а все равно четыре строчки получилось

(198) С БСП как-то легко очень получилось, а вот когда встала задача из БП 3.0 тащить некоторые объекты, вот тогда я взвыла. Сволочи они)

(200) Я не гордый. Ну так как?

(199)Хотя да. Там бардак.

(202) Так она на БСП и написана :)

+(205) тока замудрили нереально сильно...

только если это мой генетичекий код

(207) :))...тебя раздражают женщины которые лезу в твой генетический код с целью копирования?
зы
или только нелицензионного копирования?

главное чтоб работало и звери были довольные =)))

Меня не раздражает

(103) про мужчину так же можно сказать

(68) Покажи свой вариант этого куска кода.

Смотря кто и зачем лезет в мой код, насколько грамотно его правит...

А я обновляю конфигурацию периодически на одном предприятии, и наблюдаю как её постепенно огомнокодивают местные программисты. неприятно :(. Особенно когда внесут какое-нибудь сложное для анализа изменение.

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

Я на заводе один, вся конфа в моем распоряжении

Если поправленный код лучше чем был прежде, тогда

(212) Переделал под функцию:

   ЕстьОшибки = 0;
   ВыбратьСтроки();    
   Пока ПолучитьСтроку() = 1 Цикл
       Если (ПустоеЗначение(КорСчет) = 1) Или (ПустоеЗначение(КорСчетПредставление) = 1) Тогда
           Сообщить("В строке № "+НомерСтроки+" не указан корр. счет.", "!");
           ЕстьОшибки = 1;
           Продолжить;
       КонецЕсли;
       
       Если (КорСчет = СчетПоКоду("8111")) Или (КорСчет = СчетПоКоду("8211")) Или (КорСчет = СчетПоКоду("8311")) Или (КорСчет = СчетПоКоду("8411")) Тогда
           Сообщить("ВНИМАНИЕ счета затрат 8111, 8211, 8311, 8411 нельзя выбирать !!!!", "!");
           ЕстьОшибки = 1;
           Продолжить;
       КонецЕсли;
       
       Для СчСубконто = 1 По КорСчет.КоличествоСубконто() Цикл
           Атрибут = ПолучитьАтрибут("Субконто"+СчСубконто);
           Если ПустоеЗначение(Атрибут) = 1 Тогда
               Сообщить("В строке № "+НомерСтроки+" не указано субконто №"+СчСубконто+".", "!");
               ЕстьОшибки = 1;
           Иначе
               Если Атрибут.Вид() = "ВидыЗатрат" Тогда
                   Если Атрибут.СчетЗатрат <> КорСчет Тогда
                       Сообщить("ВНИМАНИЕ выбран не тот счет !!!!", "!");
                       ЕстьОшибки = 1;
                   КонецЕсли;
               КонецЕсли;
           КонецЕсли;
       КонецЦикла;
       
   КонецЦикла;
   
   Возврат ЕстьОшибки;

А меня только один раз раздразил другой код. Чувак в комментариях об**л меня и мой код и сделал свой еще хуже. Представляю что заказчику про меня наговорил

(77) http://savepic.net/3085386.jpg (С) Новая ПЯТНИЦА13 !!!!

Ну и я поделюсь мелехо  ГК
видел в одной конфе примерно следующее
есть ТЗ (тзПоказатели)состоит примерно из таких полей
Период - Показатель - Значение
и есть

Функция ОбновитьЗначениеПоказателя(Период,Показатель,Значение)
Для Каждого Строка Из тзПоказатели Цикл
Если Строка.Период = Период И Строка.Показатель = Показатель Тогда
   Строка.Значение = Значение;
КонецЕсли;
КонецЦикла;  
КонецФункции

я когда это увидел - аж прослезился - есно от смеха . :)

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

+(220) Оттуда же: http://savepic.net/3097665.jpg

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

(222) правильный код примерно такого же размера - но суть в быстродействии !!!
этот код занимал оч много вычислительного времени

(222) Если это в 7.7, то это за отсутствием НайтиСтроки(), но тут 8-ка, значит семёрошник писал.

Без примеров кода автора, ветка ниочем.

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

(222) с чего бы это больше?
Для каждого строка Из тзПоказатели.НайтиСтроки(Новый Структура("Период, Показатель", Период, Показатель)) Цикл
   строка.Значение = Значение;
КонецЦикла;

(221) а над чем особо смеяться? по сути верный подход, но мало опыта
Интересно, что будет работать быстрее: поиск и обработка строк, или поиск строки в отсортированной таблице и обработка этого участка

(0) странный вопрос. Мой код моя собственность что-ли?