Windowsformを使ったマスタメンテ画面の作成

Question

前提・実現したいこと

C#初学者です。
windowsformとSQLserver、C#を使ったユーザーマスタメンテの画面を作成しています。
下記③番の条件にて入力された、ユーザーIDとサーバーに登録されたIDを比較するコード書けず止まっています。

①ユーザーID（6桁の整数）を入力し、
②「入力完了（btnFinish_Click）」をクリックすると、
③サーバーにそのＩＤが登録されていなければ
④ユーザー名、パスワードを入力し登録する。
という流れです。

お力添えいただけますと幸いです。

発生している問題・エラーメッセージ

該当のソースコード

USER_ID   USER_NAME  PASSWORD ADMIN_FLAG CRT_DAY CRT_TIME
1         tanaka	   aaa	    NULL	    NULL	 NULL
2	       yamada	   bbb	    NULL	    NULL	 NULL

using System;
using System.ComponentModel;
using System.Data.SqlClient;
using System.Text;
using System.Windows.Forms;

namespace user_registration
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            timer1.Start();
            ActiveControl = textBox1;
            MessageBox.Show("ORG0001 START");


        }

        private void Form1_KeyDown(object sender, KeyEventArgs e)
        {
            switch (e.KeyCode)
            {

                case Keys.F9:
                    btnRevocation.Focus();
                    btnRevocation.PerformClick();
                    break;
                
                case Keys.F10:
                    btnClosing.Focus();
                    btnClosing.PerformClick();
                    break;

                case Keys.F12:
                    btnFinish.Focus();
                    btnFinish.PerformClick();
                    break;

                default:
                    return;
            }
        }

        private void checkBox1_CheckedChanged(object sender, EventArgs e)
        {

        }

        private void Form1_Load(object sender, EventArgs e)
        {
            KeyPreview = true;
        }

        private void label1_Click(object sender, EventArgs e)
        {

        }

        private void textBox1_TextChanged(object sender, EventArgs e)
        {


        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            DateTime datetime = DateTime.Now;
            lblDate.Text = datetime.ToString("yyyy/MM/dd HH:mm:ss");
        }



        private void textBox3_TextChanged(object sender, EventArgs e)
        {

        }

        private void groupBox2_Enter(object sender, EventArgs e)
        {

        }

        private void groupBox1_Enter(object sender, EventArgs e)
        {

        }

        private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {

        }

        private void btnDeletion_Click(object sender, EventArgs e)
        {
            if (body.Enabled == false)
            {
                textBox1.Clear();
            }

            if (header.Enabled == false)
            {
                
                if (!String.IsNullOrEmpty(textBox2.Text)|| !String.IsNullOrEmpty(textBox3.Text))
                {
                    textBox2.Clear();
                    textBox3.Clear();
                    checkBox1.Checked = false;
                }
                else
                {
                    checkBox1.Checked = false;
                    header.Enabled = true;
                    body.Enabled = false;
                }
            }

            
        }


        private void btnClosing_Click(object sender, EventArgs e)
        {
            if (String.IsNullOrEmpty(textBox1.Text) && String.IsNullOrEmpty(textBox2.Text) && String.IsNullOrEmpty(textBox3.Text))
            {
                Application.Exit();
            }
            else
            {
                DialogResult result = MessageBox.Show("入力中の内容は破棄されます。\nよろしいですか？",
                "確認",
                MessageBoxButtons.YesNo,
                MessageBoxIcon.Exclamation);

                if (result == DialogResult.Yes)
                {
                    MessageBox.Show("ORG0001 END");
                    Application.Exit();
                }
            }
            
        }

        private void btnFinish_Click(object sender, EventArgs e)
        {

            try 
            {
                if (header.Enabled == true)
                {
                    if (String.IsNullOrEmpty(textBox1.Text))
                    {
                        MessageBox.Show("ユーザーコードを入力してください。");
                        ActiveControl = textBox1;
                        return;
                    }
                }

                if (textBox1.Text.IndexOf(' ') > -1 && this.ActiveControl != null)
                {
                    MessageBox.Show("不正な文字列が含まれています");
                    ActiveControl = textBox1;
                    return;
                }

                body.Enabled = true;
                header.Enabled = false;

　　　　　　　　// ここから下のコードがSQL文の記載箇所です。

                SqlConnection connection = new SqlConnection(Properties.Settings.Default.sqlServer);
                connection.Open();
                MessageBox.Show("接続に成功しました");


                StringBuilder sql = new StringBuilder();
                sql.AppendLine("SELECT USER_ID");
                sql.AppendLine("FROM USER_REGISTRATION");
                sql.AppendLine("WHERE USER_ID = 1");

                SqlCommand command = new SqlCommand(sql.ToString(), connection);
                SqlDataReader dr = command.ExecuteReader();



                //while (dr.Read())
                //{
                //    for (int i = 0; i < dr.FieldCount; i++)
                //    {
                //        MessageBox.Show(dr[i].ToString());
                //    }
                //}

                //builder.DataSource = "localhost\\SQLEXPRESS";
                //builder.UserID = "test";
                //builder.Password = "test";
                //builder.InitialCatalog = "master";
                //builder.IntegratedSecurity = true;

                //using (SqlConnection connection = new SqlConnection(builder.ConnectionString))
                //{
                //    connection.Open();
                //    MessageBox.Show("接続に成功しました");

                //    SqlCommand command = new SqlCommand();


                //}



            }
            catch (SqlException ex)
            {
                MessageBox.Show(ex.Message);
            }

            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            } 
        }
    }
}

Comments

[Zuishin]

> 下記③番の条件にて入力された、ユーザーIDとサーバーに登録されたIDを比較するコード書けず止まっています。

C# ではなく SQL で解決すべき問題です。

[退会済みユーザー]

質問者さん、無言ですが、回答したのでそれに対するフィードバックを返してください。役に立った / 立たなかったぐらいはすぐ返せるのでは？　役に立たなかったならどこがダメだっらかを書くとより期待に近い回答が出てくるかも。とにかく無言は NG です。

[kawanoonigiri]

SurFerOnWww様
回答ありがとうございます。

こちらの説明に不足していた部分があり、いただいた回答は使用しませんでしたが、他の機会にいかそうと思います。
また、上記コード解決済みです。
後ほど解決コードを記載の上、本質問を閉じさせていただきます。

ご回答並びにご指摘ありがとうございました。

[退会済みユーザー]

> 後ほど解決コードを記載の上、本質問を閉じさせていただきます。

忙しいのかもしれませんが早めにそれをお願いします。自分が関わったスレッドがオープン状態のままなのは自分的にかなり気になりますので。

Answer 1

ご指摘いただいた点など踏まえ、コードの修正をしました。
以下コードです。
途中エラーチェックなどの記載は省略してあります。

SqlConnection con = null;

try
{
    // サーバーに接続
    con = new SqlConnection(Properties.Settings.Default.sqlServer);
    con.Open();

    //SQLを作成
    StringBuilder sbSql = new StringBuilder();
    sbSql.AppendLine(" SELECT ");
    sbSql.AppendLine("     COUNT(USER_ID) ");
    sbSql.AppendLine(" FROM ");
    sbSql.AppendLine("     USER_REGISTRATION ");
    sbSql.AppendLine(" WHERE ");
    sbSql.AppendLine("     USER_ID = @USER_ID ");
    SqlCommand command = new SqlCommand(sbSql.ToString(), con);

    command.Parameters.Add(new SqlParameter("@USER_ID", strUserId));

    //SQLの実行（ユーザー件数の取得）
    int nCount = (int)command.ExecuteScalar();

    //データの存在確認
    bool bRet = false;
    if (nCount == 0)
    {
        bRet = true;
    }

    command.Dispose();

    return bRet;
}
catch (Exception ex)
{
    throw(ex);
}
finally
{
    con.Close();
}

Comments

[退会済みユーザー]

無駄に回答欄を増やさないでください。訳が分からなくなります。

それではダメですよ。

int nCount = (int)command.ExecuteScalar();

でその時は nCount はゼロだったとしても、その後で誰か他の人が INSERT できるわけなので、ダブる可能性があるってことが分かりませんか？

Answer 2

自己解決しました。
以下コードです。
途中エラーチェックなどの記載は省略してあります。

            try 
            {
                if (header.Enabled == true)
                {
                    ///SQLserverに接続
                    SqlConnection connection = new SqlConnection(Properties.Settings.Default.sqlServer);
                    connection.Open();

                    ///SQL文を記載
                    StringBuilder sbSql = new StringBuilder();
                    sbSql.AppendLine(" SELECT ");
                    sbSql.AppendLine("     COUNT(USER_ID) ");
                    sbSql.AppendLine(" FROM ");
                    sbSql.AppendLine("     USER_REGISTRATION ");
                    sbSql.AppendLine(" WHERE ");
                    sbSql.AppendLine("     USER_ID = @USER_ID ");
                    SqlCommand command = new SqlCommand(sbSql.ToString(), connection)

　　　　　　　       ///TextBoxから取得した値（userId.Text）をパラメーターに追加
                    command.Parameters.Add(new SqlParameter("@USER_ID", userId.Text));
　　　　　　　
　　　　　　　       ///command.ExecuteScalar()を使い戻り値の確認をしつつSQL文を実行
                    int nCount = (int)command.ExecuteScalar();
                   
　　　　　　　       ///SQLserverとの接続を終了
                    connection.Close();
                    return;
                }
            }

Comments

[退会済みユーザー]

私の回答にも書きましたが、それでは質問にあった、

> ③サーバーにそのＩＤが登録されていなければ
> ④ユーザー名、パスワードを入力し登録する。

で、③ と ④ の間に誰か他の人が新たな「ユーザーID」を登録して、それとダブってしまう（その場合 SELECT クエリで確認した意味がない）という可能性があります。SQL Server を使うということは、そういう可能性がある環境だと思ったのですが、違うのですか？

あと、本題とは関係ない話ですが、connection.Close(); の仕方が不適切です。finally 句の中に入れるか、接続に using 句を使うなど、必ず Close されるようにしましょう。

[kawanoonigiri]

ご指摘ありがとうございます。
仰る通り、ダブりの可能性があります。
データのINSERTをする際、他の人が先に同一IDで登録した場合はエラーになるよう別メソッドに記載してあります。
言葉足らずで申し訳ございません。
また、Close();の件回答に反映いたしました。
ありがとうございました。

[退会済みユーザー]

> 仰る通り、ダブりの可能性があります。
> データのINSERTをする際、他の人が先に同一IDで登録した場合はエラーになるよう別メソッドに記載してあります。

新たに作った回答欄のコードのことを言っているのでしょうが、それではダメですよ。

int nCount = (int)command.ExecuteScalar();

でその時は nCount はゼロだったとしても、その後で誰か他の人が INSERT できるわけなので、ダブる可能性があるってことが分かりませんか？

Answer 3

①ユーザーID（6桁の整数）を入力し、
②「入力完了（btnFinish_Click）」をクリックすると、
③サーバーにそのＩＤが登録されていなければ
④ユーザー名、パスワードを入力し登録する。

SELECT クエリを投げて ③ を確認しているようですがそんなことはしないで、「ユーザーID」を主キーにするかまたはユニーク制約を付ければ、SELECT で確認などする必要はなく、即 INSERT 操作をすれば未登録なら登録されるし、登録済みなら Primary / Unique キー制約違反例外が出るので、それを catch してユーザーに知らせて訂正と再入力を促すようにしていかがですか？

そうすれば、③ と ④ の間に誰か他の人が登録した「ユーザーID」とダブってしまうというような問題も避けられます。

PK 制約違反を catch するサンプルコードを載せておきます。ユニーク制約違反の場合は sqle.Number == 2601 になります。

public bool InsertAuthors()
{
    var connection = new SqlConnection("接続文字列");
    var command = new SqlCommand("INSERT INTO authors VALUES ('172-32-1176', 'White', ...)", connection);
    try
    {
        connection.Open();
        try
        {
            command.ExecuteNonQuery();
        }
        catch(SqlException sqle)
        {
            if (sqle.Number == 2627)
            {
                return false;
            }
            else
            {
                throw;
            }
        }
    }
    finally
    {
        connection.Close();
    }

    return true;
}