【Ruby】データベースに値がある場合trueを返すメソッドの作成について

Question

現在、シェアハウスのマッチングアプリを作成中です。
イメージとして次のような動きをします。

１募集主が同居の募集する
２他者が募集に応募する
３募集主が応募に対して、許可をする

データベースに値がある場合trueを返すメソッドの作成について教えていただきたいと思います。
find_byやwhereで値を探し出しcountでtrueやfalseを返す形のメソッドを検討しているのですが、どうしても各IDごとで返す方法がわかりませんでした。テーブルごとで返すのではなく、ID個別に値を返す方法がありましたら、ぜひご教示いただければと思います。よろしくお願いいたします。

mysql> select * from shares;
+----+---------+----------+------------+---------------------+---------------------+
| id | user_id | house_id | ok_request | created_at          | updated_at          |
+----+---------+----------+------------+---------------------+---------------------+
|  4 |       2 |        2 |          0 | 2020-04-08 13:44:19 | 2020-04-25 05:37:02 |
|  7 |       1 |        4 |          0 | 2020-04-09 12:50:42 | 2020-04-22 23:06:49 |
|  9 |       3 |        4 |          1 | 2020-04-10 12:48:52 | 2020-04-20 02:49:29 |
| 10 |       3 |        1 |          0 | 2020-04-15 08:56:55 | 2020-04-15 08:56:55 |
| 11 |       1 |        3 |          1 | 2020-04-16 14:48:19 | 2020-04-17 14:11:23 |
| 12 |       2 |        5 |          0 | 2020-04-16 14:52:48 | 2020-04-20 02:35:36 |
| 13 |       3 |        3 |          0 | 2020-04-24 14:47:55 | 2020-04-24 14:47:55 |
| 15 |       1 |        5 |          0 | 2020-04-28 10:50:48 | 2020-04-28 10:50:48 |
+----+---------+----------+------------+---------------------+---------------------+
8 rows in set (0.00 sec)

●user.rb

class User < ApplicationRecord
  validates :name, presence: true, length: { maximum: 50 }
  validates :email, presence: true, length: { maximum: 255 },
                    format: {with: /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i },
                    uniqueness: { case_sensitive: false }
  has_secure_password
  
  has_many :houses #募集との紐付け
  has_many :shares #許可テーブルとの紐付け
  
  
  has_many :requestings, through: :shares, source: :house #リクエストしている募集の取得
  has_many :reverses_of_share, class_name: 'Share', foreign_key: 'ok_request' #自分の募集にリクエストしてくれているユーザーへの参照

  def request(house) #申請する動き
    self.shares.find_or_create_by(house_id: house.id)
  end
  
  def unrequest(house) #申請を取り消す動き
    request = self.shares.find_by(house_id: house.id)
    request.destroy if request
  end
  
  def requesting?(house) #申請してるかの確認
    self.requestings.include?(house)
  end

  #has_many :ok_requestings, through: :houses, source: :shares #許可している募集の取得
  has_many :ok_requestings, through: :shares, :foreign_key => 'ok_request', source: :house
  
  def ok_request?(house) #申請を許可してるかの確認
    self.ok_requestings.include?(house)
  end
  
  def permitted_appliments?(house)　#申請を許可してるかの確認
    return self.shares.where(ok_request: 1)
  end

  
  def permit_share?(house) #申請を許可してるかの確認
    count = self.shares.where(ok_request: 1).count
    return resurt = count > 0
  end
  
  def permit_share?(zero="0") #申請を許可してるかの確認
    count = Share.where(ok_request: true).count
    count > 0 ? count.to_s : zero
  end
  
  
  has_many :favorites
  has_many :likes, through: :favorites, source: :like
  
  def like(house)
    self.favorites.find_or_create_by(like_id: house.id) 
  end

  def unlike(house)
    like = self.favorites.find_by(like_id: house.id)
    like.destroy if like
  end
  
  def like?(house)
    self.likes.include?(house)
  end

end

●_ok_button.html.erb（ボタン）

<% if current_user.permit_share?(house) %>
  <%= form_with( url: unpermit_share_path, local: true, method: :delete) do |f| %>
    <%= hidden_field_tag :user_id, user.id %>
    <%= hidden_field_tag :house_id, house.id %>
    <%= f.submit '許可の取り消し', class: 'btn btn-danger btn-block' %>
  <% end %>
<% else %>
  <%= form_with( url: permit_share_path, local: true) do |f| %>
    <%= hidden_field_tag :user_id, user.id %>
    <%= hidden_field_tag :house_id, house.id %>
    <%= f.submit '申請を許可する', class: 'btn btn-primary btn-block' %>
  <% end %>
<% end %>

●_request_users.html.erb（ボタンをrenderするビューページ）

<% house.shares.each do |share| %>
    <p><%= share.user.name %></p>
    <%= render 'shares/ok_button', user: share.user, house: house %>
<% end %>

●shares_controller.rb

class SharesController < ApplicationController
  before_action :require_user_logged_in
  
  def create
    house = House.find(params[:house_id])
    current_user.request(house)
    flash[:success] = '同居申請をしました。'
    redirect_to houses_path
  end

  def destroy
    house = House.find(params[:house_id])
    current_user.unrequest(house)
    flash[:success] = '同居申請を取り消しました'
    redirect_to houses_path
  end
  
  def permit_share  
    share = Share.where(user_id: params[:user_id])
                 .where(house_id: params[:house_id])
    if share
      share.update(ok_request: true)
      flash[:success] = '申請を許可しました。'
    end
    redirect_to house_path(params[:house_id])
  end
  
  def unpermit_share  
    
    share = Share.where(user_id: params[:user_id])
                 .where(house_id: params[:house_id])
    if share
      share.update(ok_request: false)
      flash[:success] = '許可を取り消しました。'
    end
    redirect_to house_path(params[:house_id])
  end
  
end

Comments

[maisumakun]

「うまく動作しない」とありますが、どのようになるのでしょうか？

[tomohiro2525]

このメソッドを動作させて、ボタンの表示を切り替えたいのですが、ボタンが切り替わりません。

[maisumakun]

えっと、「判定メソッドの動作」と「画面表示」のどちらに問題があるか切り分けは行いましたか？

[tomohiro2525]

はい、画面表示はif-elsaの条件分岐をしないように書き換えたところ、ボタンどちらも表示させデータベースに反映させることができました。
ですので、判定メソッドの動作に問題があると判定しました。判定メソッドは、テーブル全体でok_requestカラムに値があるとtrueを返すメソッドとなっているようです。ですので、ID個別にtrueかfalseを返せるように記述したいのですが、、わかりませんでした。

[winterboum]

何が問題なのかよくわかりません。
まず、３つのrubyのプログラムですが、file名もしくは class名を併記してください。
modelが３つありそうです。それらの関連を定義している部分を載せてください。
で、なにが問題なのかですが「各IDごとで返す方法」とは
１）どのmodelのID毎に、でしょう？
2) 何を返すのでしょう？　「ok_request　が１であるレコードがあるかどうか」ですか？

[tomohiro2525]

返信が遅くなりまして申し訳ありません。modelについて追記しました。
作成中のアプリの概要は、次のとおりです。
１募集主が同居の募集する
２他者が募集に応募する
３募集主が応募に対して、許可をする

3の動きを実装するにあたり等質問の内容に陥りました。

言葉足らずで申し訳ありません。
「sharesテーブルのid4のok_request　が１であるレコードがあるかどうか」を返す方法のことを意味しておりました。テーブル内のid別に判定する方法がわからず、質問させていただきました。

[winterboum]

「sharesテーブルのid4のok_request　が１であるかどうか」
ではなくて？
というのは　id4ということは一つの share ですから、「あるかどうか」つまり「複数ある中で１のものがあるか」と言うことにはならないので

House(id=4) に応募したuserのなかに ok_requestが１のものがあるか否か
ということ？　それとも
User（is=4)が応募したhouseの中に ok_requestが１のものがあるか否か
ということ？

[tomohiro2525]

三度すみません。ご指摘のとおり「sharesテーブルのid4のok_request　が１であるかどうか？」「User（is=4)が応募したhouseの中に ok_requestが１のものがあるか否か」という動きを実装したいと思っております。

[winterboum]

sharesテーブルのid4　ってどのidのことですか？
id | user_id | house_id　と　idが３つあります。
prefix無しのidは値４なのはひとつだけですが。
「sharesテーブルのid4のok_request　が１であるかどうか？」
と
「User（is=4)が応募したhouseの中に ok_requestが１のものがあるか否か」
は意味が違います

[tomohiro2525]

→id | user_id | house_id　一番左のidのことでした。
ですので、「sharesテーブルのid4のok_request　が１であるかどうか？」を判定したいということになります。
曖昧な表現ばかりで申し訳ありません。よろしくお願いいたします。

Answer 1

課題　sharesテーブルのid4のok_request　が１であるかどうか？

でしたら、
Share.find_by(id: 4)&.ok_request == 1
です。

Share.find(4) にしないのは万一４がないときに例外がでるから
Share.find_by(id: 4)&.ok_request　と　& を入れたのは、万一４が無いときに例外がでるから

Comments

[tomohiro2525]

丁寧にご解答いただきありがとうます。質問の書き方、問題提示の簡潔化も含め今後勉強していきたいと思います。

[winterboum]

読み返してみれば「sharesテーブルのid4のok_request　が１であるかどうか？」という意味のことを言っていますね。
私が混乱したのは、このような基本のことでわからなくなっているとは思わなかったので、「絞り込んだ複数のデータの中に一つでも１のものがあるか」というようなもうちょい複雑なことを聞いているのではないのか？とおもってしまったからでした