MFC radio button - simple question

For the first radio button in group, change the group property to true and tabstop to true. For the rest of the radio buttons in the group (the 2 others in your case) set both group and tabstop to false. Now create a variable for the first radio button and you will get an integer type. Also the selection should work correctly.

i  didnt get your problem...but ive tried some codes..


ive made a new MFC dialog application(SampleApp), and then on that application ive added 3 radio buttons..
radio1 - the id is IDC_RADIO1
radio2 - the id is IDC_RADIO2
radio3 - the id is IDC_RADIO3

then on the properties of IDC_RADIO1 check the group checkbox, and dont check the group checkbox on the other radio button

then i declare it on the applications .h (SampleAppDlg.h)
CButton m_Radio1;
CButton m_Radio2;
CButton m_Radio3;

then put it on the void CSampleAppDlg::DoDataExchange(CDataExchange* pDX) function
DDX_Control(pDX, IDC_RADIO1, m_Radio1);
DDX_Control(pDX, IDC_RADIO2, m_Radio2);
DDX_Control(pDX, IDC_RADIO3, m_Radio3);

if(m_Radio1.GetCheck() == 1){
   int id = m_Radio1.GetDlgCtrlID();
}
if(m_Radio2.GetCheck() == 1){
   int id = m_Radio2.GetDlgCtrlID();
}
if(m_Radio3.GetCheck() == 1){
   int id = m_Radio3.GetDlgCtrlID();
}

i dont know if this is exactly what you need

In addition to Priyesh you also need the NEXT control in the tab order to have the group property set to true.  (I typically use a border/box for that)

As a simple alternative, you can use this solutions which uses a ListBox but with appearance of radio buttons:
http://www.codeproject.com/combobox/RadioListBox.asp

As Priyesh suggested, I set the GROUP and TABSTOP property to "true" for IDC_RADIO1, and both of those to "false" for IDC_RADIO2 and IDC_RADIO3. I right clicked on IDC_RADIO1 and created a variable (only AFTER unchecking "control variable" was I able to get "int" type available). The integer I created was m_reportType.

The results:
IDC_RADIO1 and IDC_RADIO2 both select/deselect properly, but IDC_RADIO3 appears to be part of a different group as it remains selected once I click on it (it operates independently of IDC_RADIO1 and IDC_RADIO2).

Also, the code below doesn't produce any message boxes, so it appears that the integer variable isn't getting set properly.

void CDialogName::OnBnClickedOk()
{
      if(this->m_reportType == 1)
            MessageBox(L"comp");
      if(this->m_reportType == 2)
            MessageBox(L"prog");
      if(this->m_reportType == 3)
            MessageBox(L"cust");

      OnOK();
}

I'm wondering if AndyAinscow's suggestion could solve this... but I'm not sure how to determine what control is the next one in the taborder... so I don't know how to figure out which one should have GROUP set to "true".

In looking at haroldxious' suggestion, it seems like the code haroldxious provided does what MFC should do automatically. I was under the impression that by creating an integer variable in MFC, I could reference the variable in the OK event handler without needing to check each radio button's status.

Jim
Along with that, turn the tab order on in your dialog editor and make sure that the three radio buttons are in consecutive tab order. That's all i can think about and you shouldnt have to add any code normally to get the expected mutually exclusive button states.

The mutually exclusive button states now work fine (I was able to find the tab order view finally and correct the misorder). However, it still always gives me zero (0) when I check the integer m_reportType in the OK button event handler, regardless of which radio button is selected.

I ripped out the m_reportType variable and re-inserted it... here are the steps I used:

1. In the Dialog Editor, select IDC_RADIO1.
2. Right click and select Add Variable.
3. Uncheck "control variable".
4. Select "int" from the dropdown list under Variable Type.
5. Name the variable "m_reportType".
6. Click finish.

Now, it's my understanding that when I reference this->m_reportType from the Ok button handler, it will return an integer representing the radio button ID that is selected. Or, am I missing a step?

Here's the code I have checking the variable from the button handler... and sure enough, regardless of which radio button is selected, every time it prints "zero detected" meaning that the member variable I added has a zero value.

void CDialogName::OnBnClickedOk()
{
      if(this->m_reportType == 0) MessageBox(L"zero detected.");
      OnOK();
}

Jim,
Almost there - see the following change.

void CDialogName::OnBnClickedOk()
{
      UpdateData(TRUE) ;//This transfers the control value to member var
      if(this->m_reportType == 0) MessageBox(L"zero detected.");
      OnOK();
}

I added the UpdateData(TRUE) line, but it still gives me a zero regardless of which radio button is selected. Any other suggestions to try?