RESTendpoint与注入的服务-在哪里抛出自定义WebServiceExcture？

Question

我有一个注入服务的RESTendpoint。

@Singleton
@Path("/v1/service")
public class MyService {

    @Inject
    private TokenService tokenService;

    @Inject
    private InfoService infoService;

    @GET
    @Path("/info")
    @Consumes(MediaType.APPLICATION_JSON)
    @Produces(MediaType.APPLICATION_JSON)
    public InfoResponse getInfo(@HeaderParam("Authorization") String token){

        tokenService.validateToken(token);

        List<String> info = infoService.getInfo();

        // processing info code, throwing WebServiceException if something wrong

       return new InfoResponse(info);
    }
}

其中一些会验证数据（例如令牌），如果不正确，则会抛出RestServiceExcense异常。异常由ExceptionMapper处理，以便返回适当的响应。其他服务返回/处理信息。

我的自定义异常类与其他字段：

public class RestServiceException extends WebServiceException {

    public Integer httpStatus;

    public String message;
}

示例服务方法：

public User getUser(Long userId){

    User user = userMapper.getUser(userId);

    if (user == null){
        String msg = "Invalid user: " + userId;
        logger.debug(msg);
        throw new RestServiceException(HttpStatus.SC_NOT_FOUND, InternalStatus.INVALID_USER, msg);
    }

    return user;
}

在注入的服务中抛出RestServiceExcture是好的做法吗？如何设计代码方便测试？在哪里放置验证方法-服务或静态类？

Answer 1

在注入的服务中抛出WebServiceExcture是好的做法吗？

我不会的。

>

使服务在jax-rs应用程序之外不可重用（对您来说可能没有太大问题）。

打破分层架构规则；较低的级别不应该对较高的级别有任何了解。您应该抛出一个正常的服务级别异常，在资源中捕获异常，然后从那里如果您想换行一个WebApplication ationExc0019，那么就这样做。

也许仅仅抛出一个WebApplicationException并不被认为是“了解上层”，但我个人认为是这样的，因为异常类型是特定于上层的。

如何设计代码方便测试？

使用构造函数注入代替字段注入。如果需要，向资源提供模拟服务会更容易。

public class MyService {

    private final TokenService tokenService;
    private final InfoService infoService;

    @Inject
    public MyService(TokenService tokenService,
                     InfoService infoService) {
        this.tokenService = tokenService;
        this.infoService = infoService;
    }
}

在哪里放置验证方法-服务或静态类？

这真的没有意义。如果“静态类”是指单例模式单例，那么不肯定不是。使用你目前正在做的控制反转是你更好的选择。

我想我看错你的问题了。出于某种原因，我将您的WebServiceExc0019读取为JAX-RSWebApplication ationExc0019。

我想我上面提到的规则不适用。但我可能会将异常的名称更改为webserviceeexception以外的名称。这将它与更高的“web”层联系在一起。将异常命名为特定于该服务的名称。

关于在资源级别处理异常，如果您不想，您不需要对它做任何事情，如果ExceptionMapper正在处理它。或者，如果需要，您可以捕获它并将其包装在WebApplicationException中。

您可以查看这篇文章的底部，其中列出了所有的子类。如果需要，您可以将异常包装在其中一个中并将其抛出。这些异常都映射到状态代码。