"PHP" / Говнокод #8760 Ссылка на оригинал

0

  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 mimits89, (Updated )

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

    • скорее всего из-за сочетания 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

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

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

Переведи на "PHP", guest!

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


    8