ADO.NETでUpdateができない

Question

Visual Studioのバージョンは、Microsoft Visual Studio Community 2017 Version 15.8.4です。
下記のコードでは、SQL ServerのUpdateが出来ませんでした。
結局は、SQL ServerにあるArticlesテーブルのUpdateTimeカラムの値をすべての行ごとnullにしたいのです。
最初のコードブロックの部分内の処理が出来ずに困っています。更新の仕方に間違いがありそうです。
どうしたらよいか教えて下さい。よろしくお願いします。

        public void DeleteUpdateTime(DataTable table)
        {
            using (SqlConnection conn = new SqlConnection(GetConnectionString(databasename)))
            using (da = GetDataAdapter(conn, table.TableName))
            {
                conn.Open();
                cmd = new SqlCommand(SelectTable(table.TableName));
                da = new SqlDataAdapter { SelectCommand = cmd };

                cmd = new SqlCommand("UPDATE Articles SET UpdateTime = null;", conn);

                da.UpdateCommand = cmd;

                da.Update(table);
            }
        }

// 下記は、参考程度にしてください。問題は上のコードです。↑

        public DataTable GetTheArticle(SqlConnection conn, DataSet ds)
        {
// ID, Name, InputTime, UpdateTimeのカラムを持ったArticlesテーブル(StrArticles)のUpdateTimeをnullにしようとしてます。

            string sSelect = "SELECT ID, Name, InputTime, UpdateTime FROM Articles ORDER BY ID;";

            using (conn = new SqlConnection(GetConnectionString(dataBaseName)))
            {
                cmd = new SqlCommand(sSelect, conn);

                SqlDataAdapter da = new SqlDataAdapter(cmd);
                DataTable dTbl = new DataTable(StrArticles);

                da.Fill(dTbl);
                dTbl.PrimaryKey = new DataColumn[] { dTbl.Columns["ID"] };
                return dTbl;
            }
        }

        private string GetConnectionString(string dataBaseName)
        {
            string sConn = "";
            switch (authenticName)
            {
                case WinAuthentic:
                    sConn = "Data Source=" + serverName + ";Database=" + dataBaseName + ";Integrated Security=true;";
                    break;
                case SqlAuthentic:
                    sConn = "Persist Security Info=False;User ID=" + loginName + ";Password=" + loginPass +
                        ";Initial Catalog=" + dataBaseName + ";Server=" + serverName + ";";
                    break;
            }
            return sConn;
        }

// さらに下記のように、DataSetを追加して、また、要らないと思われるコードを削ってみましたが、やはりだめでした。

        public void DeleteUpdateTime(DataTable table)
        {
// 今度は、こうやって非接続型での処理を使用との意向を明確にするため、DataSetを明示してみましたが、やはりだめでした。
            DataSet ds = new DataSet();
            string sTable = table.TableName;
            using (SqlConnection conn = new SqlConnection(GetConnectionString(databasename)))
            using (SqlDataAdapter da = new SqlDataAdapter(SelectTable(sTable), conn))
            {
                // conn.Open();
                da.Fill(ds, sTable);

                cmd = new SqlCommand("UPDATE Articles SET UpdateTime = null;", conn);

                da.UpdateCommand = cmd;

                da.Update(table);
            }
        }
// なお、tableには、ArticlesというTableのデータがすべて格納されていますので、接続状態などは問題なさそうです。

        public void DeleteUpdateTime(DataTable table)
        {
            string sTable = table.TableName;
            using (SqlConnection conn = new SqlConnection(GetConnectionString(StrNote)))
            using (SqlDataAdapter da = new SqlDataAdapter(SelectTable(sTable), conn))
            {
                cmd = new SqlCommand("UPDATE Articles SET UpdateTime = null;", conn);

                da.UpdateCommand = cmd;

// Forループを使わずに、あっさりとすべての行のUpdateTime列をnullにできるともっといいけど。
                for (int i = 0; i < table.Rows.Count - 1; i++)
                {
                    articleTable.Rows[i]["UpdateTime"] = DBNull.Value;
                }

                da.Update(table);
            }
        }

Comments

[退会済みユーザー]

どこがどう期待と違う結果になるのか詳しく書いてください。エラーメッセージが出ているならそれをコピペして質問欄に書いてください。

[FacePanel]

まず、エラーにはなりません。最後の da.Update(table); のところも、処理出来てよさそうなのですが、データベースは、変わらぬままです。で、どうしたらよいのだろう、となりました。

[退会済みユーザー]

> DataSetを追加して、また、要らないと思われるコードを削って ← 回答に書いたこと、全然理解してないですよ。ここでは改行もできないので回答欄のコメントに書きます。

[FacePanel]

DataSet を使ってそこへの変更をデータベースに反映するのが非接続型と理解しましたが、おかしいかな？DataReaderは使っていないわけだし。なんだろ？

[退会済みユーザー]

話が通じてませんよ。DataSet は関係ない。回答欄のコメントを見てください。

Answer 1

コードを拝見しましたが、はっきり言って基本的なことが分かってないように見えます。

ADO.NET の DataTable, DataAdapter を使うのは「非接続型のデータ更新」と言いますが、その場合は以下の記事の図１、図２のようにします。質問にアップされたコードは全くそうなってないですよね？

DB 設計者のための明解 ADO.NET 第 1 回
https://msdn.microsoft.com/ja-jp/library/cc482903.aspx

同じ記事に「接続型データアクセス」の説明がありますが、そちらで行うことを検討されてはいかがですか？

Comments

[FacePanel]

基本の解説をご紹介いただき、ありがとうございます。
まずは、理解するまで読んでみます。

[退会済みユーザー]

> DataSetを追加して、また、要らないと思われるコードを削って

図１、図２を見て DataSet を使うようにしたと思いますが、それはは関係ないです。DataSet 無しでも DataTable に直接 Fill できますし、Update で更新もできます。

問題は、(1) SelectCommand, UpdateCommand, DeleteCommand, InsertCommand の設定、(2) DataTable の編集です。

(1) が設定できていたとしても、(2) はどうしたのですか？ Fill するだけではダメで、図１にあるように DataTable を編集して当該列に NULL（実際は DBNull.Value）を設定し Modified 状態にしないと、Update をかけても何も変わらないことが分かりませんか？

[FacePanel]

あ、DataTableのUpdateTime列の値をDBNull.Valueにするという訳ですね。で、それはどうやるんでしょうか？ 今まで、そんなときどうしていたかを覚えておりません。
まさか、For文で、１行ずつ変更するのでしょうか？

Answer 2

// 自力ではありません。SufferOnWwwさんのおかげです。

        public void DeleteUpdateTime(DataTable table)
        {
            string sTable = table.TableName;
            using (SqlConnection conn = new SqlConnection(GetConnectionString(databasename)))
            using (SqlDataAdapter da = new SqlDataAdapter(SelectTable(sTable), conn))
            {
                cmd = new SqlCommand("UPDATE Articles SET UpdateTime = null;", conn);

                da.UpdateCommand = cmd;

// ほんとは、ループでなく一括で処理したいけど
                for (int i = 0; i < table.Rows.Count - 1; i++)
                {
                    table.Rows[i]["UpdateTime"] = DBNull.Value;
                }

                da.Update(table);
            }
        }

Comments

[退会済みユーザー]

それは結果オーライと言うだけで根本的にダメです。それで解決済みマークを付けるのはいかがなものかと思います。

[FacePanel]

いったい、どの辺りに問題があるのでしょうか？分からないままで終わってしまう訳にはいきませんね。

[FacePanel]

table.AcceptChanges();を追加しなければいけなかった、ということでしょうか？

[退会済みユーザー]

Q&A を続けたいというご希望であれば、解決済みマークは外して、上の回答の内容は質問欄に追記して「この方法ではどうか？」と質問するようにし、上の回答の内容は削除して回答欄に移した旨書いてください。

[FacePanel]

解決済みを外せません。ロックがかかっているのか、解決済みボタンが見えません。

[退会済みユーザー]

では、新たに別のスレッドを立てて、「より良い方法がないか？」というような質問してはいかが？

[FacePanel]

ありがとうございます！試してみます。