質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.91%

paramsで渡したtitleとstatusに一致した検索結果を一覧表示する(titleはあいまい検索)

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 150

kousuke24

score 29

現在Ruby on Railsでタスク管理アプリを作成しています。
form_withでtitleをあいまい検索、statusをラジオボタンで選択します。両者に該当したオブジェクトを一覧表示したいです。

現状

・選択したstatusは無視され、titleで該当したオブジェクトが一覧画面に表示されます。
・task:status=多:1 で実装しています。
・statusはrailsコンソールで既に作成しています。

Status.create(id: 1, name: "未着手")
Status.create(id: 2, name: "着手中")
Status.create(id: 3, name: "完了")

コントローラー(tasks_controller.rb)

class TasksController < ApplicationController
  def index
    @tasks = if params[:title].present? && params[:status_id].present?
               task = Task.where('title LIKE ?', "%#{params[:title]}%")
               task.each do |t|
                if t.status_id == params[:status_id]
                  t
                else
                  next
                end
               end
             elsif params[:title].present?
               Task.where('title LIKE ?', "%#{params[:title]}%")
             elsif params[:status_id] == "1"
               Task.where(status_id: 1)
             elsif params[:status_id] == "2"
               Task.where(status_id: 2)
             elsif params[:status_id] == "3"
               Task.where(status_id: 3)
             elsif params[:sort] == 'create_desc'
               Task.all.order(created_at: :DESC)
             elsif params[:sort] == 'deadline_asc'
               Task.all.order(deadline: :ASC)
             elsif params[:sort] == 'deadline_desc'
               Task.all.order(deadline: :DESC)
             else
               Task.all.order(created_at: :ASC)
             end
  end

  private

  def task_params
    params.require(:task).permit(:title, :content, :deadline, :status_id)
  end

ビュー(index.html.erb)

<h2>タスク一覧</h2>

<%= form_with url: root_path, method: :get, local: true do |f| %>
  <%= f.text_field :title %>

  <%= f.radio_button :status_id, "1" %>
  <%= f.label :status_id, "未着手", {value: "1"} %>
  <%= f.radio_button :status_id, "2" %>
  <%= f.label :status_id, "着手中", {value: "2"} %>
  <%= f.radio_button :status_id, "3" %>
  <%= f.label :status_id, "完了", {value: "3"} %>

  <%= f.submit "検索" %>
<% end %>

<%= form_with url: root_path, method: :get, local: true do |f| %>
  <%= f.select :sort, options_for_select({作成日時降順: "create_desc", 終了期限昇順: "deadline_asc", 終了期限降順: "deadline_desc"},params[:sort]), include_blank: "作成日時昇順" %>

  <%= f.submit "並べ替え" %>
<% end %>

<% @tasks.each do |task| %>
  <p><%= link_to task.title, task_path(task) %> : <%= link_to task.status.name, task_path(task) %></p>
<% end %>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

状況が曖昧なので 推定

@tasks = if params[:title].present? && params[:status_id].present?
           task = Task.where('title LIKE ?', "%#{params[:title]}%")
           task.each do |t|
              if t.status_id == params[:status_id]
                t
              else
                next
              end
             end


if-elsif のつながりの中で、このifの場合に起きている現象ですね?
task.eachの結果が戻りますが、eachの戻り値はtaskなので、task = Task.whereの結果が使われて閉まっています。
この方法で行うなら
task.map do ...... end.compact ですね。

Task.where('title LIKE ?', "%#{params[:title]}%").
     where(status_id: params[:status_id])


で。
elsif params[:status_id] が沢山並んでいますが

elsif params[:status_id]
  Task.where(status_id: params[:status_id])



orderは絞り込んだ時にも掛けたいのでしょうから、titleとstatus_idで
@tasks = if .... end を終えて、そのあとで
if params[:sort] == 'create_desc'     @tasks.all.order(created_at: :DESC)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/14 22:51

    説明が不足している中回答して頂きありがとうございます。
    where文を連結させることで解決できました!これから教えていただいた内容を加味し、リファクタリングを行いたいと思います!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.91%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る