Pygameとゲームパッドを用いた値の変更(ボタン長押しによって値が増減し続けるプログラム)

Question

実現したいこと

• Pygameとゲームパッドを用いた値の変更(ボタン長押しによって値が増減し続ける)

前提

Pythonで、ゲームパッドのボタン入力によって値が増減するプログラムを作成しております。
Pygameというモジュールを使用し、ゲームパッドのボタン入力(押した or 離した)を取得します。

下記コードで動作しないわけではないのですが、動作が不安定なため改善したいと考えております。
初めての質問となりますが、どうぞよろしくお願いいたします。

発生している問題・エラーメッセージ

• 動作が安定しません。ボタンを押した際、処理が始まる場合とそうでない場合があります。

該当のソースコード

import pygame
from pygame.locals import *
import threading
import time

#Pygame関連の初期化
pygame.init()
pygame.joystick.init()
joys = pygame.joystick.Joystick(0)
joys.init() #ゲームパッドの初期化


#ボタンを押した際に取得するe.buttonの値
"""
LBボタン：4
RBボタン：5
"""

n1 = 512 #変更する値の初期値
switch = 0 #ボタンから指が離れた状態の値を定義

#ボタンを押したか離したかの判定
def OnOff():
    while True:
        global switch
        for e in pygame.event.get():
            if e.type == pygame.locals.JOYBUTTONDOWN: #ボタンが押された場合
                switch = 1
            if e.type == pygame.locals.JOYBUTTONUP: #ボタンから指が離れた場合
                switch = 0

#0.5秒ごとに値をターミナルに表示
def Display():
    while True:
        print('現在値:{}'.format(n1)）
        time.sleep(0.5)

#ボタン押し離しの判定と現在値表示の開始
Th1 = threading.Thread(target=OnOff)
Th2 = threading.Thread(target=Display)
Th1.start()
Th2.start()

#ボタン入力による値の変更
while True:
   for e in pygame.event.get():
      if e.type == pygame.locals.JOYBUTTONDOWN:
         pushed = int(e.button)

         if pushed == 4:
            while switch == 1: #ボタンから指が離れるまで繰り返す
                n1 += -1
                if n1 < 0:
                    n1 = 0 #下限値の設定
                time.sleep(0.2) #0.2秒ごとに1減少

         if pushed == 5:
            while switch == 1: #ボタンから指が離れるまで繰り返す
                n1 += 1
                if n1 > 1023:
                    n1 = 1023 #上限値の設定
                time.sleep(0.2) #0.2秒ごとに1増加

試したこと

ボタン入力が受付けられる場合とそうでない場合があるのではと考え、以下の内容を試しましたが、改善できませんでした。
・値の増減を繰り返すtime.sleep(0.2)の前後にゲームパッド初期化(joys.init())を入れる
・各処理へのtime.sleep()の追加、および値の変更

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

Pythonのバージョン : Python 3.5.6
OS : Ubuntu 22.04 LTS 64ビット
メモリ : 16GB
プロセッサ : Intel® Core™ i7-10750H CPU @ 2.60GHz × 12

Answer 1

pygameは使ったことがないのですが、イベントの扱いに問題があるように思います。

pygame.event.get() はその時点で発生していたイベントを順に取りだすメソッドですが、これを メインループとOnOff()関数の両方で呼んでいます。
この実装だと、あるイベントはどちらかでしか取得できないはずなので、ボタン入力がどちらに入ったかによって受け付けられるかどうかが変っているのでしょう。

こういうイベント処理系のプログラムは、メインループでイベントの取得を行ない、イベントによって動作を振り分けるつくりにするのが一般的です。

Comments

[batakel]

TakaiY様

ご回答いただきありがとうございました。解決しました。
ご指摘いただいた通り、pygame.event.get()をメインループとOnOff()関数の両方で使用しているのが原因だったようです。
お書きいただいた「メインループでイベントの取得を行ない、イベントによって動作を振り分けるつくりにする」という内容とは異なるかもしれませんが、下記のコードに修正したところ動作が安定しました。
本当にありがとうございました。
---

import pygame
from pygame.locals import *
import threading
import time

#Pygame関連の初期化
pygame.init()
pygame.joystick.init()
joys = pygame.joystick.Joystick(0)
joys.init() #ゲームパッドの初期化

#ボタンを押した際に取得するe.buttonの値
"""
LBボタン：4
RBボタン：5
"""

n1 = 512 #変更する値の初期値
switch = 0 #ボタンから指が離れた状態の値を定義

#ボタンを押したか離したかの判定
def OnOff():
while True:
global switch
global e #追加箇所
for e in pygame.event.get():
if e.type == pygame.locals.JOYBUTTONDOWN: #ボタンが押された場合
switch = 1
if e.type == pygame.locals.JOYBUTTONUP: #ボタンから指が離れた場合
switch = 0

#0.5秒ごとに値をターミナルに表示
def Display():
while True:
print('現在値:{}'.format(n1)）
time.sleep(0.5)

#ボタン押し離しの判定と現在値表示の開始
Th1 = threading.Thread(target=OnOff)
Th2 = threading.Thread(target=Display)
Th1.start()
Th2.start()

#ボタン入力による値の変更
while True:
#for e in pygame.event.get():　#削除
if switch == 1: #修正
pushed = int(e.button)

if pushed == 4:
while switch == 1:
n1 += -1
if n1 < 0:
n1 = 0
time.sleep(0.2)
if pushed == 5:
while switch == 1:
n1 += 1
if n1 > 1023:
n1 = 1023
time.sleep(0.2)

Answer 2

TakaiYさんのご指摘をもとに修正・解決したコードとなります。

import pygame
from pygame.locals import *
import threading
import time

#Pygame関連の初期化
pygame.init()
pygame.joystick.init()
joys = pygame.joystick.Joystick(0)
joys.init() #ゲームパッドの初期化


#ボタンを押した際に取得するe.buttonの値
"""
LBボタン：4
RBボタン：5
"""

n1 = 512 #変更する値の初期値
switch = 0 #ボタンから指が離れた状態の値を定義

#ボタンを押したか離したかの判定
def OnOff():
    while True:
        global switch
        global e #追加箇所
        for e in pygame.event.get():
            if e.type == pygame.locals.JOYBUTTONDOWN: #ボタンが押された場合
                switch = 1
            if e.type == pygame.locals.JOYBUTTONUP: #ボタンから指が離れた場合
                switch = 0

#0.5秒ごとに値をターミナルに表示
def Display():
    while True:
        print('現在値:{}'.format(n1)）
        time.sleep(0.5)

#ボタン押し離しの判定と現在値表示の開始
Th1 = threading.Thread(target=OnOff)
Th2 = threading.Thread(target=Display)
Th1.start()
Th2.start()

#ボタン入力による値の変更
while True:
    #for e in pygame.event.get():　#削除
    if switch == 1: #修正
        pushed = int(e.button)

        if pushed == 4:
            while switch == 1:
                n1 += -1
                if n1 < 0:
                    n1 = 0
                time.sleep(0.2)
        if pushed == 5:
            while switch == 1:
                n1 += 1
                if n1 > 1023:
                    n1 = 1023
                time.sleep(0.2)

Answer 3

TakaiY様

ご回答いただきありがとうございました。解決しました。
ご指摘いただいた通り、pygame.event.get()をメインループとOnOff()関数の両方で使用しているのが原因だったようです。

本当にありがとうございました。