0

我想找到一种更有效的方法来处理这个函数：

<?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!'));

虽然这完成了工作，但我正在努力学习，因此我想让这个功能更高效，而不是硬编码。有任何想法吗？

4

3 回答 3

1

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

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

就是这样。老实说，没有太多的 php 可以优化

于 2013-05-06T00:39:17.670 回答
1

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

创建单独的错误模板：例如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您想要的任何其他参数。

于 2013-05-06T01:12:24.490 回答
0

任何一个

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

或者

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

此外，die和exit输出传递的参数 - 在这种情况下是zipError()的返回值。因此，要么从zipError返回字符串而不是回显它，要么在调用函数并退出函数时让事情变得更容易，而不是用die()包装调用。

于 2013-05-06T01:25:56.067 回答