2016-07-12 61 views
0

我正在使用Sinatra写Shop。我实现了添加到购物篮,但我无法从购物篮删除工作。Ruby,Sinatra DELETE方法

我的类应用:

get "/basket" do #working 
     products_in_basket = FetchBasket.new.call 
     erb :"basket/show", locals: { basket: products_in_basket } 
    end 

    post "/basket" do #working 
     AddToBasket.new(params).call 
     redirect "/" 
    end 


    delete "basket/:id" do # doesn't work 
    DeleteBasket.new(params).call 
    redirect "/" 
end 

我DeleteBasket:

module Shop 
    class DeleteBasket 
    attr_reader :product_id, :id 

    def initialize(params) 
     @id = params.fetch("id").to_i 
     @product_id = params.fetch("product_id").to_i 

    end 
    def call 
     basket = FetchBaskets(id) # finds Basket instance with given id 

     return unless basket 

     reduce_basket_quantity(basket) 



    def reduce_basket_quantity(basket) 
     if basket.quantity >= 1 
     basket.quantity -= 1 
     #warehouse = FetchWarehouseProduct.new.call(product_id) 
     #warehouse.quantity += quantity 
     else 
     BASKET.delete(basket) 
     end 
     end 
    end 
    end 
end 

删除观点:

<td> <form action="/basket/<%=b.id%>" method="post"> 
    <input type="hidden" name="_method" value="delete"> 
    <input type="hidden" name="product_id" value=<%= b.product_id %>> 
    <input type="hidden" name="id" value=<%= b.id %>> 
    <button type="submit">Delete</button> 
    </form> 

,因为它应该它不重定向到主页,它不会改变篮子数量1.它根本不做任何事情。

+0

? –

+0

类应该是名词,方法应该是动词处理所说的名词。有一个名为'DeleteBasket'的类没有意义。另外,什么是不工作?你的'DeleteBasket'类在语法上甚至是不正确的。 –

+0

什么是语法错误? – Jes

回答

0

我觉得最明显的原因是你是不是调用delete HTTP方法,但post代替:

<form action="/basket/<%=b.id%>" method="post"> 

通常你会使用

<form action="/basket/<%=b.id%>" method="delete"> 

解决这个问题,但是这还不根据this answer支持。

我认为最好的方法是定义你的delete路线作为post

post "delete-basket/:id" do 
    DeleteBasket.new(params).call 
    redirect "/" 
    end 

,然后您正在使用什么ORM写

<form action="/delete-basket/<%=b.id%>" method="post">