Discussion :

[Devilju69]

Bonjour à tous,

Actuellement je réalise une boucle avec un update, puis un select et enfin un insert. Mais cela sur environ 900 lignes.

Comment puis améliorer cela ?

Code :

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

1
2
3
4
5
6
7
8
9
foreach($_POST['preleve'] as $num_preleve){
        $req_edit_paiements = 'UPDATE paiements SET etat_paiements="1" WHERE id_paiements="'.$num_preleve.'"';
	$res_edit_paiements = mysql_query($req_edit_paiements);
	$sql_paiement = 'SELECT date_paiements, montant_paiements, id_facture_paiements, id_client_paiements FROM paiements WHERE id_paiements="'.$num_preleve.'"'; 
	$req_paiement = mysql_query($sql_paiement);
	$paiement = mysql_fetch_array($req_paiement);
	$req_creer_bordereaux = 'INSERT INTO bordereaux (numero_bordereaux, date_debut_bordereaux, date_fin_bordereaux, date_bordereaux, date_paiement_bordereaux, montant_paiement_bordereaux, type_bordereaux, id_paiements_bordereaux, id_facture_bordereaux, id_client_bordereaux) VALUES ("'.$numero1.'", STR_TO_DATE("'.$date_debut_bordereaux.'","%d/%m/%Y"), STR_TO_DATE("'.$date_fin_bordereaux.'","%d/%m/%Y"), "'.$date_auj.'", "'.$paiement["date_paiements"].'", "'.$paiement["montant_paiements"].'", "'.$type_paiements.'", "'.$num_preleve.'", "'.$paiement["id_facture_paiements"].'", "'.$paiement["id_client_paiements"].'")';
	$res_creer_bordereaux = mysql_query($req_creer_bordereaux);
}

Merci d'avance

[sabotage]

1 - utilise PDO au lieu de l'extension obsolète mysql_
2 - utilise des requêtes préparées et protège les contre les injections SQL
3 - tu peux mettre le SELECT directement dans la requête INSERT.

[Devilju69]

Et sur les deux derniers points cela donnerais quoi ?

[sabotage]

Que tu peux essayer et que si tu n'y arrives pas tu nous montres ton code.