Problem with % opertaor

I'm creating a computerized version of a sort of game that maths teachers play with the class sometimes, but I have a prob. This section of the code checks whether 3 is a factor of the 'count' integer and whether 5 is a factor if it, but when it is executed, the program only reacts (by changing the contents of a child control) when it is a factor of 5 and not when it is a facor of 3. Whats wrong?

Code:

                          div_test = count % 3; 
			  
			  if(div_test == 0)
			  {
				  SetDlgItemText(hDlg, IDC_RESPONSE, "Factor of three");
			  }
			  
			  div_test = count % 5;
			  
			  if(div_test == 0)
			  {
				  SetDlgItemText(hDlg, IDC_RESPONSE, "Factor of five");
			  }
			  else
			  {
				  itoa(count, count_str, 10);
			      SetDlgItemText(hDlg, IDC_RESPONSE, count_str);
			  }

Thanks

the problem is that after your test for a factor of 3, you are always testing for a factor of 5 and then setting the text of the same edit box.

so, if after your test for a factor of 3, if the contents say "Factor of three" it'll be overwritten straight away by "Factor of Five" or whatever the value of count is... you need to do something like this:

Code:

int div5_test = count % 5;
int div3_test = count % 3;

if(div5_test == 0 && div3 == 0)
{
    // 3 and 5 are both factors as both the tests are zero
    SetDlgItemText(hDlg, IDC_RESPONSE, "Factor of three and factor of five");
}
else if(div5_test == 0)
{
    // if not both, then if just 5 is a factor
    SetDlgItemText(hDlg, IDC_RESPONSE, "Factor of five");
}
else if(div3_test == 0)
{
    // if not both then if it's just a factor of 3
    SetDlgItemText(hDlg, IDC_RESPONSE, "Factor of three");
}
else
{
    // if we get here then neither of them is a factor
    SetDlgItemInt(hDlg, IDC_RESPONSE, count, false);
}

hope this helps!
U.

thanks !

Or you could simply just add an else before the second if, and just put the condition straight in like so:

if(count % 3 == 0)
{
SetDlgItemText(hDlg, IDC_RESPONSE, "Factor of three");
}

div_test = count % 5;

else if(count % 3 == 0)
{
SetDlgItemText(hDlg, IDC_RESPONSE, "Factor of five");
}
else
{
itoa(count, count_str, 10);
SetDlgItemText(hDlg, IDC_RESPONSE, count_str);
}