Question

This page a user must choose between one of 2 checkboxes 5 times. So I wrote this:

        if (box1a.isSelected() == true || box1b.isSelected() == true) {
            if (box2a.isSelected() == true || box2b.isSelected() == true) {
                if (box3a.isSelected() == true || box3b.isSelected() == true) {
                    if (box4a.isSelected() == true || box4b.isSelected() == true) {
                        if (box5a.isSelected() == true || box5b.isSelected() == true) {


                                 with some other things he does when it is true.


        } else {
            new Error("You must select an answer at all the questions");
        }

Then he only returns a error if you don’t check one of the top checkboxes. Then cleary I need a while loop in there but i don’t know how to uhm do it. I know how a while loop works but don’t know how It would look in this situation. Please help

Also now I have to do the same with text fields and using th same methode that I got answered by you guys doesn’t work. any advise?

Answer 1

if ((box1a.isSelected() || box1b.isSelected()) &&
   (box2a.isSelected() || box2b.isSelected())  &&
   (box3a.isSelected() || box3b.isSelected())  &&
   (box4a.isSelected() || box4b.isSelected())  &&
   (box5a.isSelected() || box5b.isSelected())) 
   {
      //true stuff
   }
   else 
   {
       new Error("You must select an answer at all the questions");
   }

You should never shouldn’t test for true with ==. It is poor style, better to just use the return value from isSelected()