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

    • не смейся над школьниками. которые ещё думать не научились, а кодить уже пытаютяся.
      Ответить
      • Школьники или нет - неизвестно, но на сайте написано, что продукт надёжный, гибкий, доступный, и юзается в более чем миллионе интернет-магазинов :)
        Ответить
        • и оттуда надёжно валятся миллионы ошибок в миллионах магазинов
          Ответить
    • Забавно :)
      Ответить
    • Может они забыли !
      хотели написать:
      if(!isset($_GET['catId'])){
      ...
      } else if($_GET['catId']>0){
      ...
      }
      Ответить
    • по логике тут всё нормально, если юзер отправил script.php?catId=0, isset вернет true, т.к. переменная установлена

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

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