1

巨大な if-else-if チェーン (+20 else-ifs) を置き換えるために、C# で多くの変数を大文字と小文字を切り替えて (値が null または空かどうかをチェックする) 方法。コードを Switch ステートメントに置き換えて、より効率的なコードを減らすにはどうすればよいですか。

        con.Open();
        SqlCommand cmd = new SqlCommand("usp_AddPublisher", con);
        cmd.CommandText = "usp_AddPublisher";
        cmd.CommandType = CommandType.StoredProcedure;
        if (String.IsNullOrEmpty(data.Account_Num))
        {
            cmd.Parameters.AddWithValue("@AccountNum", "");
        }
        else
            cmd.Parameters.AddWithValue("@AccountNum", data.Account_Num);
        if (String.IsNullOrEmpty(data.publisher))
        {
            cmd.Parameters.AddWithValue("@publisher", "");
        }
        else
            cmd.Parameters.AddWithValue("@publisher", data.publisher);
        if (String.IsNullOrEmpty(data.addr1))
        {
            cmd.Parameters.AddWithValue("@addr1", "");
        }
        else
            cmd.Parameters.AddWithValue("@addr1", data.addr1);

試してみましたが、長すぎます:

  con.Open();
        SqlCommand cmd = new SqlCommand("usp_AddPublisher", con);
        cmd.CommandText = "usp_AddPublisher";
        cmd.CommandType = CommandType.StoredProcedure;
switch (String.IsNullOrEmpty(data.Account_Num))
        {
            case true:
                cmd.Parameters.AddWithValue("@AccountNum", "");
                break;
            case false:
                cmd.Parameters.AddWithValue("@AccountNum", data.Account_Num);
                break;

        }
  switch (String.IsNullOrEmpty(data.publisher))
            {
                case true:
                    cmd.Parameters.AddWithValue("@publisher", "");
                    break;
                case false:
                    cmd.Parameters.AddWithValue("@publisher", data.publisher);
                    break;
            }
            switch (String.IsNullOrEmpty(data.addr1))
            {
                case true:
                    cmd.Parameters.AddWithValue("@addr1", "");
                    break;
                case false:
                    cmd.Parameters.AddWithValue("@addr1", data.addr1);
                    break;
            }
4

5 に答える 5