Richa

Right now it looks like your code loops through the ingredients looking for checked boxes, and then you run "add_ingredient" which loops through the ingredients again.

I think it would be better if 1) you only looped through the ingredients once, so it's clearer what's going on, and 2) if you make that change and keep add_ingredient, then either add_ingredient should only add ONE ingredient, or you should call it add_ingredients. A name like add_Ingredient that maybe adds multiple ingredients (or none!) is very confusing -- as it is it would better be called something like "find_and_add_new_ingredients".