Discussion :

[bigs3232]

function trie($current_category_id,$sort_by1,$titre1,$page,$row_by_page,$manufacturer,$search,$ppr_ch,$pf_ch){
$params='cPath='. $current_category_id .'&page=' . $page.'&row_by_page=' . $row_by_page . '&manufacturer=' . $manufacturer.(isset($search) ? '&search='.$search : '').'&ppr_ch='.$ppr_ch.'&pf_ch='.$pf_ch;
echo $params;
}

a chaque fois que je l'utilise
je l'appelle comme ceci

trie($current_category_id,$sort_by1='p.products_model',$titre1=TABLE_HEADING_MODEL,$page,$row_by_page,$manufacturer,$search,$ppr_ch,$pf_ch);

et comme j'appelle la fonction plusieurs fois avec des paramètres différentes .Je suis obligé d'ecrire la ligne la fonction à chaque fois

<?php trie($current_category_id,$sort_by1='p.products_model_f',$titre1=TABLE_HEADING_MODEL_F,$page,$row_by_page,$manufacturer,$search,$ppr_ch,$pf_ch)?>
<?php trie($current_category_id,$sort_by1='p.products_pa',$titre1=TABLE_HEADING_PRODUCTS_PA,$page,$row_by_page,$manufacturer,$search,$ppr_ch,$pf_ch)?>
<?php trie($current_category_id,$sort_by1='p.products_weight',$titre1=TABLE_HEADING_WEIGHT,$page,$row_by_page,$manufacturer,$search,$ppr_ch,$pf_ch)?>

j'aimerais savoir s'il y a une possibilité de simplifier la fonction trie
en lui donnant moins de parametres du genre
trie($current_category_id,$sort_by1,$titre1,$param)
et $param prend en compte les variables restantes qui sont

,$page,$row_by_page,$manufacturer,$search,$ppr_ch,$pf_ch

merci d'avance

[gene69]

C'est pas un problème de complexité c'est un problème de lisibilité.

ya bien une solution mais c'est plus compliqué que ce qui existe déjà.


tu fais un objet, qui stockes toutes les variables d'appel de la fonction.
tu poses 2 ou 3 méthodes sur l'objet: un constructeur(), ta fonction trie() et éventuellement __set() si les attributs sont protégés/privés. ensuite avant chaque appel tu modifies uniquement l'attribut qui a changé.

C'est pas moins complexe. C'est juste plus facile à lire.

[eric.quinton]

Dans une fonction :
mafonction($arg, $arg1=null, $arg2=null)
les deux derniers arguments deviennent facultatifs.
Il suffit de créer une classe qui va permettre de stocker les arguments facultatifs.
La fonction va donc devoir faire plusieurs choses :
- stocker les valeurs facultatives fournies lors de son appel ;
- récupérer les valeurs facultatives qui n'ont pas été fournies lors de l'appel ;
- exécuter le code proprement dit de la fonction.

[grunk]

Et quelque chose comme cela ? :

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
 
 
function trie ($datas)
{
    $params='cPath='. $datas['current_category']; //....
    echo $params;
}
 
$datas = array(
    'current_category'=>$current_category_id,
    'sort_by'=> 'p.products_model_f',
    'titre'    => TABLE_HEADING_MODEL 
    // ...
);
trie($datas);
// Ici tu modifie l'élément du tableau nécessaire
$datas['titre'] = 'titre 2';
trie($datas);

[bigs3232]

oh là là ca a l'air balaise pour moi.celui de grunk m'a l'air pas mal du tout