Discussion :

[Ashgenesis]

Bonjour, je ne suis pas sûr de poster dans le bon forum et je m'en excuse à l'avance si ce n'est pas le cas.

J'ai en fait un soucis d'optimisation sur une suite de condition dans un if else

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
val = soit un objet soit un entier;
if(!is_object(val)){
    if(!is_numeric(val)) {
        erreur ce n'est pas une valeur numérique
    }
} elseif (val instanceof ma classe) {
    val = val->num;
} else {
    erreur ce n'est pas une instance de ma classe
}
if(1 == val) {
    erreur val doit être différent de 1
}
retourne un entier val

Actuellement, j'ai 2 if d'imbriqué et un if à la suite, je ne suis pas satisfait de cette structure et je n'arrive pas à en concevoir une autre qui fasse la même chose mais qui soit optimisée idéalement dans un seul if si possible. Quelques conseils seraient les bienvenues

[droggo]

Fio,

Déjà, ça me paraît curieux d'utiliser une même variable pour y mettre soit un entier, soit un objet. Ça te conduit forcément à accumuler les test.
Une conception différente serait sans doute meilleure.

[PRomu@ld]

En fait, il y a un soucis de conception.

Si on s'en tient à ça :

Code :

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

val = soit un objet soit un entier;

alors ceci :

Code :

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

1
2
3
4
if(!is_object(val)){
    if(!is_numeric(val)) {
        erreur ce n'est pas une valeur numérique
    }

Te conduit à une deuxième structure if qui ne sera jamais vraie.

Mais pour parler pratique, avoir deux if imbriqués, c'est pas la mort au niveau performance. A mon avis, si tu as des soucis de performances, ça n'est pas là qu'il faut bosser.

[Ashgenesis]

Merci pour vos réponses

Fio,

Déjà, ça me paraît curieux d'utiliser une même variable pour y mettre soit un entier, soit un objet. Ça te conduit forcément à accumuler les test.
Une conception différente serait sans doute meilleure.

Une valeur ne doit contenir qu'un seul type de variable en effet mais pour des raisons d'utilisation de ma fonction le paramètre passé doit pouvoir accepter les 2. L'accumulation des tests ne me dérange pas mais je souhaiterais les effectuer de la manière la plus optimisé qu'il soit.

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
function test(val){
if(!is_object(val)){
    if(!is_numeric(val)) {
        erreur ce n'est pas une valeur numérique
    }
} elseif (val instanceof ma classe) {
    val = val->num;
} else {
    erreur ce n'est pas une instance de ma classe
}
if(1 == val) {
    erreur val doit être différent de 1
}
retourne un entier val
}

En fait, il y a un soucis de conception.

Si on s'en tient à ça :

Code :

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

val = soit un objet soit un entier;

alors ceci :

Code :

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

1
2
3
4
if(!is_object(val)){
    if(!is_numeric(val)) {
        erreur ce n'est pas une valeur numérique
    }

Te conduit à une deuxième structure if qui ne sera jamais vraie.

Euh ben la j'avouerais que j'ai pas tout saisi.

Mais pour parler pratique, avoir deux if imbriqués, c'est pas la mort au niveau performance. A mon avis, si tu as des soucis de performances, ça n'est pas là qu'il faut bosser.

Ce n'est pas un soucis de perf mais bien d'optimisation de code que je cherche à avoir. J'ai l'impression que ce code est très sale et qu'un petit nettoyage ne serait pas superflu

Pour expliquer plus clairement, ma fonction prend un paramètre qui sera soit un objet soit un entier.
Si c'est un entier et que sa valeur est différente de 1 elle retourne directement l'entier.
Si c'est un objet et que cet objet est bien une instance de maclasse alors elle va rechercher l'entier de l'objet si cet entier est différent de 1 elle retourne l'entier
Si ce n'est pas une instance de ma classe on renvois une erreur précisant que ce n'est pas une instance de ma classe
Si ce n'est pas une valeur numerique on renvois l'erreur ce n'est pas une valeur numérique
Si l'entier vaut 1 on renvoit une erreur precisant que l'entier ne doit pas valoir 1. Ma fonction fait ce que je lui demande mais pas de manière optimisé et c'est ce qui me gène un peu. Si vous avez des conseils je suis preneur

[droggo]

Fio,

Je ne vois toujours pas pourquoi tu as vraiment besoin de faire ainsi.

Pour moi, il y a tout simplement erreur de conception, et si tu veux "optimiser ton code", comme tu dis (et qui ne veut pas dire grand chose), il faut commencer par revoir la conception.

[Ashgenesis]

Tu ne dois pas connaître le proverbe qui dit on ne discute pas un ordre même quand celui-ci est débile

Il est fort possible que ce soit une erreur de conception mais la question n'est pas là. La question est d'optimiser (je ne vois pas en quoi cela ne veux rien dire) le code, le terme refactoring aurait peut être été plus adéquat mais bon...

Donc, la question n'est pas de savoir si le code est mal conçu pour le reste de l'appli (ce n'est pas moi qui l'ai commencé, j'ai repris la suite) mais juste comment arranger ou refactoriser la structure conditionnelle. Le fait que dans les conditions, il y ai un test sur le type de la variable ne doit pas être un obstacle à l'optimisation de ces tests.

Les tests dans les différents if sont importants pour les méthodes qui sont implémentées mais niveau algo je ne vois vraiment pas l'importance que cela puisse avoir. Si cela te gène, on pourrait revoir la structure du if comme cela
Prenons 4 conditions A B C et D et une valeur val
Dans ma structure, cela devrait donner

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if(A != val){
    if(B != val){
        erreur B
    }
} elseif (C == val){
    retour val
} else {
    erreur C
}
if(D != val){
    erreur D
} else {
   retour val
}

Si val est different de A et B on renvoit une erreur B
Si val == C on retourne val sinon on renvoit une erreur C
Si val == C alors val == A
Si val == A et val != C on renvoit une erreur C
Si val == B alors val != D
Si val == B et val == D on renvoit une erreur D
Sachant aussi que A != B != C != D