While I can see some value, the problem I seem to be running into the most lately is over-abstracted code. Untangling the web of naming and redirections leading to small pieces of code or single variables becomes extremely frustrating. Maybe it comes from education in an engineering background.

I would much rather see

    return foo && bar || baz;

and a comment explaining behavior (although perhaps parens so I wouldn't have to reason about order of operations)