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

    −130

    1. 1
    2. 2
    3. 3
    Если Конт.Сумма <> КонтСумма Тогда
    Конт.Сумма = КонтСумма;
КонецЕсли;

    Встретил в типовой конфигурации одной из небольших фирм-франчайзи. Много думал ...

    Запостил: SSNick, 04 Августа 2010

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

    • ava TarasB 04.08.2010 19:23 # −3

      Воткнули для отладки, потом убрать забыли.
      Ответить

      • ava SSNick 04.08.2010 22:39 # −6

        показать все, что скрытоА почему было просто не написать одной строкой: Конт.Сумма = КонтСумма; ?
        Какой тайный смысл несёт в себе проверка условия?
        Ответить

        • ava Kornew 04.08.2010 22:56 # +8

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

          а сейчас стоит: "у нас чот не сходится?: Тссс... оно всё сходится, то вам показалось..."
          Ответить

          • ava 3.14159265 05.08.2010 14:19 # −2

            думаю просто в 1С никогда не слышали о ассертах
            или скорее всего ничего не планировалось
            ведь это 1С, детка.
            Ответить

            • ava sergeante 06.04.2011 09:19 # −2

              Что такое ассерт, отец?
              Ответить

              • ava 3.14159265 19.01.2012 14:15 # +1

                Извиняй, опечтался.
                Имел в виду Ассрет. Ass - это жопа. Короче assret - "жопа срет".
                Ответить

        • ava TarasB 05.08.2010 17:37 # +2

          Блин, для отладки. Чтобы пославить точку прерывания, которая сработает только по условию.
          Ответить

          • ava bugmenot 15.08.2010 14:42 # −1

            Лично я предпочёл бы не знать из чего там сделан отладчик...
            Ответить

    • ava rudolff 05.08.2010 12:11 # +1

      Оптимизация :)
      Ответить

    • ava yudakov 18.08.2010 22:52 # +11

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

      Это отнюдь не говнокод, это тест на профпригодность :)
      Ответить

      • ava Dimly 25.11.2010 09:08 # +1

        +1
        Рад что среди 1С програмистов остались люди с головой.
        Ответить

      • ava sergeante 06.04.2011 09:23 # 0

        Ээээ, а позвольте узнать, что такое "Конт". Предполагаю что контрагент, но какое тогда отношение сумма контрагента имеет к модифицированность формы?
        Ответить

      • ava guest 10.04.2011 02:03 # 0

        С чего вдруг это баг? Присвоение было? Было. Форма модифицирована? А то ж!
        Ответить

      • ava guest 16.08.2011 17:00 # 0

        +1 Абсолютно здоровый код
        Ответить

      • ava bugmenot 19.01.2012 11:12 # +1

        перевожу с 1С на русский: чтобы сеттер с побочным эффектом зря не дергать
        Ответить

        • ava 3.14159265 19.01.2012 14:08 # 0

          >перевожу с 1С на русский
          Сурово.

          Ну а сахарок в виде неявных сеттеров чудесным образом превращается в такое.
          Короче говно в языке.
          Ответить

    • ava Xbrut 19.01.2012 05:06 # 0

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

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