Лучшее тройное состояние

Question

Метод find_something может вернуть nil. В следующем коде

something = find_something(id) ? find_something(id) : create_something(foo)

find_something(id) вызывается дважды. Это запах, которого я хочу избежать. Есть ли способ избежать избыточности в этом выражении?

Answer 1

Недостаточно подробностей, чтобы сказать это с уверенностью, хотя, возможно, это относится к find_or_create_by.

Если это подходит, вы бы просто сделали:

something = YourModel.find_or_create_by(id: id)

Вы также можете предоставить для этого блок, который передается методу create, если запись не найдена. Например:

something = YourModel.find_or_create_by(id: id) do |instance|
  # this block only gets executed on create
  instance.some_new_attribute = 'goes here'
end

Надеюсь, это полезно - дайте мне знать, подходит ли оно для вашего варианта использования.

Answer 2

Что-нибудь подобное?

something = find_something(id) || create_something(foo)