2

如果可能的话，我需要一些帮助。

我创建了两个函数，以便在重定向后设置 $_GET 时显示一些消息。代码如下：

function display(){
if(isset($_GET['cnf_upd']) && $_GET['cnf_upd'] == '1'){
  $value = "The update was successful!";
  $type = "confirm";
  construct_the_div($value, $type);
}
if(isset($_GET['err_upd']) && $_GET['err_upd'] == '1'){
  $value = "The Update failed.";
  $type = "error";
  construct_the_div($value, $type);
}
if(isset($_GET['cnf_del']) && $_GET['cnf_del'] == '1'){
  $value = "Deleted completely.";
  $type = "confirm";
  construct_the_div($value, $type);
}
if(isset($_GET['err_del']) && $_GET['err_del'] == '1'){
  $value = "Unable to delete.";
  $type = "error";
  construct_the_div($value, $type);
}
}
function construct_the_div($value, $type){
// creating a div to display the message results
$div = "<div class=\"{$type}Msg\">\n";
$div .= "<p>{$value}</p>\n";
$div .= "</div><!-- end of {$type}Msg -->\n";
echo $div;
}

我想做的是尝试改进显示功能，因为它越来越长，所以如果可能的话，应该只有一个（或最多两个）if语句。因此，GET 的值将动态地在 if 条件内，如果它具有前缀“cnf_”，它将是一个“confirmMsg”，如果它具有前缀“err_”，它将是一个“errorMsg”。

有可能做这样的事情吗？？？

4

4 回答 4

1 
function display() {
    $messages = array(
        'cnf_upd' => 'The update was successful!',
        'cnf_err' => 'The Update failed.!',
        // ...
        // add all error and confirm there
        // ...
    );
    foreach($_GET as $key => $value) {

        if(strpos($key, 'cnf_')===0) {
            $type = 'confirm';
            $value = isset($messages[$key])
                ? $messages[$key]
                : $key;
            construct_the_div($value, $type);
        }

        if(strpos($key, 'err_')===0) {
            $type = 'error';
            $value = isset($messages[$key])
                ? $messages[$key]
                : $key;
            construct_the_div($value, $type);
        }

    }
}
于 2013-02-04T14:34:49.963 回答
0

我将提出一个不同的解决方案。不是$_GET根据要发送的消息设置不同的参数，而是设置一个参数并解析其值。

// Start by setting integer constants:
define(CNF_UPD, 1);
define(ERR_UPD, 2);
define(CNF_DEL, 3);
define(ERR_DEL, 4);

然后，当您设置值时 un $_GET，使用常量：

// Build the URL with a deletion error...
header("Location: http://example.com/script.php?msg=" . ERR_DEL);

最后，使用 aswitch来解析它们

if (isset($_GET['msg'])) {
  switch ($_GET['msg']) {
    case CNF_UPD:
      // Updated...
      break;
    case ERR_UPD:
      // failed...
      break;
    // etc...
    default:
      // invalid code.
  } 
}

如果您confirm/error/confirm/error对整数常量使用 的模式，则可以通过取$_GET['msg'] % 2. 奇数是确认，偶数是错误。当然，还有很多其他的方式可以列出，我只是碰巧按照你使用的交替顺序输入了它们。例如，您还可以对确认使用正整数，对错误使用负数。

$type = $_GET['msg'] % 2 == 1 ? $confirm : $error;

这很容易扩展到使用多个消息。由于它们是整数值，因此您可以安全地构造一个逗号分隔的列表，explode()并在收到它们时构造它们。

$messages = implode(array(ERR_DEL,CNF_UPD));
header("Location: http://example.com/script.php?msg=$messages");
于 2013-02-04T14:30:02.613 回答
0

该方法不正确，似乎一次只能出现一条消息（不能同时“完全删除”和“无法删除”）。尝试以这种方式构造参数：?msg=upd&msgType=cnf

function display(){
if (isset($_GET['msg']) && isset($_GET['msgType']))
{
  $messages = array('cnf_upd'=>'The update was successful!',
    'err_upd'=>'The update failed!',
    'cnf_del'=>'The deletion was successful!',
    'cnf_upd'=>'The deletion failed!',
  );
  if (isset($messages[$_GET['msgType'].'_'.$_GET['msg']))
    construct_the_div($messages[$_GET['msgType'].'_'.$_GET['msg']], htmlspecialchars($_GET['msgType']));
}

还有很多需要改进的地方，但首先这更清洁、更安全。

于 2013-02-04T14:33:50.673 回答
0

除非您可以基于 $_GET 参数以某种方式生成 $value 和 $type （我看不出您会怎么做），否则您可以执行以下操作：

$messages = array();
$messages[] = array('id' => 'cnf_upd', 'value' => 'The update was successful!', 'type' => 'Confirm');
$messages[] = array('id' => 'err_upd', 'value' => 'The Update failed.', 'type' => 'error');
...
foreach ($messages as $message) {
    if(isset($_GET[$message['id']]) && $_GET[$message['id']] == '1'){
        construct_the_div($message['value'], $message['type']);
    }
}
于 2013-02-04T14:37:02.667 回答