Question

I’m working on a fairly simple site that allows users to choose recipe ingredients, their quantities and then shows them nutritional info based on their recipe and a large database.

Right now, I feel like I’m repeating myself a bit. I want to be able to make this “DRY” by having one method each in the Recipe and Recipe_Ingredient model that will do the same thing only accept the right parameter, which will be the type of nutrient.

Here is the relevant code in my view that currently calls two different methods (and will call more when extended to the other nutrients):

<ul>Calories <%= @recipe.total_calories %></ul>
<ul>Fat (grams) <%= @recipe.total_fat %></ul>

In my recipe model, I have methods that iterate over each of the ingredients in the recipe:

def total_calories
   recipe_ingredients.to_a.sum { |i| i.total_calories }
end

def total_fat
  recipe_ingredients.to_a.sum { |i| i.total_fat }
end

In the block, we call two separate methods that actually calculate the nutrients for each individual recipe ingredient:

def total_calories
  ingredient.calories*ingredient.weight1*quantity/100
end

def total_fat
  ingredient.fat*ingredient.weight1*quantity/100
end

This last piece is where we reference the database of ingredients. For context, here are the relationships:

class RecipeIngredient < ActiveRecord::Base
  belongs_to :ingredient
  belongs_to :recipe

class Recipe < ActiveRecord::Base
  has_many :recipe_ingredients

Thanks in advance for any help.

• Lev

Answer 1

The send method with a symbol parameter works well for that kind of DRY.

<ul>Calories <%= @recipe.total :calories %></ul>
<ul>Fat (grams) <%= @recipe.total :fat %></ul>

Recipe

def total(type)
  recipe_ingredients.to_a.sum { |i| i.total type }
end

RecipeIngredient

def total(type)
  ingredient.send(type) * ingredient.weight1 * quantity / 100
end