関数を呼び出す時って他の文字に置き換えてから使えたほうがいいんですか？

例えばadd関数を呼び出してif文に入れるときの場合
ｘ＝add（ｎ）；
if（ｘ＝＝１）
みたいな感じか
if（add（ｎ）＝＝１）
どちらがいいんでしょうか？
下のほうでやっていたら友達にそのやり方は見にくくてありえないしプロがそんなことやったら確実に怒られるといわれたんですがどうなんでしょうか？