トップに関する質問 [Ruby on Rails]複数条件でAND検索し、その結果をビューで表示させたい

編集履歴

質問編集履歴

追記　part3

2019/12/14 12:37

投稿

clora

スコア72

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -479,3 +479,29 @@
 ただ、まだ検索結果のビューでは、@production_resulはnilclassになってしまいますね、、、。
 うまく変数が渡せていないだけだと思うのですが、、、。
+### 追記 part3
+なんとか解決しました。
+ビューの方の表記を変えたところきちんと検索結果が表示されました。
+```html
+間違い
+<%= @production_result.date %>
+正解
+<%= (@production_result.to_a)[0].date %>
+```
+また、asmさんのアドバイス通りwhereでなく、find_byを使えば@production_result.dateという形でも取り出せます。whereやfind_byの違いや原理を理解していないのがエラーにハマった原因でした。

誤字の修正

2019/12/14 12:37

投稿

clora

スコア72

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -442,42 +442,34 @@
 @production_resultがnilになってしまうのはproduct_idの指定の仕方が間違えていたことによるものでした。
-上の方にも書いた通り、product_idは他のテーブルに値を探しにいって変数product_idに入れて、searchメソッドに渡しています。その時にフォームから直接受け取ったgroup_idは"1"となっていたのに対し、product_idは 1 となっていました。そのため、そのまま、product_idを検索条件に入れてしまうと、条件に引っかからず、検索結果が0となり、```ruby
+上の方にも書いた通り、product_idは他のテーブルに値を探しにいって変数product_idに入れて、searchメソッドに渡しています。その時にフォームから直接受け取ったgroup_idは"1"となっていたのに対し、product_idは 1 となっていました。そのため、そのまま、product_idを検索条件に入れてしまうと、条件に引っかからず、検索結果が0となり、@production_resulがnilになっているのでしたそこで下記のように、searchメソッドを書き直したところ、きちんと検索結果が、@production_resultに入ってくれました。
+```ruby
+#production.rb
+def self.search(date,line_id,product_id)
+    Production.where(date: date).
+              where(group_id: group_id).
+              where(product_id: "#{product_id}")
+end
+```
+```ruby
 [1] pry(#<Productions::SearchesController>)> product_id
-=> 1
+=> 1 #こっちはただの数字
 [2] pry(#<Productions::SearchesController>)> params[:production][:group_id]
-=> "1"がnilになっているのでしたそこで下記のように、searchメソッドを書き直したところ、きちんと検索結果が、@production_resultに入ってくれました。
-```ruby
-#production.rb
-def self.search(date,line_id,product_id)
-    Production.where(date: date).
-              where(group_id: group_id).
-              where(product_id: "#{product_id}")
-end
-```
-```ruby
-[1] pry(#<Productions::SearchesController>)> product_id
-=> 1 #こっちはただの数字
-[2] pry(#<Productions::SearchesController>)> params[:production][:group_id]
 => "1" #こっちはダブルクォーテーションで囲まれている
 ```

追記　part2

2019/12/14 11:51

投稿

clora

スコア72

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -433,3 +433,57 @@
 ### 試したこと
 binding pryをしてみたところ、seaechメソッドをすると@production_resultが[]となってしまいます。
+### 追記 part2
+上記の状況から進展がありました。
+@production_resultがnilになってしまうのはproduct_idの指定の仕方が間違えていたことによるものでした。
+上の方にも書いた通り、product_idは他のテーブルに値を探しにいって変数product_idに入れて、searchメソッドに渡しています。その時にフォームから直接受け取ったgroup_idは"1"となっていたのに対し、product_idは 1 となっていました。そのため、そのまま、product_idを検索条件に入れてしまうと、条件に引っかからず、検索結果が0となり、```ruby
+[1] pry(#<Productions::SearchesController>)> product_id
+=> 1
+[2] pry(#<Productions::SearchesController>)> params[:production][:group_id]
+=> "1"がnilになっているのでしたそこで下記のように、searchメソッドを書き直したところ、きちんと検索結果が、@production_resultに入ってくれました。
+```ruby
+#production.rb
+def self.search(date,line_id,product_id)
+    Production.where(date: date).
+              where(group_id: group_id).
+              where(product_id: "#{product_id}")
+end
+```
+```ruby
+[1] pry(#<Productions::SearchesController>)> product_id
+=> 1 #こっちはただの数字
+[2] pry(#<Productions::SearchesController>)> params[:production][:group_id]
+=> "1" #こっちはダブルクォーテーションで囲まれている
+```
+ただ、まだ検索結果のビューでは、@production_resulはnilclassになってしまいますね、、、。
+うまく変数が渡せていないだけだと思うのですが、、、。

誤字の修正

2019/12/14 11:50

投稿

clora

スコア72

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -322,7 +322,7 @@
 Rails.application.routes.draw do
-  root 'lines#index'
+  root 'groups#index'
   namespace :productions do

誤字の修正

2019/12/14 06:47

投稿

clora

スコア72

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -366,7 +366,7 @@
  product_id = Product.find_by(code: params[:production][:product_id]).id
- @production_result = Production.search(params[:production][:date], params[:production][:line_id], product_id)
+ @production_result = Production.search(params[:production][:date], params[:production][:group_id], product_id)
@@ -388,11 +388,11 @@
 def self.search(date,line_id,product_id)
-  if date and line_id and product_id
+  if date and group_id and product_id
     Production.where
-    ("(date = ?) AND (line_id = ?) AND (product_id = ?)", "%#{date}%", "%#{line_id}%", "%#{product_id}%")
+    ("(date = ?) AND (group_id = ?) AND (product_id = ?)", "%#{date}%", "%#{group_id}%", "%#{product_id}%")
   else

誤字の修正

2019/12/14 06:46

投稿

clora

スコア72

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -356,18 +356,6 @@
 ```
-resources :groups do
-resources :productions
-end
-end
 ```ruby
 #/controllers/productions/searches_controller.rb

コメントを受けて追記

2019/12/14 06:44

投稿

clora

スコア72

test CHANGED Viewed

	@@ -1 +1 @@
1	- [Ruby on Rails]複数条件でAND検索したい
1	+ [Ruby on Rails]複数条件でAND検索し、その結果をビューで表示させたい

test CHANGED Viewed

@@ -289,3 +289,159 @@
 rails 5.0.7
 ruby 2.5.1
+### 追記
+コメントで指摘を受けた部分を変更しました。
+それに加えて、resultアクションを追加して、@production_resultに検索結果を挿入し、resultアクションで検索結果を表示させるようにしました。
+しかし、@production_resultがnilになってしまい検索結果が表示されません。
+どうもsearchメソッドを使用すると、@production_resultがから([])となってしまいます。
+これは、検索結果がないということなのでしょうか?配列の中の値がnilとして返ってくるのならわかるのですが、配列自体が空になってしまいます。まだ、searchメソッドの記述がおかしいのでしょうか。
+下記はsearchメソッドを使用した後、binding pryで確かめた結果です。
+```
+[1] pry(Production)> @production_result
+=> nil
+```
+```ruby
+#route.rb
+Rails.application.routes.draw do
+  root 'lines#index'
+  namespace :productions do
+    resources :searches, only: :index do
+      collection do
+        get 'search'
+      end
+      collection do
+        get 'result'
+      end
+    end
+  end
+  resources :groups do
+    resources :productions
+  end
+end
+```
+resources :groups do
+resources :productions
+end
+end
+```ruby
+#/controllers/productions/searches_controller.rb
+def search
+ @production_result = Production.new
+ product_id = Product.find_by(code: params[:production][:product_id]).id
+ @production_result = Production.search(params[:production][:date], params[:production][:line_id], product_id)
+ redirect_to result_productions_searches_path(@production_result)
+end
+def result
+end
+```
+```ruby
+def self.search(date,line_id,product_id)
+  if date and line_id and product_id
+    Production.where
+    ("(date = ?) AND (line_id = ?) AND (product_id = ?)", "%#{date}%", "%#{line_id}%", "%#{product_id}%")
+  else
+    redirect_to productions_searches_path
+  end
+end
+```
+```html
+-#views/productions/searches/result.html
+検索結果
+<%= @production_result.date %>
+<%= @production_result.group_id %>
+<%= @production_result.product_id %>
+```
+| id | date | group_id | product_id |
+| ---- | ---- | ---- | ---- |
+| 26 | 2019-12-13 00:00:00 | 1 | 1 |
+### 試したこと
+binding pryをしてみたところ、seaechメソッドをすると@production_resultが[]となってしまいます。

誤字の修正

2019/12/14 06:42

投稿

clora

スコア72

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -182,7 +182,7 @@
     <%= f.number_field :products_id %> #productsのid(数字)を入力
-    <%= f.select :group_id, [["8F-16", 1],["8F-9", 2]]  %> #グループidの入力
+    <%= f.select :group_id, [["グループ1", 1],["グループ2", 2]]  %> #グループidの入力
     <%= f.submit '検索', class: "btn" %>