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

    +156

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    case "delete":
       $id=$_GET['id'];
              if(!isset($id)){
              // no id? GTFO   
             header('Location: index.php?go=manage_costumes&act=view'); 
             exit();
             }else{
            // id exists
    ...

    )))

    Запостил: mimits89, 07 Декабря 2011

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

    • вы из-за 4ой строчки запостили?
      Ответить
      • скорее всего из-за сочетания 2-ой и 3-ей (ну и пятой до кучи)
        Ответить
    • Может я еще сплю. Но говна не вижу.
      Ответить
      • Ну как так?
        Данный свич думаю обрабатывает переменную $_GET['act'], и если она будет 'delete', то мы зайдем в данный кусок кода, и что мы видим? Если в адресной строке index.php?act=delete&id=5 мы принудительно уберем id=5, то нажав энтер увидим ошибку на экране о том, что было обращение к переменной $_GET['id'], и потому сначала должна быть проверка if(isset($_GET['id'])), а уж потом обращение к ней. Говно, плюсую!
        Ответить
        • >то нажав энтер увидим ошибку на экране о том, что было обращение к переменной $_GET['id'
          Вывод ошибок мог быть подавлен.
          по мне автор просто прозевал.
          Ответить
      • вы еще спите
        Ответить
      • case "delete":
        $id=(int)$_GET['id'];
        if($id <= 0){
        // no id? GTFO
        header('Location: index.php?go=manage_costumes&act=view');
        exit();
        }else{
        // id exists

        Так я думаю правильнее, в данном случае ...
        Ответить

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