- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 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
...
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+156
case "delete":
$id=$_GET['id'];
if(!isset($id)){
// no id? GTFO
header('Location: index.php?go=manage_costumes&act=view');
exit();
}else{
// id exists
...
)))
Lure Of Chaos 07.12.2011 23:24 # +1
istem 07.12.2011 23:50 # +1
Vasiliy 08.12.2011 08:22 # −3
vkontakte 08.12.2011 10:57 # +3
Данный свич думаю обрабатывает переменную $_GET['act'], и если она будет 'delete', то мы зайдем в данный кусок кода, и что мы видим? Если в адресной строке index.php?act=delete&id=5 мы принудительно уберем id=5, то нажав энтер увидим ошибку на экране о том, что было обращение к переменной $_GET['id'], и потому сначала должна быть проверка if(isset($_GET['id'])), а уж потом обращение к ней. Говно, плюсую!
Vasiliy 08.12.2011 19:23 # −1
Вывод ошибок мог быть подавлен.
по мне автор просто прозевал.
jokz 08.12.2011 13:01 # +1
nobody 08.12.2011 13:11 # +1
$id=(int)$_GET['id'];
if($id <= 0){
// no id? GTFO
header('Location: index.php?go=manage_costumes&act=view');
exit();
}else{
// id exists
Так я думаю правильнее, в данном случае ...