SQL serverデータベースの特定の行の値を更新したい（C#、Visual Studio）

Question

英単語帳を作成していて、現在SQL serverデータベースへ値の登録までできたのですが、Update文での値の更新がうまくいかないので質問します。

やりたいことは、データベースよりランダムで行を取得したものを
その行を編集したい時に、UIから編集できるようにしたいです。

一応調べてエラーがで出ないように書けたのですが、データベースが更新されていません。
おかしいところがあれば教えてください。

　//接続文字列の初期化はしています。
  //データベースの更新 新しい４つの値を引数へ入れて更新させる。
        public void EditRow(string jp, string en, string addi, string audi)
        {
            SqlCommand Updcom = new System.Data.SqlClient.SqlCommand();

            connection.Open();
            try
            {
                Updcom.Connection  = connection;
                Updcom.CommandType = CommandType.Text;

                //現在の主キーId番号の各列の値を変更する
                Updcom.CommandText = "UPDATE [Sentence] " +
                    "SET [JpSentence] = @JpSentence, " +
                    "[EnSentence]     = @EnSentence, " +
                    "[Addition]       = @Addition," +
                    "[AudioLink]      = @AudioLink " +
                    "WHERE [Id]       = SCOPE_IDENTITY()";

                Updcom.Parameters.AddWithValue("@jpSentence", en);
                Updcom.Parameters.AddWithValue("@EnSentence", en);
                Updcom.Parameters.AddWithValue("@Addition", addi);
                Updcom.Parameters.AddWithValue("@AudioLink", audi);

                Updcom.ExecuteNonQuery();
            }
            catch (Exception exception)
            {
                Console.WriteLine(exception.Message);
                throw;
            }
            finally
            {
                connection.Close();
            }

        }

データの取り出しは以下のようにしています。

   //ランダムで１行のデータを取り出す
        public void GetRow()
        {
            connection.Open();
            try
            {　　//ランダムでひとつ取り出す
                string sqlstr = "select top(1) * from Sentence order by NEWID()";

                SqlCommand com = new SqlCommand(sqlstr, connection);

                //SQLクエリの結果を読み出すSqlDataReader オブジェクトを取得
                SqlDataReader sdr = com.ExecuteReader();
                while (sdr.Read() == true)
                {
                    //インデクサーに列名を与え値を取得する
                    ID = (int)(DBNull.Value.Equals(sdr["Id"]) ? null : sdr["Id"]);
                    JpSentence = (string)(DBNull.Value.Equals(sdr["JpSentence"]) ? null : sdr["JpSentence"]);
                    EnSentence = (string)(DBNull.Value.Equals(sdr["EnSentence"]) ? null : sdr["EnSentence"]);
                    Addition   = (string)(DBNull.Value.Equals(sdr["Addition"]) ? null : sdr["Addition"]);
                    AudioLink  = (string)(DBNull.Value.Equals(sdr["AudioLink"]) ? null : sdr["AudioLink"]);
                }
            }
            catch (Exception exception)
            {
                Console.WriteLine(exception.Message);
                throw;
            }
            finally
            {
                connection.Close();
            }
        }

Answer 1

ざっと見ただけで他にも何かあるかもしれませんが、気になったのは update 文で主キー id に設定している scope_identity() です。編集したレコードの元々の(表題で言う「特定の行」の)主キー値を設定するはずですが？

Comments

[nabesi]

回答ありがとうございました。Where部分でご指摘のあった部分に取り出したIDを入れると、目的通りのデータ更新ができました。感謝します。

where部分を右に変えて　　　　　　　"WHERE [Id] = @Id";
この１行を加えるとできました！　　　Updcom.Parameters.AddWithValue("@Id", ID);

Answer 2

SCOPE_IDENTITYはスコープ内で、最後に挿入されたIDENTITY列の値を取得するものです。

読み込んだデータに対しての更新ですから、読み込んだIDをwhere条件にする必要があります。

Comments

[nabesi]

回答ありがとうございました。Where部分でご指摘のあった部分に取り出したIDを入れると、目的通りのデータ更新ができました。感謝します。

where部分を右に変えて　　　　　　　"WHERE [Id] = @Id";
この１行を加えるとできました！　　　Updcom.Parameters.AddWithValue("@Id", ID);