我有一个php页面,我正在尝试简化它,但遇到了一些我无法独自解决的问题。我的表单获取用户数据,发布到自己,验证字段是否已填写,然后将表单内容/发布显示到mysql数据库。
我遇到的问题是,我不想有20个if()/elf语句,而是想将变量名加载到一个数组中,循环遍历该数组,如果表单中没有填充变量,则会产生错误消息。不幸的是,无论字段中是否有值,我的代码都会显示错误消息。
作为附加说明,我可以将$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
}
}
?>
提前谢谢。
我怀疑语法$$ENote[$a]
可能被解释为($$ENote)[$a]
而不是$($ENote[$a])
(括号不是合法语法,只是为了演示)。
所以我建议使用一个中间变量,或者复杂的语法:
foreach ($ENote as $a => $b) {
$varname = $ENote[$a];
if(!$$varname){
或者(没有测试,只是预感它可能会起作用):
foreach ($ENote as $a => $b) {
if(!${$ENote[$a]}){
这里有几个问题。
首先,每个$ENote
条目必须省略前导$
。变量不能包含$
部分。
$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
}
}
为什么不创建一个数组或所有字段,并将该数组传递给一个函数,然后在该函数中循环遍历每个项,如果为空,则抛出一个错误?
类似这样的东西:
function checkEmpty($some_array) {
foreach($some_array as $key=>$value) {
if($value=="") {
or if(empty($key)) { ///throw error }
//put error message and exit
}
}
}