編集履歴

質問編集履歴

説明追加

2020/02/17 05:02

投稿

1234567

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -332,6 +332,108 @@
 ```
+application_controller
+class ApplicationController < ActionController::Base
+  before_action :set_current_user
+  def top
+  end
+  def set_current_user
+    @current_user = User.find_by(id: session[:user_id])
+  end
+  def authenticate_user
+  end
+  def forbid_login_user
+    if @current_user
+      flash[:notice] = "すでにログインしています"
+      redirect_to("/posts/index")
+    end
+  end
+end
+```
+```
+home_controller
+class HomeController < ApplicationController
+before_action :forbid_login_user, {only: [:new, :create, :login_form, :login]}
+  def top
+  end
+end
+```
+```
 users/login_form.html.erb

説明追加

2020/02/17 05:02

投稿

1234567

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -516,6 +516,80 @@
+```
+user/edit
+<div class="user-index">
+  <h1>ユーザー情報の編集</h1>
+    <% @user.errors.full_messages.each do |message| %>
+  <div class="form-error">
+    <%= message %>
+  </div>
+    <% end %>
+  <div class="user-edit-container">
+      <%= form_for @user do |f| %>
+    <div class="user-edit-item">
+      <%= f.label :name, "ユーザー名(※必須)", class: "form" %><br>
+      <%= f.text_field :name %><br>
+      <%= f.label :image, "画像", class: "form" %><br>
+      <%= f.file_field :image %><br>
+      <%= f.label :email, "メールアドレス(※必須)", class: "form" %><br>
+      <%= f.text_field :email %><br>
+      <%= f.label :password, "パスワード(※必須)", class: "form" %><br>
+      <%= f.text_field :password %><br>
+      <%= f.submit "新規登録", class: "form" %>
+    </div>
+      <% end %>
+  </div>
+</div>
+```

説明追加

2020/02/17 04:57

投稿

1234567

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -456,14 +456,14 @@
     t.string "email"
-    t.string "password"
     t.string "image"
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
+    t.string "password_digest"
   end

説明追加

2020/02/17 04:48

投稿

1234567

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -208,25 +208,75 @@
   def login
-    @user = User.find_by(email: params[:email], password: params[:password])
+    @user = User.find_by(email: params[:email])
-    if @user
+      if @user && @user.authenticate(params[:password])
-      session[:user_id] = @user.id
+        session[:user_id] = @user.id
-      flash[:notice] = "ログイン成功"
+        flash[:notice] = "ログイン成功"
-      redirect_to("/posts/index")
+        redirect_to("/posts/index")
+      else
+        @error_message = "メールアドレスかパスワードが違います"
+        @email = params[:email]
+        @password = params[:password]
+        render("users/login_form")
+      end
+  end
+  def logout
+    session[:user_id] = nil
+    flash[:notice] = "ログアウトしました"
+    redirect_to("/login")
+  end
+  def edit
+      @user = User.find_by(id: params[:id])
+  end
+  def update
+    @user = User.find_by(id: params[:id])
+    if @user.update(user_params)
+      flash[:notice] = "ユーザー情報を変更しました"
+      redirect_to("/users/#{@user.id}")
     else
-      @error_message = "メールアドレスかパスワードが違います"
-      @email = params[:email]
-      @password = params[:password]
-      render("users/login_form")
+      render("/users/edit")
     end
@@ -236,82 +286,34 @@
-  def logout
-    session[:user_id] = nil
-    flash[:notice] = "ログアウトしました"
-    redirect_to("/login")
-  end
-  def edit
+  def destroy
-      @user = User.find_by(id: params[:id])
-  end
-  def update
     @user = User.find_by(id: params[:id])
-    if @user.update(user_params)
+    @user.destroy
-      flash[:notice] = "ユーザー情報を変更しました"
-      redirect_to("/users/#{@user.id}")
+    redirect_to("/users/index")
+    flash[:notice] = "削除しました"
-    else
+  end
+  private
+    def user_params
-      render("/users/edit")
+      params.require(:user).permit(:name, :email, :password, :image)
     end
-  end
-  def destroy
-    @user = User.find_by(id: params[:id])
-    @user.destroy
-    redirect_to("/users/index")
-    flash[:notice] = "削除しました"
-  end
-  private
-    def user_params
-      params.require(:user).permit(:name, :email, :password, :image)
-    end
@@ -474,6 +476,46 @@
+```
+models/User
+class User < ApplicationRecord
+  has_secure_password
+  mount_uploader :image,ImageUploader
+  validates :name, {presence: true}
+  validates :email, {presence: true}
+  validates :password, {presence: true}
+def posts
+  return Post.where(user_id: self.id)
+end
+  paginates_per 8
+end
+```

説明追加

2020/02/17 04:47

投稿

1234567

スコア7

test CHANGED Viewed

	@@ -1 +1 @@
1	- ~~ArgumentError in Users#login_~~form　　~~First argument in~~ for~~m cannot contain nil or be empty　エラー~~
1	+ ログインフォームをform_forで作成できず、困っています

test CHANGED Viewed

File without changes

説明追加

2020/02/13 04:55

投稿

1234567

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -480,6 +480,24 @@
 自分で試したこととしては、form_forの@userがnilなので、login_formアクション内で@userオブジェクトを生成したりもしたのですが、特に変わらず、、
+(具体的には、
+・@user = User.find_by(email: params[:email], password: params[:password])
+・@user = User.find_by(id: params[:id])
+など。)
+sessionコントローラーなどは用いずに実装したいと考えています。（できないのであれば仕方ないとは考えています）。
+また、form_tagで実装すると機能するので、form_tagのままでいいかなとも思いつつ、全体としてform_forで揃えるべきだよな、、とも考えています。
-もしお分かりの方がいらっしゃいましたら、アドバイスいただけると幸いです。
+login_formアクション内で@userオブジェクトに何を定義すればいいのでしょうか？もしお分かりの方がいらっしゃいましたら、アドバイスいただけると幸いです。