正确使用static

Question

目前我使用的是mongodb和java。我不想每次都重新打开到数据存储的连接，所以我想我应该创建一个静态变量：

package Config;

import java.net.UnknownHostException;

import com.google.code.morphia.*;

import com.mongodb.Mongo;
import com.mongodb.MongoException;

public class Config {

    static String dbUrl = "url";
    static int dbPort = portnumber;
    static String username = "user";
    static String password = "pw";
    static String dbName = "dbname";
    public static Datastore ds;

    public static Datastore getDatastore() throws UnknownHostException,
            MongoException {
        if (ds == null) {
            ds = connect();
        }
        return ds;
    }

    public static Datastore connect() throws UnknownHostException,
            MongoException {
        Mongo m = new Mongo(dbUrl, dbPort);
        Datastore ds = new Morphia().createDatastore(m, dbName);
        boolean con = ds.getDB().authenticate(username, password.toCharArray());// todo
        return ds;
    }
}

我会这样使用它：

Config.getDatastore().doSomthing();

除了那些未处理的异常之外，这是使用static变量的正确方式吗？

Answer 1

给定的Config似乎表明这个类应该是单例的，如果是这样，那么对所有成员使用static变量都是可以的。你会有多个Configs，还是只有一个？

如果Config不打算成为单例，那么在这种情况下static就不是必需的，或者如果是这样的话，只有在可以与Config类的每个实例共享Datastore的情况下才需要。你可以通过一个简单的非静态成员变量来实现你的目标，并且你仍然可以拥有，例如一个getDataStore方法。但是每个Config类都有它自己的Datastore

只有当您还想在Config类的多个实例之间共享该Datastore时，才使用static。

Answer 2

将Config类作为singleton会更好。

Answer 3

这是使用静态变量的正确方式吗？

好吧，你的代码看起来像是编译的，所以按照语言来说是正确的。就语义而言，用法视情况而定。这取决于问题中的类，以及你想要实现的设计。

我个人会选择connection pool。如果不需要多个连接，如果我只需要维护一个连接，我可能会使用singleton。如果不需要单个连接，我会使用成员变量。

在您的示例中，这将取决于您希望如何使用config类。在一般的设计术语中，您所做的是problematic，原因如下：

• 类中的一切都是静态的，它在面向对象方面没有任何意义。除非使用powermock或某些其他framework.
• what，否则无法对此类进行单元测试。当有人多次调用connect()时，会发生这种情况？

*注： Singleton是一个anti-pattern。一般来说，任何全局状态都是不好的。我会尽可能地避免它。