未定义的索引错误（PHP）

Question

我在此代码的第3,6和7行中收到未定义索引“id”的通知。我想不通，我做错了什么：

if (isset($_POST['action']) && $_POST['action'] == 'save') { 
    foreach ($options as $value) { 
     if(($value['type'] === "checkbox" or $value['type'] === "multiselect") and is_array($_REQUEST[ $value['id'] ])) 
      { $_REQUEST[ $value['id'] ]=implode(',',$_REQUEST[ $value['id'] ]); //This will take from the array and make one string 
      } 
     $key = $value['id']; 
     $val = $_REQUEST[$key]; 
     $settings[$key] = $val; 
    } 

我想这是解决一个小的事情，但还没有与任何东西我已经试过所有的运气。我做的一件事是运行var_dump（$ key），有时$ key是空的，有时不是。所以我假设它与此有关。顺便说一下，这是WordPress主题的选项页面的一部分。这是作为“保存”功能的一部分运行的代码。

如果调试模式关闭，脚本实际上可以正常工作，但是当调试模式打开时，当点击选项页面上的保存按钮时，弹出这些通知。预先感谢任何见解，并让我知道是否需要发布更多代码以提供更多上下文。

编辑：我已经在pastebin中发布了选项页面。它的长。它包含t13lo的修复程序。该问题的代码是在线1957感谢：http://pastebin.com/NGX6qzgr

Answer 1

您可以更改第3行：

if (($value['type'] === "checkbox" || $value['type'] === "multiselect") && isset($_REQUEST['id']) && is_array($_REQUEST[ $value['id'] ])) 

然而，在你的foreach你的代码的其余部分似乎取决于$value['id']。此值并不总是被设置，因此它可能使这些线路：

$key = $value['id']; 
$val = $_REQUEST[$key]; 
$settings[$key] = $val; 

不着边际的话$value['id']未设置。如果是这种情况，将当前在foreach中的所有代码放在if (isset($value['id'])) {之内可能是合适的。

您需要进一步分析和理解代码（以及需要做）以确定正确的逻辑。

Answer 2

试试这个：

if(($value['type'] === "checkbox" || $value['type'] === "multiselect") && isset($value['id']) && is_array($_REQUEST[ $value['id'] ])){ ... 

Answer 3

不知道你的代码是假设做，但基于您已经发布了什么给这个一杆..（更换显然只是的foreach循环的一部分代码）..

foreach ($options as $value) { 
    if(!isset($value['id'])) 
     continue; 
    if(!isset($_REQUEST[$value['id']])) { 
     $settings[$value['id']] = ''; 
     continue; 
    } 
    if($value['type'] === "checkbox" || $value['type'] === "multiselect") { 
     // No isset check here, the top conditional catches non-set items 
     if(is_array($_REQUEST[$value['id']])) 
      $_REQUEST[$value['id']] = implode(',', $_REQUEST[$value['id']]); 
    } 
    $settings[$value['id']] = $_REQUEST[$value['id']]; 
} 

应该做完全像以前一样，有异常检查瓦尔首先设置，在$设置数组中的值也设置为空字符串时匹配$ _REQUEST VAR ISN” t设置..（假设我跟着什么代码正在做的是）..

编辑：继意见讨论中，我已经更新上面的代码中，应该确定在1957年线这是有道理的，该键不为你的代码中设置的通知，并非所有的数组项$options实际上是选项，一些是指在你的主题的选项页面上获得输出的标题/选项卡（无论什么）（所以在上面的循环中跳过它们是有意义的）。

我认为这样可以解决您的问题，但是我曾经说过，我认为指出该函数中的代码并不理想，整个存储机制看起来并没有很好地整体消毒。我倾向于建议将该页面所需的功能设置为manage_options，因此它至少在管理员被限制到可以实施替换之前（建议应该在单独的问题中，并且理想地在WPSE上）。