Question

我有一个PHP页面，我正在尝试简化，并遇到一些我无法单独解决的问题。我的表单将用户数据，帖子发送到自身，验证已填写的字段，然后将表单内容/帖子显示到mysql数据库。

我遇到的问题是，我没有使用20个if（）/ elseif语句，而是将变量名称加载到数组中，循环遍历该数组，如果未在表单中填充变量，则生成它错误消息。不幸的是，无论字段中是否包含值，我的代码都会显示错误消息。

作为补充说明，我可以将$ ShippingCo添加到我的表单中并回复它，但仍然显示未完成的通知。

另外，如果脚本输入if语句，我希望它在关闭后停止执行页面的其余部分我已经尝试退出;没有成功。

这就是我所拥有的：

<?php

$ShippingCo = $_POST['ShippingCo'];
$ShipAcct = $_POST['ShipAcct'];
$ShipService = $_POST['ShipService'];
$FOB = $_POST['FOB'];
$Terms = $_POST['Terms'];

$ENote[] = '$Terms';
$ENote[] = '$FOB';
$ENote[] = '$ShippingCo';
$ENote[] = '$ShipAcct';
$ENote[] = '$ShipService';

$Emessg[] = 'Shipping Terms';
$Emessg[] = 'FOB Method';
$Emessg[] = 'Shipping Company';
$Emessg[] = 'Shipping Account';
$Emessg[] = 'Shipping Service Type';


foreach ($ENote as $a => $b) {

if(!$$ENote[$a]){       //I intentionally put the '$$' in this line otherwise none of the messages show. . . with data in the variables or not.

$error = "Error!  Please Add the $Emessg[$a]!";
?>
<table width="800" align="center">
<tr>
<td align="center">

<h2>Sales Order Entry Form</h2>
</td>
</tr>
<tr>
<td align="center">
<h3>
<font color="red">
<?php
echo "$error";
?>
</font>
</h3>
</td>
</tr>
<tr>
<td align="center">
Please press back to properly complete the form</td>
</tr>
</table>

<?php
}
}
?>

提前谢谢。

Answer 1

我怀疑语法$$ENote[$a]可能被解释为($$ENote)[$a]而不是$($ENote[$a])（括号不是合法语法，仅用于演示）。

所以我建议使用一个中间变量，或者复杂的语法：

foreach ($ENote as $a => $b) {
  $varname = $ENote[$a];
  if(!$$varname){

或（未经测试，只是预感它可能有效）：

foreach ($ENote as $a => $b) {
  if(!${$ENote[$a]}){

Answer 2

为什么不直接创建数组或所有字段并将该数组传递给函数，并在该函数中循环遍历每个项目，如果为空则抛出错误？

这样的事情：

function checkEmpty($some_array) {
    foreach($some_array as $key=>$value) {
      if($value=="") {
         or if(empty($key)) {   ///throw error }
         //put error message and exit

       }

    }
  }

Answer 3

这里有几个问题。

首先，每个$ENote条目必须省略前导$。 Variable variables不得包含$部分。

$ENote[] = 'Terms';
$ENote[] = 'FOB';
...

其次，正如ctrahey所说，你需要将变量名存储到一个变量中，以便用另一个$扩展它。

以下是一些代码清理提示：

将两个数组合并为一个（地图）。
从循环内的$_POST分配给命名变量。
使用break退出循环。

这将消除大部分冗余。

$ENote = array(
    'Terms' => 'Shipping Terms',
    'FOB' => 'FOB Method',
    ...
);

foreach ($ENote as $field => $title) {
    $value = $$field = $_POST[$field];
    if (!$value) {
        ... display error using $title ...
        break;  // exit loop
    }
}

如果在Foreach循环中声明

3 个答案: