Бухгалтерия / Говнокод #2498 Ссылка на оригинал

0

  1. 1
Выбрать * из РегистрСведений.Таблица Где Таблица.ДатаПодтверждения >= ДАТАВРЕМЯ(1, 1, 1, 0, 0, 0))

Это вот условие писал мой бывший начальник..

Запостил: KingOfShit KingOfShit, (Updated )

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

  • ava guest # Ссылка на оригинал 0

    лучшие тому подтверждение, начальник? руководи!
    Ответить

  • ava totparen # Ссылка на оригинал 0

    Что не так? Поясниет. )) ХОчу знать говнокод в лицо!
    Ответить

    • ava Трактор # Ссылка на оригинал 0 ⇈

      Условие ненужно. Совсем. Оно всегда будет истинно.
      Можно просто написать
      Выбрать * из РегистрСведений.Таблица

      Но начальнику, ИМХО, простительно. Человек развлекается.

      А вот что за гвонокод коцает русские буквы на этой странице? Вот в этом сообщении слово Таблица покоцано. Кривая расцветка синтаксиса, похоже.
      Ответить

      • ava nil # Ссылка на оригинал 0 ⇈

        А могло так быть, что иногда надо там что-то полезное ставить? Скажем, большую часть времени, чтобы синтаксис не менять, не комментарить, там вот это, а иногда реальная дата?
        Я 1С не знаю, но в сишном коде при отладке в условии if(some_check) можно делать if(0 && some_check), чтобы временно отшибить ветвь кода (или 1 || чтобы наоборот, включить всегда).
        И, например, код в таком состоянии и попал на рассмотрение?
        Ответить

        • ava KingOfShit # Ссылка на оригинал 0 ⇈

          Этот код находился не на стадии отладки. Он был внедрен и функционировал
          Ответить

        • ava Трактор # Ссылка на оригинал 0 ⇈

          Такое возможно, но тогда запрос выглядел бы иначе. Например так:
          Запрос.Текст = "Выбрать * из РегистрСведений.Таблица Где Таблица.ДатаПодтверждения >= ДАТАВРЕМЯ(&НекаяДата))";
          Запрос.УстановитьПараметр("НекаяДата", '00010101000000');
          Это бы подразумевало что в будущем мы заменим константу '00010101000000' на переменную, которую ещё не придумали.
          Ответить

          • ava KingOfShit # Ссылка на оригинал 0 ⇈

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

            • ava Трактор # Ссылка на оригинал 0 ⇈

              Ну, блин, если ты так придираешься, то в твоём примере количество открываемых/закрываемых скобок не совпадает :-)~
              И 1С не пропустит твой запрос без слова КАК.
              ПереписАл
              Запрос.Текст = "Выбрать * из РегистрСведений.Таблица КАК Таблица Где Таблица.ДатаПодтверждения >= &НекаяДата";
Запрос.УстановитьПараметр("НекаяДата", '00010101000000');
              Ответить

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓