1. 1C / Говнокод #1723

    −117.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    Если Приказ.Выбран() = 1 Тогда
			Если Приказ.Вид() = "Приказ" Тогда
				// это все-таки приказ
				Если Приказ.ТипПриказа <> Перечисление.ТипыПриказов.НаВнутреннееСовмещение Тогда
					// но приказ другой, не внутреннее совмещение
					Продолжить;
				КонецЕсли;
			Иначе
				Продолжить;
			КонецЕсли;
		Иначе
			Продолжить;
		КонецЕсли;

    Долой минимализм!

    Наговнокодили: Ингресс-лайн

    Запостил: Полупрограммисты, 29 Августа 2009

    Комментарии (49) RSS

    • ava Nagg 29.08.2009 21:44 # −2

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

    • ava guest 30.08.2009 09:37 # −4

      Nagg, опередил)
      Ответить

    • ava farid 30.08.2009 13:18 # 0

      А что здесь не так?
      Ответить

      • ava viktorious 31.08.2009 15:43 # 0

        Я не знаю язык в 1С, но мне кажется, вместо кучки Если и тройного "продолжить", можно было бы использовать логическую конструкцию, что-то вроде:
        Если Приказ.Выбран() = 1 И Приказ.Вид() = "Приказ" И Приказ.ТипПриказа = Перечисление.ТипыПриказов.НаВнутреннееСо вмещение Тогда
        Иначе
        Продолжить
        КонецЕсли;
        Ответить

        • ava farid 31.08.2009 17:21 # 0

          Строка Приказ.Выбран() означает проверку того что объект типа документ или справочник спозиционирован и существует. Если мы ее исключим, существует вероятность схвотить эксепшен, так как объект не спозиционирован.
          Далее после определения вида документа или справочника «Приказ» - из кода этого не понятно, идет обращение непосредсвенно к реквизиту объетка. Если мы определение вида исключим, тогда есть вариант, что у этого объекта нет такого реквизита, и мы опять сможем схватить эксепшен.
          Привидений код может и не является красивым, но он ГАРАНТИРОВАНО отработает то, что ему полагается.
          Ответить

          • ava farid 31.08.2009 17:27 # 0

            Максимум, что можно свернуть, это два первых условия, и то надо смотреть по контексту.
            Ответить

          • ava Полупрограммисты 01.09.2009 15:26 # 0

            Если при создании конфигурации ты определяешь тип своего реквизита, как "Документ.Приказ" - то проверять приказ ли это - это паранойя.
            Ответить

            • ava guest 06.10.2009 09:56 # 0

              Ну в коде то не видно, какой тип у этого реквизита
              Ответить

          • ava viktorious 04.09.2009 11:14 # 0

            Разве 1С не умеет анализировать логические конструкции?
            По идее, если Приказ.Выбран() = 1 не будет истинным, то остальные условия и не должны выполняться в любом более-менее вменяемом современном языке, и эксепшиона не будет.
            Конечно, если 1С принудительно выполняет все элементы логической конструкции, тогда согласен.
            Ответить

            • ava guest 18.09.2009 15:25 # 0

              Это код для 1с 7.7 - она не оптимизирует вычисление булевых выражений. В 8.х такого уже нет.
              Ответить

              • ava guest 06.10.2009 09:56 # 0

                Ух ты, не знал, что в 8.x уже все лучше.
                Ответить

              • ava guest6 10.01.2023 18:33 # 0

                Обоже мой


                То есть если сказать "if FOO and BAR" то проверятся оба условия, даже если FOO ложно?

                Ужасно. Хорошо, что 7.7 уже семнадцать лет как неактуальна
                Ответить

                • ava HoBorogHuu_nemyx 10.01.2023 18:38 # 0

                  Когда восьмая версия только появилась, её закидывали какашками.
                  Ответить

                  • ava guest6 10.01.2023 18:44 # 0

                    Потому что тупила наверное, ну и сам факт разворота SQL сервера всех смущал

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

                    • ava Desktop 10.01.2023 18:46 # 0

                      > в сторону нормального приложения
                      > с веб-интерфейсным клиентом
                      – странно это слышать от тебя
                      Ответить

                      • ava guest6 10.01.2023 18:50 # 0

                        Мы живем в мире постправды, и я сам не знаю, что я думаю. Я слышал много нахрюков на 1С-овые десктопные клиенты как то:
                        * Реализовать доступ пяти тыщ питухов проще на веб-сайт, чем на RDP ферму с клиентами (сильно проще)
                        * Десктопный клиент всегда тормозит
                        * Используется конечно какой-то пизданутый порт, который поди еще открой везде
                        * Для дебага там чуть ли не сервер подключается к клиенту, так что через NAT вообще плохо
                        Ответить

            • ava guest 24.12.2009 16:14 # 0

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

              • ava guest6 10.01.2023 18:34 # 0

                >Возможно, я тебя шокирую, но во многих языках порядок проверки предикатов неопределён.

                Расскажи пожалуйста подробнее
                Ответить

      • ava guest6 10.01.2023 18:33 # 0

        Во-первых сравнение со строковым литералом как-то не очень (я правда не знаю, насколько это не очень в 1С)
        Во-вторых все три условия можно совместить.
        Ответить

    • ava guest 30.08.2009 14:53 # −1

      ненавидь 1с
      Ответить

    • ava Полупрограммисты 01.09.2009 15:35 # 0

      Вся эта полная ересь может быть заменена такой конструкцией:
      Если ПустоеЗначение(Приказ) = 1 Тогда
      продолжить;
      КонецЕсли;
      Если Приказ.ТипПриказа <> Перечисление.ТипыПриказов.НаВнутреннееСо вмещение Тогда
      продолжить;
      КонецЕсли;
      Ответить

      • ava guest 24.12.2009 16:16 # 0

        Первое условие в вашем коде неверно.
        А если в целом - не такая уж и ересь, вы не согласны?
        Ответить

      • ava cmepmop 18.04.2019 01:31 # 0

        Как ты зарегала такой ник?
        Ответить

        • ava AHCKuJlbHblu_nemyx 19.04.2019 00:43 # 0

          Как ты понял, что она тян?
          Ответить

          • ava gost 19.04.2019 00:45 # 0

            Всё просто: у неё на аватарке в Бормандостоке — тян.
            Ответить

            • ava AHCKuJlbHblu_nemyx 19.04.2019 00:51 # −1

              Привет. Не желаешь со мной замутить? Ты из какого города?
              Ответить

              • ava gost 19.04.2019 01:19 # 0

                Прости, но у нас ничего не выйдет. Я — 2D-тян, а ты —петух, да ещё и анскильный.
                Ответить

                • ava Xyu_HE_3HAET 19.04.2019 04:24 # 0

                  Ауменя?
                  Ответить

                  • ava gost 19.04.2019 14:42 # 0

                    Хуй зна… А, ну да. Тогда никто больше не знает.
                    Ответить

                • ava Desktop 10.01.2023 18:05 # 0

                  минусминуспетух?
                  Ответить

                  • ava guest6 10.01.2023 18:29 # 0

                    MASHENKA. Плюс-реал. Я поимела двинуть на "Tsunami", я поимела плюс-похорошо в hoчу, я поимела 25% -free-пяо, я двинула с "Медведково" на "Olimpus", я поимела до-до-pauza-yebi-их, я поимела видеть много-много чоуди-policе-шагуа, я поимела видеть много-много mobi-robi, я поимела минус-похорошо в hoчу, я поимела много-много до-до-svalka-uyebi-их, я поимела razрыв-по-юйи-obo-еbo, я поимела много-много stopin-klopin, я поимела krik-по-трейсу, я поимела минус-минус-минус-похорошо в hoчу, я поимела porvaть пяо!
                    Ответить

                • ava guest6 10.01.2023 18:16 # 0

                  Петух с два-дэ тянкой
                  Они, если честно,
                  Ответить

    • ava Holden 03.09.2009 10:45 # 0

      (Почитал. Вспомнил. Всплакнул.)
      А програмер молодец - разумные коменты пишет.
      Ответить

    • ava guest 18.09.2009 15:14 # 0

      Мдя, высокоинтеллектуальный код. Похож на код Апокалипсиса.
      Ответить

    • ava informatikmarazmatik 16.05.2012 18:38 # 0

      Зарплата и кадры?
      Ну она же?!
      Не? Точно зарплата и управление персоналом!
      Ответить

    • ava guest6 10.01.2023 18:24 # 0 

      КонецПоложить;
      Ответить

    Добавить комментарий