質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 936
退会済みユーザー

退会済みユーザー

前提・実現したいこと

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
        #5件のランキングデータを取得
        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
        #5件のランキングデータを取得
        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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • winterboum

    2019/08/02 10:36

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

    キャンセル

  • chelsy7110

    2019/08/02 10:39

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

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2019/08/02 11:11

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

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

    キャンセル

回答 1

checkベストアンサー

+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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/02 11:17

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

    おっしゃるように@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アクションの時点でポストは削除できているように感じるので非常に困惑しています。
    引き続き自分で考えますが、何かアドバイスを頂ければ幸いです。

    キャンセル

  • 2019/08/02 11:31

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

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

    キャンセル

  • 2019/08/02 11:35

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

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

    さきほど、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さんの回答が無ければ絶対にここまでたどり着けていませんでした。本当にありがとうございます!

    キャンセル

  • 2019/08/02 11:55

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る