やろうとしたことを満たすコードを書いたのですが、BaseNutritionクラスのメソッドnutrition_calculatorがなんとなく、汚い(冗長なコードのような気がする)のですが、もっときれいに書く方法はありますか?初心者で正直、関数、クラスの使い訳もうまくできないと思います。正直このようなコードをクラスで書くべきかどうかもわかりません。何がいいコードで何が悪いのかよくわからないので、見ていただきたいと思いました。改善できるところがありましたら教えていただけると嬉しいです。(あいまいな質問ですいません。)
やろうとしたこと。
まず、BaseNutritionクラスで食べ物の総量、総量グラムに対するタンパク質、炭水化物、脂質の属性を作った。
そして、BaseNutritionオブジェクトに栄養価の辞書データを与えることで、その食べ物を何グラムかを、nutrition_calculatorメソッドに渡すことによって、渡した重さに対する栄養価がわかるようなコードを書きました。
class BaseNutrition():
def __init__(self, amount, kcal, carbohydrates, protein, fat):
self.amount = amount
self.kcal_per_gram = kcal / amount
self.carbohydrates_per_gram = carbohydrates / amount
self.protein_per_gram = protein / amount
self.fat_per_gram = fat / amount
def nutrition_calculator(self, grams):
kcal = round(grams * self.kcal_per_gram,1)
carbs = round(grams * self.carbohydrates_per_gram,1)
protein = round(grams * self.protein_per_gram, 1)
fat = round(grams * self.fat_per_gram, 1)
result_calculation = []
result_calculation.append(kcal)
result_calculation.append(carbs)
result_calculation.append(protein)
result_calculation.append(fat)
return result_calculation
chiken_nutrition = {"kcal":116,
"carbohydrates": 0,
"protein": 19,
"fat": 4 ,
"amount": 100}
chiken = BaseNutrition(**chiken_nutrition)
print(chiken.nutrition_calculator(200)) # [232.0, 0.0, 38.0, 8.0]