array_unique重複した値を削除しないph

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オプションを試してみましたが、それが動作するように表示されません。私はそれを間違って使っているかもしれないと思いますか？

DISTINCTを使用すると、mysqlクエリでは機能しません。照会されている「セット」のうちのいくつかが同じ結果になります（意味がある場合）。

ご協力いただきありがとうございます。

Answer 1

最初に：あなたのコードはSQL injectionに脆弱です。これを避けるにはprepared statementsを使用してください。

第2に、ループの各繰り返しでクエリを実行することはよくありません。そしてこの場合、それは避けることができます。 where句の等価比較の代わりに、in演算子を使用して、すべてのギアセットを一度に比較することができます。

これにより、別個の値を取得する問題も解決します。 1つのクエリのみを実行すると、はを使用して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); 
}