Лучший ответ

    1.  1 0

    contact_us (38) 7 (20588)458158 12 лет  

    switch($_GET['module'])
    {
        case "module3": include ('module3.php'); break;
        case "module4": include ('module4.php'); break;
        case "module5": include ('module5.php'); break;
        case "module6": include ('module6.php'); break;
        case "module8": include ('module8.php'); break;
        case "module9": include ('module9.php'); break;
        case "module10": include ('module10.php'); break;
        case "module1":
        case "module2":
        case "module7":
        default:
            include ('left.php');
            include ('main.php');
            include ('right.php');
        break;
    }


    И вообще, это говнокод какой-то. Можно всё это улучшить. Если интересно, пиши в личку.

Ответы

    1.  0 2

    fafanja 7 (20590)51760 12 лет  

    ты неправильно делаешь... можно хотябы так:

    if ($_GET['module'] < 1 || $_GET['module'] > 10) { die(); }

    if ($_GET['module'] == 1 || $_GET['module'] == 2 || $_GET['module'] == 7) {
    include ('left.php');
    }
    include('module'.$_GET['module'].'.php');
    if ($_GET['module'] == 1 || $_GET['module'] == 2 || $_GET['module'] == 7) {
    include ('right.php');
    }

    но лучше использовать массивы

Похожие вопросы

Рейтинг@Mail.ru Top.LV PULS.LV Professional rating system