How to use break;

Question

Am I using the break statment and the corrosponding
peranthesis in a way that is causing a logic error.
* Return true if this set and the parameter contain
* exactly the same elements
public boolean equals(SetADT setADT)
boolean result=true;
if(this.size()!=setADT.size())
result= false;
Iterator scan=setADT.iterator();
while(scan.hasNext())
if(!(this.contains(scan.next())))
result= false;
break;
return result;
}

Answer 1

If you write the method and indent the code you'll see that the break is outside of the if statement. The code will always hit the break statement on the first pass through the while loop and exit the loop.
Instead of a break you may want to put a return statement just after setting result to false.
public boolean equals(SetADT setADT)
  boolean result=true;
  if(this.size()!=setADT.size()) { result= false; }
  Iterator scan=setADT.iterator();
  while(scan.hasNext()) {
     if(!(this.contains(scan.next()))) {
         result= false; }
     break; //Will always hit this line and exit while loop
   }//End While
  return result;
}