1

我正在通过 Jetty/CometD 运行一个长时间运行的 Web 服务，并且我正在使用 Redisson 库连接到 redis。我正在使用单例模式来获取我的 RedissonClient/连接，但我不确定这是否是最好的方法。

类看起来像这样：

public class RedisClient {
    // singleton instance of our RedisonClient/connection
    private static RedissonClient _redissonInstance;
    public static String REDIS_HOST = "my.redishost.com:6379";


    private static RedissonClient setupRedis() {
        org.redisson.Config config = new org.redisson.Config();

        config.useSingleServer()
                .setAddress(REDIS_HOST)
                .setConnectionPoolSize(200);

        return Redisson.create(config);
    }

    public static RedissonClient getRedis() {
        if (_redissonInstance == null) {
            _redissonInstance = setupRedis();
        }
        return _redissonInstance;
    }

    public static void setRedisHost(String redisHost) {
        _logger.warn("Setting REDIS_HOST to: " + redisHost);
        REDIS_HOST = redisHost;
    }
}
4

1 回答 1

4

我会说这是个坏主意。我不认为单身人士一般来说是一个好主意，但即便如此这也不是一个好方法。您的代码不是线程安全的，似乎您想要支持多个主机。

如果你真的不想将你的 redis 客户端传递给每个组件，并且你的主机不会改变并且想要快速而肮脏的东西，试试这个：

public class Redis {
    public static final RedissonClient CLIENT;
    static {
        Config config = new Config();
        config.useSingleServer()
            .setAddress("my.redishost.com:6379")
            .setConnectionPoolSize(200);
        CLIENT = Redisson.create(config);
    }
}

这具有线程安全的好处，在获取引用时无需任何同步。

于 2016-03-02T18:37:20.923 回答