1. Список говнокодов пользователя mikhailu

    Всего: 1

  2. PHP / Говнокод #4176

    +158

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    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 не имеют смысла, если редирект выполнен.

    mikhailu, 06 Сентября 2010

    Комментарии (5)