État au milieu ou à la queue d'une chaîne

Question

Il est facile d'avoir une condition à la tête d'une chaîne et partager le reste:

if condition 
    get_this 
else 
    get_that 
end 
.foo.bar.baz 

Mais souvent, je veux une condition au milieu ou à la queue d'une chaîne. Le mieux que je peux penser est d'utiliser instance_eval:

foo.bar 
.instance_eval{ 
    if condition 
     get_this 
    else 
     get_that 
    end 
} 
.baz 

Mais je me inquiète que l'appel instance_eval est lourd, donc je finis réellement ne pas le faire. Cela vaut-il la peine de le faire? Y at-il une meilleure façon, ou devrais-je écrire tout simplement:

if condition 
    foo.bar.get_this 
else 
    foo.bar.get_that 
end 
.baz 

Answer 1

Pourquoi ne pas fournir une méthode sur bar qui accepte une valeur booléenne et e en actes en conséquence? Cela encapsule la logique et peut même protéger une logique complexe ou des conditions multiples, sans devenir trop salissant.

class Bar 
    def get_what(condition) 
    case condition 
     when qux; get_this(param1) 
     when quux; get_that(param1, param2) 
     else get_other(param1) 
    end 
    end 
end 

# assuming `bar` is an instance of `Bar` 
foo.bar.get_what(cond).baz 

PS: En fonction du cas d'utilisation, j'essaie souvent d'éviter des choses comme des chaînes de messages trop longs (car selon le Law of Demeter (Wikipedia) cela pourrait être considéré comme une mauvaise pratique :)), mais lorsque vous travaillez avec hash et des tableaux en rubis C'est plutôt utile. Donc, je suppose que vous avez un cas d'utilisation valide pour utiliser une longue chaîne de messages.

Answer 2

Comment l'utilisation Object#send appeler conditionnellement une méthode:

foo.bar.send(condition? ? :get_this : :get_that).baz 

Dans un cas plus complexe, avec de multiples/différents paramètres, on pourrait utiliser le splat operator pour déballer un tableau dans les paramètres:

foo.bar.send(
    * case conditional 
    when qux 
     [:get_this, param1] 
    when quux 
     [:get_that, param1, param2] 
    else 
     [:get_other, param1, param2] 
    end 
).baz 

Answer 3

Parfois, une solution low-tech est le meilleur:

my_bar = foo.bar 
if condition 
    my_thing = my_bar.get_this 
else 
    my_thing = my_bar.get_that 
end 
my_thing.baz 

Sérieusement, je considère beaucoup plus important que le sens du code source est 100% clair pour tout le monde (y compris vous un peu de temps à l'avenir) qu'il est raccourci par une ligne ou deux.