VisualStudio C# プロジェクトの分割

Question

WindowsFormで画面を作成しています。
①生徒一覧画面　②登録画面　③編集画面　の３画面を作成しています。
C#でSQLServerに接続し、データを操作できる画面です。　
ここではSQL文を扱うプロジェクト(SQL)と画面の操作をするプロジェクト(Operation)でわけたいと思っています。

id：生徒番号

//(Operation)

//削除ボタン押下時
        private void DeleteBt_Click(object sender, EventArgs e)
        {
            //メッセージボックスの表示
            DialogResult result = MessageBox.Show("削除します。よろしいですか？", "確認", MessageBoxButtons.OKCancel);
            //OKボタン押下時
            if (result == DialogResult.OK)
            {
               
                // 接続文字列の取得
                var connectionString = ConfigurationManager.ConnectionStrings["sqlsvr"].ConnectionString;

                // データベース接続の準備
                using (var connection = new SqlConnection(connectionString))
                using (var command = connection.CreateCommand())
                {
                    try
                    {
                        // データベースの接続開始
                        connection.Open();

                        // SQLの準備
                        command.CommandText = @"DELETE MASTER WHERE id = @id";
                        command.Parameters.Add(new SqlParameter("@id", dgv.Cells[0].Value.ToString()));

                        // SQLの実行
                        command.ExecuteNonQuery();
                    }
                    finally
                    {
                        // データベースの接続終了
                        connection.Close();
                    }

現在、上記のようにコードを書いています。
それを

 if (result == DialogResult.OK)
            {
　　　　　　　　　MemberInformation member = new MemberInformation();
                MemberInformationManager.Delete(member.Id);
                member.Id = Convert.ToInt32(dgv.Cells[0].Value);
　　　　　　　｝

で書いてみたのですが、実行できませんでした。
ちなみにSQL文のプロジェクトとクラスは下記です。

//(SQL)

        public static Boolean Delete(Int32 id)
        {
            // 接続文字列の取得
            var connectionString = ConfigurationManager.ConnectionStrings["sqlsvr"].ConnectionString;

            // データベース接続の準備
            using (var connection = new SqlConnection(connectionString))
            using (var command = connection.CreateCommand())
            {
                try
                {
                    // データベースの接続開始
                    connection.Open();

                    // SQLの準備
                    MemberInformation resultMember = new MemberInformation();
                    command.CommandText = @"DELETE MASTER WHERE id = @id";
                    command.Parameters.Add(new SqlParameter("@id", resultMember.Id));

                    // SQLの実行
                    command.ExecuteNonQuery();

                }
                finally
                {
                    // データベースの接続終了
                    connection.Close();
                }
            }

            return true;
        }

//クラス

public class MemberInformation
    {
        public int Id { get; set; }
｝

何かアドバイスをいただけないでしょうか。
よろしくお願い致します

Comments

[y_waiwai]

実行できない、とはどういうふうになって実行できないんでしょうか。
エラーが出るならエラーメッセージを提示しましょう

[maam]

エラーが出るのではなく、削除が実行されないのでSQLServerに情報が残ったままになります。

[pontaq]

関係なくて申し訳ないのですが、usingでくくってるのでconnection.Close();は不要だと思います

Answer 1

public static Boolean Delete(Int32 id)の引数idが当該関数の中で使われておらず、SQLのデリート文ではその場でnewしたMemberInformationクラスのidが渡されている。

これではおそらく、id=0しか消すことができない。

newのときにコンストラクタに引数のidをわたすか、resultMember.Id = id;をSQL発行より前にするかしないと、選択したメンバを削除することはできないと思われる

--

 if (result == DialogResult.OK)
            {
　　　　　　　　　MemberInformation member = new MemberInformation();
                MemberInformationManager.Delete(member.Id);
                member.Id = Convert.ToInt32(dgv.Cells[0].Value);
　　　　　　　｝

ここでもDelete()を実行した時点で、member.Idは空っぽなので、default(int)の0が入るはず。

 if (result == DialogResult.OK)
            {
　　　　　　　　　MemberInformation member = new MemberInformation();
//                MemberInformationManager.Delete(member.Id);
                member.Id = Convert.ToInt32(dgv.Cells[0].Value);
                MemberInformationManager.Delete(member.Id);
　　　　　　　｝

場所が下では？

Comments

[maam]

ご回答ありがとうございます。
【　】の部分を付け加えてみたのですが、削除されずにデータがSQLServerに残ったままになります…。対応が違いますか？？

// データベースの接続開始
connection.Open();

// SQLの準備
MemberInformation resultMember = new MemberInformation();
【　resultMember.Id = id;　】
command.CommandText = @"DELETE MASTER WHERE id = @id";
command.Parameters.Add(new SqlParameter("@id", resultMember.Id));

// SQLの実行
command.ExecuteNonQuery();

[papinianus]

そこのところの意図は回答と合っておりますが、私の見落しがありました。回答追記します。申し訳ないです

[maam]

成功しました！ありがとうございます！