Спагетти код и что с ним делать #228291

что делать с множеством вложенных друг в друга if'ов? сейчас дошел до такого кода :

вам непонятно, что написано в этой процедуре?

Дошел? Тогда разворачивайся и возвращайся....

Переделай в:

От этого я и ухожу..

А зачем уходишь? Тебе это непонятно?

ага.. потом хрен разберешь что зачем..

Пиши пояснения в начале

Ну это наверное все индивидуально, мне например больше понравился бы

ЧЕМ??

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

-------------------------------------------------------------------------------- Запрещается использование вместо вложенных конструкций Если...КонецЕсли конструкций типа Если...ИначеЕсли...ИначеЕсли... с «обратными» условиями, как сложные для анализа, модификации и отладки. Пример Конструкция: должна быть заменена на конструкцию:

это откуда?

Система стандартов и методик разработки конфигураций для платформы 1С Предприятие 8.0 Версия 8.1.0.12

Стесняюсь спросить... А где в вложенные if ?

а НЕТУ! я от них ухожу!

Интересный термин "обратные" условия. как родился интересно... Они ему не нравились - - то как он от них избавился

бугага... валяюсь...

ф двадцатке..

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

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

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

не так забавно как кажется..

Если ты в трех соснах разобраться не можешь, как же ты в реально сложном коде разбираешся?...

1) реальный код не дожен быть сложным 2) в реальном коде комментариев должно быть столько же сколько кода

Наивный

В 1С код - "самодокументируемый" :)

Тебе не пофиг (положа руку на сердце)?

а разве вы так не делаете?

неа.. потом мне с этим разибраться...

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

>>реальный код не дожен быть сложным LOL! Поучи пожалуйста программистов фирмы 1С реальный код в типовых писать ;)...

если заплатят - с удовольствием..

Наивный.:) Как тебе такое творчество (последние несколько минут)? //______________________ Разбираться кто-то будет потом... Хотя надо работать, а не разбираться. Если код неподьемный для тебя - переписываешь все нафиг. Бывает такое, особенно в нетиповых, где код разными людьми, переписывался и обрастал костылями.

тут бы у меня было 5 строчек + 2 процедуры + 10 строчек комментов..

Если количество строк внутри условия или цикла больше 10, то пишу текст условия в конце условия / цикла    Если Условие2 Тогда

А тут?

Было бы... А я уже запустил - все почти работает. Сейчас подрихтую и через пару минут все пойдет в эксплуатацию. 1С - это быстро-грязная разработка. Редко бывает, что нужен действительно хороший код, да и то, в основном из-за производительности.:)

Модуль Закрытия месяца - вообще памятник.:)

я бы уволил...

LOL! А Нуралиев то не знает ;)))...

За, что? Позвольте поинтересоваться.

за безграмотность...

я валяюся

мой начальник отдела так в закрытии месяца не разобрался..

Давайте я у вас буду начальником, удаленно? Не помню сколько раз уже этот модуль правил. Да и судя по всему тоже.:) Так, что модуль тут не при чем.

Выньте меня из под стула...

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

Маски шоу прям

Клиенты ведь разные все.

Ему делать более некуй в жизни... По воробьям стреляете...

+ 1 жесть...

:) Работать не будет. Че бум в док присваивать, если не нашли по коду? - наш человек! Идиоты. Слово Case видимо не слышали. Общий подход - совершенно правильный. Обратная логика. Если цикл - все что не нужно - перебирать выносится в начало цикла. После блока фильтров - совершенно понятно, что тут, только то, что нужно обрабатывать. Весь фильтр со всей логикой виден сразу, а не размазан по пятистам страницам кода.

>Весь фильтр со всей логикой виден сразу, а не размазан по пятистам страницам кода. эт смотря какой фильтр... :(

>> мой начальник отдела так в закрытии месяца не разобрался заповедник какой-то ;)))...

а еще у нас есть линуксоид.. который дома 1С запускает под VM ware

Бывает. Что до стиля кода 1С - увы, не пример для подражания. В нормальных конторах, за два вложенных ифа - с программером не разговаривают, а за три - ваще морду бьют! :)

В нормальных конторах ифы не считают...

В нормальных - смотрят за стилем.

В нормальных  есть своя нотация и мозги друг другу количеством if на квадратный сантиметр не канифолят.

Значит Нуралиев не доглядывает...

а у вас как с сабжем?

1. Нашли - Спр.ТекущийЭлемент; Не нашли, создали, все равно -  Спр.ТекущийЭлемент;  Я вот тоже линуксоид, только под Wine семерку запускаю. И при чем здесь код?

>И при чем здесь код? >заповедник какой-то ;)))...

Ты странный какой-то. О каком качестве кода в среде 1С может идти речь если предназначение большинства проектов это не создание нетленки а подписание акта и вышибание денег из заказчика? Тут все средства хороши, и даже goto.

Ниче не понял...

хм.. а потом еще взять денег за рефакторинг кода?

А ваапчето Шаблон

теперь я не понял..

От ситуации зависит. Есть книжка хорошая "Совершенный код" - стараюсь следовать её рекомендации. Если условия сложные - упрощаем, а если простые - то и так сойдет - меньше шансов заблудиться в переменных (не создавайте сущности без необходимости).

часто помогает ... Правдо ошибки "сыпать" будет, но отработает

+

+ если надо очень

спасибо, помедитируюю...

Либо сразу так ... Попытка    ВсеПринтеры.УдалитьЗначение(ВыбранныеПринтеры.НайтиЗначение(ВсеПринтеры.ПолучитьЗначение(ВСеПРинтеры.ТекущаяСтрока))) ;Исключение

жесть. ;)

поддерживаю автора :) мне тоже проще читать код в , чем в .

Развод! Стопудово! :) Чтоб разбираться, нужно комментировать при написании, и все понятно будет позже.

структура кода и комментарий суть разные понятие.

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

ап

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

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

триллер прям какой-то...

Правило №0: есть код работает правильно - не трожь. Займись тем, что еще не автоматизировано.

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

И дались вам эти IF! Пишите как умеете до комментарий оставляйте c ФИО, чтоб было кого вспоминать матами ;)

А я в комментах вообче анекдоты травлю... када сам не могу обьяснить что я тама(в коде) понаписал... %)

Ага. примерно такое: