質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.64%

ADO.NETでUpdateができない

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 3,384

FacePanel

score -4

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);
            }
        }
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • SurferOnWww

    2018/10/07 23:35 編集

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

    キャンセル

  • FacePanel

    2018/10/07 23:53

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

    キャンセル

  • SurferOnWww

    2018/10/07 23:55

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

    キャンセル

回答 2

+2

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/10/07 19:03

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

    キャンセル

  • 2018/10/07 23:52 編集

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

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

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

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

    キャンセル

  • 2018/10/08 00:41

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

    キャンセル

check解決した方法

0

// 自力ではありません。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);
            }
        }

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/10/08 12:04

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

    キャンセル

  • 2018/10/08 12:39

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

    キャンセル

  • 2018/10/08 12:44

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.64%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る