Bonjour à tous,
Je travail actuellement sur de la factorisation de code, exercice que je trouve particulièrement difficile, et je galère totalement. J'ai créer un petit script qui permet d'ajouter des produits dans un panier et de les supprimer. Le code fonctionne très bien, mais il n'est pas vraiment propre, et dans un soucis d'optimisation, je dois le transformer en une fonction. Mais je n'ai pas l'habitude de faire ça et je ne comprend pas comment je pourrais m'y prendre.
Voici le code en question :
Je ne comprend pas bien comment je dois m'y prendre pour simplifier tout ça étant donné que je fais déjà appel à des fonctions dans ce code, votre aide est la bienvenue ! Merci d'avance !
Code : Sélectionner tout - Visualiser dans une fenêtre à part
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31 if (isset($_POST) && !empty($_POST)) { $post = $_POST; if(!isset($action)){ $tab = [$_POST["product_id"] . "a" => ["qt" => $_POST["qt"]]]; if (existPanier($post, $panier)) { $index = $_POST["product_id"] . "a"; unset($panier[$index]); $tab2 = array_merge($panier, $tab); $test = $ordersModel->addPanier($user->id,serialize($tab2)); redirect("/commandes"); } else { $tab2 = array_merge($panier, $tab); $test = $ordersModel->addPanier($user->id,serialize($tab2)); redirect("/commandes"); } } if(isset($_POST['delete_product'])) { if (existPanier($post,$panier)){ $index = $_POST["product_id"] . "a"; unset($panier[$index]); $test = $ordersModel->addPanier($user->id,serialize($panier)); redirect("/commandes/panier"); } } }
Partager