コントローラ変数へのRails変数の受け渡し

Question

私はしばらくの間作業してきたプログラムを持っています。これは基本的なバーコードスキャンシステムです。 、私は起こるしたいもの、

今

class RawsController < ApplicationController 
    def new 
     @raw = Raw.new 
    end 

     def create 
     @raw = Raw.new(params[raw_params]) 
     if @raw.save 
      receive(@raw) 
      redirect_to new_raw_path 
     end  
    end 

    def receive(raw) 
     scanned = Raw.find_by(code: raw.code) 
     if scanned.quantity.nil? 
      flash[:danger] = "no match" 
     else 
      scanned.quantity += 1 
      if scanned.save 
       respond_to do |format| 
        format.html {} 
        format.json{} 
       end 
      end 
     end 
    end 
private 
    def raw_params 
     params.require(:raw).permit(:code, :product, :quantity) 
    end 

end 

誰かがフォームにバーコード番号を入力したとき、それはのインスタンスとして@raw作成することです：生のこだわり

私のコントローラは以下の通りです生の（生のための）クラス。その後、受信機能では、在庫内の一致する商品を検索してから、1つのユニットを商品在庫に追加します。

しかし、いったんscanned.quantity += 1行になると、このメソッドはnilクラスでは定義されていないというエラーが発生し続けます。

私はscanned.quantity.nil?ラインに追加しました。そのフラッシュメッセージが表示されるので、問題は数量がゼロであることと関係しています。しかし、私はどのようにこれを修正するか分からない。

助けが必要ですか？

Answer 1

問題は、あなたがthis commentにポストされたデータを見るとRaw.new(...)

であなたの初期化であり、私はアクションが正しい属性を持つRawオブジェクトを作成していないことを言うことができます。保存する場合は、許可された属性を使用する必要があります。あなたはnewをparams[raw_params]と呼んでおり、nilとなります。

def create 
    @raw = Raw.new(raw_params) 
    if @raw.save 
     receive(@raw) 
     redirect_to new_raw_path 
    end 
    end 

Answer 2

scanned.quantity.nil?のif節が成功したため、quantityはnilですが、scannedオブジェクトが存在することを意味します。私はあなたのデフォルト値が0ではないと仮定します。

理想的には、その列をデータベースに追加した移行内のquantity列にデフォルト値の0を追加します。

ただし、インクリメントを実行する前に0に設定することができます。また、すでにRawオブジェクトを渡しているので、実際にはfind_byを実行する必要はありません。アクションは、そのオブジェクト自体に対して実行できます。

def receive(raw) 
    scanned = raw 

    if scanned.nil? 
     # your flash message 
    else 
     scanned.quantity ||= 0 
     scanned.quantity += 1 
     if scanned.save 
     respond_to do |format| 
      format.html {} 
      format.json{} 
     end 
     end 
    end 
end 

Answer 3

あなたがそのエラーを取得している理由ですnil値に+=をしようとしています。

あなたが値@raw.quantity @raw新しいインスタンスを作成している量のデフォルト値を変更するには、マイグレーションを書く0

class Raw 
    after_initialize :set_quantity 

    def set_quantity 
    self.quantity = 0 
    end 
end 

それとも

に数量を設定するために使用after_initializenil

です〜0

def change 
    change_column :raw, :quantity, :integer, default: 0 
end