在C#中连接内容之前无所事事的最佳方式

时间:2011-06-13 19:10:11

标签: c# database-connection wait

我正在检查代码以连接到我正在处理的其中一个应用程序中的数据库,我看到了这个

 if (_dbConnection == null)
     _dbConnection = GetConnection();

 while (_dbConnection.State == ConnectionState.Connecting)
 {
     //Do Nothing until things are connected.
 }

 if (_dbConnection.State != ConnectionState.Open)
     _dbConnection.Open();

 var command = GetCommand(commandType);
 command.Connection = _dbConnection;
 return command;

while循环让我担心。在事情发生变化之前,有没有更好的方法可以做任何事情?

编辑:

连接如下:

private static IDbConnection GetConnection()
{
     return new SqlConnection(ConfigurationManager.ConnectionStrings["CoonectionStringName"].ConnectionString);
}

4 个答案:

答案 0 :(得分:5)

虽然循环确实有效并且是等待某些后台操作的有效策略,但其他答案似乎错过了一个关键点;你必须让后台操作做一些工作。通过while循环进行搅拌效率不高,但是Windows会考虑应用程序的主线程,这可能是等待的重要线程,并且在后台操作之前会在循环中旋转数百或数千次。获得一个CPU时间的时钟。

为了避免这种情况,请使用Thread.Yield()语句告诉处理器旋转等待CPU时间的所有其他线程,并在完成后返回。这允许计算机在您等待后台进程时完成一些工作,而不是独占CPU以旋转基本上空的循环。这很简单;这是贾斯汀的答案修订:

var startTime = DateTime.Now;
var endTime = DateTime.Now.AddSeconds(5);
var timeOut = false;

while (_dbConnection.State == ConnectionState.Connecting)
{
    if (DateTime.Now.CompareTo(endTime) >= 0)
    {
        timeOut = true;
        break;
    }
    Thread.Yield(); //tells the kernel to give other threads some time
}

if (timeOut)
{
    Console.WriteLine("Connection Timeout");
    // TODO: Handle your time out here.
}

答案 1 :(得分:2)

编辑:请注意,这适用于DbConnection而非IDbConnection

您始终可以使用DbConnection类的StateChange事件而不是while循环。

检查this

答案 2 :(得分:1)

考虑到这是一个Web应用程序,最好的办法是计算自开始尝试连接以来经过的持续时间,如果它超过了超时时间。显然,抛出异常或处理当时的情况。

var startTime = DateTime.Now;
var endTime = DateTime.Now.AddSeconds(5);
var timeOut = false;

while (_dbConnection.State == ConnectionState.Connecting)
{
    if (DateTime.Now.Compare(endTime) >= 0
    {
        timeOut = true;
        break;
    }
}

if (timeOut)
{
    // TODO: Handle your time out here.
}

答案 3 :(得分:0)

在StateChange事件上挂钩处理程序。 当国家开放时,做你需要的。

            m_SqlConnection = new SqlConnection(ConnectionStringBuilder.ConnectionString);
            m_SqlConnection.StateChange += new System.Data.StateChangeEventHandler(m_SqlConnection_StateChange);
            m_SqlConnection.Open();



    void m_SqlConnection_StateChange(object sender, System.Data.StateChangeEventArgs e)
    {
        try
        {
            if (m_SqlConnection.State == ConnectionState.Open)
            {
                //do stuff
            }
            if (m_SqlConnection.State == ConnectionState.Broken)
            {
                Close();
            }
            if (m_SqlConnection.State == ConnectionState.Closed)
            {
                Open();
            }
        }
        catch
        {

        }
    }