I find it readable if you're trying to test both sides of the && to be true -- if(foo() && bar()){ // do this }
However, the way it's written in the article, the intent is to merely test foo() and then execute bar() if foo() returns true. Even though I'm familiar with &&, it took me a minute to understand this as well. I'd be eqaully annoyed seeing this in production code.