Ruby商品の合計金額を表示させるプログラム

Question

Rubyの商品注文カリキュラムで詰まってしまい、質問させていただけないでしょうか。

やりたいこと：
出力下記の結果を出力させるための注文プログラムを作成したい

#実行結果
山田は2個の商品を購入し、全部で2500円支払いました。
結果として残りの所持金は7500円です。

わからない部分：
cartメソッドの記述
試したこと：
出力時にcartメソッド内にcountとtotal_priceを定義して購入個数と合計金額を表示させる処理を記述する必要があるということは分かるのですが、処理を行うための考え方がわからず詰まっています・・・

class Product
  attr_accessor :name, :price

  def initialize(name, price)
    @name = name
    @price = price
  end
end

class Book < Product
  attr_accessor :author, :publisher, :page_count
end

class CD < Product
  attr_accessor :artist, :year, :songs
end

class ShoppingCart
  attr_accessor :count, :total_price
  #商品情報と個数
  def initialize
    self.count = 0
    self.total_price = 0
  end
  def add_product(product)
    self.count += 1
    self.total_price += product.price
  end  
end

class User
  attr_accessor :last_name, :first_name, :gender, :age
   
  def initialize(last_name, first_name, gender, age )
    self.last_name = last_name
    self.first_name = first_name
    self.gender = gender
    self.age = age
  end
   
  def print_greetings
     puts "今日は。#{self.last_name} #{self.first_name} です。性別は#{self.gender}です。 歳は#{self.age}です。"
  end  
   
  def male?
    return self.gender == "男"

  end
end


class ShoppingUser < User
  # 実装してください
  attr_accessor :money
  #所持金の追加
  def initialize(money)
    @money = money
  end
  
  #カートに入れる
  def into_cart(product)
    #ショッピングカートクラスを変数cartに格納
    cart = ShoppingCart.new
    #cart内の商品ごとの追加情報を表示
    cart.add_product(product)
  end
  
  
  def cart
    #購入する個数(count)
   count = 1
    #合計金額を表示(total_price)
  end  
  
  
  #支払い金額と残り所持金の表示
  def checkout
    cart = ShoppingCart.new
    #支払い金額
    @money -= cart.total_price
    #残り所持金
  end  
end

book1 = Book.new('吾輩は猫である', 1000)
book1.author = '夏目漱石'
book1.publisher = 'Vitalize出版'
book1.page_count = 400


cd1 = CD.new('SMAP', 1500)
cd1.artist = 'スマップ'
cd1.year = 2018
cd1.songs = ["青い稲妻", "世界に一つだけの花", "オレンジ"]

user1 = ShoppingUser.new(10000)
user1.last_name = '山田'
user1.first_name = '花子'
user1.gender = '女'
user1.age = 30

user1.into_cart(book1)
user1.into_cart(cd1)

if user1.checkout
  puts "#{user1.last_name}は#{user1.cart.count}個の商品を購入し、全部で#{user1.cart.total_price}円支払いました。"
  puts "結果として残りの所持金は#{user1.money}円です。"
else
  puts "#{user1.last_name}はチェックアウトできませんでした。"
  puts "なぜなら買い物の合計金額が#{user1.cart.total_price}円なのに対し、所持金は#{user1.money}円しかなかったからです。"
end

申し訳ありませんが、助言をいただけないでしょうか。

Answer 1

修正しました。
修正個所はdiffの結果を参照してください。
不明点があれば、補足してください。
def cartはコメントアウトしています。

class Product
  attr_accessor :name, :price

  def initialize(name, price)
    @name = name
    @price = price
  end
end

class Book < Product
  attr_accessor :author, :publisher, :page_count
end

class CD < Product
  attr_accessor :artist, :year, :songs
end

class ShoppingCart
  attr_accessor :count, :total_price,:count
  #商品情報と個数
  def initialize
    self.count = 0
    self.total_price = 0
  end
  def add_product(product)
    self.count += 1
    self.total_price += product.price
  end
end

class User
  attr_accessor :last_name, :first_name, :gender, :age

  def initialize(last_name, first_name, gender, age )
    self.last_name = last_name
    self.first_name = first_name
    self.gender = gender
    self.age = age
  end

  def print_greetings
     puts "今日は。#{self.last_name} #{self.first_name} です。性別は#{self.gender}です。 歳は#{self.age}です。"
  end

  def male?
    return self.gender == "男"

  end
end


class ShoppingUser < User
  # 実装してください
  attr_accessor :money,:cart
  #所持金の追加
  def initialize(money)
    @money = money
    #ショッピングカートクラスを変数cartに格納
    @cart = ShoppingCart.new
  end

  #カートに入れる
  def into_cart(product)
    #cart内の商品ごとの追加情報を表示
    @cart.add_product(product)
  end

=begin
  def cart
    #購入する個数(count)
   count = 1
    #合計金額を表示(total_price)
  end
=end

  #支払い金額と残り所持金の表示
  def checkout
    #支払い金額
    @money -= cart.total_price
    #残り所持金
    if @money >= 0
      return true
    else
      return false
    end
  end
end

book1 = Book.new('吾輩は猫である', 1000)
book1.author = '夏目漱石'
book1.publisher = 'Vitalize出版'
book1.page_count = 400


cd1 = CD.new('SMAP', 1500)
cd1.artist = 'スマップ'
cd1.year = 2018
cd1.songs = ["青い稲妻", "世界に一つだけの花", "オレンジ"]

user1 = ShoppingUser.new(10000)
user1.last_name = '山田'
user1.first_name = '花子'
user1.gender = '女'
user1.age = 30

user1.into_cart(book1)
user1.into_cart(cd1)

if user1.checkout
  puts "#{user1.last_name}は#{user1.cart.count}個の商品を購入し、全部で#{user1.cart.total_price}円支払いました。"
  puts "結果として残りの所持金は#{user1.money}円です。"
else
  puts "#{user1.last_name}はチェックアウトできませんでした。"
  puts "なぜなら買い物の合計金額が#{user1.cart.total_price}円なのに対し、所持金は#{user1.money}円しかなかったからです。"
end

変更箇所

19c19
<   attr_accessor :count, :total_price
---
>   attr_accessor :count, :total_price,:count
54c54
<   attr_accessor :money
---
>   attr_accessor :money,:cart
57a58,59
>     #ショッピングカートクラスを変数cartに格納
>     @cart = ShoppingCart.new
62,63d63
<     #ショッピングカートクラスを変数cartに格納
<     cart = ShoppingCart.new
65c65
<     cart.add_product(product)
---
>     @cart.add_product(product)
68c68
<
---
> =begin
74c74
<
---
> =end
78d77
<     cart = ShoppingCart.new
81a81,85
>     if @money >= 0
>       return true
>     else
>       return false
>     end

Answer 2

わからない部分：
cartメソッドの記述

色々な考え方があると思いますが、cartはカートを表している訳で、ShoppingUser、すなわち買い物客が実施するメソッドとしては若干不適切です。買い物をしている客自身が、「カート！」と叫んで何かするようなもので、違和感があります。実際に買い物をする客は、買い物を始めるときに買い物カートをまず用意するでしょうから、ShoppingUserクラスがインスタンス化したときにcartを持つようにするのがより適切に思います。

class ShoppingUser < User
  # 実装してください
  attr_accessor :money
  attr_accessor :cart
  #所持金の追加
  def initialize(money)
    @money = money

    # 最初に買い物カートを用意
    @cart = ShoppingCart.new
  end
...

カートに商品を入れる処理、into_cartもちょっとおかしいですね。これではカートに入れるたびに商品が消えてしまいます。

  #カートに入れる
  def into_cart(product)
    #ショッピングカートクラスを変数cartに格納
    cart = ShoppingCart.new
    #cart内の商品ごとの追加情報を表示
    cart.add_product(product)
  end

それらを考慮して修正してみたのが以下です。元のコードと見比べてみてください。

class Product
  attr_accessor :name, :price

  def initialize(name, price)
    @name = name
    @price = price
  end
end

class Book < Product
  attr_accessor :author, :publisher, :page_count
end

class CD < Product
  attr_accessor :artist, :year, :songs
end

class ShoppingCart
  attr_accessor :count, :total_price
  #商品情報と個数
  def initialize
    self.count = 0
    self.total_price = 0
  end

  def add_product(product)
    self.count += 1
    self.total_price += product.price
  end
end

class User
  attr_accessor :last_name, :first_name, :gender, :age

  def initialize(last_name, first_name, gender, age )
    self.last_name = last_name
    self.first_name = first_name
    self.gender = gender
    self.age = age
  end

  def print_greetings
     puts "今日は。#{self.last_name} #{self.first_name} です。性別は#{self.gender}です。 歳は#{self.age}です。"
  end  

  def male?
    return self.gender == "男"

  end
end


class ShoppingUser < User
  # 実装してください
  attr_accessor :money
  attr_accessor :cart
  #所持金の追加
  def initialize(money)
    @money = money

    # 最初に買い物カートを用意
    @cart = ShoppingCart.new
  end

  #カートに入れる
  def into_cart(product)
    #ショッピングカートクラスを変数cartに格納
    # cart = ShoppingCart.new
    #cart内の商品ごとの追加情報を表示
    @cart.add_product(product)
  end

# 不要です。ShoppingUserクラスはcartインスタンスフィールドを持っているので、
# ややこしいことになっています。
# def cart
#   #購入する個数(count)
#  count = 1
#   #合計金額を表示(total_price)
# end

  #支払い金額と残り所持金の表示
  def checkout
    # 支払うときに買い物カートをまた用意しちゃダメです。
    # cart = ShoppingCart.new
    #支払い金額
    @money -= @cart.total_price
    #残り所持金
  end  
end

book1 = Book.new('吾輩は猫である', 1000)
book1.author = '夏目漱石'
book1.publisher = 'Vitalize出版'
book1.page_count = 400


cd1 = CD.new('SMAP', 1500)
cd1.artist = 'スマップ'
cd1.year = 2018
cd1.songs = ["青い稲妻", "世界に一つだけの花", "オレンジ"]

user1 = ShoppingUser.new(1000)
user1.last_name = '山田'
user1.first_name = '花子'
user1.gender = '女'
user1.age = 30

user1.into_cart(book1)
user1.into_cart(cd1)

if user1.checkout
  puts "#{user1.last_name}は#{user1.cart.count}個の商品を購入し、全部で#{user1.cart.total_price}円支払いました。"
  puts "結果として残りの所持金は#{user1.money}円です。"
else
  puts "#{user1.last_name}はチェックアウトできませんでした。"
  puts "なぜなら買い物の合計金額が#{user1.cart.total_price}円なのに対し、所持金は#{user1.money}円しかなかったからです。"
end

実行してみます。意図通り動いているようです。

$ ruby -v
ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux-gnu]

$ ruby t2.rb
山田は2個の商品を購入し、全部で2500円支払いました。
結果として残りの所持金は7500円です。
$

ちなみに所持金を少なくするとダメですね。1000円に修正して実行してみます。

$ ruby t2.rb
山田は2個の商品を購入し、全部で2500円支払いました。
結果として残りの所持金は-1500円です。
$

ShoppingUser#checkoutメソッドの実装が不充分なように思いますが、色んな解決策があるはずですので、質問者さんご自身で解決してみてください。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
すでに自身で解決するために解決策を探して2日間悩んでおり、自己解決が難しいと判断してこちらで質問しました。
ご回答いただくことが難しいようでしたら、参考になる考え方でも構いませんのでご教授いただけないでしょうか。
すでに自分でググって見つかる情報から考えてここまで辿り着きました。

[dodox86]

> すでに自身で解決するために解決策を探して2日間悩んでします。

どの部分を言っていますか？　そもそも、探すというより、本来は自分で考え、コードにする部分なのですが。

[dodox86]

私の回答の部分、
> ShoppingUser#checkoutメソッドの実装が不充分なように思いますが、色んな解決策があるはずですので、質問者さんご自身で解決してみてください。

の解決策が分からないということでしょうか。

[hentaiman]

過去質問から考えて就職先の課題ですよね？現在の進捗を職場の人に相談した方が良くないですか？理路整然と質問するのも仕事する上では重要ですよ

[dodox86]

> ShoppingUser#checkoutメソッドの実装が不充分なように思いますが、色んな解決策があるはずですので、質問者さんご自身で解決してみてください。

についてだったら、そんなに難しい話をしているつもりはなくて、もともとある以下のコードに適合するようにShoppingUser#checkoutメソッドの返り値をセットすれば良いだけなのですが。
if user1.checkout
# 買える場合
else
# 買えない場合
end

買えるならtrue, 買えないならfalse、ということで成立しますね。もし本当に課題なのであれば、出題者に相談しましょう。cartメソッドは私の回答で削除しましたが、あらかじめある程度コードが用意されていて、cartメソッドは無理やりにでも使え、という課題であったのかもしれませんし。

[退会済みユーザー]

ご回答ありがとうございます！
見返してみるとShopping_user.newを無駄に二回記述してました・・・
initializeで定義しておいくべきでした。
また、cartも無駄にメソッドを作成していました。attr_accessorで最初から使えるようにしておけばよかったんですね・・