🚀 go-pugleaf

"Dillan Fernando" <dillan@geocities.com> writes:

> public boolean dummy()
> {    for(a=0;a<x;a++)
>     {    if (a==b) return true
>     }
> return false;
> }

Procedural programmers will balk at it and mumble something about
"single exit point": to please them, rewrite it like this:

public boolean dummy() {
  boolean retVal = false;
  loop: for (int a = 0; a < x; a++) {
    if (a == b) {
      retVal = true;
      break loop;
    }
  }
  return retVal;
} 
  
Note: The "loop" label here is redundant, and only there for
readability purposes.

(I assume your sample code was just an illiustration: The method below
is a much faster version:

 public void dummy() {
   return ((b >= 0) && (b < x));
 }

)