継承関係にない他のクラス内の値(変数データ)を使う方法はありますか？

Question

[a.rb]
class A  #様々なメソッドが格納されているクラス
  attr_accessor :calorie_of_main_dish, :calorie_of_side_dish
  
  def initialize
    @calorie_of_main_dish = 0
    @calorie_of_side_dish = 0
  end
  
  def suggest_main_dish
    puts "料理の提案をします"
    main_dish_and_calorie_key = list_of_main_dish_and_calorie.keys.sample
    @calorie_of_main_dish = list_of_main_dish_and_calorie[main_dish_and_calorie_key]
    puts "・メイン料理: #{main_dish_and_calorie_key}(#{calorie_of_main_dish} kcal)"
  end
  def suggest_side_dish
    side_dish_and_calorie_key = list_of_side_dish_and_calorie.keys.sample
    @calorie_of_side_dish = list_of_side_dish_and_calorie[side_dish_and_calorie_key]
    puts "・副菜: #{side_dish_and_calorie_key}(#{calorie_of_side_dish} kcal)"
  end
  def output_calorie_of_suggested_dishes
    total_calorie_of_suggested_dishes = calorie_of_main_dish + calorie_of_side_dish
    puts "これらの料理の総カロリーは#{total_calorie_of_suggested_dishes} kcalです"
  end
end

[b.rb]
class B  #class A で使う、様々な料理やカロリーなどのデータが格納されているクラス
  attr_accessor :list_of_main_dish_and_calorie, :list_of_side_dish_and_calorie

  def initialize
    @list_of_main_dish_and_calorie = nil
    @list_of_side_dish_and_calorie = nil
  end
  
  def calorie_labeling
    @list_of_main_dish_and_calorie= {
      "ハンバーグ" => 437,
      "ラーメン" => 443,
        :
    }
    
    @list_of_side_dish_and_calorie = {
      "シーザーサラダ" => 287,
      "キャベツサラダ" => 194,
        :
    }
  end
end

これらのクラスは継承関係でもないと思うので
class B < A
とするのはおかしいと思います。

そこで下記のようにすべてのクラス名を"A"で統一してしまい、元のclass B 内の変数データをclass A に追加(上書き)するという形をとっても良いのでしょうか。

[a.rb]
require "./b"
class A

[b.rb]
class A  #元のclass B

それともいくらクラス内で使う変数データを格納してあるとしても、同じクラス名を使うと混乱する可能性があるため、やめたほうがいいのでしょうか。

やめたほうがいい場合、他に良い方法があれば教えていただきたいです。

Comments

[maisumakun]

class 1の「メソッド」は、通常のインスタンスメソッドでしょうか？それとも、クラスメソッドでしょうか？

[dentiste]

通常のインスタンスメソッドです。

Answer 1

csvファイルに料理のリスト(class B の内容)をまとめ、class A で読み込む設計にしました。

[dishes_list.csv]

dishes,calorie,genre
ハンバーグ,437,main-dish
ラーメン,443,main-dish
シーザーサラダ,287,side-dish
キャベツサラダ,194,side-dish

Answer 2

csvファイルに料理のリスト(class B の内容)をまとめ、class A で読み込む設計にしました。

[dishes_list.csv]
dishes,calorie,genre
ハンバーグ,437,main-dish
ラーメン,443,main-dish
シーザーサラダ,287,side-dish
キャベツサラダ,194,side-dish

Answer 3

rails ではなく 純rubyですね？
ですと、他のclass内の値(変数データ)（多分classの変数ではなくインスタンスの変数ですね？）を用いるには、
そのインスタンスを特定して　例えば recipeに代入し、recipeに定義したmethodを参照する（recipe.calorie) というのが正道かと。

Answer 4

# 1+2 プロフィール+基礎代謝計算
class User
  # 外部から見られるデータを公開する
  attr_reader :weight, :height, :age, :sex
  # 基礎代謝を計算し返すメソッド
  def bee
  end
  def initialize(sex, age, height, weight)
    @sex = sex
    @age = age
    @height = height
    @weight = @weight
  end
end

# 3 レシピ(特にメソッド生やす必要性を感じなかった為Structを採用)
Recipe = Struct.new(:name, :energy)

レシピオススメ部分の実装はいろいろあるが
オススメしてくれるレシピ本クラスを作ったりするのがわかりやすそう

class RecipeBook
  # レシピを追加
  def add(recipe)
    @book << recipe
  end
  # オススメ
  def recommend(user)
    @book.select{|recipe| recipe.energey <= user.bee}
  end
end

Answer 5

同じクラス名を使うと混乱する可能性があるため、

同じクラス名というより、同じクラスです。

これらのクラスは継承関係でもないと思うので

class 2 < 1
class 3 < 1
とするのはおかしいと思います。

と言うことであれば、同じクラスにするのもおかしいでしょう。

「変数データが格納されている」という意味が何を言わんとしているのかわかりませんが、使用目的によって適切なやり方があるかと思います。

Comments

[dentiste]

少し詳細を追記させていただきました。
上記のようなクラスの場合、やはり継承関係もなく、同じクラスにするのもおかしいのでしょうか。

[otn]

具体性が無いため、何を言わんとしているのか、さっぱりです。

[dentiste]

申し訳ありません。質問を変えます。
class 3 内で定義されているハッシュ(下記のような料理名とカロリーが格納されているもの)
@dishes = {
"ハンバーグ" => 437,
"ラーメン" => 443,
:
}
これをclass 1 内で下記のように使いたい
a = dishes.keys.sample #料理名をランダムに取得
b = dishes[a] #選ばれた料理のカロリー取得
puts "#{a} #{b} kcal" #料理名とカロリー出力

とすると、下記のうちどのような方法が良いのでしょうか
1. 継承する (class 3 < 1)
2. 同じクラスにする (class 3 から class 1 にする)
3. その他

[otn]

class1は何のクラスなんでしょうか？（class1オブジェクトは何を表しているのか？）
class3は何のクラスなんでしょうか？（class3オブジェクトは何を表しているのか？）

[dentiste]

class1 は料理を提案するクラス
class3はデータ(料理名とカロリー)管理用のクラス
としています。

[otn]

class1オブジェクトは「料理の提案」を表すと言うことですか？
class3オブジェクトは何を表しているのでしょうか？
「料理」を表すクラスは無い？

[dentiste]

詳細なソースコードを追記させていただきました。
class1オブジェクトは「料理の提案」を表します。
class3オブジェクトは「料理とカロリー」を表します。
「料理」を表すクラスがclass3のつもりでした

[otn]

クラスに対する基本的な理解が間違っている気がします。

・オブジェクト指向でゼロから設計し直す
・非オブジェクト指向で、クラスを使わないで書く

のどちらかがいいと思います。現状ベースなら、後者が近道です。

[dentiste]

理解したいので、再度勉強し直してみることにします。ご回答ありがとうございました。

[otn]

・1つの料理を表すクラス
・料理のリストを表すクラス
を作ると良いと思います。2つは継承関係に無いです。