Path: csiph.com!news.swapon.de!eternal-september.org!feeder.eternal-september.org!news.eternal-september.org!.POSTED!not-for-mail From: Andreas Leitgeb Newsgroups: comp.lang.java.programmer Subject: Re: I am unable to run my Bluej Code Date: Sat, 14 Jan 2017 20:23:28 -0000 (UTC) Organization: A noiseless patient Spider Lines: 17 Message-ID: References: <7bbab5d9-5eca-406b-bd0d-bb7caa28525a@googlegroups.com> Reply-To: avl@logic.at Injection-Date: Sat, 14 Jan 2017 20:23:28 -0000 (UTC) Injection-Info: mx02.eternal-september.org; posting-host="b25bc1c2b925ca07c60e294da0b786ef"; logging-data="7873"; mail-complaints-to="abuse@eternal-september.org"; posting-account="U2FsdGVkX19Yb8oJMo+utZNqSE9Mj99a" User-Agent: slrn/1.0.2 (Linux) Cancel-Lock: sha1:grO01yBzc9a4ajgitZfH7qZuQK4= Xref: csiph.com comp.lang.java.programmer:36193 lewbloch@gmail.com wrote: > On Friday, January 13, 2017 at 7:16:32 PM UTC-8, Yesi Gebre wrote: >> class ChkNum{ >> //return true if x is even >> boolean isEven(int x){ >> if((x % 2) == 0) return true; >> else return false; >> } >> } > `(x % 2) == 0` is already a boolean expression, so making it > a conditional to turn it into a boolean is rather gilding the > lily, wouldn't you say? I wouldn't say. OP's way makes it easier to add logging code lateron. Even better would be, if the statements were even "embraced".