Discussion :

[renaud26]

Bonjour à tous,

Je ne parviens pas à articuler mon code logiquement, sachant que j'ai une boucle dans laquelle passent des ID de produits. Pour chacun d'entre eux, 3 types de remises peuvent s'appliquer, mais pas se cumuler. Et je dois respecter un ordre de priorité des remises :

- La remise "code-promo" prime sur les 2 autres
- La remise "promoprod" prime sur la dernière
- La remise "promocat" ne s'applique que si aucune n'a été appliquée.

Mais rien n'empêche que 3 produits soient dans le panier et que chacun ait une des trois remises. L'important, c'est le non-cumul pour un même produit.

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
 
 
foreach( $tab as $id_prod=>$prix){
 
   if(is_code_promo($id_prod)){
 
       $remise = is_code_promo($id_prod);
       $prix = applique_remise($prix, $remise);
 
  }else{
 
         if(is_promoprod($id_produit))
        {
 
             $remise = is_promoprod($id_prod);
             $prix = applique_remise($prix, $remise);
 
         }else{
                if(is_promocat($id_prod))
                {
 
                    $remise = is_promocat($id_prod));
                    $prix = applique_remise($prix, $remise);
 
                }
 
         }
    }
}

Mais ce code ne fonctionne que si le produit soumis à "code-promo" se présente en 1er dans la boucle. Or je ne connais pas l'ordre des produits.
Je ne sais pas si j'ai bien expliqué le truc (j'ai beaucoup simplifié) mais si oui, je veux bien une piste sur la façon d'organiser ça.
Merci d'avance.

[rawsrc]

Bonjour,

Pour suivre l'ordre de préférence tu as if () elseif () elseif ()...
Cela donnerait :

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
<?php
 
foreach($tab as $id_prod => &$prix) {
 
   if (is_code_promo($id_prod)) {
      $remise = is_code_promo($id_prod);
 
    } elseif (is_promoprod($id_produit)) { // il semblerait que tu ais une faute $id_produit != $id_prod
      $remise = is_promoprod($id_prod);
 
   } elseif (is_promocat($id_prod)) {
      $remise = is_promocat($id_prod));
   }
 
   $prix = applique_remise($prix, $remise); 
}
 
?>

[micetf]

Essaye ceci :

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
32
33
34
35
36
37
38
39
40
41
<?php 
function is_code_promo($id) 
{
	return ($id==1) ? 10 : false;
}
function is_promoprod($id) 
{
	return ($id==2) ? 20 : false;
}
function is_promocat($id) 
{
	return ($id==3) ? 30 : false;
}
function applique_remise($prix, $remise) 
{
	return $prix-$remise;
}
 
$tab=array(100,100, 100);
 
foreach( $tab as $id_prod=>$prix){ 
	if(is_code_promo($id_prod)){
	   $remise = is_code_promo($id_prod);
	   $prix = applique_remise($prix, $remise);
	   continue;
	}
	if(is_promoprod($id_prod)) {
		$remise = is_promoprod($id_prod);
		$prix = applique_remise($prix, $remise);
		continue;
	}
	if(is_promocat($id_prod))
	{
		$remise = is_promocat($id_prod);
		$prix = applique_remise($prix, $remise);
		continue;
	}
}
echo '<pre>';
var_dump($tab);
echo '</pre>';

ça devrait t'aider à comprendre ton problème :
Tu verras que aucun des prix n'est changé.
Dans ta boucle $prix contient bien $tab[$id_prod], mais dans une zone mémoire différente ce qui fait que lorsque tu appliques ta remise à $prix, cela n'affecte en rien le contenu du tableau.
PUIS regarde et essaye ceci :

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
foreach( $tab as $id_prod=>$prix){ 
	if(is_code_promo($id_prod)){
	   $remise = is_code_promo($id_prod);
	   $tab[$id_prod] = applique_remise($prix, $remise);
	   continue;
	}
	if(is_promoprod($id_prod)) {
		$remise = is_promoprod($id_prod);
		$tab[$id_prod] = applique_remise($prix, $remise);
		continue;
	}
	if(is_promocat($id_prod))
	{
		$remise = is_promocat($id_prod);
		$tab[$id_prod] = applique_remise($prix, $remise);
		continue;
	}
}

ça devrait résoudre ton problème.

MiCetF

[renaud26]

Bonjour à vous, et merci de vos précieuses indications !

En fait, j'ai solutionné ce problème avant de voir vos réponses car le suivi email n'a pas fonctionné. Mais finalement, je suis sur le même principe que ce qu'à écrit micetf : je range tout dans un nouveau tableau (id_prod est la clé) dans les ordres de priorité des remises au fur et à mesure ce qui donne :

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
 
$new_tab = array();
 
if(is_code_promo($id_prod)){
$remise = is_code_promo($id_prod);
$tot_remise = $prix - calcule_remise($prix, $remise);
$new_tab[$id_prod] = array("type_promo"=>"CP", "remise"=>$remise, "tot_remise"=>$tot_remise);
}
 
//si le produit n'est pas déjà dans le new_tab pour cause de remise //code-promo et qu'une remise produit peut s'appliquer :
if(!in_array($id_prod, array_keys($new_tab)) && is_promoprod($id_prod)){
$remise = is_promoprod($id_prod);
$tot_remise = $prix - calcule_remise($prix, $remise);
$new_tab[$id_prod] = array("type_promo"=>"P", "remise"=>$remise, "tot_remise"=>$tot_remise);
}
//et pareil pour les remises catégories...

Le nouveau tableau contient bien les infos que je désire.
Mais ensuite, hors de la boucle, je souhaite le ré afficher sous forme de synthèse en groupant les montants de remise par type de remise.
Or, j'ai simulé un exemple ou 2 produits ont une remise code_promo et le 3ème une remise produit : voici le print_r de ce tableau :

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
 
Array
(
    [189] => Array
        (
            [remise] => 10
            [tot_remise] => 4.6
            [type_promo] => CP
        )
 
    [190] => Array
        (
            [remise] => 10
            [tot_remise] => 9
            [type_promo] => CP
        )
 
    [596] => Array
        (
            [remise] => 25.00
            [tot_remise] => 3.2
            [type_promo] => P
        )
 
)

Ma question est donc : comment je peux fusionner les 2 premiers tableaux en se basant sur le fait qu'ils ont une clé commune qui m'intéresse : [type_promo]

Ce qui donnerait :

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
 
Array
(
    [CP] => Array
        (
            [tot_remise] => 13.6
 
        )
 
     [P] => Array
        (
            [tot_remise] => 3.2
 
        )
 
)

Passer par un tableau $tmp ?

[micetf]

Il y a peut-être mieux, mais voici une proposition :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$tab = array(
	189 => array('remise' => 10,'tot_remise' => 4.6,'type_promo' => 'CP'),
    190 => array('remise' => 10,'tot_remise' => 9,'type_promo' => 'CP'),
	596 => array('remise' => 25.00,'tot_remise' => 3.2,'type_promo' => 'P'));
 
$synthese=array();
foreach ($tab as $id => $promo) {
	$synthese[$promo['type_promo']] = (isset($synthese[$promo['type_promo']])) ? $synthese[$promo['type_promo']]+$promo['tot_remise'] : $promo['tot_remise'];
}
 
echo '<pre>';
var_dump($synthese);
echo '</pre>';

MiCetF

[renaud26]

Merci beaucoup ! je vois que tu manies toujours très bien les tableaux.
Je te souhaite une bonne journée.