duh!!! as soon as I read the response I realized my error.
Thanks Ted --- On Tue, 1/4/11, Farrukh Ijaz <[email protected]> wrote: > From: Farrukh Ijaz <[email protected]> > Subject: Re: a simple Java question > To: "Theodore Petrosky" <[email protected]> > Cc: [email protected] > Date: Tuesday, January 4, 2011, 7:01 AM > 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]
