rails delete関数にif条件を追加し、表示する警告メッセージを受け取ります

Question

指定された条件が満たされていないときに警告メッセージを表示します。私のコードは

def delete 
    @items = PrdItem.find(params[:id]) 
    @item_odr=OdrOrderLine.where(prd_item_id: @items.id).pluck(:id) 

    if @item_odr!=null 
     @items.destroy 
     @vendor_id = UsrContactVendor.find_by_usr_contact_id(current_usr_contact.id).usr_vendor_property_id 
     redirect_to usr_vendor_property_path(@vendor_id) 
    else 
     flash[:notice] = "Cannot be deleted" 
     @vendor_id = UsrContactVendor.find_by_usr_contact_id(current_usr_contact.id).usr_vendor_property_id 
     redirect_to usr_vendor_property_path(@vendor_id) 

Answer 1

OdrOrderLine.where(prd_item_id: @items.id).pluck(:id)の応答は常に条件@item_odr!=nullリターンは本当でしょう....です。

最初に、nullはRubyキーワードではないことに注意してください。nilを代わりに使用することをお勧めします。ローカル変数nullが定義されていないため、nullが単にnilを返すため、これはエラーを発生させません。しかし、この行は他の開発者と混乱するかもしれません。

第2に、一致するレコードがデータベースにない場合、OdrOrderLine.where(prd_item_id: @items.id).pluck(:id)は空の配列を返します。したがって、あなたはelseブランチに入ることはできません。

おそらく、このようなものは、より読みやすいです：

def delete 
    @item = PrdItem.find(params[:id]) 

    if OdrOrderLine.where(prd_item_id: @item.id).any? 
    flash[:notice] = "Cannot be deleted" 
    # ... 

    else 
    @item.destroy 
    # ... 

はところで、私は1つのレコードのみ@itemfindので、戻りに@items変数の名前を変更しました。

Answer 2

フォームの先頭にこれがある場合は、エラーメッセージが表示され、コントローラのスキンを維持できます。

<% if @item.errors.any? %> 
     <h3><%= pluralize(@item.errors.count, "error") %> prohibited this record from being saved:</h3> 
     <ul> 
     <% @item.errors.full_messages.each do |message| %> 
     <li> 
      <%= message %> 
     </li> 
     <% end %> 
     </ul> 
     <% end %>