As the others already mentioned, it is about personal taste. But maybe more: it is how the people that maintain the code prefer to read it. If you are the only one, do what you prefer best. If it is a team, please follow the guidelines of the team and be consistent in that.

I never use the func () if $expr; syntax, because that is - for my mind - exactly the wrong way around.

I also never use && and || to do so, because it then doesn't differ enough with expressions: