Hi Ted,

You should use only estimates().

if(estimates()) {
        // your code.
}

jobIsClosed() || !jobIsClosed() will always return true as at one given moment 
one of them will be true so anding it to estimates() doesn't make sense as if 
estimates() is false the evaluation will quit with false due to java precedence 
which is left to right.

Farrukh

On 2011-01-04, at 2:51 PM, Theodore Petrosky wrote:

> 
> 
> if (estimates() && jobIsClosed() || !jobIsClosed()) {}
> 
> I have these accessors (estimates()  jobIsClosed()). Do I need to 
> parenthesize the second part of this to be correct:
> 
> if (estimates() && (jobIsClosed() || !jobIsClosed())) {}
> 
> 
> the first example is working as I intend, but I originally had:
> 
> if (estimates() && jobIsClosed() || estimates() && !jobIsClosed()) {}
> 
> 
> I was reading and searching on Java precedence and couldn't find anything 
> that specifically answered the question. If I leave my code as the first 
> example, will it come back to bite me later? or is the second example more 
> correct?
> 
> Ted
> 
> 
> 
> _______________________________________________
> Do not post admin requests to the list. They will be ignored.
> Webobjects-dev mailing list      ([email protected])
> Help/Unsubscribe/Update your Subscription:
> http://lists.apple.com/mailman/options/webobjects-dev/farrukh.ijaz%40fuegodigitalmedia.com
> 
> This email sent to [email protected]

 _______________________________________________
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list      ([email protected])
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/webobjects-dev/archive%40mail-archive.com

This email sent to [email protected]

Reply via email to