[POO] simplification de syntaxe

Version imprimable

07/11/2006, 13h04
mussara

[POO] simplification de syntaxe
bonjour à tous

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

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:

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
07/11/2006, 13h11
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:

$table = $item = $object;

Voilà une solution :

Code:

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

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

je vais en prendre un autre...

Code:

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

07/11/2006, 13h37
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)
07/11/2006, 13h54
Mr N.

Citation:

Envoyé par mussara

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 :
http://img165.imageshack.us/img165/6...ot54bw7.th.png
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 ;)
07/11/2006, 14h09
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