Discussion :

[mussara]

bonjour à tous

je me demandais... si je pouvais simplifier ce type de script:

Code :

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

1
2
3
4
5
6
if ($objet=="type") {
  $table="type";$item="type";
}
if ($objet=="genre") {
  $table="genre";$item="genre";
}

avec quelque chose du genre

Code :

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

1
2
3
4
5
$valeur=in_array("type","genre")
...
{
  $table=$valeur;$item=$valeur;
}

l'exemple que je donne est basique, mais en fait j'ai des parties de programme qui ne sont que des redondances, avec seulement des variables qui changent en fonction de tests de conditions...

ou alors le mieux est de créer des fonctions qui, elles, reprendront ces changements de variables?

si quelqu'un a des pistes...

merci beaucoup

[Mr N.]

J'ai pas tout compris mais ce qui est sur c'est que les deux bouts de code ne sont pas équivalents !
Si tu veux simplifier ton premier bout de code, tu peux le remplacer par :

Code :

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

$table = $item = $object;

[Kerod]

Voilà une solution :

Code :

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

1
2
3
4
5
$valeur = array("type","genre");
if(in_array($objet,$valeur))
{
 // code
}

[mussara]

oui je suis d'accord avec toi, ce n'est qu'un exemple... mal choisi soit...

je vais en prendre un autre...

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
if ($objet=="type") {
	$liste_decoupe_type = preg_split("[, ]", $type, -1, PREG_SPLIT_NO_EMPTY);
	$liste_decoupe = explode(", ", $type);
	$compteurbis = "0";$compteur_trouve="0";
	$liste_type = 'array("';
	foreach ( $liste_decoupe as $element) {
		$liste_type = $liste_type.$element.'","';
		$compteurbis++;		
	}
	$liste_type = $liste_type.'"';
	$liste_type = str_replace(',""',')',$liste_type);
	$pos = strpos($type, $findme);//echo $pos;
	if ($pos === false) {
	} else {
		$apres_autres = substr($type,$pos); 
		$valeur_autres_type = str_replace("autres (","", $apres_autres);$valeur_autres_type = str_replace(")","", $valeur_autres_type);
	   	$test_valeur_autres_type = "1";
		$valeur_autres=$valeur_autres_type;
	}
}
if ($objet=="genre") {
	$liste_decoupe_genre = preg_split("[, ]", $genre, -1, PREG_SPLIT_NO_EMPTY);
	$liste_decoupe = explode(", ", $genre);
	$compteurbis = "0";$compteur_trouve="0";
	$liste_genre = "'";
	foreach ( $liste_decoupe as $element) {
		$liste_genre = $liste_genre.$element."','";
		$compteurbis++;		
	}
	$liste_genre = $liste_genre."'";
	$liste_genre = str_replace(",''","",$liste_genre);
	$pos = strpos($genre, $findme);//echo $pos;
	if ($pos === false) {
	} else {
		$apres_autres = substr($genre,$pos); 
		$valeur_autres_genre = str_replace("autres (","", $apres_autres);$valeur_autres_genre = str_replace(")","", $valeur_autres_genre);
		$test_valeur_autres_genre = "1";
		$valeur_autres=$valeur_autres_genre;
	}
}

j'ai donc ici une 40aine de lignes
je souhaite n'en avoir que la moitié car je me dis que les seules choses qui changent dans ce script sont la variable $objet qui prend la valeur type puis genre

la seule question que je me pose pour diminuer la taille de mon code, est ce de créér une fonction et l'appeler en mettant en variable $objet ou bien existe t il mieux au niveau codage?

merci

[mussara]

désolé Kerod, j'étais en train de rédiger quand tu as publié ta réponse...

du coup à partir de ce qu'a écrit Kerod, j'appelle une fonction ou je mets directement le code dans les accolades? ( en utilisant bien entendu $valeur)

[Mr N.]

j'ai donc ici une 40aine de lignes
je souhaite n'en avoir que la moitié car je me dis que les seules choses qui changent dans ce script sont la variable $objet qui prend la valeur type puis genre

Houla attention ! Il y a quand même quelques différences :

Tu ne peux simplement supprimer 20 lignes de ton code comme ça, au petit-bonheur-la-chance.
Tu dois factoriser ton code, mais fais gaffe à ne rien oublier

[mussara]

oui je suis d'accord avec toi...
mais justement ça me permettra d'éviter ce genre de bétise que tu as vue lors de la comparaison que tu as faite...

bien entendu qu'il faut factoriser...

merci à tous