Question

我想找到一种更有效的方法来实现这个功能：

<?php
function zipError($title, $desc) {
    echo '<style type="text/css">';
    echo 'body { margin:0;font-family:Arial, sans-serif;font-size:13px;background:#DDD;color:#1A1A1A; }';
    echo 'h1 { color:#FFF;font-family:Arial;font-weight:bold;text-align:center; }';
    echo 'h2 { color:#1C1C1C;font-family:Arial;font-weight:bold;text-align:center;padding-bottom:4px;border-bottom:1px solid #AAA; }';
    echo '#error#head { background:#1C1C1C;padding:10px;;border-bottom:5px solid #FFF; }';
    echo '#error#content { border:1px solid #AAA;background:#FFF;padding:20px;width:780px;margin:30px auto; }';
    echo 'p { text-align:center;padding:10px;color:#1A1A1A;font-family:verdana; }';
    echo '</style>';
    echo '<title>' . $title . '</title>';
    echo '<div id="error head"><h1>An error has occurred.</h1></div>';
    echo '<div id="error content"><h2>'.$title.'</h2><p>'.$desc.'</p></div>';
}

此代码用于抛出错误。例如：

die(zipError('Session Not Found', 'Your session has not been found! Please re-login now!'));

虽然这可以完成工作，但我正在努力学习，所以我想让这个功能更有效，而不是硬编码。有什么想法吗？

Answer 1

我要做的第一件事是退出所有HTML代码的PHP并跳回来回收数据。从维护的角度来看，它只是让它更具可读性。

接下来，使用单独的CSS文件以便可以共享它。它可以是它自己的小文件，也可以合并为更大的文件。

这就是真的。实际上并没有很多PHP可以优化

Answer 2

这看起来像错误模板，我会做这样的事情：

创建sparate错误tamplate：例如error.php：

<html>
   <head>
     <style>
      //style here...
     </style>
     <title><?= $title?></title>
   </head>
   <body>
       //and soon ...
   </body>

和一个功能：

public function zipError($title, $desc){
    include('error.php');
}

和用法：

 ... die(zipError('Session Not Found', 'Your session has not been found! Please re-login now!'));

现在，您可以轻松编辑/更改错误模板，只要它能够回显$title和$desc以及您想要的任何其他参数。

Answer 3

无论

echo "Output text/HTML using one statement
     instead of many, which can include\n
     line breaks and can span multiple lines";

OR

Only drop into PHP to output variables like <?=$var?> or <?php echo $var; ?>

此外，死和退出输出传递的参数 - 在本例中为 zipError（）的返回值。因此要么从 zipError 返回字符串而不是回显它，要么在调用函数时使事情变得更容易，而在函数内部退出而不是用 die包裹调用（）

如何让这个PHP函数更高效？

3 个答案: