Каков предпочтительный способ перечисления нескольких элементов?

Question

Что является более эффективным (или в противном случае предпочтительнее): отделить where условия запятыми, как это:

previous_rsvps = Rsvp 
    .where(user_id: array_of_ids, 
    waitlist_position: nil, 
    and so on...) 

или цепь их, как это:

previous_rsvps = Rsvp 
    .where(user_id: array_of_ids) 
    .where(waitlist_position: nil) 
    and so on... 

или лучше использовать SQL Анды, как это:

query = <<-SQL 
    user_id IN (array_of_ids) 
    AND waitlist_position IS NULL 
    AND so on... 
SQL 
previous_rsvps = Rsvp.where(query) 

у меня есть некоторые ? параметров в моем запросе, если это делает никакой разницы.

Answer 1

Это скорее вопрос мнения и предпочтения, чем что-либо еще. Я предпочитаю подход в вашем первом примере, так как он немного более краток и легко читается. Они также должны переводить на очень похожий SQL, поэтому эффективность должна быть одинаковой во всех случаях.

Однако существуют случаи, когда использование синтаксиса Ruby не будет работать для более сложных запросов, например, для нетрадиционных имен таблиц, объединения нескольких моделей и т. Д. Slo, безусловно, важно знать синтаксис третьего примера.

Answer 2

Давайте используем силу ActiveRecord всякий раз, когда можем и используем первый или второй (я буду использовать их взаимозаменяемо в зависимости от читаемости, которую они мне дают).

Лично я считаю, что это проблема с читабельностью в первую очередь и проблема ремонтопригодности впоследствии (поэтому я не буду использовать третий, если это действительно необходимо).

Answer 3

На мой взгляд, первый вариант лучше в случае, если вы хотели, чтобы реорганизовать свой код и обойти еще несколько параметров, или более сложные запросы, например, если вам необходимо для объединения таблиц и посмотреть в совместных таблицах:

condition = { waitlist_position:nil, user: {email:"some_email"} } 
Rsvp.joins(:users).where(condition) 

Учитывая, что условие динамически генерируется где-то в другом месте. Таким образом, вы сохраняете переменную condition в месте и просто корректируете критерии где-то еще.

Answer 4

Я предлагаю избегать прикованных там, где вы звоните за пределы своей модели, а также поддерживать логику запросов внутри методов видимой области вашей модели.

Rsvp.for_user(user_id) 

class Rsvp 
    scope :for_user ->(user_id) do 
    where(user_id: user_id, waitlist_position: nil) 
    end 
end 

Это добавит большей гибкости в вашу реализацию, сделает ее чистой и позволит избежать ненужного дублирования кода.

Вот очень хорошая статья относительно упомянутых идей:

http://craftingruby.com/posts/2015/06/24/say-no-to_chained-scopes.html

Answer 5

Обычно можно было бы использовать первый вариант. Class.where(attr1: "lala", attr2: "lelo")