Discussion :

[gotcha5832]

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 :

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
[...]
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

[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 :

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

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)

[rawsrc]

Bonjour,

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

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
<?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));
 
?>

[gotcha5832]

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

Bon allez je part faire une sieste.