Discussion :

[flow-bolox]

Bonjour, j'ai réalisé un scripts qui compare une base en SQL et la table correspondante en xml. Ensuite, il supprimme de la table SQL les entrées qui ne sont pas dans la base xml.

Avec plus de 3000 produits dans les tables, Ce code fonctionne mais est bien trop lourd !

Comment puis-je l'optimiser ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
//selection de tous les produits de la table SQL
$exist_query = mysql_query(" SELECT * from products ");
while($tableau = mysql_fetch_array($exist_query ))
{
$exist = 0;
//selection de tous les produits de la table XML
	foreach($xml->Data->Exportation_XML->Ligne_XML as $name){
//Comparaison 	
		if ($name->GA_ARTICLE == $tableau['products_id']){$exist = 1;}
	}
	if ($exist == 0){mysql_query("DELETE FROM products WHERE products_id= '".$tableau['products_id']."'")or die('Impossible de se connecter : ' . mysql_error());} 
}

Merci

[mathieu]

déjà il suffit que tu parcours le fichier XML un seule fois et tu stockes tous les "products_id" dans un tableau PHP
ensuite avec ce tableau tu construis une requête DELETE avec NOT IN

Code SQL :

Sélectionner tout - Visualiser dans une fenêtre à part

DELETE FROM products WHERE products_id NOT IN ('2', '3', '4')

[flow-bolox]

Merci ca a l'air beaucoup mieux comme ca.
Donc j'ai crée mon array mais je n'arrive pas a l'utiliser dans ma requete :
( Les array c'est pas mon fort ^^ )
J'ai essayé ca mais ca ne marche pas

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

DELETE FROM customers WHERE products_id NOT IN $tiers_comp[]

Merci

[ThomasR]

Bonjour,

il ne faut pas mélanger PHP ET SQL, ce sont deux langages différents, avec des synthaxes et éléments de langages tout aussi différents :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
<?php
mysql_query('DELETE FROM customers WHERE products_id NOT IN ('.implode(',',$tiers_comp).')');
?>

[flow-bolox]

Merci ca marche parfaitement bien et c'est bien plus rapide ^^

Mathieu, il manquait juste a ton code les guillemets pour l'affichage du tableau dans la requete :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

DELETE FROM customers WHERE customers_id NOT IN ("'.implode('","',$tiers_comp).'")

Merci a vous deux.

[mathieu]

Attention, n'oublies pas de vérifier qu'il n'y ait pas de possibilités d'injection SQL :
http://php.developpez.com/faq/?page=...rite-injection