ajouter une condition de condition de suppression des rails et passer un message d'alerte à afficher

Question

Je souhaite afficher un message d'alerte lorsque la condition donnée est en échec. Mon code est ....

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

La réponse de OdrOrderLine.where(prd_item_id: @items.id).pluck(:id) fera toujours la condition @item_odr!=null return true. Tout d'abord, il est important de noter que null n'est pas un mot-clé Ruby, vous souhaitez probablement utiliser nil à la place. Cela ne génère pas d'erreur car il n'y a pas de variable locale null définie et donc null renvoie simplement nil. Mais cette ligne pourrait dérouter les autres développeurs. Deuxièmement, s'il n'y a pas d'enregistrement correspondant dans la base de données, OdrOrderLine.where(prd_item_id: @items.id).pluck(:id) retournera un tableau vide. Par conséquent, vous ne serez jamais capable d'entrer dans la branche else.

Peut-être quelque chose comme ceci est plus lisible:

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

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

    else 
    @item.destroy 
    # ... 

Btw Je rebaptisés la variable @items à @item parce que find ne renvoie qu'un seul enregistrement.

Answer 2

Si vous avez cela au début d'un formulaire, cela peut générer des messages d'erreur pour vous et vous permettre de garder votre contrôleur skinnier.

<% 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 %>