巨大な 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;
}