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

вот чего не выношу - так это неряшливости в коде, в смысле форматирования. И плохочитаемо, и скачки эти раздражают.
Хорошо 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) странный вопрос. Мой код моя собственность что-ли?