Discussion :

[Hadafang]

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 :

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");
        }
    }
 
}

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 !

[rawsrc]

salut,

houlà !
tu peux poster le code de ta fonction existPanier(), stp

[Hadafang]

Oui bien sur ! La voici :

Code :

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

1
2
3
4
5
function existPanier($post, $panier)
{
    return isset($post["product_id"]) && array_key_exists($post["product_id"] . "a", $panier);
 
}

[rawsrc]

d'où provient $action ?

[Hadafang]

Cela provient d'une fonction qui fait le travail d'un écouteur d'événement, il détecte si le $_POST est soumis et exécute le reste du script.

[rawsrc]

tu gères un écouteur d’événement et tu as du mal à factoriser ton code ?!?? Ce n'est pas très cohérent. M'enfin...