Railsアプリ上で他人としてログインするとエラーになる(Couldn't find Post with 'id' =)

Question

前提・実現したいこと

Railsで作成したマイクロポストアプリにRedisを用いてランキング機能を実装しました。無事に記事タイトルとPV数の表示を完了することができたのですが、次に他人としてログインして他の投稿が編集削除出来ないか確認しようとしたところエラーが発生してしまいました。

ちなみに現在のユーザーは二人作成していて、Redisでのランキング機能を実装した時は、ユーザーIDが1の方でログインをしていました。

Railsは一ヶ月間勉強しており、**Redisに関してはほぼ素人です。Redisに関する下記のコードは理解が乏しいかと思います。**どうしてもアプリを完成させたいので、助言をいただけたら幸いです。

追記：Idが存在していないポストをコンソール上で作成したら一旦、動くようにはなったのですがそれでも投稿を削除しようとすると同じエラーが発生してしまいます。
**私の予測ですが、どうやら削除した瞬間にランキングに表示すべきポストが入った変数がおかしくなっている気がします。**そのままランキング表示から削除したポストは外したいのですが、それもどうやったらいいのか見当がつきません…。引き続き自分でのリサーチは続けます。

追記：案の定、データを全て削除するとランキングの一番目にあったポストのIdが見つからないと言う風にエラーが表示されました。原因はわかったのですが解決方法がいまだに掴めていません。

追記：考えた結果、削除を実行した際にモデルのデータからは情報が削除されて、一方でRedisからはデータが削除されていないためエラーになることが分かりました。そこで下記のコードをpostsコントローラのdestroyアクションに追加したのですが、再びエラーが発生しています。

発生している問題・エラーメッセージ

destroyアクション等にコードを追加するまでは下のエラーが発生していました。

Couldn't find Post with 'id'=7

そしてランキングデータを取得するアクションの下の行を
@ranking_posts = ids.map{| id |Post.find(id)}
次のように書き換えました。
@ranking_posts = ids.map{| id |Post.find_by(id: id)}

また、Redisのソート済セット型のデータから日付をキー、ポストのIdをメンバーを指定してスコアを削除するようにpostsコントローラーのdestroyアクションに下の行を追加しました（@postはビフォーアクションで取得できています。）
REDIS.zrem "posts/daily/#{Date.today.to_s}", @post.id

そしてその結果発生しているエラーが下です。

undefined method `user_id' for nil:NilClass

エラーの該当箇所は間違ったユーザーに編集削除を許可するためのアクションです。

def ensure_correct_user
        @post = Post.find_by(id: params[:id])
        if @post.user_id != current_user.id
            flash[:notice] = "Not yours"
            redirect_to "/"
        end
end

該当のソースコード

下はHomeコントローラーです。

class HomeController < ApplicationController
    before_action :set_ranking_data, only:[:top]
    
    def top
        @posts = Post.all.order(created_at: :DESC)
    end
    
    def set_ranking_data
        #５件のランキングデータを取得
        ids = REDIS.zrevrangebyscore "posts/daily/#{Date.today.to_s}", "+inf", 0, limit: [0, 5]
        @ranking_posts = ids.map{| id |Post.find_by(id: id)}
    end
end

下はPostsコントローラーです。

class PostsController < ApplicationController
    before_action :set_post, only: [:show, :edit, :update, :destroy]
    before_action :authenticate_user!, only:[:edit, :update, :destroy]
    before_action :ensure_correct_user, only:[:edit, :update, :destroy]
    before_action :set_ranking_data, only:[:show, :edit]
    
    # GET /posts
    # GET /posts.json
    def index
        @posts = Post.all.order(created_at: :DESC)
    end
    
    # GET /posts/1
    # GET /posts/1.json
    def show
        @post = Post.find_by(id: params[:id])
        REDIS.zincrby "posts/daily/#{Date.today.to_s}", 1 ,@post.id
    end
    
    def set_ranking_data
        #５件のランキングデータを取得
        ids = REDIS.zrevrangebyscore "posts/daily/#{Date.today.to_s}", "+inf", 0, limit: [0, 5]
        @ranking_posts = ids.map{| id |Post.find_by(id: id)}
    end

    # GET /posts/new
    def new
        @post = Post.new
    end
    
    # GET /posts/1/edit
    def edit
        @post = Post.find_by(id: params[:id])
    end
    
    # POST /posts
    # POST /posts.json
    def create
        @post = Post.new(post_params)
        @post.user_id = current_user.id
        respond_to do |format|
            if @post.save
                format.html { redirect_to @post, notice: 'Post was successfully created.' }
                format.json { render :show, status: :created, location: @post }
                else
                format.html { render :new }
                format.json { render json: @post.errors, status: :unprocessable_entity }
            end
        end
    end
    
    # PATCH/PUT /posts/1
    # PATCH/PUT /posts/1.json
    def update
        respond_to do |format|
            if @post.update(post_params)
                format.html { redirect_to @post, notice: 'Post was successfully updated.' }
                format.json { render :show, status: :ok, location: @post }
                else
                format.html { render :edit }
                format.json { render json: @post.errors, status: :unprocessable_entity }
            end
        end
    end
    
    # DELETE /posts/1
    # DELETE /posts/1.json
    def destroy
        @post.destroy
        respond_to do |format|
            format.html { redirect_to "/", notice: 'Post was successfully destroyed.' }
            format.json { head :no_content }
        end
        
#　この部分でRedisのデータからポストを削除したいと思っています
        REDIS.zrem "posts/daily/#{Date.today.to_s}", @post.id
    end
    
    private
    # Use callbacks to share common setup or constraints between actions.
    def set_post
        @post = Post.find_by(id: params[:id])
    end
    
    # Never trust parameters from the scary internet, only allow the white list through.
    def post_params
        params.require(:post).permit(:title, :content)
    end
    
    def ensure_correct_user
        @post = Post.find_by(id: params[:id])
        if @post.user_id != current_user.id
            flash[:notice] = "Not yours"
            redirect_to "/"
        end
    end
end

ちなみにランキングを表示しようとしているHTMLは下のように記述しています。

<% @ranking_posts.each do |post| %>
            <li><%= link_to post.title, "/posts/#{post.id}" %><span>（<%= "#{REDIS.zscore("posts/daily/#{Date.today.to_s}", post.id).to_i}" %>PV）</span></a></li>
        <% end %>

補足情報（FW/ツールのバージョンなど）

ruby 2.5.3
rails 5.2.3

Comments

[winterboum]

これだけの情報では助言難しいです。
そのエラーはどのfileのどこで起きたのか
エラーが起きたaction（method)はどのviewのどこから呼ばれたのか
そのviewを書きだしたのはどのcontrollerか
を開示して下さい。
コメント欄ではなく質問本文にお願いしまs

[chelsy7110]

Redisはわかりませんが、論理削除できるようにparanoia等のgemを使っていて、Redisはそんなもの関係なくランキングを取得しているように思えるのですが、論理削除のgemを使っていますか？

[退会済みユーザー]

ご返信ありがとうございます。

質問の方を編集してなるべく多くの情報を追加しました。非常に長い質問文になってしまい恐縮ですが、目を通していただけたら幸いです。

Answer 1

findではidが見つからない時エラーになりますがfind_byでは要素がない場合にnilが返ります。

@ranking_posts = ids.map{| id |Post.find_by(id: id)}

これで単純なエラーの回避はできますが、ランキングに削除済みPostが残ります。

ちゃんとやるならPostを削除するとき一緒に
Redisのランキングのメンバーから当該Postを削除する必要があります。

REDIS.srem  "posts/daily/#{Date.today.to_s}", post_id

Comments

[退会済みユーザー]

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

おっしゃるように@ranking_postsの方はそのまま
@ranking_posts = ids.map{| id |Post.find_by(id: id)}
に書き換えました。

そして、REDISからの削除についてはソート済セット型を使っていること、またメンバーとして@post変数のIdを使っていることを考慮してpostsコントローラのdestroyアクションに以下のように記述しました。
```
REDIS.zrem "posts/daily/#{Date.today.to_s}", @post.id
```

しかし、次はfind_byによってnilクラスが返されたことによって
undefined method `user_id' for nil:NilClass
といったエラーが返されるようになりました。

destroyアクションの時点でポストは削除できているように感じるので非常に困惑しています。
引き続き自分で考えますが、何かアドバイスを頂ければ幸いです。

[hellomartha]

都度Redis削除していればnilデータがランキングに含まれることはないはずですが・・・
現状のREDIS内のランキングデータを確認して
destroyアクション変更前に削除したデータが残っていないか確認してください。

もしPostコントローラー以外でPost削除しているような部分があれば
Postモデルのbefore_destroyとかでRedis削除してください。

[退会済みユーザー]

素早いご回答ありがとうございます！

参考にさせていただきます。

さきほど、destroyアクションの
REDIS.zrem "posts/daily/#{Date.today.to_s}", @post.id
という記述を
　　@post.destroy
respond_to do |format|
format.html { redirect_to "/", notice: 'Post was successfully destroyed.' }
format.json { head :no_content }
end
の上に移動させたところ期待通りに動作するようになりました。
REDISに関する記述は@post変数に関するアクションより上に記載しないといけないルールなどは存在するのでしょうか？

おっしゃる通りで、このコードがきちんと動いてなかったためにランキング表示の方でエラーが発生していたのだと思います…。
hellomarthaさんの回答が無ければ絶対にここまでたどり着けていませんでした。本当にありがとうございます！

[hellomartha]

>REDISに関する記述は@post変数に関するアクションより上に記載しないといけないルールなどは存在するのでしょうか？
このようなルールは存在しないはずです。
なぜ記述位置が関係するのかは不可解ですが動作したならよかったです