如何在Spring MVC中处理RESTful删除

时间:2012-04-26 20:01:44

标签: java spring rest spring-mvc

如何在Spring MVC控制器中正确使用RESTful删除?我有DAO在尝试删除某个项目时返回布尔值。

我正在尝试删除项目。如果一切正常,只需显示项目列表(已删除的项目将不再存在)。如果无法删除项目,请重定向到详细信息页面并说明无法删除的原因。

我是否需要一些特殊的响应状态或类似的东西?我的方法是RESTful吗?

@RequestMapping(value = "items/{id}", method = RequestMethod.DELETE)
public String delete(@PathVariable("id") int itemId, Model model) {
    Item item = itemDao.get(id);

    // true -> can delete
    // false -> cannot delete, f.e. is FK reference somewhere
    boolean wasOk = itemDao.delete(item); 

    if (wasOk) {
        return "redirect:/items";
    }

    // will write to user which item couldn't be deleted
    model.addAttribute("item", item);
    return "items/error";
}

2 个答案:

答案 0 :(得分:19)

您应该考虑使用HTTP status codes来指示删除操作是否成功而不是重定向。例如HTTP 200 OK(或HTTP 204 No Content)表示操作成功,HTTP 404 Not Found如果您尝试访问的资源不存在,HTTP 405 Method Not Allowed如果删除操作不允许等等。根据响应状态,客户端可以决定是否保留引用的资源(在您的情况下是item/{id}引用的对象)。

由于您使用的是Spring,因此可以使用适当的@ResponseStatus注释控制器方法,例如 @ResponseStatus(value = HttpStatus.NO_CONTENT)

此外,Spring HandlerExceptionResolver已经按default提供了一些状态代码。

答案 1 :(得分:10)

如果用户可以修复删除问题,那么这似乎没问题。如果用户无法做任何事情,那么错误代码状态可能更正确。我可以想象删除的唯一失败是授权失败,这将是一个401.这可以通过在方法'HttpServletResponse response'中添加一个参数来设置。您的代码将变为:

@RequestMapping(value = "items/{id}", method = RequestMethod.DELETE)
public String delete(@PathVariable("id") int itemId, Model model, HttpServletReponse response) {
    Item item = itemDao.get(id);

    // true -> can delete
    // false -> cannot delete, f.e. is FK reference somewhere
    boolean wasOk = itemDao.delete(item); 

    if (!wasOk) {
        // will write to user which item couldn't be deleted
        response.setStatus(HttpServletResponse.SC_UNAUTHORIZED);
        model.addAttribute("item", item);
        return "items/error";   
    }

    return "redirect:/items";
}

您可以根据需要替换其他状态代码,但这是一般概念。

您还可以执行以下操作:

    if (!wasOk) {
        throw new DataAccessException("Unable to delete item: " + item);
    }

然后在同一个类中有一个带注释的错误处理程序

@ExceptionHandler(DataAccessException.class)
@ResponseStatus(HttpStatus.BAD_REQUEST)
public String handleDataAccessException(DataAccessException ex) {
    // Do some stuff
    return "errorView";
}