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

    • ava Lure Of Chaos 07.12.2011 23:24 # +1

      вы из-за 4ой строчки запостили?
      Ответить

      • ava istem 07.12.2011 23:50 # +1

        скорее всего из-за сочетания 2-ой и 3-ей (ну и пятой до кучи)
        Ответить

    • ava Vasiliy 08.12.2011 08:22 # −3

      Может я еще сплю. Но говна не вижу.
      Ответить

      • ava vkontakte 08.12.2011 10:57 # +3

        Ну как так?
        Данный свич думаю обрабатывает переменную $_GET['act'], и если она будет 'delete', то мы зайдем в данный кусок кода, и что мы видим? Если в адресной строке index.php?act=delete&id=5 мы принудительно уберем id=5, то нажав энтер увидим ошибку на экране о том, что было обращение к переменной $_GET['id'], и потому сначала должна быть проверка if(isset($_GET['id'])), а уж потом обращение к ней. Говно, плюсую!
        Ответить

        • ava Vasiliy 08.12.2011 19:23 # −1

          >то нажав энтер увидим ошибку на экране о том, что было обращение к переменной $_GET['id'
          Вывод ошибок мог быть подавлен.
          по мне автор просто прозевал.
          Ответить

      • ava jokz 08.12.2011 13:01 # +1

        вы еще спите
        Ответить

      • ava nobody 08.12.2011 13:11 # +1

        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

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

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