Ruby On Rails — как отрефакторить (массив Ruby)?

Question

[Ilya Lozer]

1.)tournament = Tournament.where("date = ?", date)"
2.)arr = []
3.)tournament.map do |x|
4.) arr += [x.id] if x.tournament_locales.where(:locale=>"ru").count > 0
5.)end
6.)return arr

Строчки 2-6. Часто приходится делать подобное, особенно не нравится это явное объявление переменной. Возможно, есть какое-то более не костыльное решение?

Answer 1

[mgyk]

tournament.select{|x| x.tournament_locales.where(:locale => "ru").count > 0 }

Хотя в этом примере гораздо лучше на стороне БД это сделать.

Comments

[Ilya Lozer]

Отлично подошло. Спасибо большое за ответ.

[Борис Пеньковский]

вот ха что я люблю ruby)

Answer 2

[Борис Пеньковский]

Юзай DataMapper

arr = Tournament.all(:conditions => {Tournament.date => date, Tournament.tournament_locales.locale => 'ru'})

Answer 3

[Кирилл Платонов]

results = Tournament.select("tournaments.id, (select count(*) from tournament_locales where locale = 'ru') locales_count").joins(:tournament_locales).where("date = ? AND locales_count > 0", date).pluck('tournaments.id')
results

Мог где-то ошибиться в коде, будут ошибки - кидайте, поправим.

Comments

[Ilya Lozer]

С точки зрения построения и правильно хорошо, но с точки зрения чистоты кода, больше чисто AR запросы нравятся. Спасибо большое за ответ.