データをランダムに複数件取得する際の(たぶんID落ちによる)エラーへの対処

Question

複数データのランダム取得

以下のコードでランダムに4冊の本のデータを取得しています。

def sample
  @books = Book.offset(rand(Book.count)).first(4)
end

取得した4冊の本のタイトルをviewで出します。

<ol>
  <% @books.each do |book| %>
  <li>
    <%= book.title %>
  </li>
  <% end %>
</ol>

ここまでは問題なく表示できます。
表示例）
0. 罪と罰
0. こころ
0. 地獄変
0. 斜陽

リロードすると…
0. 地獄変
0. 砂の器
0. 3匹のおっさん
0. 吾輩は猫である

(たぶんID落ちによる)エラー

上記のviewではeachを使って取得したデータ全て(4冊)を表示しました。
今度は取得したデータそれぞれについて個別に表示させようと思い、以下のように追記しました。

<ol>
  <% @books.each do |book| %>
  <li>
    <%= book.title %>
  </li>
  <% end %>
</ol>

　　　↓　以下を追記
<%= @books.first.title %>
<%= @books.second.title %>
<%= @books.third.title %>
<%= @books.fourth.title %>

すると、基本的には問題なく表示できるのですが、リロードした際、時折エラーが起きるようになりました。エラー内容は以下のものです。

undefined method `title' for nil:NilClass

このエラー文に該当する箇所は、先ほど追記したうちの
<%= @books.second.title %>か
<%= @books.third.title %>か
<%= @books.fourth.title %>です。
secondで引っ掛かったり、fourthで引っ掛かったり、その時々です。

エラーになる場合、おそらくcontrollerがデータを集める時点で空のIDを含んでいるのだと思いました。

例）Bookデータのidが1~10あり、そのうち5,6,7が削除済みだったとする。　controllerでランダムに取得した４つのデータのidが1,4,6,9だった場合、削除済みのid=6はnilなので　<%= @books.third.title %>　でエラーとなる。これより上記のeach文は、取得できた1,4,6,9のうちtitleのある1,4,9のみ表示している(エラーなので確認できないが)。

こんなことだろうな、これはIDに抜けがないときだけ使おう、と思っていたのですが…

いや待てよ、たしか…　エラーが起きたのは　<%= @books.second.title %>　などを追記してからだから、
追記部分を削除してeach文だけ残すと…

あれ？　不思議とエラーは起こらず毎回必ず4件のtitleがランダムに取得できます。

もうわかりません。

• ※Bookデータは計20件入っています。
• ※20件のうちいくつか削除したデータがあります。
• ※リロードすると4～5回に1回のペースで上記エラーが起こります。
• ※追記した箇所を削れば(each文だけでは)エラーは起きません。
• ※追記があっても、each文の箇所ではエラーは起きません。
• ※<%= @whgs.first.title %>でエラーになったことはありません。

教えていただきたいこと

1. エラーの起きる原因と、なぜeach文だけではエラーにならず毎回4件取れるのか。
2. エラーの起きる原因がIDが欠けていることだった場合、欠けたIDをうまく避けて4件のレコードを取得するには、controller側にどう書けばいいか。

controllerの記述を以下のものに変えてみても結果は全く同じでした。

def sample
  @books = Book.where('id >= ?', rand(Book.first.id..Book.last.id)).first(4)
end

プログラミング初心者です。よろしくお願いいたします。

Answer 1

エッジケースを満たせていませんね。
Book.count == 20のときにrand(20)が19に評価された場合

Book.offset(19).first(4)

となり、最後の1冊しか取得できませんよね。

4 / (0..19).size = 1/5となり、現象が4〜5回に1回起きるのも順当でしょう。

batch_size = 4
Book.offset(rand(Book.count-batch_size)).first(batch_size)

などとするのが妥当ではないでしょうか。
しかしながら、Book.countがそもそも4未満のときも考えなければならないです。

(あと「ランダムにn件取得」は、厳密にはLIMIT, OFFSETでは実現できませんね。Railsでは何もしなければ自動でidによるsortが入る様になっているので、連続したレコードしか取得できないですし。)

Comments

[Gr.]

takumiabeさん
回答ありがとうございます！
batch_sizeの記述でエラーはなくなりました。
batch_size、初見でした…。

取得した4件のidを見てみたところ、確かに連番になっており、ご指摘の通り厳密には「ランダムにn件取得」は果たせていませんでした（データが少なく、かつeach文の並びをshuffleしていたため気づきませんでした）。

では、自動でidによるsortが入る様になっているRailsで「ランダムにn件取得」するためにはどういった処理が有効でしょうか。
もしご存知でしたらヒントだけでもいいのでお与えください。

[takumiabe]

動作の保証はしませんが
https://qiita.com/yancya/items/b963c3c0b3e3de646b6b
などが参考になるのではないでしょうか。

[Gr.]

takumiabeさん
返答していただきありがとうございます！
色々調べていくうちに.sampleという書き方を見つけました。
ランダムに(n個)を取ってこれるようで、
@books = Book.sample(4)
これだけで十分でした。

回答をいただけたおかげで、railsによって自動でidによるsortがかかる仕様を知れたので、それをヒントに調べて見つけられました。
たいへん助かりました！ありがとうございました！

[takumiabe]

解決して良かったです〜