array_unique 不删除重复值 php

Question

抱歉，如果这是重复的，我已尝试搜索但似乎找不到答案。我可能只是把这段代码放错了地方。

我已经计算了从各种选择框中输入的重复值，这些值是通过 $_GET 发送过来的。使用这些重复项，如果超过（无论设置的数量是多少），那么它将通过 mysql 查询运行。这一切都很好。

问题是我需要删除从 mysql 查询返回的重复项。这是我的代码：

if ($countGearSelected >= 2) {      
        $gearSets = array_keys(array_filter(array_count_values($_GET['gearPiece']), function($v) {
            return $v > 1;
        }));

        foreach ($gearSets as $gearSetKey => $gearSetValue) {

            $result = mysqli_query($con,"SELECT twoPieceBonus FROM sets WHERE setName='".$gearSetValue."';");
            while($row = mysqli_fetch_array($result)){

                $twoPieceBonus .= urldecode($row['twoPieceBonus']).'</br></br>';

            }
            $twoPieceBonus = implode(',',array_unique(explode(',', $twoPieceBonus)));
            $twoSelected = substr($twoPieceBonus, 0, -10);

        }


    }else{
        $twoSelected = '';
    }

如您所见，我在 SE 上的其他各种帖子中尝试了 array_unique 选项，但它似乎不起作用。我想我可能用错了？

在 mysql 查询中使用 DISTINCT 不起作用，因为正在查询的一些“集合”具有相同的结果（如果这有意义？）。

非常感谢任何帮助。

Answer 1

首先：您的代码容易受到SQL injection 的攻击：使用prepared statements 来避免这种情况。

其次，在循环的每次迭代中执行查询通常是一个坏主意。在这种情况下，它可以避免。您可以使用 in 运算符并一次性比较所有齿轮组，而不是在您的 where 子句中进行相等比较。

这也将解决获取不同值的问题。只执行一个查询，您现在可以使用distinct。

代码如下所示。我无法对此进行测试，但我希望可以轻松修复错误（如果有）：

$twoSelected = '';
if ($countGearSelected >= 2) {      
    $gearSets = array_keys(array_filter(
        array_count_values($_GET['gearPiece']), function($v) {
            return $v > 1;
        }
    ));

    // Create comma separated list of question marks
    $placeHolders = implode(",", array_fill(0, count($gearSets), "?"));
    // Prepare SQL statement with it
    $stmt = mysqli_prepare($con, 
            "SELECT DISTINCT twoPieceBonus 
             FROM sets 
             WHERE setName IN ($placeHolders);");

    // All gearSet values are strings:
    $types = str_repeat("s", count($gearSets));
    // Turn the gearSets into references
    $gearSetRefs = [];
    foreach ($gearSets as $i => $_) {
         $gearSetRefs[] = &$gearSets[$i];
    }
    // Bind arguments
    mysqli_stmt_bind_param($stmt, $types, ...$gearSetRefs); // the splat operator
    // Now we are all set to (safely) execute the query 
    mysqli_stmt_execute($stmt);
    $result = mysqli_stmt_get_result($stmt);
    // Let the result of the URL decoding still be an array
    $twoPieceBonus = [];
    while ($row = mysqli_fetch_array($result)) {
        $twoPieceBonus[] = urldecode($row['twoPieceBonus']);
    }
    mysqli_stmt_close ($stmt);
    // ... and then use implode to insert those HTML breaks
    $twoSelected = implode("</br></br>", $twoPieceBonus);
}