检查数据库是否存在 MSSQL C#

Question

我有这样的代码：

private bool CheckDatabase(string databaseName, bool bRet)
    {
        string connString = "Server=localhost\\SQLEXPRESS;Integrated Security=SSPI;database=master";
        string cmdText = "select * from master.dbo.sysdatabases where name=\'" + databaseName + "\'";



        using (SqlConnection sqlConnection = new SqlConnection(connString))
        {
            sqlConnection.Open();
            using (SqlCommand sqlCmd = new SqlCommand(cmdText, sqlConnection))
            {
                int nRet = sqlCmd.ExecuteNonQuery();
                // regMessage.Text = nRet.ToString();
                if (nRet <= 0)
                {
                    bRet = false;
                }
                else
                {
                    bRet = true;
                }
            }
        }
        return bRet;
    }

然而

nRet

结果总是-1，就好像数据库不存在（确实存在）。 是因为数据库是空的吗？ 或者如果创建了数据库，它是否应该返回 >0，即使它是空的？ 顺便说一句，字符串 databaseName 是正确的。

编辑> 编辑： 我在“CREATE DATABASE @database”ssql 命令上也遇到了意外错误：

''@userDatabase' 附近的语法不正确

代码是这样的：

    var connString = "Server=localhost\\SQLEXPRESS;Integrated Security = SSPI; database = master";
                string cmdText = "CREATE DATABASE @userDatabase";
                using (var sqlConnection = new SqlConnection(connString))
                {
                    using (var sqlCmd = new SqlCommand(cmdText, sqlConnection))
                    {
                        sqlCmd.Parameters.Add("@userDatabase", System.Data.SqlDbType.NVarChar).Value = databaseName;=
                        sqlConnection.Open();
                        sqlCmd.ExecuteNonQuery();
                    }
                }

Answer 1

1. 使用参数化查询。
2. 使用Select count(*) 而不是Select *。
3. 使用ExecuteScalar 而不是ExecuteNonQuery
4. 注意代码上的注释，它们解释了我所做的更改。

// No point of passing a bool if all you do is return it...
private bool CheckDatabase(string databaseName)
{
    // You know it's a string, use var
    var connString = "Server=localhost\\SQLEXPRESS;Integrated Security=SSPI;database=master";
    // Note: It's better to take the connection string from the config file.

    var cmdText = "select count(*) from master.dbo.sysdatabases where name=@database";

    using (var sqlConnection = new SqlConnection(connString))
    {
        using (var sqlCmd = new SqlCommand(cmdText, sqlConnection))
        {
            // Use parameters to protect against Sql Injection
            sqlCmd.Parameters.Add("@database", System.Data.SqlDbType.NVarChar).Value = databaseName;

            // Open the connection as late as possible
            sqlConnection.Open();
            // count(*) will always return an int, so it's safe to use Convert.ToInt32
            return Convert.ToInt32( sqlCmd.ExecuteScalar()) == 1;
        }
    }

}

Answer 2

不，您不能使用 ExecuteNonQuery 从 SELECT 语句中返回某些内容，您指向该 MSDN 论坛的链接包含不正确的答案（正如您已经尝试过的那样）

MSDN 文档解释说 ExecuteNonQuery 仅针对 UPDATE/INSERT/DELETE 查询返回受 sql 语句影响的行数。要计算受 SELECT 影响的行数，它应该通过更多工作来检索结果。这可能不利于性能。

所以你可以改变你的功能

private bool CheckDatabase(string databaseName)
{
    string connString = "Server=localhost\\SQLEXPRESS;Integrated Security=SSPI;database=master";
    string cmdText = @"if Exists(select 1 from master.dbo.sysdatabases where name=@db) 
                       select 1 else select 0";
    using (SqlConnection sqlConnection = new SqlConnection(connString))
    {
        sqlConnection.Open();
        using (SqlCommand sqlCmd = new SqlCommand(cmdText, sqlConnection))
        {
            sqlCmd.Parameters.Add("@db", SqlDbType.NVarChar).Value = databaseName;
            int nRet = Convert.ToInt32(sqlCmd.ExecuteScalar());
            return (nRet > 0);
        }
    }
}

我改变了什么：

• 不需要作为参数传递的 bool 变量
• 检查数据库是否存在的最高效的方法是通过 IF EXIST 语句
• 数据库名称应作为参数而不是字符串传递 串联
• 不要调用 ExecuteNonQuery 而只是调用 ExecuteScalar 来获得简单的 IF EXIST(... condition...) SELECT 0/1 语句返回的标量值
• 返回值可以简化

Answer 3

如果你知道数据库文件在哪里，可以使用File.Exists()：

if (File.Exists(DatabaseFile))

{
 //DoSomething
}