【rails6】before_actionで使用するshuffleメソッドを一度だけ実行されるようにしたい

Question

前提

RoR6である複数のレコードを取得し、その順番をランダム順にするためにshuffleメソッドを使用しています。
しかし、newアクションのときとcreateアクションの両方でbefore_actionをしているため、createアクションが実行されるタイミングでnewアクションのときと異なる順番になってしまいます。

実現したいこと

newメソッドだけでレコードをランダム順に並び替え、createアクションではnewメソッドで並び替えられたものと全く同じ並び順のレコードを使いたい。

該当のソースコード

before_action :random_articles, only: [:new, :create]

def new
end
def create
end

private
def random_articles
  @articles = Article.all.shuffle
end

ソースコードは超簡易的に記述していますが、要するにnewアクションの時にArticle.allがランダム順で@articlesに格納されますが、createアクションに移った時@articlesの順番がnewメソッドの時と同じになればよいです。

Answer 1

コントローラーのインスタンス変数は、その名の通りコントローラーのインスタンスごと、つまりリクエストごとに再作成されます。newで作った変数をcreateのときも使う、というようなことはできません。

一度作った順序をセッションなどに保存して、2回目はそれを引っ張り出す、というような流れにする必要があります。

Comments

[greenwalk]

回答ありがとうございます。

def new
@articles = Article.all.shuffle
session[:articles] = @articles
end

def create
@articles = session[:articles]
end

のようにして、上手くいかないのはなぜでしょうか・・
sessionメソッドの使い方が間違っていますか？

[maisumakun]

> 上手くいかないのはなぜでしょうか・・

現状ではどうなっているのですか？

[greenwalk]

すみません勘違いしていました。
ちゃんと渡せてるみたいでしたが、cookieのサイズが大きすぎたみたいです。
ActionDispatch::Cookies::CookieOverflow
こちらのエラーが出てました。

[maisumakun]

idだけ入れる、などの工夫も考えられます。

Answer 2

シャッフルした、というフラグを用意しておいて、
コレが立っているとシャッフルしないようにすればいいかと

Comments

[greenwalk]

回答ありがとうございます。
あくまでもDBの中身は変更せずに実装したいのですが、具体的にどのように記述すればよいか教えていただけないでしょうか。