PHP Frontpage/页面控制器

Question

我使用以下内容作为 Frontpage/Page 控制器，到目前为止它工作正常，除了我面临的两个问题，如您所见，$pages 数组和开关，它们实际上比我的要长得多已经粘贴到这里了每次需要新的页面控制器时，我都必须将其添加到 $pages 数组中并进行切换，这使得该列表非常长。您将如何克服这个问题？您是否看到此代码有任何其他改进？页面控制器中的loadLogic()用于获取pages/controllername/logic/function.php下的函数。

Frontpage 控制器 - index.php：

include 'common/common.php';
if(!isset($_GET['p']) OR $_GET['p'] == ''){
    $_GET['p'] = 'home';
    header('Location: index.php?p=home');
}

$pages = array('home','register','login','logout','page1','page2','page3');

$_GET['p'] = trim($_GET['p']);

if(isset($_GET['p'])){
    if(in_array($_GET['p'], $pages)){
        switch ($_GET['p']) {
            case 'home':
                include 'home.php';
                break;
            case 'register':
                include 'register.php';
                break;
            case 'login':
                include 'login.php';
                break;
            case 'logout':
                include 'logout.php';
                break;
            case 'page1':
                include 'page1.php';
                break;
            case 'page2':
                include 'page2.php';
                break;
            case 'page3':
                include 'page3.php';
                break;
        }
    }else{
        echo '404!';
    }
}

页面控制器 - {home,register,login,logout,page1,page2,page3}.php：

include 'tpl/common/header.php';
contentStart();

if(isset($_SESSION['logged'])){
    loadLogic('dashboard');

}else{
    loadLogic('nologin');

}


//Display login form in logic page instead links
//
if(!isset($_SESSION['logged'])){
    contentEnd();
    loadLogic('nologinForm');
}else{
  contentEnd();
  include'tpl/common/rcol.php';

}
include 'tpl/common/footer.php';

函数 loadLogic()：

function loadLogic($logic) {
    $path = dirname(__DIR__) . '/pages';

    $controller = preg_split('/&/',$_SERVER['QUERY_STRING']);
    $controller = trim($controller[0],"p=");
    $logicPath = 'logic';
    $logic = $logic . '.php';
    $err = 0;
    $logicFullPath = $path.'/'.$controller.'/'.$logicPath.'/'.$logic;

    if($err == '0'){
        include "$logicFullPath";

    }
}

文件夹结构：

projectName
  |
   ---> common
  |
   ---> pages
  |   |
  |    --->home
  |   |
  |    --->register
  |   |
  |    --->login
  |   |
  |    --->logout
  |   |
  |    --->page1
  |   |
  |    --->page2
  |   |
  |    --->page3
  |
   ---> tpl
  |   |
  |    ---> common
  |
   --> home.php
  |
   --> register.php
  |
   --> login.php
  |
   --> logout.php
  |
   --> page1.php
  |
   --> page2.php
  |
   --> page3.php

Answer 1

对于frontpage控制器，为什么有这么多case语句。您已经知道哪些页面可以有效包含，并检查它是否在有效页面中。

你可以这样做：

if(isset($_GET['p']))
{
    if(in_array($_GET['p'], $pages))
    {
         include($_GET['p'] . '.php');
    }
}

如果你希望它有不同的名称传递给 $_GET 进行混淆，以及不同的潜在扩展，那么你可以这样做：

$pages = array('home'=>'index.php','register'=>'registerpage.htm','page1'=>'one.html');

if(isset($_GET['p']))
{
    if(array_key_exists($_GET['p'],$pages))
    {
          include($pages[$_GET['p']]);
    }
}

如果你想让页面数组更容易管理，你可以将它分成多个lines:

$pages = array(
'home'=>'index.php',
'register'=>'registerpage.htm',
'etc'=>'/home/user/public_html/directory/etc.php'
);

哦，还有，因为主页 http://www.domain 有点难看.com/?p=home，如果 p 的值不在数组中或不是数组键（具体取决于您使用的值），则只需将 home 设为默认包含即可。

所以：

if(isset($_GET['p']) && in_array($_GET['p'],$pages)) //You can combine these like this as well, same functionality.  If p isn't set, it won't even try the in_array()
{
    include($_GET['p'] . '.php');
}
else
{
    include('home.php');
}

那么你可以去掉 if !isset $_GET['p'] or $_GET['p'] == '' 在顶部。您还可以使用 && 将 isset 和 in_array/array_key_exists 组合到同一 if 语句中。如果第一次评估是错误的并且它命中了 &&然后它只是停止并且不评估其余部分，所以没有错误或任何东西，这也意味着您可以轻松地设置一次默认响应，因为嵌套它们意味着您必须为两个 if 都有一个默认值。

更多编辑。如果您确实希望在用户尝试访问不存在的 ap= 时出现 404，而不只是引导到主页，您可以在顶部执行此操作：

if(isset($_GET['p']))
{
$fourohfour = true; 
}

然后在 If 结构中向下包含页面，在 else include home.php 之前执行 else if ，例如：

else if(isset($fourohfour))
{
    include('404.php');
}

这样，如果 p 已设置但未签出，则它将包含 404，但如果未设置，它将转到 home.php