質問編集履歴

追加情報

2019/08/02 02:10

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -24,10 +24,16 @@
+追記：考えた結果、削除を実行した際にモデルのデータからは情報が削除されて、一方でRedisからはデータが削除されていないためエラーになることが分かりました。そこで下記のコードをpostsコントローラのdestroyアクションに追加したのですが、再びエラーが発生しています。
 ### 発生している問題・エラーメッセージ
+destroyアクション等にコードを追加するまでは下のエラーが発生していました。
 ```
@@ -36,17 +42,65 @@
 ```
+そしてランキングデータを取得するアクションの下の行を
+ @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コントローラーです。
 ```Ruby
 class HomeController < ApplicationController
-    before_action :set_ranking_data
+    before_action :set_ranking_data, only:[:top]
@@ -64,9 +118,7 @@
         ids = REDIS.zrevrangebyscore "posts/daily/#{Date.today.to_s}", "+inf", 0, limit: [0, 5]
-        # この下の行がエラーを発生させています
-        @ranking_posts = ids.map{| id |Post.find(id)}
+        @ranking_posts = ids.map{| id |Post.find_by(id: id)}
     end
@@ -74,7 +126,7 @@
 ```
-ホームコントローラーと同様のアクションがポストコントローラーにも書かれているので、おそらく投稿詳細ページでも同じようなエラーが起きると思います。
+下はPostsコントローラーです。
@@ -88,7 +140,7 @@
     before_action :ensure_correct_user, only:[:edit, :update, :destroy]
-    before_action :set_ranking_data
+    before_action :set_ranking_data, only:[:show, :edit]
@@ -124,11 +176,157 @@
         ids = REDIS.zrevrangebyscore "posts/daily/#{Date.today.to_s}", "+inf", 0, limit: [0, 5]
-        @ranking_posts = ids.map{| id |Post.find(id)}
+        @ranking_posts = ids.map{| id |Post.find_by(id: id)}
-    end
+    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
 ```

追加情報

2019/08/02 02:10

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -17,6 +17,12 @@
 追記：Idが存在していないポストをコンソール上で作成したら一旦、**動くようにはなったのですがそれでも投稿を削除しようとすると同じエラーが発生してしまいます。**
 **私の予測ですが、どうやら削除した瞬間にランキングに表示すべきポストが入った変数がおかしくなっている気がします。**そのままランキング表示から削除したポストは外したいのですが、それもどうやったらいいのか見当がつきません…。**引き続き自分でのリサーチは続けます。**
+追記：案の定、データを全て削除するとランキングの一番目にあったポストのIdが見つからないと言う風にエラーが表示されました。原因はわかったのですが解決方法がいまだに掴めていません。

追加情報

2019/08/01 14:34

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

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

追加情報

2019/08/01 14:23

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -2,7 +2,7 @@
-Railsで作成したマイクロポストアプリにRedisを用いてランキング機能を実装しました。無事に記事タイトルとPV数の表示を完了することができたのですが、次に他人としてログインして他の投稿が編集削除出来ないか確認しようとしたところエラーが発生してしまいました。
+Railsで作成したマイクロポストアプリにRedisを用いてランキング機能を実装しました。無事に記事タイトルとPV数の表示を完了することができたのですが、**次に他人としてログインして他の投稿が編集削除出来ないか確認しようとしたところエラーが発生してしまいました。**
@@ -10,7 +10,11 @@
-Railsは一ヶ月間勉強しており、Redisに関してはほぼ素人です。Redisに関する下記のコードは理解が乏しいかと思います。どうしてもアプリを完成させたいので、助言をいただけたら幸いです。
+Railsは一ヶ月間勉強しており、**Redisに関してはほぼ素人です。Redisに関する下記のコードは理解が乏しいかと思います。**どうしてもアプリを完成させたいので、助言をいただけたら幸いです。
+追記：Idが存在していないポストをコンソール上で作成したら一旦、**動くようにはなったのですがそれでも投稿を削除しようとすると同じエラーが発生してしまいます。**