I first want to apologize because I am not sure if it will make complete sense but I want to provide you with as much context as possible. I working on a lesson that is trying to teach me iterating over hashes and nested hashes. I have been working on this for a couple days now and have been struggling and now I think I have put in too many lines of code, it might look a bit sloppy.
So first off the method is tested against the following items and tests:
describe "Grocer" do
let(:items) do
[
{"AVOCADO" => {:price => 3.00, :clearance => true}},
{"KALE" => {:price => 3.00, :clearance => false}},
{"BLACK_BEANS" => {:price => 2.50, :clearance => false}},
{"ALMONDS" => {:price => 9.00, :clearance => false}},
{"TEMPEH" => {:price => 3.00, :clearance => true}},
{"CHEESE" => {:price => 6.50, :clearance => false}},
{"BEER" => {:price => 13.00, :clearance => false}},
{"PEANUTBUTTER" => {:price => 3.00, :clearance => true}},
{"BEETS" => {:price => 2.50, :clearance => false}},
{"SOY MILK" => {:price => 4.50, :clearance => true}}
]
end
let(:coupons) do
[
{:item => "AVOCADO", :num => 2, :cost => 5.00},
{:item => "BEER", :num => 2, :cost => 20.00},
{:item => "CHEESE", :num => 3, :cost => 15.00}
]
end
So these are the items it they go through. Along with the coupons and their values. Below is the tests apply_coupons will go through:
describe "#apply_coupons" do
context "base case - with perfect coupon (number of items identical):" do
before(:each) do
@avocado = find_item('AVOCADO')
@avocado_coupon = coupons.find { |coupon| coupon[:item] == "AVOCADO" }
@cart = [@avocado, @avocado]
@consolidated_cart = consolidate_cart(@cart)
@avocado_result = apply_coupons(@consolidated_cart, [@avocado_coupon])
end
it "adds a new key, value pair to the cart hash called 'ITEM NAME W/COUPON'" do
expect(@avocado_result.keys).to include("AVOCADO W/COUPON")
end
it "adds the coupon price to the property hash of couponed item" do
expect(@avocado_result["AVOCADO W/COUPON"][:price]).to eq(2.50)
end
it "adds the count number to the property hash of couponed item" do
expect(@avocado_result["AVOCADO W/COUPON"][:count]).to eq(2)
end
it "removes the number of discounted items from the original item's count" do
expect(@avocado_result["AVOCADO"][:price]).to eq(3.00)
expect(@avocado_result["AVOCADO"][:count]).to eq(0)
end
it "remembers if the item was on clearance" do
expect(@avocado_result["AVOCADO W/COUPON"][:clearance]).to eq(true)
end
end
context "more advanced cases:" do
it "accounts for when there are more items than the coupon allows" do
cheese = find_item('CHEESE')
cart = Array.new(5, cheese)
consolidated_cart = consolidate_cart(cart)
cheese_coupon = find_coupon("CHEESE")
cheese_result = apply_coupons(consolidated_cart, [cheese_coupon])
expect(cheese_result["CHEESE"][:price]).to eq(6.50)
expect(cheese_result["CHEESE"][:count]).to eq(2)
expect(cheese_result["CHEESE W/COUPON"][:price]).to eq(5.00)
expect(cheese_result["CHEESE W/COUPON"][:count]).to eq(3)
expect(cheese_result["CHEESE W/COUPON"][:clearance]).to eq(false)
end
it "doesn't break if the coupon doesn't apply to any items" do
cheese = find_item('CHEESE')
cart = Array.new(2, cheese)
consolidated_cart = consolidate_cart(cart)
irrelevant = apply_coupons(consolidated_cart, [find_coupon("AVOCADO")])
expect(irrelevant["CHEESE"][:price]).to eq(6.50)
expect(irrelevant["CHEESE"][:count]).to eq(2)
expect(irrelevant.keys).to_not include("AVOCADO W/COUPON")
expect(irrelevant.keys).to_not include("AVOCADO")
end
it "can apply multiple coupons" do
avocado = find_item('AVOCADO')
cheese = find_item('CHEESE')
cart = Array.new(4, cheese)
3.times { cart << avocado }
consolidated_cart = consolidate_cart(cart)
coupons = [find_coupon("AVOCADO"), find_coupon("CHEESE")]
multiple_coupons = apply_coupons(consolidated_cart, coupons)
["AVOCADO", "CHEESE"].each { |item| expect(multiple_coupons[item][:count]).to eq(1) }
expect(multiple_coupons["CHEESE"][:price]).to eq(6.50)
expect(multiple_coupons["AVOCADO"][:price]).to eq(3.00)
expect(multiple_coupons["CHEESE W/COUPON"][:price]).to eq(5.00)
expect(multiple_coupons["CHEESE W/COUPON"][:count]).to eq(3)
expect(multiple_coupons["CHEESE W/COUPON"][:clearance]).to eq(false)
expect(multiple_coupons["AVOCADO W/COUPON"][:price]).to eq(2.50)
expect(multiple_coupons["AVOCADO W/COUPON"][:count]).to eq(2)
expect(multiple_coupons["AVOCADO W/COUPON"][:clearance]).to eq(true)
end
it "doesn't break if there is no coupon" do
cheese = items.find { |item| item['CHEESE'] }
cart = [cheese, cheese]
consolidated_cart = consolidate_cart(cart)
no_coupon_result = apply_coupons(consolidated_cart, [])
expect(no_coupon_result["CHEESE"][:price]).to eq(6.50)
expect(no_coupon_result["CHEESE"][:count]).to eq(2)
end
it "can increment coupon count if two are applied" do
avocado = find_item("AVOCADO")
coupon = find_coupon("AVOCADO")
consol_cart = consolidate_cart([avocado, avocado, avocado, avocado, avocado])
two_coupon_result = apply_coupons(consol_cart, [coupon, coupon])
expect(two_coupon_result["AVOCADO"][:count]).to eq(1)
expect(two_coupon_result["AVOCADO W/COUPON"][:price]).to eq(2.50)
expect(two_coupon_result["AVOCADO"][:price]).to eq(3.00)
expect(two_coupon_result["AVOCADO W/COUPON"][:count]).to eq(4)
end
end
end
Now, my code so far passes up until the advance case of applying multiple coupons but also
can apply multiple coupons (FAILED - 1)
doesn't break if there is no coupon (FAILED - 2)
can increment coupon count if two are applied (FAILED - 3):
def apply_coupons(cart, coupons)
#cart is a hash with 2 keys with a nested hash with 3 values
#coupons is an array with a hash with 3 keys and 3 values
#if coupons[0].values.include?(cart.keys[0]) will find avocado in coupon
#coupons[0].values[0] is AVOCADO in coupon
#cart.keys[0] is AVOCADO in cart
#coupons[0].values[0] + " W/COUPON" creates "AVOCADO W/COUPON"
#coupons[0].values[1] = 2 the number of coupons
#coupons[0].values[2] = 5.0 which is the number off need to be divded by 2(number of coupons)
applyCoupons = {}
coupons.each do |k|
if coupons.include?(" W/COUPON")
applyCoupons[coupons[0].values[0] + " W/COUPON"]
applyCoupons[coupons[0].values[0] + " W/COUPON"][:price] = coupons[0].values[2] / coupons[0].values[1]
applyCoupons[coupons[0].values[0] + " W/COUPON"][:clearance] = cart.values[0].has_value?(true)
applyCoupons[coupons[0].values[0]][:count] += 1
end
if cart.keys[0] == coupons[0].values[0]
applyCoupons[cart.keys[0]] = cart.values[0]
applyCoupons[coupons[0].values[0] + " W/COUPON"] = {}
applyCoupons[coupons[0].values[0] + " W/COUPON"][:price] = coupons[0].values[2] / coupons[0].values[1]
applyCoupons[coupons[0].values[0] + " W/COUPON"][:clearance] = cart.values[0].has_value?(true)
applyCoupons[coupons[0].values[0] + " W/COUPON"][:count] = coupons[0].values[1]
applyCoupons[coupons[0].values[0]][:count] -= coupons[0].values[1]
else
applyCoupons[cart.keys[0]] = cart.values[0]
end
end
applyCoupons
end
The error I get: Failures:
1) Grocer #apply_coupons more advanced cases: can apply multiple coupons Failure/Error: ["AVOCADO", "CHEESE"].each { |item| expect(multiple_coupons[item][:count]).to eq(1) }
NoMethodError:
undefined method `[]' for nil:NilClass
# ./spec/grocer_spec.rb:121:in `block (5 levels) in <top (required)>'
# ./spec/grocer_spec.rb:121:in `each'
# ./spec/grocer_spec.rb:121:in `block (4 levels) in <top (required)>'
This is a big ask but I am learning so any help is appreciated. Would love to learn methods that will make my code simpler too.
def apply_coupons(cart, coupons)
coupons.each do |coupon|
if cart.keys.include?(coupon[:item])
if cart[coupon[:item]][:count] >= coupon[:num]
itemwithCoupon = "#{coupon[:item]} W/COUPON"
if cart[itemwithCoupon]
cart[itemwithCoupon][:count] += coupon[:num]
cart[coupon[:item]][:count] -= coupon[:num]
else
cart[itemwithCoupon] = {}
cart[itemwithCoupon][:price] = (coupon[:cost] / coupon[:num])
cart[itemwithCoupon][:clearance] = cart[coupon[:item]][:clearance]
cart[itemwithCoupon][:count] = coupon[:num]
cart[coupon[:item]][:count] -= coupon[:num]
end
end
end
end
cart
end