Simplifier un petit code

[sff]

Bonjour,

J'aimerais savoir si ce petit bout de code est simplifiable.

$table = !is_numeric($userrow['bag'.$bag.'id']) ? 'equips' : 'items';

]switch ($table) {
			case 'equips': $type = 1; break;
			case 'items': $type = 2; break;
			default: $type = 3;
			}

Merci

 

[Sullyvan]

peut -être comme ça

$types = array('equips'=>1,'items'=>2);
$type = $types[$table];

 

[sff]

En terme d'exécution, c'est plus rapide ou ça revient au même ?

 

[jcaron]

S'il n'y a que ça dans ton code, et que $table n'est pas utilisé ou modifié par ailleurs, tu peux faire un raccourci comme ça:

$type = is_numeric($userrow['bag'.$bag.'id'])?2:1;

Mais j'ai tendance à penser que si c'est écrit comme ça c'est qu'il doit y avoir une raison, non?

Jacques.

 

[Sullyvan]

En terme d'exécution, c'est plus rapide ou ça revient au même ?

en théorie ça devrait être plue rapide puisqu'il n'y a pas de test switch, mais bon ça doit se jouer à quelques millièmes de secondes je pense

La solution de jcaron me semble bonne aussi en partant du principe que $table ne sert pas ailleurs.

 

[kitten13]

Franchement, prise de tete pour rien, ta gagner 000000000000000000000.1 ms

 

[jcaron]

Sans connaître le contexte, c'est difficile de dire si c'est important ou pas... Si c'est au coeur d'une boucle qui est exécutée 10000 fois par requête et qu'il se prend des milliers de requêtes par seconde, ça peut faire toute la différence...

Jacques.