1. SQL / Говнокод #1027

    −855

    1. 1
    2. 2
    if (nvl(pSubject_ID, 0) = 0) then
   ...

    Вот так матёрые банковские программеры пишут "is null" на PL/SQL

    Запостил: guest, 12 Мая 2009

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

    • ava guest 13.05.2009 07:00 # 0

      cyberone:
      Возможно это связано с тем что в oracle кроме null есть еще что-то типа undef, много раз встречал такое и не в банках.
      Ответить

    • ava guest 13.05.2009 22:03 # +3

      Formidable:
      Матерые банковские программеры пишут (pSubject_ID is NULL) OR (pSubject_ID = 0), а nvl короче и понятней для тех кто в теме
      Ответить

    • ava guest 25.05.2009 10:33 # 0

      Дмитрий:
      Не факт, что это IS NULL. Может быть сплошь и рядом ситуация, что действие выполняется, когда значение пусто или равно нулю. Согласен с предыдущим аффтаром - тут nvl к месту.
      Ответить

    • ava guest 21.07.2009 23:24 # 0

      Ivan:
      Если выбралось 0 строчек с pSubject_ID - то результат не null: is null не подойдет
      Ответить

    • ava guest 10.08.2009 16:32 # 0

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

    • ava guest 11.08.2009 10:46 # 0

      У нас на проекте такой код используется генератором запросов чтобы не делать лишнюю проверку и сохранить структуру запроса независимо от пришедшего значения. Тоесть чтоб не писать is null когда приходит NULL а всегда писать where nvl(field, -1) = nvl(value, -1)
      ИМХО говнокодом тут не пахнет
      Ответить

    • ava guest 13.08.2009 15:43 # 0

      аффтар, включи моск
      Ответить

    • ava guest 18.08.2009 00:30 # 0

      Не говнокод. Негодую.
      Ответить

    • ava guest 27.10.2009 19:19 # 0

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

      вывод - из говнокода убрать, автора забанить к ебениматери придурка
      Ответить

    • ava guest 12.12.2009 01:10 # 0

      Поддержу негодование.
      Ответить

    • ava guest 25.12.2009 16:14 # 0

      автор, 0 IS NOT NULL
      Ответить

    • ava guest 15.03.2010 10:29 # 0

      Скорее всего для того чтобы index - nvl(pSubject_ID, 0) работал так как на is null индекс не построить
      Ответить

    • ava guest 21.05.2010 11:51 # 0

      скорее всего это вместо if (pSubject_ID is null or pSubject_ID=0)
      ничего говнокодерского
      Ответить

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