raspberry piを使ってボタンが押されたらランダムで音楽再生される方法

Question

raspberry piを使ってボタンが押されたらリストの中の音楽がランダムで再生される物を作っています。

現在下記のコードでボタンが押されたらリストの中の音楽がランダムで再生されるところまでは行ったのですが、続けてボタンを押してしまうと重ねて音楽が再生されてしまいます。

プログラミングを習いたてでまだ応用が効かずこちらに質問させていただきました。

#!/usr/local/bin/python
# -*- coding: utf-8 -*-

from __future__ import print_function

import RPi.GPIO as GPIO
import os
import time
import random
import shlex
import subprocess

# Pin Number
PIN = 19

GPIO.setmode(GPIO.BCM)
GPIO.setup(PIN, GPIO.IN, pull_up_down=GPIO.PUD_UP)
last_pin_status = 0

SONG_DIR = '/home/pi/Music/'
SONG_LIST = ['song1.wav', 'song2.wav']


def play_song():
    song = random.choice(SONG_LIST)
    song_path = os.path.join(SONG_DIR, song)
    command = 'aplay %s' % (song_path)
    print(command)
    subprocess.Popen(shlex.split(command))


while True:
    pin_status = GPIO.input(PIN)
    if last_pin_status == 1 and pin_status == 0:
        play_song()

    last_pin_status = pin_status
    time.sleep(0.1)

GPIO.cleanup()

再度ボタンが押されたら

1 再生している音楽を停止する
２ リストの中からランダムで再生する

という２つの条件を入れれば良いと考えて、まずボタンが押されたら音楽を停止するために
if last_pin_status == 1 and pin_status == 0:
play_song()
の後に
if last_pin_status == 1 and pin_status == 0:
stop_song()
を入れました。

 if last_pin_status == 1 and pin_status == 0:
        play_song()
        if last_pin_status == 1 and pin_status == 0:
            stop_song()

こちらで試したところ、停止するどころか、重ねて音楽も再生されませんでした。

どこが問題なのでしょうか？

---

def stop_songを定義していませんでした。

Comments

[t_obara]

ちなみに、停止する機能はきちんと機能しているのかは確認ができているのでしょうか？ こちらのURLなどが参考になるかと思いますが http://shinogasa.hatenablog.com/entry/2016/01/07/112738

Answer 1

ボタンの状態を保持しているステータスはありますが、
曲の再生状況を保持しているステータスが無いので現在の再生状況がわからないのが問題でしょうね。

Python、殆ど書いたことないんですけど、これでイケるんじゃないかと思います。
（一応確認ですが、def stop_song() は定義してあるんですよね？）

last_pin_status = 0
is_playing      = False  # ← 再生ステータス（bool型）

      :

while True:
    pin_status = GPIO.input(PIN)

    if last_pin_status == 1 and pin_status == 0:

        if is_playing:    # 再生ステータスの検査
            stop_song()

        else:
            play_song()
        
        is_playing = not(is_playing)    # 再生ステータスの反転

    last_pin_status = pin_status
    time.sleep(0.1)

GPIO.cleanup()

Comments

[退会済みユーザー]

subprocess.Popen() って、新しいプロセスを立ち上げるインターフェイスみたいなので、
旧コードの場合、ボタン押して離される度にどんどん複数の新プロセスが立ち上がってそのプロセスごとに曲を再生してしまう、って事でしょうね、きっと。

[退会済みユーザー]

だから、stop_song() 内で、さっき立ち上げた別プロセスを終了する方法を知っているのかな？って事が次に問題になりそうな気がしています。

[退会済みユーザー]

夜中に起きたので書いた回答ですが、「停止するどころか、重ねて音楽も再生“されません”でした。」だったんですね。で、コードを読み直してみたら、当然そうなりますね。last_pin_status と pin_status の値が変わる前に play_song() して stop_song() していますから。
そうすると、最低限 stop_song() は記述され機能していることを期待してよさそうですね。

[MasahikoHirata]

丁寧な回答お疲れ様です。非常に良いと考えます。