Python OSコマンド処理実行後のreturn値が返ってこない

Question

閲覧ありがとうございます。

windowsのコマンドを使用したプログラムを作成しています。
外部関数にOSコマンドを使用した処理の後にreturn値を設定しているのですが
呼び出し側にreturn値が返ってこなくて困っています。

ご回答よろしくお願いいたします。

def func1():
    def func2():
        OS_Command = subprocess.Popen('shutdown /i', shell=True, universal_newlines=True)
        print(OS_Command)
        
        return OS_Command

def main():
    print (func2())

Answer 1

他の方が指摘されているコーディングの間違いを直した上でのことですが、

subprocess.Popen()はPopenクラスのインスタンスを返すので、これだけでは外部コマンドの終了コードは取れません。これを使うのならだいたい次のようにします。

OS_Command = subprocess.Popen(['shutdown', '/i'])
OS_Command.wait()
return OS_Command.returncode

subprocess.call()を使う簡便な方法もあります。

return subprocess.call(['shutdown', '/i'])

なお、どちらの例も引数にコマンドラインではなく、引数のリストを与えています。コマンドラインが定数文字列であれば関係ないですが、動的に変わる場合のコマンド注入攻撃を未然に防ぐために、引数リストの形式が推奨されています。またそういうわけで、コマンドラインを文字列としてしか与えられないos.system()は、もはや推奨されません。

Comments

[anmitu]

回答ありがとうございます。
ちゃんと戻り値が返ってきました！
詳しく書いてくださったのでベストアンサーに選ばせていただきました。

Answer 2

import subprocess


def func():
    OS_Command = subprocess.Popen('shutdown /i', shell=True, universal_newlines=True)
    return OS_Command


def main():
    print(func())

if __name__ == '__main__':
    main()

Comments

[anmitu]

回答ありがとうございます。
ちゃんと戻り値が返ってきました！