質問編集履歴

読みやすく修正

2020/08/31 11:02

投稿

75ks

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -4,7 +4,7 @@
 検索機能を実装したいのですがうまく動きません。
-具体的には`ALL`,` MEN`, `WOMEN`の3つのボタンを作り、`ALL`の時には全てのレコード、`MEN`の時には男性のレコード、`WOMEN`の時には女性のレコードを表示する機能です。
+具体的には`ALL`,`MEN`, `WOMEN`の3つのボタンを作り、`ALL`の時には全てのレコード、`MEN`の時には男性のレコード、`WOMEN`の時には女性のレコードを表示する機能です。
@@ -14,7 +14,9 @@
 ユーザーは`usersテーブル`という名前で、投稿は`postsテーブル`という名前を使っていて、ユーザーは`devise`を使っています。
+`gender_id`は`Active Hash`を使っています。
-`user`と`post`は、「1:N」の関係です。
+`user`と`post`は、`1:N`の関係です。
 検索のために`searchアクション`を定義しています。

本文の修正

2020/08/31 11:02

投稿

75ks

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -8,7 +8,7 @@
-現在のコードでは、`ALL`の時には全てのレコード、`MEN`の時には男性のレコード、`WOMEN`の時にも男性のレコードが表示されてしまいます。
+現在のコードでは、`ALL`の時には全てのレコード、`MEN`の時には男性のレコード、なぜか`WOMEN`の時にも男性のレコードが表示されてしまいます。

わかりやすく文章とコードを編集

2020/08/31 11:00

投稿

75ks

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,68 +1,82 @@
-#Ruby on Rails
-一覧ページにある`link_to`を押した時に、子モデル(postモデル)のレコードを、親モデル(userモデル)のカラムである性別(gender_id)で検索する方法を教えていただきたいです。
-gender_idはAcrtive Hashを使っています。
+Ruby on Railsで写真とテキストを投稿できるSNSを作っています。
+検索機能を実装したいのですがうまく動きません。
-検索のための`search`アクションを作成し、`post`モデルに`joins`と`where`を使って検索のコードを記述をしましたが、上手くいかず、原因がわかりません。
+具体的には`ALL`,` MEN`, `WOMEN`の3つのボタンを作り、`ALL`の時には全てのレコード、`MEN`の時には男性のレコード、`WOMEN`の時には女性のレコードを表示する機能です。
-```indexhtmlerb
+現在のコードでは、`ALL`の時には全てのレコード、`MEN`の時には男性のレコード、`WOMEN`の時にも男性のレコードが表示されてしまいます。
-<ul class="gender-lists">
-     <li>
-      <%= link_to "ALL", root_path, class:"gender" %>
+ユーザーは`usersテーブル`という名前で、投稿は`postsテーブル`という名前を使っていて、ユーザーは`devise`を使っています。
-     </li>
+`user`と`post`は、「1:N」の関係です。
-     <li>
+検索のために`searchアクション`を定義しています。
-      <%= link_to "MEN", search_posts_path(gender_id:2), class:"gender" %>
+`search.html.erb`は`index.htl.erb`と同じコードにしています。
-     </li>
-     <li>
-      <%= link_to "WOMEN", search_posts_path(gender_id:3), class:"gender" %>
-     </li>
+###ソースコード
-    </ul>
+```controller
+-----省略
+def search
+    @posts = Post.search(params[:gender_id])
+  end
+-----省略
 ```
-```gendermodel
+```routes
-class Gender < ActiveHash::Base
+Rails.application.routes.draw do
-  self.data = [
+  devise_for :users
-    { id: 1, name: '---' },
+  root "posts#index"
-    { id: 2, name: 'MEN' },
+  resources :posts, only: [:new, :create, :show, :destroy, :edit, :update] do
-    { id: 3, name: 'WOMEN' },
+    resources :comments, only: :create
-    { id: 4, name: '選択しない' }
+    collection do
+      get "search"
-  ]
+    end
   end
+  resources :users, only: :show
+end
 ```
-```postmodel
+```model
 class Post < ApplicationRecord
@@ -92,17 +106,13 @@
   def self.search(search)
-    if search == 2
+    if search = 2
       Post.joins(:user).where("users.gender_id = 2")
-    elsif search == 3
+    elsif search = 3
       Post.joins(:user).where("users.gender_id = 3")
-    else
-      Post.includes(:user).order("created_at DESC")
     end
@@ -116,242 +126,36 @@
-```usermodel
+```indexhtmlerb
-class User < ApplicationRecord
+-------省略
-  # Include default devise modules. Others available are:
-  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
-  devise :database_authenticatable, :registerable,
-         :recoverable, :rememberable, :validatable
-  extend ActiveHash::Associations::ActiveRecordExtensions
+<ul class="gender-lists">
-  belongs_to_active_hash :gender
+     <li>
-  has_many               :posts, dependent: :destroy
+      <%= link_to "ALL", root_path, class:"gender" %>
+     </li>
+     <li>
-  has_many               :comments, dependent: :destroy
+      <%= link_to "MEN", search_posts_path(gender_id:2), class:"gender" %>
+     </li>
+     <li>
+      <%= link_to "WOMEN", search_posts_path(gender_id:3), class:"gender" %>
+     </li>
+    </ul>
-  with_options presence: true do
-    validates :nickname
+-------省略
-    validates :gender_id
-  end
-  validates :email, uniqueness: true
-  validates :nickname, length: { maximum: 8, message: "は8文字以下にしてください" }
-  validates :gender_id, numericality: { other_than: 1, message: "を選択してください" }
-  half_width_alphanumeric = /\A(?=.*?[a-z])(?=.*?\d)[a-z\d]{6,100}+\z/i
-  validates :password, presence: true, format: { with: half_width_alphanumeric, message: "に半角英数字を使用してください" }
-end
 ```
-```postcontroller
-class PostsController < ApplicationController
-  before_action :set_post, only: [:show, :destroy, :edit, :update]
-  before_action :correct_user, only: [:edit, :update]
-  def index
-    @posts = Post.includes(:user).order("created_at DESC")
-  end
-  def new
-    @post = Post.new
-  end
-  def create
-    @post = Post.new(post_params)
-    if @post.valid?
-      @post.save
-      flash[:notice] = "投稿が完了しました"
-      redirect_to root_path
-    else
-      flash.now[:alert] = "投稿に失敗しました"
-      render :new
-    end
-  end
-  def show
-    @comment = Comment.new
-    @comments = @post.comments.includes(:user)
-  end
-  def destroy
-    if @post.destroy
-      flash[:notice] = "削除が完了しました"
-      redirect_to root_path
-    else
-      flash.now[:alert] = "削除に失敗しました"
-      render :show
-    end
-  end
-  def edit
-  end
-  def update
-      if @post.update(post_params)
-      flash[:notice] = "編集が完了しました"
-      redirect_to post_path(@post.id)
-    else
-      flash.now[:alert] = "編集に失敗しました"
-      render :edit
-    end
-  end
-  def search
-    @posts = Post.search(params[:gender_id])
-  end
-  private
-  def post_params
-    params.require(:post).permit(:text, images: []).merge(user_id: current_user.id)
-  end
-  def set_post
-    @post = Post.find(params[:id])
-  end
-  def correct_user
-    unless user_signed_in? && current_user.id == @post.user.id
-      redirect_to root_path
-    end
-  end
-end
-```
-.
-.
-.
---------------追記---------------------
-`postmodel`の`self.search`の記述を下記のように`search == 2`から `search = 2`のように書き換えたところ、
-`MEN`のボタンを押したときは男性だけの投稿になりましたが、`WOMEN`のボタンを押しても男性の投稿になってしまいます。
-```postmodel
-def self.search(search)
-    if search = 2
-      Post.joins(:user).where("users.gender_id = 2")
-    elsif search = 3
-      Post.joins(:user).where("users.gender_id = 3")
-    end
-```

誤字の修正

2020/08/31 10:52

投稿

75ks

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -336,7 +336,7 @@
 `postmodel`の`self.search`の記述を下記のように`search == 2`から `search = 2`のように書き換えたところ、
-`MEN`のボタンを押したときは男性だけの投稿になりましたが、`WEMEN`のボタンを押しても男性の投稿になってしまいます。
+`MEN`のボタンを押したときは男性だけの投稿になりましたが、`WOMEN`のボタンを押しても男性の投稿になってしまいます。

情報の更新

2020/08/31 09:35

投稿

75ks

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -323,3 +323,35 @@
 ```
+.
+.
+.
+--------------追記---------------------
+`postmodel`の`self.search`の記述を下記のように`search == 2`から `search = 2`のように書き換えたところ、
+`MEN`のボタンを押したときは男性だけの投稿になりましたが、`WEMEN`のボタンを押しても男性の投稿になってしまいます。
+```postmodel
+def self.search(search)
+    if search = 2
+      Post.joins(:user).where("users.gender_id = 2")
+    elsif search = 3
+      Post.joins(:user).where("users.gender_id = 3")
+    end
+```

情報の追加

2020/08/31 09:27

投稿

75ks

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,6 +1,6 @@
 #Ruby on Rails
-子モデル(postモデル)のレコードを、親モデル(userモデル)のカラムである性別(gender_id)で検索する方法を教えていただきたいです。
+一覧ページにある`link_to`を押した時に、子モデル(postモデル)のレコードを、親モデル(userモデル)のカラムである性別(gender_id)で検索する方法を教えていただきたいです。
 gender_idはAcrtive Hashを使っています。
@@ -10,6 +10,34 @@
+```indexhtmlerb
+<ul class="gender-lists">
+     <li>
+      <%= link_to "ALL", root_path, class:"gender" %>
+     </li>
+     <li>
+      <%= link_to "MEN", search_posts_path(gender_id:2), class:"gender" %>
+     </li>
+     <li>
+      <%= link_to "WOMEN", search_posts_path(gender_id:3), class:"gender" %>
+     </li>
+    </ul>
+```
 ```gendermodel
 class Gender < ActiveHash::Base