problème de comparer deux tableaux

Question

J'essaie de comparer le contenu de 2 tableaux, fondamentalement, je suis en utilisant un panier et je dois vérifier les prix des formulaires soumis par rapport à une base de données, le problème est quand j'en ai un incorrect prix dans le panier il me donne un message d'erreur mais quand j'ai 1 prix correct et 1 incorrect il continue à la caisse, je ne sais pas ce que je fais mal toute aide serait appréciée.

foreach ($cart->get_contents() as $item) 
    { 
    $item_id = $item['id']; 
    $item_name = $item['name']; 
    $item_price = $item['price']; 
    $item_qty = $item['qty']; 

$connection = mysql_connect($dbhost,$dbuser,$dbpass) or die("Error connecting to mysql"); 
mysql_select_db($dbname); 

$query = "select * from products where product_name = '$item_name'"; 

$result = mysql_query($query); 
if (!$result) { 
    echo mysql_error(); 
} 

while ($row = mysql_fetch_assoc($result)) { 
    $sql_price[] = $row['product_price']; 
    $qty[] = $row['product_qty']; 
    $name = $row['product_name']; 
} 
foreach($sql_price as $price) { 
    $price = $price; 
    if ($price !== $item_price) { 
     $valid_prices = false; 
    }else{ 


     $valid_prices = true; 
    } 
    } 
    } 

    if ($valid_prices !== true) 
    { 
    // KILL THE SCRIPT 
    die($jcart['text']['checkout_error']); 
    } 

Answer 1

le problème est que vous marquer l'entrée du chariot comme valide si le dernier élément du tableau de $sql_price est égal à $item_price.

Réécrire la boucle comme:

$valid_prices = true; 

foreach($sql_price as $price) { 
     $price = $price; 
     if ($price !== $item_price) { 
       $valid_prices = false; 
     } 
} 

Pour éviter itérer supplémentaire, ajoutez une break dans la if intérieure pour arrêter la boucle après avoir trouvé un prix non valide.

Ou encore ceci:

$valid_prices = (array_search($price, $sql_price) !== false); 

Vous pouvez avoir MySQL faire votre travail pour vous, même:

$query = 'select 1 from products where product_name = "' . mysql_real_escape_string($item_name) . '" and product_price = ' . (int)$item_price; 

$result = mysql_query($query); 
if (!$result) { 
     echo mysql_error(); 
} 

if (mysql_num_rows($result) > 0) { 
     $valid_prices = true; 
     echo 'price is good!'; 
} else { 
     $valid_prices = bad; 
     echo 'price is bad!'; 
} 

Answer 2

Vous définissez $valid_prices true ou false à chaque itération de la boucle. Donc, après la boucle, il ne correspond qu'au dernier élément de la boucle.

Il est peut-être préférable de le définir sur true à l'extérieur (avant) de la boucle, puis de le définir comme faux lorsque vous trouvez un mauvais cas - de cette façon, il ne peut pas être remis à true. Alternativement, vous pouvez simplement lancer l'erreur dans la boucle dès que vous trouvez le prix invalide.

Answer 3

Le problème principal semble être votre boucle foreach interne et le réglage de valid_prices. Votre boucle continue à travers tous les prix, donc $ valid_prices dépendra du dernier prix seulement. Vous devriez sortir de votre boucle dès que vous détectez un prix invalide.

Quelques autres petites choses:

• vous ne semblez pas utiliser le nom qté $ ou des variables $ name
• $ n'utilise pas [] comme sql_price de $, qté $
• Vous pourrait utiliser! = pas! ==