関数にif文を入れた時の動作について

Question

test関数のif a == 0:がtrueの時だけtest３関数の中でtest2関数を動かすことはできますでしょうか？（test関数のprintの下にtest2関数を入れる以外で探しております。test3関数の中でtest関数の後にtest2関数をif a==0:がtrueの時だけ動作させたいです。）
ご存知の方おられましたらご教示お願いいたします。

a = 1
def test():
    if a == 0:
        print(location(), a)

def test2():
    print(location(), a)

def test3():
    test()
    test2()

test3()

理由としましては、test4, test5と今後関数が増えていく予定なので、下記のような運用ができればより管理がしやすくなるからです。
test関数は共通で、実際にはコードがもっと長いです。

def test3_2():
    test()
    test4()

def test3_3():
    test()
    test5()

from collections.abc import Callable
a = 0

def test(callback):
    assert isinstance(callback, Callable), f"`callback` is not Callable,
    if a == 0:
        print(location(), a)
        callback()

def test2():
    print(location())

def test3_2():
    print(location())
    test(test2)

test3_2()

SyntaxError: unterminated string literal (detected at line 135)

Answer 1

なにをしたいのかよーわかりませんが、

test関数のprintの下にtest2関数を入れる以外で探しております。

ということなら、そこに、test2関数の中身をコピペすればいいです。

Answer 2

def test3():
    test()
    test2()

質問は、上記の記述のままで、 test関数の内部の条件によって、test2関数が実行されたりされなかったりするようにしたいということだと理解しましたが、そのような処理にすべきではないと思います。
この記述を見て、test2が実行されるかどうかわからないということでは可読性が0です。

def test3():
    if test():
        test2()

このように、testは何らかの結果を呼び出し元に返すべきで、呼び出し元はその結果にもとづいて、test2を実施するかどうか判定するような作りにすべきです。

Answer 3

理由としましては、test4, test5と今後関数が増えていく予定

では，探している

test関数のprintの下にtest2関数を入れる以外で探しております

を無視して

test3関数の中でtest関数の後にtest2関数をif a==0:がtrueの時だけ動作させたいです。

を実現するしかありません．

from collections.abc import Callable
a = 0
def test(callback):
    assert isinstance(callback, Callable), f"`callback` is not Callable, its {type(callback)}"
    if a == 0:
        print(location(), a)
        callback()

def test2():
    print(location(), a)

def test3():
    test(test2)

def test3_2():
    test(test4)

def test3_3():
    test(test5)

test3()
test3_2()
test3_3()

test関数が共通の処理をもっており，その中のifの条件に一致した際に可変な関数の実行がしたいという要件であればCallableを渡すようにしてやればよいでしょう．
このように実装すれば，関数がtest4(), test5()と増えていっても，test()がどのCallableを受け取るかはtest3_2(), test3_3()次第なので，要件が満たされているはずです．

Comments

[fuku-chann]

ありがとうございます。
本文に修正後のコードとその結果を追記しましたが、下記コードをコメントアウトすれば大丈夫そうですが、下記コードを入れるとエラーになります。

assert isinstance(callback, Callable), f"`callback` is not Callable,

[ps_aux_grep]

ちゃんとcallbackがCallableであるかをチェックする文です．

もしAssertion Errorが出るならcallbackはCallableじゃないので，どのみちif文中のcallback()でエラーが起きます．
もしName Errorが出るなら最初のimport文が欠けています．
質問に追記されているようなエラーであればコピペミスです

確実にcallbackがCallableであるならコメントアウトなどで消しても大丈夫です．ただのチェックする構文でしかありません．

[fuku-chann]

pythonではダブルクオテーションで始まったらダブルクオテーションで終わらないといけないというルールがあるようです。下記でエラーがなくなりましたが、いかがでしょうか？

assert isinstance(callback, Callable), f"callback is not Callable"

[ps_aux_grep]

はい．それでも良いですね．
もしかして解答のコードが見切れたりしてるのでしょうか？私からはダブルクォーテーションで囲まれており，不足ないですが

[fuku-chann]

とても勉強になります。
コードが右側に隠れていたことに気が付きませんでした。
おっしゃる通り全文コピペしたらエラーがなくなりました。
大変失礼いたしました。

勉強のため、質問させていただきますが、callableかどうか事前に確認することには、どのようなメリットがあるのでしょうか？callableではない時はどのみちif文中のcallback()でエラーが出るので、エラーが出るタイミングが早くなるメリットはありますが、これだけだとimportも含めて2行増える方が気になってしまいます。現在はcallbackが確実にcallableですが、今後は確実とは言えませんし、折角教えていただきましたので、理解して適切に使い分けをしたいと思い、ご質問させていただいた次第です。

[ps_aux_grep]

Callableでない かつ if文の分岐に入らない場合では，エラーにならずチェックできないからです．もしif文等の分岐が存在せず，毎回呼ばれるのならassertは確実に冗長です．
動作確認でif文に入る/入らないという状態を確認するのはコスパが悪いですが，好みの範囲だと思います

[fuku-chann]

早急なご回答ありがとうございました。なるほどです。if文があるときはcollableであることを先に確認しておいた方が良さそうですね。とても勉強になりました。ありがとうございました!!

Answer 4

普通にifを書いてはいかがでしょうか

a = 0

def test():
    if a == 0:
        print("test1")

def test2():
    print("test2")

def test3():
    print("test3")
    test()
    if a == 0:
        test2()

test3()

Comments

[fuku-chann]

ご回答ありがとうございます。
test関数にはifが他にも複数あるので、別の方法で考えています。