PHP嵌套Foreach代码逻辑优化求助：如何实现输出1110？

嘿，我来帮你解决这个问题！你的代码当前输出100010001000，和预期的1110不符，核心原因是嵌套循环的逻辑方向错了：你现在对每个全局事件，都会遍历一遍用户已接受的所有事件，每对比一次就输出一个字符，导致每个全局事件对应3次输出（用户有3个接受事件），最终得到12个字符的结果。而我们需要的是每个全局事件只输出一次判断结果——只要该事件在用户接受列表里就输出1，否则输出0。

下面给你两种简单可行的修正方案：

方案一：用in_array()直接判断（直观易懂） #

把嵌套循环改成单层循环，对每个全局事件，直接用in_array()检查它是否存在于用户接受列表中，然后输出对应的字符：

<?php 
$user_accepted_events = [1,2,3]; 
$all_events = [1,2,3,4]; 

foreach ($all_events as $single_row) {
    // 检查当前事件是否在用户接受列表里
    if (in_array($single_row, $user_accepted_events)) {
        echo "1";
    } else {
        echo "0";
    }
}
// 最终输出：1110
?>

方案二：数组键映射（大数据量更高效） #

如果后续用户接受的事件数量很多，in_array()的效率会有点低（每次遍历整个列表），这时可以把用户接受列表转换成键为事件ID的数组，这样判断存在性的时间复杂度是O(1)，更快：

<?php 
$user_accepted_events = [1,2,3]; 
$all_events = [1,2,3,4]; 

// 把接受列表转成键为事件ID的数组，值随便设（这里用array_flip翻转键值）
$accepted_event_map = array_flip($user_accepted_events);

foreach ($all_events as $single_row) {
    // 检查键是否存在，存在就输出1，否则0
    echo isset($accepted_event_map[$single_row]) ? "1" : "0";
}
// 最终输出：1110
?>

为啥原来的代码会输出100010001000？ #

给你拆解下原来的执行过程，你就明白啦：

• 第一个全局事件1：和用户的1、2、3依次对比 → 输出1、0、0 → 拼接成100
• 第二个全局事件2：和用户的1、2、3依次对比 → 输出0、1、0 → 拼接后总输出变成100010
• 第三个全局事件3：和用户的1、2、3依次对比 → 输出0、0、1 → 拼接后总输出变成100010001
• 第四个全局事件4：和用户的1、2、3依次对比 → 输出0、0、0 → 最终输出100010001000

这样就清楚了，原来的逻辑是每个全局事件输出3次判断结果，而我们需要的是每个事件只输出一次最终判断结果~

内容的提问来源于stack exchange，提问作者DataMiner