Python Tkinterボタンからの外部アプリKillの方法教えてください。

Question

前提・実現したいこと

Raspberry pi,pythonともに初心者です。
GUIのボタンまたは、GPIOからの物理スイッチで
PDFアプリのevinceを起動と終了をしたいと思いっております。
evinceは立ち上がるんですが、終了でKillしたいのですが
方法がわかりません。
どなたか、ご指導お願い致します。

該当のソースコード

# -*- coding: utf-8 -*-

import RPi.GPIO as GPIO
import sys,os,time
import subprocess
import tkinter as tk

def STR_procss (event):
    path = ("evince")
    evince_GO = subprocess.Popen(path)

def END_procss(event):
    print("end")
    evince_GO.kill()

GPIO.setmode(GPIO.BCM)
GPIO16 = 16
GPIO21 = 21
GPIO.setup(GPIO16, GPIO.IN, pull_up_down = GPIO.PUD_UP)#プルアップ抵抗　有効
GPIO.setup(GPIO21, GPIO.IN, pull_up_down = GPIO.PUD_UP)#プルアップ抵抗　有効
GPIO.add_event_detect(GPIO16, GPIO.RISING, callback=STR_procss, bouncetime=200)
GPIO.add_event_detect(GPIO21, GPIO.RISING, callback=END_procss, bouncetime=200)

#Tkinter
root = tk.Tk()
root.title("TEST Launcher")
root.geometry("300x120+10+10")

#ボタンを作る
button1 = tk.Button(root, text="evince起動", bg="YELLOW",bd=5,font=("","14"),command= lambda : STR_procss(button1))
button1.pack(side="top")
button2 = tk.Button(root, text="evince終了", bg="RED",bd=5,font=("","14"),command= lambda : END_procss(button2))
button2.pack(side="bottom")

root.mainloop()


try:
    while True:
        time.sleep(1)
except KeyboardInterrupt:#Ctrl+Cで終了

    GPIO.remove_event_detect()
    GPIO.cleanup()

補足情報（FW/ツールのバージョンなど）

Rsapberry　Pi4
Python3

Answer 1

エラーメッセージは出てませんか？

コードを見て目に付いた部分ですが、
evince_GO はローカル変数なので、別の関数からは参照出来ません。
NameError: name 'evince_GO' is not defined と出る場合は、
STR_procss 関数内で global 宣言で対応できます。

他のエラーの場合は、質問文にエラーメッセージを記載してください。

---

グローバル変数の使いかた

#1 関数の外で初期化
evince_GO = None

#2 各関数内でグローバル宣言
# 変数に対して = で直接「代入」を行う関数でのみ宣言
# 参照のみの場合は不要です。

def STR_procss (event):
    global evince_GO

    #3 利用前に Noneである事を確認
    # 既に evince_GO がある場合、多重起動を防ぎます
    if not evince_GO:

        # XXX: グローバル変数とは別問題ですが、Popenの引数はリストにします。
        # 注意点 ("evince") はタプルにはなりません。末尾にカンマが必要です。
        # https://docs.python.org/ja/3/library/subprocess.html#subprocess.Popen
        path = ["evince"]

        evince_GO = subprocess.Popen(path)

def END_procss(event):
    global evince_GO

    #4 利用後に後始末
    if evince_GO:
        print("end")
        evince_GO.kill()
        evince_GO = None

Comments

[shootinghead]

回答有難うございます。
仰る通りエラーが「NameError: name 'evince_GO' is not defined」
小生、知識不足なのでぶしつけながら重ねて質問させてください。
関数の外でglobal宣言してそれをコード化するイメージが湧きません。
参考になるURLや考え方を御指導いただけませんか。

[teamikl]

> 関数の外でglobal宣言して

(Pythonでは) global 宣言は関数の中で行います。回答欄に追記

----
参考URL

9.2.1. スコープと名前空間の例
> 異なるスコープと名前空間がどのように参照されるか、また global および nonlocal が変数の束縛にどう影響するか、この例で実演します:
https://docs.python.org/ja/3/tutorial/classes.html#scopes-and-namespaces-example

[shootinghead]

参考URL有難うございます
global宣言の使い方が根本から間違っていました。
非常に参考になりました。

また、参考コードとおり修正したら
GUIのボタンでもevinceが起動＆終了
GPIOの物理スイッチでもevinceが起動＆終了できました。
狙った動きが出来るようになったので次のステップへ進めます。
本当に有難うございました。