如何实现CloseableHttpClient可复用？下述代码实现是否正确？

你的现有实现存在几处明显问题，无法直接投入使用，具体问题如下：

• 线程安全缺陷：静态client变量的初始化放在构造方法的无锁非空判断里，多线程场景下同时创建ConnectionPool实例时，会触发多次client实例化，无法保证单例。如果要保留懒加载逻辑，需要使用双重检查锁，同时给client变量加volatile关键字禁止指令重排序。
• 配置重复冗余：你同时在PoolingHttpClientConnectionManager和HttpClients.custom()构造器里设置了最大连接数、单路由最大连接数，两处配置重复，若数值不一致会以HttpClientBuilder的配置为准，建议统一在连接管理器中配置即可，删除构造器里的重复配置项。
• 编译不通过问题：代码中引用的requestConfig、keepAliveStrategy两个变量没有定义，直接运行会报错。你需要手动初始化这两个对象：比如RequestConfig可以设置连接超时、读取超时、获取连接超时等核心参数，无特殊需求的话KeepAliveStrategy可以直接用默认实现，不需要额外设置。
• 无资源释放逻辑：没有提供连接池和HttpClient的销毁方法，应用停止时无法主动释放连接资源，会出现资源泄漏。建议新增静态销毁方法，在应用关闭钩子或者容器销毁回调中调用client.close()释放资源。
• 初始化逻辑不合理：client是全局静态变量，却绑定到类的构造方法初始化，只有实例化ConnectionPool时才会触发初始化，不符合全局复用的设计初衷，建议改为静态块初始化或者单例模式实现，不需要依赖类的实例化。

优化后参考实现 #

public class HttpClientPool {
    // 加volatile禁止指令重排序
    private static volatile CloseableHttpClient client;
    
    // 私有构造方法禁止外部实例化
    private HttpClientPool() {}

    // 对外提供全局获取方法，双重检查锁保证线程安全
    public static CloseableHttpClient getInstance() {
        if (client == null) {
            synchronized (HttpClientPool.class) {
                if (client == null) {
                    // 初始化请求配置，按需调整参数
                    RequestConfig requestConfig = RequestConfig.custom()
                            .setConnectTimeout(5000)
                            .setSocketTimeout(10000)
                            .setConnectionRequestTimeout(3000)
                            .build();
                    
                    PoolingHttpClientConnectionManager connectionManager = new PoolingHttpClientConnectionManager();
                    connectionManager.setDefaultMaxPerRoute(200);
                    connectionManager.setMaxTotal(200);
                    connectionManager.setValidateAfterInactivity(50000);

                    client = HttpClients.custom()
                            .setConnectionManager(connectionManager)
                            .setConnectionManagerShared(true)
                            .setDefaultRequestConfig(requestConfig)
                            // 无特殊需求可以用默认的keepAlive策略
                            .setConnectionReuseStrategy(new DefaultConnectionReuseStrategy())
                            .setUserTokenHandler(context -> null)
                            .setConnectionTimeToLive(50000, TimeUnit.MILLISECONDS)
                            .build();
                }
            }
        }
        return client;
    }

    // 提供销毁方法，应用停止时调用
    public static void destroy() throws IOException {
        if (client != null) {
            client.close();
        }
    }
}

内容的提问来源于stack exchange，提问作者average.joe