Привет, ребята, мне было интересно, может ли кто-нибудь помочь мне, мне нужно очистить этот контроллер, потому что полученный код просто обновляет количество элементов, если он уже существует, кажется слишком сложным.Clean Up Controller, Update Item Количество в корзине
class LineItemsController < ApplicationController
def create
@product = Product.find(params[:product_id])
if LineItem.exists?(:cart_id => current_cart.id)
item = LineItem.find(:first, :conditions => [ "cart_id = #{@current_cart.id}"])
LineItem.update(item.id, :quantity => item.quantity + 1)
else
@line_item = LineItem.create!(:cart => current_cart, :product => @product, :quantity => 1, :unit_price => @product.price)
flash[:notice] = "Added #{@product.name} to cart."
end
redirect_to root_url
end
end
`
Как всегда любая помощь очень ценится, код должен быть достаточно понятны, спасибо :)
PS разместил его здесь так же как это выглядит немного смешно здесь http://pastie.org/994059
Возможно, вы должны быть немного более конкретными, лично я не буду увлекаться вопросами «очистить свой код». –
@ Джозеф Сильваши, я не согласен. Это не «очищает мой код» так сильно, как «как я могу использовать рельсы лучше, чтобы мой код выглядел более чистым». Часть изучения лучшего программирования - это то, что кто-то показывает вам лучший способ сделать что-то. Карл достаточно проницателен, чтобы понять, что то, что он делает, может быть сделано как-то лучше. – DJTripleThreat