C# 重複があったらエラーメッセージを表示させる

Question

タイトル通り、重複があったらエラーメッセージを表示させるようにするコードを
現在書いているのですが、思うようにいかないのです。
やりたいことは、主キーとなっている担当者コードとnumeric_担当者コード.Valueで指定した数値が一致したときに
エラーメッセージを表示させるということです。
SQL文でCOUNT とDISTINCTを使っているのですが、
そのためにデータグリッドビューのデータにあるコードとnumeric.valueの値が一致したときに
エラーメッセージを出すようにしたいのにstaff_idはデータ内の個数を取ってくるため、一致させることができません。
以下、コードになります。

private void button1_Click(object sender, EventArgs e)
        {
            // 登録ボタン押下時の処理
            // 重複のない担当者コードを取る
            command.CommandText = "SELECT COUNT(DISTINCT 担当者コード) AS 担当者コード FROM 担当者マスタ";
            command.Connection = common.connection;
            SqlDataReader reader = command.ExecuteReader();
            while (reader.Read())
            {
                staff_id = (int)reader.GetValue(reader.GetOrdinal("担当者コード"));
            }
            reader.Close();
            // 半角英数記号のチェック
            System.Text.RegularExpressions.Regex r = new System.Text.RegularExpressions.Regex("^[a-zA-Z0-9!-/:-@¥[-`{-~]+$");

            if (txt_担当者名.Text == "")
            {// 省略   }

            else if (staff_id == numeric_担当者コード.Value)
            {
                //メッセージボックスを表示する
                DialogResult result = MessageBox.Show("担当者コードが重複しています。", "エラー",
                    MessageBoxButtons.OK,
                    MessageBoxIcon.Error);
            }
            else if (!r.IsMatch(txt_メールアドレス.Text) && !r.IsMatch(txt_メールアドレス.Text))
            {
               // 省略
            }
            else
            {
              // 省略
            }

Comments

[coco_bauer]

「numericUpDownで指定した数値」とありますが、コードにUpDownとの語は含まれていません。質問とコードの内容は、ちゃんと関連が判るようにしてください。

Answer 1

select 担当者コード from 担当者マスタ where 担当者コード in (select 担当者コード from 担当者マスタ group by 担当者コード having count(*) > 1)

Comments

[退会済みユーザー]

having を使って実行してみたところ、うまくいきました。
ありがとうございました。

[Zuishin]

この SQL 文だと重複のあるすべての担当者コードが抽出されますので、重複が複数ある場合には抽出された最初のものが望む担当者のものとは限りません。
お書きになったソースでは、最初のデータが求めるデータと決め打ちされていますが、これだとうまくいかない場合もあることにお気を付けください。

TAKA_0921 さんのご指摘のように、SQL 文の中に求める担当者コードを含めるか、あるいは得られたデータを使ってループすることをお勧めします。

Answer 2

担当者マスタから全件取得した後にクライアント側で重複があるかをチェックするのではなく、
NumericUpDown.Valueが既に担当者マスタに存在するかを問い合わせるSQLを記述するのはいかがでしょうか？

SELECT COUNT(担当者コード) AS "COUNT" FROM 担当者マスタ WHERE 担当者コード = :CODE

例えばOracleだったらこんな感じで、COUNT > 0の場合は重複があると考えられると思います。

Comments

[退会済みユーザー]

こちらの方でも試してみて、numeric_担当者コード.Valueのレコード数を取得したのちにif文でレコード数のチェックを行うことで解決となりました。
自分のコードではこちらの方が適しているように思いました。
ありがとうございました。