c# - 检查数据表是否为空

Question

以下代码是我一直用来从 sql 数据库中检索用户信息的代码。

            string userName = LoginUser.UserName;
            string password = LoginUser.Password;
            string comm = "SELECT Username,Password,Clientname,Role FROM Users WHERE Username = '" + userName + "';";
            bool rememberUserName = LoginUser.RememberMeSet;

            SqlConnection conn = new SqlConnection(connstring);
            conn.Open();

            SqlCommand command = new SqlCommand(comm, conn);
            SqlDataAdapter da = new SqlDataAdapter(command);
            DataTable dt = new DataTable();
            da.Fill(dt);
            DataRow dr = dt.NewRow();
            if (dt != null)
            {
                //logic
            }

但是，当数据库中没有用户名等于 LoginUser.Username 的条目时，(dt != null) 不会返回 false。是否有不同的方法来检查 sql 命令是否成功？

score 36 · Accepted Answer

如果没有记录匹配，您将得到一个空DataTable，因此您可以检查返回的记录数：

if (dt.Rows.Count > 0)

而且，稍微偏离主题，请阅读您问题下方的评论，然后搜索术语 SQL 注入和参数化 SQL 语句。尝试从这个开始。

score 13 · Accepted Answer

为什么不稍微改变一下语句来查看 DataTable 是 null 还是没有行：

if(dt != null && dt.Rows.Count > 0)

此外，在旁注中，您还应该查看参数化查询，而不是动态构建 SQL。它将减少试图破坏您的应用程序的攻击者的攻击向量数量。

score 2 · Accepted Answer

从 C# 6.0 开始，您可以使用Null 条件运算符 ?.（或?[]用于数组）。

空条件运算符将语句简化为：

if (dt?.Rows?.Count > 0)

这在以下情况下返回 false：

数据表dt为空
数据表行为dt.Rows空
行数dt.Rows.Count为 0

使用 null 条件运算符可以避免手动检查数据表和计数属性，例如if (dt != null && dt.Rows.Count > 0)

score 1 · Accepted Answer

在我看来，使用 DataTable 和 SqlDataAdapter 对于这项任务来说有点太重了。

您可以在这里使用 DataReader：

        SqlCommand command = new SqlCommand(comm, conn);
        using (var reader = command.ExecuteQuery()) 
        {
            if (reader.Read())
            {
                //logic
                var userName = reader.GetString(0);
                var password = reader.GetString(1);
                // etc
            }
        }

score 0 · Accepted Answer

如果try catch用于检查表是否为空会更好，只需处理IndexOutOfRangeException异常即可。

使用以下逻辑：

try
{
   //dataTable operations
}
catch(IndexOutOfRangeException)
{
}

对我来说它的工作。

score 0 · Accepted Answer

我知道这个问题是旧的，并且在它发布的那一刻得到了帮助，但是现在有一种简单的方法可以解决这个问题，如下所示：

if ((dataTableName?.Rows?.Count ?? 0) > 0)

score 0 · Accepted Answer

0

DataSet你可以这样检查：

if (ds.Tables[0].Rows.Count > 0)

于 2018-10-03T10:27:54.693 回答

c# - 检查数据表是否为空

7 回答 7

Related

Reference