Сначала я хочу извиниться, потому что я не уверен, что это будет иметь смысл, но я хочу предоставить вам как можно больше контекста.Я работаю над уроком, который пытается научить меня перебирать хэши и вложенные хэши.Я работал над этим в течение нескольких дней и изо всех сил пытался, и теперь я думаю, что я добавил слишком много строк кода, это может выглядеть немного неаккуратно.
Итак, во-первых, метод проверяется на следующих предметах и тестах:
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
Итак, через эти элементы они проходят.Вместе с купонами и их значениями.Ниже приведены тесты, через которые будет проходить apply_coupons:
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
Теперь мой код проходит до тех пор, пока предварительный случай применения нескольких купонов, но также
не может применить несколько купонов (FAILED - 1)t break, если купона нет (FAILED - 2) может увеличить счет купона, если применены два (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
Ошибка, которую я получаю: Сбои:
1)Grocer #apply_coupons более сложные случаи: может применить несколько купонов Сбой / Ошибка: ["AVOCADO", "CHEESE"]. Each {| item |ожидайте (множественные купоны [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)>'
Это большая просьба, но я учусь, поэтому любая помощь приветствуется.Хотелось бы изучить методы, которые также упростят мой код.