К сожалению это работает: Сравнение ROR в Activerecord - обновление

Question

К сожалению, этот беспорядок работает: у вас есть предложения по очистке этого кода: я пытаюсь использовать activerecord для сравнения двух столбцов таблицы, «required» и «amount», затем обновляю логический столбец, в зависимости от возвращаемых данных , Полное нарушение кодирования без повторов.

def update
    @inventory = Inventory.find(params[:id])

    respond_to do |format|
      if @inventory.update_attributes(params[:inventory])
            unless @inventory.needed.nil?
                if @inventory.needed < @inventory.amount then
                @inventory.instock = true
                @inventory.update_attributes(params[:inventory])
                else
                @inventory.instock = false
                @inventory.update_attributes(params[:inventory])
                end
            end
        flash[:notice] = 'Inventory was successfully updated.'
        format.html { redirect_to(@inventory) }
        format.xml  { head :ok }
      else
        format.html { render :action => "edit" }
        format.xml  { render :xml => @inventory.errors, :status => :unprocessable_entity }
      end
    end
  end

Ура! * * 1004

Answer 1

Во-первых, не допускайте посторонней логики в свой контроллер:

def update
  @inventory = Inventory.find(params[:id])

  respond_to do |format|
    if @inventory.update_attributes(params[:inventory])
      flash[:notice] = 'Inventory was successfully updated.'
      format.html { redirect_to(@inventory) }
      format.xml  { head :ok }
    else
      format.html { render :action => "edit" }
      format.xml  { render :xml => @inventory.errors, :status => :unprocessable_entity }
    end
  end
end

Затем обработайте атрибут instock с помощью обратного вызова для модели Inventory. Что-то вроде:

before_update :check_instock, :unless => Proc.new { |inventory| inventory.needed.nil? } 

def check_instock
  if needed < amount
    instock = true
  else
    instock = false
  end
end

Answer 2

Очень смущен вашими многочисленными звонками на @inventory.update_attributes. Вы должны вызывать это только один раз, когда записи должны быть сохранены в базе данных. Вы должны использовать @inventory.attributes = params[:inventory] и if @inventory.save, чтобы поразить базу данных.

Возможно, что-то вроде (не проверено):

@inventory.attributes = params[:inventory]
unless @inventory.needed.nil?
  @inventory.instock = (@inventory.needed < @inventory.amount) ? true : false
  @inventory.save
end