优化 PHP If/Else 语句

Question

我正在尝试优化以下 PHP If/Else 语句。我可以重写代码以使用 case 和 switch，还是应该保持原样，或者什么？

代码：

if(empty($_GET['id'])){
    include('pages/home.php');
}elseif ($_GET['id'] === '13') {
    include('pages/servicestatus.php');
}elseif(!empty($_GET['id'])){
    $rawdata = fetch_article($db->real_escape_string($_GET['id']));
    if(!$rawdata){
        $title = "";
        $meta['keywords'] = "";
        $meta['description'] = "";
    }else{
        $title = stripslashes($rawdata['title']);
        $meta['keywords'] = stripslashes($rawdata['htmlkeywords']);
        $meta['description'] = stripslashes($rawdata['htmldesc']);
        $subs = stripslashes($rawdata['subs']);
        $pagecontent = "<article>" . stripslashes($rawdata['content']) . "</article>";
    }
    include("includes/header.php");
    echo $pagecontent;
    if(!$rawdata){
        error_404();
    }
}

谢谢

Answer 1

我讨厌 switch 语句，但说实话，这是我个人的偏好。至于进一步优化，我建议看看某种形式的汇编语言。它将为您提供一些关于如何使条件语句更有效的一般想法。也就是说，它会让你对事物有不同的看法。

if(!empty($_GET['id'])) 
    {

    if($_GET['id'] == '13')
    {
        include('pages/servicestatus.php');
    }
    else
    {
        $rawdata = fetch_article($db->real_escape_string($_GET['id']));

        if (!$rawdata) {

            $title = "";
            $meta['keywords'] = "";
            $meta['description'] = "";
        } else {

            $title = stripslashes($rawdata['title']);
            $meta['keywords'] = stripslashes($rawdata['htmlkeywords']);
            $meta['description'] = stripslashes($rawdata['htmldesc']);
            $subs = stripslashes($rawdata['subs']);
            $pagecontent = "<article>" . stripslashes($rawdata['content']) . "</article>";
        }

        include("includes/header.php");
        echo $pagecontent;
        if (!$rawdata) {

            error_404();
        }
    }
} 
else 
{
    include('pages/home.php');
}

Answer 2

如果您要检查的 $_GET['id'] 有多个离散值，则 switch 是合适的。

为了可读性，我可以提出的一个建议是，

} elseif (!empty($_GET['id'])) {

只需要

} else {

Answer 3

好吧，我认为没有必要切换到开关
但你可以

} elseif (!empty($_GET['id'])) {

改为

}else{

Answer 4

您可能想要考虑将代码分解为 MVC 形式；这将使维护代码变得更加容易。至少将最后一个子句放入另一个文件中，可能名为 default.php 并 include 它。另外，您可以创建一个 id => 的数组文件键/值集，查找 id，并包含该文件。

if (isset($_GET['id'])) {
    $pages = array(
        0 => 'home.php',
        13 => 'servicestatus.php'
    );
    if (isset($pages[$_GET['id']])) {
        include('pages/' . $pages[$_GET['id']]);
    } else {
        include('pages/default.php');
    }
}

Answer 5

是的，switch 只计算一次，比 if elseif 高效，
并且使用这个给定的结构更容易维护

switch ($_GET['id'])
{
  case 13: ... break;
  case 0 : ... break;
  default: ... break;
}

Answer 6

我不知道你是否应该这样做，但在这里我不会。主要原因是，至少有一个语句，您可以省略，然后，您将只有一个 if-elseif-else - 语句

if (empty($_GET['id'])) { /* code */ }
elseif ($_GET['id'] === '13') { /* code */ }
elseif (!empty($_GET['id'])) { /* code* }

相同

if (empty($_GET['id'])) { /* code */ }
elseif ($_GET['id'] === '13') { /* code */ }
else { /* code* }

与之后的块中 ，语句 if(!$rawdata) 也是重复的。