カレントユーザーが投稿した音楽ではなくかつ音楽購入している場合に「SOLD OUT」を表示する。

Question

■解決したいこと
Ordersテーブルより任意のuser_idを取得する。
カレントユーザーが投稿した音楽ではなくかつ音楽購入している場合は「SOLD OUT」を表示する。

■仮説と検証作業の結果
カレントユーザーが投稿した音楽の場合は金額を表示。（できている）

カレントユーザーが投稿した音楽ではなくかつ音楽を購入していない場合は
購入ページへのリンクになっている金額を表示。（できている）

Ordersテーブルのレコードにアクセスできていない。
musicsControllerにOrdersテーブルからレコードを取得する記述ができていない

@order = Order.new(order_params)

def order_params
    params.permit(:user_id, :music_id).merge(user_id: current_user.id, music_id: params[:music_id])
  end



この記述だとテーブルの中身ではなくカレントユーザーIDのみが取得できている状態にあり、
@order.music_idを見るとわかるように
music_idの値もnilのままである。

def index
    @order = Order.new(order_params)
    @musics = Music.includes(:user).order('created_at DESC')
    query = 'SELECT * FROM musics'
    # @musics = Music.find_by_sql(query)
  end

private

  def order_params
    params.permit(:user_id, :music_id).merge(user_id: current_user.id, music_id: params[:music_id])
  end

<div class="info_show4">
    <% if user_signed_in? && current_user.id != music.user_id %>
        <% if user_signed_in? && music.id == @order.music_id  %> 
            <%= @order.user_id %>
        <% else %>
            <%= link_to "¥#{music.price}", music_orders_path(music.id) %>
        <% end %>
    <% else %>
    <%= "¥#{music.price}" %>
    <% end %>
      </div>

class Music < ApplicationRecord
  belongs_to :user
  has_many :tracks, dependent: :destroy
  has_many :comments, dependent: :destroy
  has_many :favorites
  has_many :order
  has_one_attached :image

  with_options presence: true do
    validates :title, length: { maximum: 22 } 
    validates :image
    validates :artist_name, length: { maximum: 22 } 
    validates :price, format: { with: /\A[0-9]+\z/ },
                      numericality: { only_integer: true, greater_than_or_equal_to: 500, less_than_or_equal_to: 50_000 }
  end

  validates :cd_type_id, numericality: { other_than: 1 }

  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to :cd_type

  validate :image_content_type, if: :was_attached?

  def image_content_type
    extension = ['image/png', 'image/jpg', 'image/jpeg']
    errors.add(:image, 'の拡張子が間違っています') unless image.content_type.in?(extension)
  end

  def was_attached?
    image.attached?
  end

  def favorited_by?(user)
    favorites.where(user_id: user.id).exists?
  end
end

class Order < ApplicationRecord
  has_one_attached :image
  attr_accessor :token
  belongs_to :music
  belongs_to :user
  
  
  with_options presence: true do
    validates :token
    validates :music_id
    validates :user_id
  end
end

ordersテーブル

https://gyazo.com/e44ce1f035a448db91a7bc9410af5818



musicテーブル
https://gyazo.com/4509c471019f6180145e6f639e34cd7a



ビュー
https://gyazo.com/6e15b1d6a8ca8c9b489530443a91b02a

Answer 1

着信音は、携帯電話の所有者の個性やスタイルに影響を与える重要な要素の 1 つです。 音楽を購入する必要はありませんが Gratis Ringtones Samsung で携帯電話の着信音をお気に入りの音楽に簡単に変更できます。

Answer 2

「Ordersテーブルのレコードにアクセスできていない。」
ではなく
「Ordersテーブルのレコードにアクセスしていない。」
ですね。
Musicテーブルにはアクセスしていますが

##追記
has_many :order は has_many :orders に直してください。
ここの単複型は重要です。

そうしてあれば、「ある music」 の orderは music.orders で全部取れます。
その orders の中に user_id が current_user のものがあるかどうか、で

        <%= order.user_id %>  と
        <%= link_to "¥#{music.price}", music_orders_path(music.id) %>

を切り分けてください。
order.user_id よりも current_userであることは与件ですから、current_user.id でよいのでは、と思いますが。

なお、
このままでは DB へのアクセスが多くなるとかありますが、まだそこを議論するレベルではないと思うので、まず動くものができてから、ですね

Comments

[sia_yuki_f717]

ご回答ありがとうございます。
こちらどのような記述をすることでOrdersテーブルにアクセスすることができますでしょうか？
お手数ですがご教示いただきたいです。
よろしくお願いします！

[winterboum]

Oderからどういう条件で取り出すか、によります。
find, find_by, where を調べてください。
全部纏めてでしたら all でいけます。

[sia_yuki_f717]

musicsControllerのindexに

@order = Order.find(params[:id])
@order = Order.find(params[:music_id])
＠order = Order.find_by(:music_id music.id)
＠order = Order.where(:music_id music.id)
＠order = Order.find_by(:music_id params[:music_id])
＠order = Order.where(:music_id params[:music_id])

などいろんな記述を記入しましたがやはりorderテーブルが取れませんでした💦
私の記述やメソッドの使い方が間違っているのでしょうか？
music.idに対応するOrdersテーブルのmusic_idを取り出したいです。
何度も申し訳ございません。　
ご教示いただきたいです。
よろしくお願いいたします。

[winterboum]

？？？「music.idに対応するOrdersテーブルのmusic_idを取り出したいです。」
music.idに対応するOrdersテーブルのデータの music_id は music.id と同じですよ？
どういう Order を取り出したいのですか？

[sia_yuki_f717]

言葉足らず申し訳ございません。
下記のようにユーザーがログインしていてかつmusicsテーブルのidとordersテーブルのmusic_idが同じである場合
（ordersテーブルにあるのはもうすでに購入済みの音楽であり、カレントユーザーが特定の音楽を購入していたら）SOLDOUTと表示させるというのを実装したいと思っております。
<% if user_signed_in? && music.id == @order.music_id %>
現在のこの記述ではordersテーブルのmusic_idを取り出せておりません。
musicsControllerへの記述にテーブルにアクセスして任意のmusic_idを取得する記述を
記入しなければならないと仮定したのですが一向にordersテーブルのmusic_idが取り出せません。
どのように記述することでordersテーブルのmusic_idを取り出すことができますでしょうか？

,,,ruby <music/index>
<div class="info_show4">
<% if user_signed_in? && current_user.id != music.user_id %>
<% if user_signed_in? && music.id == @order.music_id %>
<%= "SOLD OUT" %>
<% else %>
<%= link_to "¥#{music.price}", music_orders_path(music.id) %>
<% end %>
<% else %>
<%= "¥#{music.price}" %>
<% end %>
</div>
```

[winterboum]

まずmusicの一覧があって、
それに対する login user がした Order を あれば 取り出すということですか？

[sia_yuki_f717]

そうです！
ログインユーザーがorderしたmusic_idを取り出したいです！

[winterboum]

MusicとOrderのmodelのcode載せて

[sia_yuki_f717]

ありがとうございます！フルで載せます
よろしくお願いします。

<code>
class Music < ApplicationRecord
belongs_to :user
has_many :tracks, dependent: :destroy
has_many :comments, dependent: :destroy
has_many :favorites
has_many :order
has_one_attached :image

with_options presence: true do
validates :title, length: { maximum: 22 }
validates :image
validates :artist_name, length: { maximum: 22 }
validates :price, format: { with: /\A[0-9]+\z/ },
numericality: { only_integer: true, greater_than_or_equal_to: 500, less_than_or_equal_to: 50_000 }
end

validates :cd_type_id, numericality: { other_than: 1 }

extend ActiveHash::Associations::ActiveRecordExtensions
belongs_to :cd_type

validate :image_content_type, if: :was_attached?

def image_content_type
extension = ['image/png', 'image/jpg', 'image/jpeg']
errors.add(:image, 'の拡張子が間違っています') unless image.content_type.in?(extension)
end

def was_attached?
image.attached?
end

def favorited_by?(user)
favorites.where(user_id: user.id).exists?
end
end
</code>

```
class Order < ApplicationRecord
has_one_attached :image
attr_accessor :token
belongs_to :music
belongs_to :user

with_options presence: true do
validates :token
validates :music_id
validates :user_id
end
end
```

[winterboum]

読みにくい
codeは質問欄偏執して<code>で書いて

[sia_yuki_f717]

お疲れ様です。
記入が終わりました！
後で消しますがGITも載せます。
https://github.com/m81-ito-keisuke/musia_app
よろしくお願いします。

[sia_yuki_f717]

お疲れ様です！
購入したもののみSOLD OUTを表示することができました！
長きにわたり親身に回答くださいましてありがとうございました！