Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5698)
2. C++ (2733)
3. Куча (2326)
4. JavaScript (2027)
5. C# (1925)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (197)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   −132

   1. 1
   2. 2

   if isnull(@ProfileID, 0) <> 0
   ...

   bullshit

   Запостил: fr0mrus, 16 Марта 2012

   Tweet

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

   • DBdev 16.03.2012 15:56 # +3

     I don't think so
     It's just to be ensure that 0 and NULL have been compared in the same way.

     Ответить

     • N21 21.09.2012 16:59 # 0

       Глупости. 0 в принципе не может быть равно NULL, так что вполне достаточно if @ProfileID <> 0.

       Ответить

       • bormand 21.09.2012 17:05 # 0

         > 0 в принципе не может быть равно NULL
         Ну вот автор и добивается этим кодом, чтобы NULL и 0 приводили к одинаковому результату.
         
         > так что вполне достаточно if @ProfileID <> 0
         if @ProfileID <> 0 and @ProfileID is not null

         Ответить

         • bormand 21.09.2012 17:21 # 0

           Хотя я походу зафейлил, NULL <> 0 это NULL, NULL в if зачтется как false. Т.е. код @ProfileID <> 0 корректен. Но тогда есть вариант еще проще:

           if @ProfileID

           Или так можно не во всех субд?

           Ответить

   • guest 30.03.2012 00:57 # 0

     По-моему все нормально.
     Это эквивалентно:
     if (@ProfileID<>0) or (@ProfileID is not null)

     Ответить

     • 3.14159265 23.04.2012 21:20 # 0

       > (@ProfileID<>0) or (@ProfileID is not null)
       Во-первых это длиннее, во-вторых сейчас оно в ife, а завтра может переехать в where. А вот там OR нежелателен.

       Ответить

     • bormand 21.09.2012 17:10 # 0

       > or
       and

       Ответить

   • guest 26.05.2012 16:58 # 0

     Может проще так?:
     

     if @ProfileID <> 0
     ...

     
     
     Хотя второй ответ, тоже достоин уважения. :-)

     Ответить

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

   Ошибка компиляции комментария:

   1. Гости могут высказаться только в понедельник, среду, четверг или воскресение

   Помни, guest, за тобой могут следить!

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

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

   Проверочный код: *