【Redis】【Rails】 有効期限の設定の仕方と記述場所

Question

記事のアクセス数をRedisに保存しているのですが、そのkeyに対して有効期限を設定する場合、以下のようなコードで問題ないのでしょうか？

def show

    @article = Article.find(params[:id])
    REDIS.multi do
      # 1)総合ランキング用
      REDIS.zincrby "articles/daily/#{Date.today.to_s}", 1, @article.id
      REDIS.expire( "articles/daily/#{Date.today.to_s}", 172800)
      # 2)カテゴリランキング用
      REDIS.zincrby "#{@article.category_id}/articles/daily/#{Date.today.to_s}", 1, @article.id
      REDIS.expire( "#{@article.category_id}/articles/daily/#{Date.today.to_s}", 172800)
    end

end

上記の場合、記事ページにアクセスする度にexpireで有効期限を設定する事になるので、若干無駄な気がしています。

しかしながら、他にどこに記述すればいいのかもわからず。

一般論として、このようなexpireの記載方法および記載場所で問題ないものでしょうか？？

いかんせんRedisは今まで使ったことがなく勘所がわかっておりません。

教えていただけると嬉しいです。

Answer 1

最終更新(およそ翌日以降)から172800秒後に有効期限を設定されているということですよね。
その日の一番最初のアクセスがあった時だけexpireを設定するのはどうでしょうか？

#最初のアクセス
REDIS.zadd ("articles/daily/#{Date.today.to_s}", 1, @article.id)
REDIS.expire( "articles/daily/#{Date.today.to_s}", 172800)

#それ以降
REDIS.zincrby "articles/daily/#{Date.today.to_s}", 1, @article.id

railsあまり使わないのでコード間違ってたらすみません；
質問で書かれている有効期限よりおよそ1日前に設定することになるので、有効期限は172800秒+246060みたいにする感じでしょうか。

私ならこう書くかなという感じなので参考程度にしていただけたら；