booleanを返す関数からの戻り値をもとに条件分岐をするとき、boolean変数を使うべき？

Question

何らかの処理を行って、その処理が成功したか否かをbooleanとして返す関数someFunc(arg)があるとします。
この関数を用いて呼び出し側で、不成功の場合のみエラー処理をする場合、boolean変数を使うべきなのでしょうか？

具体的には

if not someFunc(arg):
   エラー処理

is_success = someFunc(arg)

if not is_success:
   エラー処理

のどちらがいいのでしょうか？

こういうのは好み？の問題なのでしょうか？
（個人的な好みでは変数を導入するほうが読みやすいですが、この関数を呼び出し側で何度も呼び出すのであれば変数を導入するのは微妙かもなあとは思います

Answer 1

こういうのは好み？の問題なのでしょうか？

明確な運用ルールを定めているプロジェクトでない限り好き好きでしょうね。
ご指摘ある通り再利用するなら変数に取っておくほうが効率的ですが
そうでない場合でもフラグを変数におさめておくと、
検査した箇所と分岐する箇所が離れていたりする場合見やすいですし、
デバッガから確認したり条件付きのブレークポイントを設定したりするのにも
役立ちそうですね。

逆にコールバック関数に無名関数をつかうように、その場限りの利用なら
無駄に変数を増やしすぎるのもデバッグの際邪魔になるかもしれないので
状況次第としか言えないですが・・