在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) 失败时我是否应该处理测试中的案例。当这个 gorm 调用失败时，会出现这种情况吗？我很感激任何帮助！谢谢！

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与其他域有关系，则失败；在这种情况下，您需要验证关系是否存在并删除该关系或告诉您的用户关系阻止删除。但这取决于您的域设计。