Discussion :

[galagann]

Salux à tous,

Je m'inquiète sur la consommation mémoire de l'opérateur ternaire par rapport à la fonction if.

En effet, j'ai remplacé ce bout de code qui consommé 4495.17 Ko :

Code :

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

1
2
3
4
5
6
if (isset($_GET['complr']) AND isset($_GET['complc'])) {
	$voiture = $_GET['complr'];
	$bus = $_GET['complc'];
}else {
	$voiture = $reunion_url;
	$bus = $course_url;

par ce bout de code qui consomme maintenant 4495.3 Ko :

Code :

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

1
2
$voiture = (isset($_GET['complr'])) ? $_GET['complr'] : $reunion_url;
$bus = (isset($_GET['complc'])) ? $_GET['complc'] : $course_url;

C'est une légère augmentation, mais avec 54 fonction if à remplacer par des opérateurs ternaires, l'augmentation au finale va être plus importante ?

Avez-vous déjà remarquer cette différence ?

Cordialement,
GaLaGaNN

[sabotage]

La micro-optimisation est inutile : regarde ta mesure, tu as une différence de 0,003% ; même répété sur des centaines de lignes, tu n'as rien a gagner.

Visiblement l'operateur ternaire est moins rapide sur des grandes quantités de données car il effectue une copie.

[laurentSc]

A noter que le code équivalent exact est :

Code php :

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

1
2
$voiture = (isset($_GET['complr'])&&($_GET['complc'])) ? $_GET['complr'] : $reunion_url;
$bus = (isset($_GET['complr'])&&($_GET['complc'])) ? $_GET['complc'] : $course_url;

[Invité]

Bonjour,

Moins rapide... peut-être...
Mais quelle économie de code !

Moi, ça me va.

[Tsilefy]

Salux à tous,

Je m'inquiète sur la consommation mémoire de l'opérateur ternaire par rapport à la fonction if.

Sérieusement, la lisibilité de ton code est plus important qu'une micro-optimisation comme ça. Regarde quelle expression est la plus lisible pour toi, et utilises-la.

[galagann]

Salux à tous pour vos réponses.

Merci et bien vu laurentSc !

[rawsrc]

Salut,

allez parce que c'est noël, en une ligne :

Code :

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

list($voiture, $bus) = (isset($_GET['complr'], $_GET['complc'])) ? [$_GET['complr'], $_GET['complc']] : [$reunion_url, $course_url];

de rien

[sabotage]

C'est beau en rouge et vert comme un sapin de noël.

[laurentSc]

allez parce que c'est noël, en une ligne :

Une seule ligne, OK, mais la lisibilité en prend un coup...

[galagann]

Re,

Alors ça rawsrc, c'est de l'optimisation de code !

Je vois que j'ai encore du chemin à parcourir...

Merci à tous,
GaLaGaNN