2

我有这个问题。我用谷歌搜索了很多关于这个问题的网站，但他们都建议关闭所有连接，或者只是使用连接和 dataReader 的“使用”。但！我的问题在于我无法打开第一个连接！我在连接附近设置了一个断点，并看到没有其他连接，所以，有第一个。当我重新制作从静态到单例的打开连接的类时，这个问题就出现了，这里是代码：

public class Storage
{
private static Storage instance;

public static Storage Instance
{
    get
    {
        if (instance == null)
        {
            instance = new Storage();
        }
        return instance;
    }
}

private Storage()
{
    Manager man = new Manager();
    products = man.LoadProducts();
    components = man.LoadComponents();
    man.LoadProductComponents();
}

public Dictionary<int, Product> Products
    {
        get { return products; }
        set { products = value; }
    }

public Dictionary<int, Component> Components
    {
        get { return components; }
        set { components = value; }
    }

private Dictionary<int, Product> products;
private Dictionary<int, Component> components;

}

这是一个 Manager 构造函数

    public Manager()
    {
        connection = new SqlConnection(@"Persist Security Info=False;User ID=user;Password=pass;Initial Catalog=Products;Server=(local)");
        if (connection.State != ConnectionState.Open) connection.Open();
    }

当出现异常时，连接为Closed. 有人有想法吗？

更新：

如果我关闭池 - 我"System.StackOverflowException" in System.Data.dll在同一条线上。

4

1 回答 1

1

您的Manager班级创建并打开一个连接：

public Manager()
{
    connection = new SqlConnection(@"Persist Security Info=False;User ID=user;Password=pass;Initial Catalog=Products;Server=(local)");
    if (connection.State != ConnectionState.Open) connection.Open();
}

但是，如果我们看看你是如何使用它的，很明显没有任何东西会关闭这个连接：

private Storage()
{
    Manager man = new Manager();
    products = man.LoadProducts();
    components = man.LoadComponents();
    man.LoadProductComponents();
}

我希望Manager实现IDisposable，并让Dispose()方法关闭并释放连接：

class Manager : IDisposable
{
    ...
    public void Dispose()
    {
        if(connection != null) connection.Dispose();
        connection = null;
    }
}

然后将通过以下方式使用using：

private Storage()
{
    using(Manager man = new Manager())
    {
        products = man.LoadProducts();
        components = man.LoadComponents();
        man.LoadProductComponents();
    }
}

我担心的是你的经理只是一个更广泛问题的一个例子：你自己没有清理连接。当它Manager是静态的时，这可能是不可见的，但是当切换到Manager实例时，很容易旋转多个Manager对象。这些中的每一个都会在 GC 之前建立一个连接。

于 2013-02-26T08:21:20.617 回答