"Foreach + if“仅适用于最后一个元素
提问于 2017-07-16 03:34:32
上下文：
我有一个带有复选框的表单和一个php页面来处理数据。我检查我的数据:一切都好。
为了管理复选框，我执行一个"FOREACH“操作，根据表单中选中的框，其中有几个包含指令和sql查询的"IF”循环。
我尝试在我的数组上循环，以便为用户选中的每个框输入循环"IF“。
发行：
当我运行脚本时，我检查3或5个框，只考虑最后一个循环"IF“. :(
我查看了HTML和PHP测试帖子的侧面:一切都很好。
，所以我的问题是：
我是在哪里搞砸的，这样外汇只会在"IF“循环中传递数组的最后一个元素？
，这是代码：
if ( isset($_POST['ModalMultipleColorEdit']) && !empty($_POST['Planning_ID2']) && !empty($_POST['CaseColor2']) && !empty($_POST['CaseWeek2']) )
{
//------- Extract Needed Posted Data
$TargetID = $_POST['Planning_ID2'];
$TargetColor = $_POST['CaseColor2'];
$TargetMonth = $_POST['CaseMonth2'];
//----- GET OLD DATA
$REQ = $DB->query("SELECT * FROM `Planning` WHERE `ID_Planning` = ".$TargetID." ;")
or die(var_dump($REQ->errorInfo()));
$DATA = $REQ->fetch();
// Explore Targeted Weeks Array
foreach( $_POST['CaseWeek2'] as $CheckedWeek )
{
//----------- IF TARGET MONTH IS JANUARY
if ( $TargetMonth == "Janvier" && $CheckedWeek == "(S-1)" )
{
//------- Extract Color String Period
$GetColor = explode("." , $DATA['Planning_M1']);
//------- Change Period Color
$GetColor[0] = $TargetColor;
//------- Build New Color String
$NewCaseMonth = $GetColor[0].".".$GetColor[1].".".$GetColor[2].".".$GetColor[3].".".$GetColor[4];
// ------ UPDATE REQUEST
$REQ2 = $DB->prepare('UPDATE `Planning` SET Planning_M1 = ? WHERE `ID_Planning` = ?');
$REQ2->execute(array($NewCaseMonth, $TargetID)) or die(var_dump($REQ2->errorInfo()));
$REQ2 = NULL;
}
if ( $TargetMonth == "Janvier" && $CheckedWeek == "(S-2)" )
{
//------- Extract Color String Period
$GetColor = explode("." , $DATA['Planning_M1']);
//------- Change Period Color
$GetColor[1] = $TargetColor;
//------- Build New Color String
$NewCaseMonth = $GetColor[0].".".$GetColor[1].".".$GetColor[2].".".$GetColor[3].".".$GetColor[4];
// ------ UPDATE REQUEST
$REQ2 = $DB->prepare('UPDATE `Planning` SET Planning_M1 = ? WHERE `ID_Planning` = ?');
$REQ2->execute(array($NewCaseMonth, $TargetID)) or die(var_dump($REQ2->errorInfo()));
$REQ2 = NULL;
}
if ( $TargetMonth == "Janvier" && $CheckedWeek == "(S-3)" )
{
//------- Extract Color String Period
$GetColor = explode("." , $DATA['Planning_M1']);
//------- Change Period Color
$GetColor[2] = $TargetColor;
//------- Build New Color String
$NewCaseMonth = $GetColor[0].".".$GetColor[1].".".$GetColor[2].".".$GetColor[3].".".$GetColor[4];
// ------ UPDATE REQUEST
$REQ2 = $DB->prepare('UPDATE `Planning` SET Planning_M1 = ? WHERE `ID_Planning` = ?');
$REQ2->execute(array($NewCaseMonth, $TargetID)) or die(var_dump($REQ2->errorInfo()));
$REQ2 = NULL;
}
if ( $TargetMonth == "Janvier" && $CheckedWeek == "(S-4)" )
{
//------- Extract Color String Period
$GetColor = explode("." , $DATA['Planning_M1']);
//------- Change Period Color
$GetColor[3] = $TargetColor;
//------- Build New Color String
$NewCaseMonth = $GetColor[0].".".$GetColor[1].".".$GetColor[2].".".$GetColor[3].".".$GetColor[4];
// ------ UPDATE REQUEST
$REQ2 = $DB->prepare('UPDATE `Planning` SET Planning_M1 = ? WHERE `ID_Planning` = ?');
$REQ2->execute(array($NewCaseMonth, $TargetID)) or die(var_dump($REQ2->errorInfo()));
$REQ2 = NULL;
}
if ( $TargetMonth == "Janvier" && $CheckedWeek == "(S-5)" )
{
//------- Extract Color String Period
$GetColor = explode("." , $DATA['Planning_M1']);
//------- Change Period Color
$GetColor[4] = $TargetColor;
//------- Build New Color String
$NewCaseMonth = $GetColor[0].".".$GetColor[1].".".$GetColor[2].".".$GetColor[3].".".$GetColor[4];
// ------ UPDATE REQUEST
$REQ2 = $DB->prepare('UPDATE `Planning` SET Planning_M1 = ? WHERE `ID_Planning` = ?');
$REQ2->execute(array($NewCaseMonth, $TargetID)) or die(var_dump($REQ2->errorInfo()));
$REQ2 = NULL;
}
}
//------- REDIRECT WHEN SUCCESS AND NO MORE TARGETS
header('Location: PlanningNow.php?Process=Success');
exit();
}
else
{
header('Location: PlanningNow.php?Process=ErrorForm');
exit();
}谢谢你的帮助！)
回答 1
Stack Overflow用户
回答已采纳
发布于 2017-07-16 05:09:45
我得编些数据来做测试。考虑这种方法，将查询调用减少到一个，并在很大程度上合并您的代码.
代码：(演示)
$_POST['CaseWeek2']=['(S-2)','(S-3)','(S-5)']; // target checkboxes
$_POST['CaseColor2']='purple'; // target color
$_POST['CaseMonth2']='Janvier'; // target month
$checkedWeeks=['(S-1)','(S-2)','(S-3)','(S-4)','(S-5)']; // possible checkboxes
$DATA['Planning_M1']='red.orange.yellow.green.blue'; // old data from db
$oldcolors=explode('.',$DATA['Planning_M1']);
if($_POST['CaseMonth2']=='Janvier'){
// var_export(array_intersect($checkedWeeks,$_POST['CaseWeek2']));
$keys=array_keys(array_intersect($checkedWeeks,$_POST['CaseWeek2'])); // get appropriate keys
// var_export($keys);
$values=array_fill(0,sizeof($keys),$_POST['CaseColor2']); // generate equal number of values
// var_export($values);
$replacements=array_combine($keys,$values); // combine keys and values to make correct array
// var_export($replacements);
$newcolors=array_replace($oldcolors,$replacements); // replace old values with new values
var_export($newcolors);
//$REQ2=$DB->prepare('UPDATE `Planning` SET Planning_M1 = ? WHERE `ID_Planning` = ?');
//$REQ2->execute(array(implode('.',$newcolors), $TargetID));
}输出：
array (
0 => 'red',
1 => 'purple',
2 => 'purple',
3 => 'green',
4 => 'purple',
)哦，为了记录，您以前的迭代被“忽略”的原因是：
$GetColor = explode("." , $DATA['Planning_M1']);每次调用更新时，都会有效地提取旧缓存的$DATA['Planning_M1']数据，以单一颜色交换，并覆盖早期迭代的更新。
如果您想保持代码基本相同，可以编写：
$GetColor = explode("." , $DATA['Planning_M1']);在foreach() 循环之前的循环，然后有条件地修改$GetColor，就像您当前对$GetColor[n]所做的那样。然后，循环完成后，可以使用完全修改的$GetColor数据对数据库进行一次更新调用。
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接：
https://stackoverflow.com/questions/45128079
复制相关文章
点击加载更多
腾讯云开发者
