1. PHP / Говнокод #1894

    +173

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if(isset($_GET['catId'])){
...
} else if($_GET['catId']>0){
...
}

    CubeCart v3.0.15

    Запостил: Ghost, 24 Сентября 2009

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

    • ava guest 25.09.2009 09:42 # −1

      не смейся над школьниками. которые ещё думать не научились, а кодить уже пытаютяся.
      Ответить

      • ava Ghost 25.09.2009 13:34 # 0

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

        • ava guest 25.09.2009 13:39 # 0

          и оттуда надёжно валятся миллионы ошибок в миллионах магазинов
          Ответить

    • ava nechin 25.09.2009 11:29 # 0

      Забавно :)
      Ответить

    • ava guest 29.09.2009 16:13 # −1

      Может они забыли !
      хотели написать:
      if(!isset($_GET['catId'])){
      ...
      } else if($_GET['catId']>0){
      ...
      }
      Ответить

    • ava Crec 30.09.2009 11:43 # 0

      по логике тут всё нормально, если юзер отправил script.php?catId=0, isset вернет true, т.к. переменная установлена

      другое дело что возможно лучше было бы заменить два условия на одно:
      if( !$_GET['catId'] )
      но тут уже зависит от задачи
      Ответить

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