If-Else-IfループよりもRadioButtonListを読み取る良い方法はありますか？

Question

大きなRadioButtonList（約20項目）があり、送信された電子メール（電子メール形式）でその値を返すために選択されたものを確認する必要があります。

今私が実行している大規模な場合は、そうでない場合、ループは1が選択されたかを調べるために、それぞれのRadioButtonを解析するために：

  if (PriMsg_AP_1.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_AP_1.Text; 
      } 
      else if (PriMsg_AP_2.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_AP_2.Text; 
      } 
      else if (PriMsg_Devices_1.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_Devices_1.Text; 
      } 
      else if (PriMsg_SMB_1.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_SMB_1.Text; 
      } 
      else if (PriMsg_SMB_2.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_SMB_2.Text; 
      } 
      else if (PriMsg_SMB_3.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_SMB_3.Text; 
      } 
      else if (PriMsg_Vertical_1.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_Vertical_1.Text; 
      } 
      else if (PriMsg_Vertical_2.Checked) 
      { 
       message.Body += "<b>Primary Message:</b> " + PriMsg_Vertical_2.Text; 
      } 

など

あなたはそれがかなりだ見ることができるように長い間、私はこのリストを解析する簡単な方法があると思います。 Forループを通してすべてを実行するのと同じですか？しかし、それぞれのRadioButtonが異なる名前を持っているので、それを行う方法がわからない...

いくつかの注意点：リストは複数のカテゴリで壊れているので、すべてのラジオボタンが1つの画面に表示されるわけではないので、リスト全体をGroupNameユーザーが1つのアイテムのみを選択できること。

思考？提案？

私はこのプロジェクトでASP.NETで作業しています。

〜アレン

編集：私は、個々のラジオボタンを使用してとグループ名と一緒にそれらを結びますよ。 RadioButtonListを使用していません。混乱をおかけして申し訳ありません。

<asp:RadioButton ID="PriMsg_AP_1" GroupName="PriMsg" runat="server" text="Promo 1" CssClass="radiobutton" /> 

Answer 1

あなたはコードがたくさん短くなるだろう条件演算子を、使用することができます

message.Body += "<b>Primary Message:</b> " + 
    PriMsg_AP_1.Checked ? PriMsg_AP_1.Text : 
    PriMsg_AP_2.Checked ? PriMsg_AP_2.Text : 
    PriMsg_Devices_1.Checked ? PriMsg_Devices_1.Text : 
    PriMsg_SMB_1.Checked ? PriMsg_SMB_1.Text : 
    PriMsg_SMB_2.Checked ? PriMsg_SMB_2.Text : 
    PriMsg_SMB_3.Checked ? PriMsg_SMB_3.Text : 
    PriMsg_Vertical_1.Checked ? PriMsg_Vertical_1.Text : 
    PriMsg_Vertical_2.Checked ? PriMsg_Vertical_2.Text : 
    "No message"; 

別の方法は、あなたがそれらをループができるように、コレクション内のすべてのラジオボタンを置くことであろう：

RadioButton[] radios = new RadioButton[] { 
    PriMsg_AP_1, PriMsg_AP_2, PriMsg_Devices_1, 
    PriMsg_SMB_1, PriMsg_SMB_2, PriMsg_SMB_3, 
    PriMsg_Vertical_1, PriMsg_Vertical_2 
}; 
foreach (RadioButton radio in radios) { 
    if (radio.Checked) { 
    message.Body += "<b>Primary Message:</b> " + radio.Text; 
    break; 
    } 
} 

Answer 2

ラジオボタンリストを試すことができます。

は、ラジオボタンのテキストを選択します。

<asp:RadioButtonList runat="server" ID="rlist1"> 
     <asp:ListItem Text="a"> 
     </asp:ListItem> 
     <asp:ListItem Text="b"> 
     </asp:ListItem> 
    </asp:RadioButtonList> 

Answer 3

私はあなたが

foreach (Control c in myPanel.Controls) 
      { 
       if(c.GetType() == typeof(RadioButton)) 
       { 
        if(((RadioButton)c).Checked) 
        { 
         message.Body += "<b>Primary Message:</b> " + ((RadioButton)c).Text; 
        } 
       } 
      } 

が、これは確かに動作します、次のコードを使用して、このパネル上でそれらすべてのループを囲み、ASPパネルを置くことを示唆しているが、あなたが使用していない場合は、それを向上させることができますマスターページでは、単にmpPane.Controlsをthis.Controlsに置き換えることができます