remplacer un multi - IF

Version imprimable

Bonjour à tous,
j'ai un bout de code que j'aimerais améliorer et j'aimerais avoir l'avis de developpeur.
J'ai imbriquer de nombreux if, et je pense que ce n'est pas la bonne solution.
Quelle serais la facon la plus propre de remplacer ce bout de code:

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
[...]
if ((basename($PHP_SELF) == FILENAME_CLASSIFICATION)&&($_GET['cid']=='')){
  $urltheorique = tep_href_link(FILENAME_CLASSIFICATION, '');
} elseif (isset($_GET['cid'])){
  $urltheorique = tep_href_link(FILENAME_CLASSIFICATION, 'cid=' . (int)
    ($_GET['cid']==56?13:
      ($_GET['cid']==54?66:
	($_GET['cid']==65?18:
	  ($_GET['cid']==64?52:
	    ($_GET['cid']==63?13:
	      ($_GET['cid']==67?20:
		($_GET['cid']==50?13:
		  ($_GET['cid']==71?18:
		    ($_GET['cid']==40?7:
		      ($_GET['cid']==62?66:$_GET['cid']
		      )
		    )
		  )
		)
	      )
	    )
	  )
	)
      )
    )
);
} 
[...]
if (($urldemande!=$urltheorique)&&($urltheorique != '')) {
  header("HTTP/1.1 301 Moved Permanently");
  header("Location: $urltheorique");
  exit();
}

]
Merci à vous

03/04/2012, 05h05
ascito
je viens de regarder ton code

faut il que $_GET['cid']==56?13 soit obligatoirement vérifié avant $_GET['cid']==63?13 ?

de plus je ne connais pas le résultat de tep_href_link(), voudrais en donner l'idée stp

sinon rapidos
Code:

1 2 3 if(in_array( $collectionLink,$dispatcher['url'])){ $posIndex = array_search( $collectionLink,$dispatcher['url']); }
Après il y'a des fonctions comme array_walk_recursive qui permettent de découvrir la valeur correspondant à une clé dans un tableau à n*n (*) clé =>valeurs et de stopper ( virtuellement) le parcours infini du tableau : cad avec la fonction tu cherche une clé, quand tu la trouve tu mets la valeur de coté et tu laisse la boucle finir son boulot ( et vis et versa tu cherche une valeur tu stop, tu garde la clef)

Bonjour,

Le type qui t'as pondu ça doit vite aller faire une sieste :mouarf:
Plus simple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
<?php
 
$cid_data = array(
   40 => 7,
   50 => 13,
   56 => 13,
   54 => 60,
   62 => 66,
   63 => 13,
   64 => 52,
   65 => 18,
   67 => 20,
   71 => 18
);
 
$cid = $_GET['cid'];
$urltheorique = tep_href_link(FILENAME_CLASSIFICATION, 'cid='.(isset($cid_data[$cid]) ? $cid_data[$cid] : $cid));
 
?>

04/04/2012, 04h37
gotcha5832

Merci

Je te remercie.
c'est certain c'est beaucoup plus propre.

Bon allez je part faire une sieste.