grailsで削除機能を実装する正しい方法は？

Question

私は、コントローラのメソッドは次のように見えるサービスメソッドを呼び出します

コントローラメソッド

def deletemap(Long id){ 


    try { 

     mapService.deleteMap(id) 

    } 
    catch (ValidationException e) { 


     flash.message = "Sorry an error occured when deleting map!!" 

     redirect(action: "maps", id: event.id) 

     return 


    } 


    flash.message = "Map was deleted!!" 

    redirect(action: "maps", id: event.id) 

    return 


} 

ようになり、削除する方法があります。

サービスメソッド

def deleteMap(id){ 


    def map = Map.get(id) 



    if(map == null){ 

      throw new org.springframework.security.access.AccessDeniedException("Id doesn't exist!!!") 
      return 
    } 



    map.delete(flush: true) 

} 

を

この機能を可能な限り完璧に実装することが目標です。私は現在、いくつかのテストを書いており、map.delete（flush：true）が失敗したときにテストのケースを処理する必要があるかどうか疑問に思っています。このゴームコールが失敗した場合はありますか？私は助けていただきありがとうございます！ありがとう！

Answer 1

それを行うにはきれいな方法：

コントローラ

def deletemap(Long id){ 

    Map mapInstance = Map.get(id) 

    if (!mapInstance) { 
     flash.message = "Map not found" 
    } 
    else { 
     mapService.deleteMap(mapInstance) 
     flash.message = "Map was deleted!!" 
    } 

    redirect(action: "maps") 
} 

サービス

def deleteMap(Map mapInstance){ 
    map.delete(flush: true) 
} 

Mapは、他のドメインとの関係を持っている場合、それは失敗します。この場合、関係が存在するかどうかを検証し、関係が存在するかどうかを確認するか、関係が削除を妨げることをユーザに伝える必要があります。しかしそれはあなたのドメイン設計に依存します。