質問編集履歴

コントロールを変更

2019/12/09 03:16

投稿

Yuukou

スコア5

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -114,7 +114,7 @@
   validates :user_id, {presence: true}
+  belongs_to :user
   def user
@@ -122,7 +122,7 @@
   end
+  # self.user_idのユーザーを戻り値として返す
 end

エラーメッセージの内容を編集しました。

2019/12/09 03:15

投稿

Yuukou

スコア5

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -20,6 +20,14 @@
 NoMethodError in Posts#index
+/app/views/posts/index.html.erb where line #7 raised:
+undefined method `image_name' for nil:NilClass
 ```

usersのコントローラーを追加しました。

2019/12/08 12:25

投稿

Yuukou

スコア5

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -231,3 +231,185 @@
 ```
+usersのコントローラーはこちらになります。
+```
+class UsersController < ApplicationController
+  before_action :authenticate_user,{only: [:index,:show,:edit,:update]}
+  before_action  :forbid_login_user,{only:[:new,:create,:login_form,:login]}
+  before_action  :ensure_correct_user, {only: [:edit, :update]}
+  def index
+    @users = User.all
+  end
+  def show
+    @user = User.find_by(id: params[:id])
+  end
+  def new
+    @user = User.new
+  end
+  def create
+    @user = User.new(
+      name: params[:name],
+      email: params[:email],
+      image_name: "default_user.jpg",
+      password: params[:password]
+      )
+    if @user.save
+      session[:user_id] = @user.id       # 登録されたユーザーのidを変数sessionに代入
+    flash[:notice] = "新規登録しました。"
+     redirect_to("/users/#{@user.id}")
+    else
+      render("users/new")
+    end
+  end
+  def edit
+    @user = User.find_by(id: params[:id])
+  end
+  def update
+    @user = User.find_by(id: params[:id])
+    @user.name = params[:name]
+    @user.email = params[:email]
+    if params[:image]
+      @user.image_name = "#{@user.id}.jpg"
+      image = params[:image]
+      File.binwrite("public/user_images/#{@user.image_name}", image.read)
+    end
+    if @user.save
+      flash[:notice] = "登録内容を編集しました。"
+      redirect_to("/users/#{@user.id}")
+    else
+      render("users/edit")
+    end
+  end
+  def login_form
+  end
+  def login
+    @user = User.find_by(
+      email: params[:email],
+      password: params[:password]
+    )    #フォームに入力されたメールアドレスとパスワードを取得
+    if @user
+      session[:user_id] = @user.id # 変数sessionに、ログインに成功したユーザーのidを代入
+      flash[:notice] ="ログインしました"
+      redirect_to("/posts/index")
+    else
+      @error_message = "メールアドレスまたはパスワードが間違っています"
+      @email = params[:email]
+      @password = params[:password]
+      render("users/login_form")
+    end #@userが存在するかどうかを判定するif文作成
+  end
+    def logout
+      session[:user_id] = nil
+      flash[:notice] = "ログアウトしました"
+      redirect_to("/login")
+    end
+    def ensure_correct_user
+      if @current_user.id != params[:id].to_i
+        flash[:notice] = "権限がありません"
+        redirect_to("/posts/index")
+      end
+    end    # ログイン中のユーザー」と「編集しようとしているユーザー」が正しくない場合の処理
+end
+```

users_controller.rb をposts_controller.rbに変更しました。

2019/12/07 14:11

投稿

Yuukou

スコア5

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -124,172 +124,110 @@
 ```ruby
-class UsersController < ApplicationController
+class PostsController < ApplicationController
-  before_action :authenticate_user,{only: [:index,:show,:edit,:update]}
+  before_action :authenticate_user
-  before_action  :forbid_login_user,{only:[:new,:create,:login_form,:login]}
-  before_action  :ensure_correct_user, {only: [:edit, :update]}
   def index
-    @users = User.all
+    @posts = Post.all.order(created_at: :desc)
-  end
+  end
   def show
-    @user = User.find_by(id: params[:id])
+    @post = Post.find_by(id: params[:id])
+    @user = @post.user
-  end
+  end
   def new
-    @user = User.new
+    @post = Post.new
-  end
+  end
   def create
-    @user = User.new(
+    @post = Post.new(
-      name: params[:name],
+      content: params[:content],
-      email: params[:email],
-      image_name: "default_user.jpg",
-      password: params[:password]
+      user_id: @current_user.id
-      )
+    )
-    if @user.save
+    if @post.save
-      session[:user_id] = @user.id       # 登録されたユーザーのidを変数sessionに代入
-    flash[:notice] = "新規登録しました。"
+      flash[:notice] = "投稿を作成しました"
-     redirect_to("/users/#{@user.id}")
+      redirect_to("/posts/index")
     else
-      render("users/new")
+      render("posts/new")
     end
   end
   def edit
-    @user = User.find_by(id: params[:id])
+    @post = Post.find_by(id: params[:id])
-  end
+  end
   def update
-    @user = User.find_by(id: params[:id])
+    @post = Post.find_by(id: params[:id])
-    @user.name = params[:name]
+    @post.content = params[:content]
-    @user.email = params[:email]
-    if params[:image]
+    if @post.save
-      @user.image_name = "#{@user.id}.jpg"
+      flash[:notice] = "投稿を編集しました"
-      image = params[:image]
-      File.binwrite("public/user_images/#{@user.image_name}", image.read)
+      redirect_to("/posts/index")
+    else
+      render("posts/edit")
     end
-    if @user.save
-      flash[:notice] = "登録内容を編集しました。"
-      redirect_to("/users/#{@user.id}")
-    else
-      render("users/edit")
-    end
+  end
+  def destroy
+    @post = Post.find_by(id: params[:id])
+    @post.destroy
+    flash[:notice] = "投稿を削除しました"
+    redirect_to("/posts/index")
-  end
+  end
-  def login_form
-  end
+end
-  def login
-    @user = User.find_by(
-      email: params[:email],
-      password: params[:password]
-    )    #フォームに入力されたメールアドレスとパスワードを取得
-    if @user
-      session[:user_id] = @user.id # 変数sessionに、ログインに成功したユーザーのidを代入
-      flash[:notice] ="ログインしました"
-      redirect_to("/posts/index")
-    else
-      @error_message = "メールアドレスまたはパスワードが間違っています"
-      @email = params[:email]
-      @password = params[:password]
-      render("users/login_form")
-    end #@userが存在するかどうかを判定するif文作成
-  end
-    def logout
-      session[:user_id] = nil
-      flash[:notice] = "ログアウトしました"
-      redirect_to("/login")
-    end
-    def ensure_correct_user
-      if @current_user.id != params[:id].to_i
-        flash[:notice] = "権限がありません"
-        redirect_to("/posts/index")
-      end
-    end    # ログイン中のユーザー」と「編集しようとしているユーザー」が正しくない場合の処理
- end
-```
+```

ご指摘いただきありがとうございます。コントローラを記載しました。よろしくお願いいたします。

2019/12/06 14:30

投稿

Yuukou

スコア5

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -124,10 +124,172 @@
 ```ruby
+class UsersController < ApplicationController
+  before_action :authenticate_user,{only: [:index,:show,:edit,:update]}
+  before_action  :forbid_login_user,{only:[:new,:create,:login_form,:login]}
+  before_action  :ensure_correct_user, {only: [:edit, :update]}
   def index
-    @posts = Post.all.order(created_at: :desc)
+    @users = User.all
-  end
+  end
+  def show
+    @user = User.find_by(id: params[:id])
+  end
+  def new
+    @user = User.new
+  end
+  def create
+    @user = User.new(
+      name: params[:name],
+      email: params[:email],
+      image_name: "default_user.jpg",
+      password: params[:password]
+      )
+    if @user.save
+      session[:user_id] = @user.id       # 登録されたユーザーのidを変数sessionに代入
+    flash[:notice] = "新規登録しました。"
+     redirect_to("/users/#{@user.id}")
+    else
+      render("users/new")
+    end
+  end
+  def edit
+    @user = User.find_by(id: params[:id])
+  end
+  def update
+    @user = User.find_by(id: params[:id])
+    @user.name = params[:name]
+    @user.email = params[:email]
+    if params[:image]
+      @user.image_name = "#{@user.id}.jpg"
+      image = params[:image]
+      File.binwrite("public/user_images/#{@user.image_name}", image.read)
+    end
+    if @user.save
+      flash[:notice] = "登録内容を編集しました。"
+      redirect_to("/users/#{@user.id}")
+    else
+      render("users/edit")
+    end
+  end
+  def login_form
+  end
+  def login
+    @user = User.find_by(
+      email: params[:email],
+      password: params[:password]
+    )    #フォームに入力されたメールアドレスとパスワードを取得
+    if @user
+      session[:user_id] = @user.id # 変数sessionに、ログインに成功したユーザーのidを代入
+      flash[:notice] ="ログインしました"
+      redirect_to("/posts/index")
+    else
+      @error_message = "メールアドレスまたはパスワードが間違っています"
+      @email = params[:email]
+      @password = params[:password]
+      render("users/login_form")
+    end #@userが存在するかどうかを判定するif文作成
+  end
+    def logout
+      session[:user_id] = nil
+      flash[:notice] = "ログアウトしました"
+      redirect_to("/login")
+    end
+    def ensure_correct_user
+      if @current_user.id != params[:id].to_i
+        flash[:notice] = "権限がありません"
+        redirect_to("/posts/index")
+      end
+    end    # ログイン中のユーザー」と「編集しようとしているユーザー」が正しくない場合の処理
+ end
-```
+```