CommandText属性未初始化。

Question

CommandText属性未初始化。

4

我发现只有在将我的工作代码更改为使用数据视图来显示单行数据而不是文本框后，才会出现这个问题。

我有以下内容：

static SqlConnection dbConnection = new SqlConnection
                       (DBConnection.DBConnection.connectionString);        
    SqlDataAdapter holdone = new SqlDataAdapter(getCommand, dbConnection);      

    DataSet holdall = new DataSet();
    DataSet updateall = new DataSet();
    DataTable invoiceTable = new DataTable();

    DataView invoiceView = new DataView();

这是由IT技术所使用的。

public void GetOne(/* connectionString, string invref, string tableref*/)
    {

getCommand = "select *redacted* from " + tableref +  
                                      "where *redacted* = " + invref;

        using (SqlConnection dbConnection = new SqlConnection
                               (DBConnection.DBConnection.connectionString))
        {
                dbConnection.Open();

            holdone.Fill(holdall);

            invoiceTable = holdall.Tables[0];

            dbConnection.Close();


        }

        DataRowView rowView = invoiceView.AddNew();

            rowView["*redacted*"] = invoiceTable;

            rowView.EndEdit();
    }

错误报告指出 holdone.fill(holdall) 这一行存在问题，但我不确定原因，因为我没有将 SQLCommand 作为参数使用，而是使用了 SQLDataAdapter 的参数。

我不太清楚自己哪里出错了。

- Wolfish

1

@Jonny ExecuteReader：CommandText 属性未初始化。 - Wolfish

1

@Jonny Ah。这是一个“InvalidOperationException”。 - Wolfish

1

@AlexBell，命令文本未初始化和无效的命令语法之间存在差异。如果他只将 “一些” 文本分配给命令，他将不会得到他现在所遇到的错误。 - Thorsten Dittmar

1

@AlexBell 抱歉，我的意思是那段文字已被我从帖子中删除，而不是我实际在使用 *。对于任何困惑感到抱歉。 - Wolfish

1

@AlexBell 这里有一个误解 - 我试图指出你的评论没有帮助到提到的问题 :-) - Thorsten Dittmar

显示剩余8条评论

3个回答

1

看起来你在将getCommand传递给SqlDataAdapter构造函数之前，还没有为其分配实际的SQL命令。首先初始化getCommand字符串，然后再构造SqlDataAdapter对象。

- Zoran Horvat

1

我需要将命令与其他内容隔离开来，因为我可能会重新使用适配器。此外，我刚刚尝试过同样的操作，结果也是一样的。 - Wolfish

1

@Wolfish 那你做错了。在 GetOne 方法中，确实设置了 getCommand，但是你没有将其赋值给 holdone SQL 适配器的 CommandText。请记住，您不是设置对 getCommand 的引用，而是在这一行中将内部命令文本设置为 getCommand 的当前值：SqlDataAdapter holdone = new SqlDataAdapter(getCommand, dbConnection);。改变 getCommand 不会更改 SqlDataAdapter 的命令文本！ - Thorsten Dittmar

0

您的CommandText是按值传递给SqlDataAdapter的，而不是按引用传递，因此在创建适配器后更改变量getCommand将不会影响数据适配器。一个非常简单的演示是：

static void Main(string[] args)
{
    string r = "Test";
    using (var adapter = new SqlDataAdapter(r, new SqlConnection("")))
    {
        r = "Test2";
        Console.WriteLine(adapter.SelectCommand.CommandText);
    }

}

输出

测试

不是

测试2

您需要显式更改适配器的SelectCommand.CommandText：

holdone.SelectCommand.CommandText = getCommand;

就算这样重复使用对象也没有什么用，它几乎不会节省代码。.NET 使用连接池，因此使用多个 SQL 连接/适配器并不一定意味着对数据库有多个管道。您还应该使用参数化查询：

string sql = "select *redacted* from " + tableref + " where *redacted* = @InvoiceParam";
using (var adapter = new SqlDataAdapter(sql, DBConnection.DBConnection.connectionString))
{
    adapter.SelectCommand.Parameters.AddWithValue("@InvoiceParam", invRef);
    adapter.Fill(holdall);
    invoiceTable = holdall.Tables[0];
}

- GarethD

1

我感谢有关参数化查询的建议，我会进一步研究。至于SelectCommand，@ThorstenDittmar上面的答案似乎有效。 - Wolfish

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- Thorsten Dittmar · Accepted Answer

问题在于，您在此行代码中将SqlDataAdapter的选择命令字符串设置为getCommand的当前值：

。

SqlDataAdapter holdone = new SqlDataAdapter(getCommand, dbConnection);

然而，由于字符串不是真正的指针，改变getCommand后将不会更改SqlDataAdapter的选择命令。

你需要做的是：

public void GetOne(/* connectionString, string invref, string tableref*/)
{

    getCommand = "select *redacted* from " + tableref + "where *redacted* = " + invref;
    using (SqlConnection dbConnection = new SqlConnection(DBConnection.DBConnection.connectionString))
    {
        dbConnection.Open();

        holdone.SelectCommand = new SqlCommand(getCommand, dbConnection);
        holdone.Fill(holdall);

        invoiceTable = holdall.Tables[0];
        //dbConnection.Close(); // This line is unnecessary, as the connection will be closed by `using`
    }

    DataRowView rowView = invoiceView.AddNew();
    rowView["*redacted*"] = invoiceTable;
    rowView.EndEdit();
}