- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
if ($_REQUEST['name']){
$nameProduct = $_REQUEST['name'];
$products = $db->products($_REQUEST['name']);
} else {
$products = 'error';
}
if ($products == 'error'){
$http->redirect('index.php');
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+158
if ($_REQUEST['name']){
$nameProduct = $_REQUEST['name'];
$products = $db->products($_REQUEST['name']);
} else {
$products = 'error';
}
if ($products == 'error'){
$http->redirect('index.php');
}
Из частного проекта.
Во-первых почему-бы не использовать $nameProduct второй раз вместо $_REQUEST['name']?
Во-вторых вместо $products = 'error'; можно сразу было сделать редирект, т.к. все последующие использования $products не имеют смысла, если редирект выполнен.
cheef 06.09.2010 12:34 # +1
и это тоже доставляет
Анонимус 06.09.2010 13:26 # 0
Morgan 06.09.2010 15:58 # 0
а если скрипт заканчивает работу не сразу после редиректа?
mikhailu 06.09.2010 17:16 # 0
Анонимус 06.09.2010 17:42 # 0
Браузер может перейти на новый локейшен, а может и не перейти (если бажный) или перейти но не сразу.