在RESTendpoint上使用throws子句是否被认为是一种糟糕的设计？[副本]

Question

我想从我的RESTendpoint抛出异常。不过，我对好的REST设计技术还不是太熟悉。请考虑以下内容...

//note the throws clause
@POST
public Response saveNewActivity(@HeaderParam("sessionTokenString") String sessionTokenString, Activity activity) throws Exception {
    Activity result = as.saveNewActivity(activity);
    if (result == null) {
        throw new DuplicateDataException("blah blah blah");
    }
    return Response.ok(result).build();
}

而不是处理异常和显式地只返回响应

@POST
public Response saveNewActivity(@HeaderParam("sessionTokenString") String sessionTokenString, Activity activity) {
    try {
        Activity result = as.saveNewActivity(activity);
        if (result == null) {
            throw new DuplicateDataException("blah blah blah");
        }
        return Response.ok(result).build();
    } catch (Exception e) {
         return Response.status(Response.Status.SOME_STATUS).build();
    }
}

我可以使用ExceptionMapper映射DuplicateDataException,如下所示

public class DuplicateDataExceptionMapper implements ExceptionMapper<DuplicateDataException> {

    @Override
    public Response toResponse(DuplicateDataException e) {
        ErrorMessage errorMessage = new ErrorMessage ("Activity names must be unique.", <HttpStatusNumber>, "<add documentation here>");
        return Response.status(Status.NOT_FOUND).entity(errorMessage).build();
    }
}

虽然最终还是会返回响应，但是处理异常的一种方式（不管它们是否是RuntimeExceptions）是优先于另一种方式，还是这并不重要？我从来没有在RESTendpoint上看到过throws语句，这就是为什么我问。

Answer 1

如果父类正在捕获抛出的异常并返回适当的HTTP响应错误代码(4xx)，那么初始代码就可以了。

如果没有父类捕获这些，使它们变成4xx而不是500，那么您的代码--将响应代码更改为适合这个特定错误的代码--似乎是一个很好的主意。