- 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');
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 1
+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 не имеют смысла, если редирект выполнен.