自定义Exception Mapper中的实例变量为何必须设为静态？

我在应用中实现了一个自定义Exception Mapper用于抛出Bad Request错误，代码如下：

package com.test.exceptions;
import javax.ws.rs.core.MediaType;
import javax.ws.rs.core.Response;
import javax.ws.rs.ext.ExceptionMapper;
import java.util.Date;
public class CustomFilterBadRequest extends Exception implements ExceptionMapper<CustomFilterBadRequest> {
    private String uriInfo;
    public CustomFilterBadRequest() {
        super("Invalid Request. Pleas...");
    }
    // 省略其他方法
}

现咨询：为何自定义Exception Mapper中的实例变量必须设为静态？

其实这个问题的核心在于JAX-RS对ExceptionMapper的实例管理策略——默认情况下，每个ExceptionMapper实现类在应用中只会被创建一个单例实例，它的生命周期和整个应用绑定。

咱们来拆解一下为什么实例变量会有问题：

• 当你的CustomFilterBadRequest既是异常类又是ExceptionMapper时，这个Mapper的单例实例会被JAX-RS容器初始化并复用。如果它有非静态的实例变量uriInfo，那么所有请求触发的异常处理都会共享这个变量。
• 举个例子：如果两个请求同时抛出CustomFilterBadRequest异常，第一个请求刚把uriInfo设为/api/user，第二个请求立刻把它改成/api/order，那么第一个请求的响应里可能会错误地使用第二个请求的uri值，这就是典型的线程安全问题。

那为什么有人会觉得“必须设为静态”？其实这是一种无奈的妥协，但并不是正确的解决方案。静态变量属于类本身，同样会面临线程安全问题（多个请求同时修改静态变量依然会导致数据混乱）。

正确的做法应该是： #

把异常类和ExceptionMapper职责分开，并且将请求相关的变量（比如uriInfo）放在异常实例中，而不是Mapper里：

1. 单独定义异常类：

public class CustomFilterBadRequestException extends Exception {
    private String uriInfo;

    public CustomFilterBadRequestException(String message, String uriInfo) {
        super(message);
        this.uriInfo = uriInfo;
    }

    public String getUriInfo() {
        return uriInfo;
    }
}

2. 单独实现ExceptionMapper：

public class CustomFilterBadRequestMapper implements ExceptionMapper<CustomFilterBadRequestException> {
    @Override
    public Response toResponse(CustomFilterBadRequestException exception) {
        // 从异常实例中获取uriInfo，每个异常实例对应一个请求，不会有线程安全问题
        String uri = exception.getUriInfo();
        ErrorResponse error = new ErrorResponse(
            new Date(),
            Response.Status.BAD_REQUEST.getStatusCode(),
            "Bad Request",
            exception.getMessage(),
            uri
        );
        return Response.status(Response.Status.BAD_REQUEST)
            .type(MediaType.APPLICATION_JSON)
            .entity(error)
            .build();
    }
}

这样一来，每次抛出异常时都会创建一个新的CustomFilterBadRequestException实例，每个实例持有自己的uriInfo，Mapper在处理时从对应的异常实例中读取数据，完全不存在线程安全问题，也不需要使用静态变量。

总结一下：不是“必须设为静态”，而是你的当前设计把异常类和Mapper混在一起，加上Mapper的单例特性，导致实例变量无法安全地存储请求相关数据。拆分职责才是正确的解决思路。

内容的提问来源于stack exchange，提问作者Ashwini