「GPSの記録」と「人感センサでⅬチカ」の2つのプログラムを1つにまとめたい

Question

前提・実現したいこと

Pythonで、
「常にGPSモジュールからの緯度経度の情報を記録しながら、人感センサで人を検知したらLEDを点滅」
というプログラムを書いこうとしています。

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

「GPSモジュールからの緯度経度の記録」と、「センサで人を検知したらLED点滅」という2つをそれぞれ別々に書くことはできたのですが、一つのプログラムにまとめるのに、どうしたらよいのか分かりません。

下の2種類のコードを、一つにまとめて書くにはどう書けばいいのか、ご教示ください。お願いします。

該当のソースコード①

import serial
import micropyGPS
import threading
import time
import codecs

gps = micropyGPS.MicropyGPS(9, 'dd') # MicroGPSオブジェクトを生成する。
                                     # 引数はタイムゾーンの時差と出力フォーマット

def rungps(): # GPSモジュールを読み、GPSオブジェクトを更新する
    s = serial.Serial('/dev/serial0', 9600, timeout=10)
    s.readline() # 最初の1行は中途半端なデーターが読めることがあるので、捨てる
    while True:
        sentence = s.readline().decode('utf-8', errors='ignore') # GPSデーターを読み、文字列に変換する
        for x in sentence: # 読んだ文字列を解析してGPSオブジェクトにデーターを追加、更新する
            gps.update(x)

gpsthread = threading.Thread(target=rungps, args=()) # 上の関数を実行するスレッドを生成
gpsthread.daemon = True
gpsthread.start() # スレッドを起動

while True:
    if gps.clean_sentences > 5: # ちゃんとしたデーターがある程度たまったら出力する
        h = gps.timestamp[0] if gps.timestamp[0] < 24 else gps.timestamp[0] - 24
        with open('gps.txt', "a", encoding='utf-8')as f:
            print(('%2d:%02d:%04.1f' % (h, gps.timestamp[1], gps.timestamp[2])), file=f)
            print(('海抜: %f' % (gps.altitude)), file=f)
            print(('緯度経度: %2.8f, %2.8f' % (gps.latitude[0], gps.longitude[0])), file=f)
            print((''), file=f)
    time.sleep(5.0)

該当のソースコード②

import threading
import RPi.GPIO as GPIO
import sys
import os
import shutil
import time
from time import sleep


#PIN NO SET
LED1 = 16
PIR = 11

GPIO.setmode(GPIO.BOARD)
GPIO.setup(LED1, GPIO.OUT)
GPIO.setup(PIR, GPIO.IN)



#LED ON OFF Thread
def ledmain():
    global stopstr1
    stopstr1 = " "
    while True:
        sleep(1)
        if ( stopstr1 != " "):
                GPIO.cleanup()
                sys.exit()

        #LED ON OFF
        if ( GPIO.input(PIR) == GPIO.HIGH):
               led01()

#INTERRUPT Thread
def stopinput():
       global stopstr1
       

#led01
def led01():
       GPIO.output(LED1, True)
       time.sleep(0.5)
       GPIO.output(LED1, False)
       time.sleep(0.5)


#SET Thread
th=threading.Thread(target=stopinput)
th.start()
th2=threading.Thread(target=ledmain)
th2.start()

試したこと

一方のコードにもう一方を無理やりぶち込んだりしましたが、案の定うまくいきませんでした……

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

Raspberry Pi zero w の、Python3.5.3です。よろしくお願いします。

Answer 1

while True:
   A
   sleep 1

と

while True:
   B
   sleep 0.5

を

while True:
   B
   sleep 0.5
   A
   sleep 0.5
   B

とすればよいです。
さらにもうすこし工夫すれば、A, B の実行間隔をもっと汎用的にすることもできます。
(スレッドをつかって A, B をそれぞれ別のスレッド並列実行するのがベターです。
でも while で回す上の方法でも工夫次第で複数の処理を別々のインターバルでそれなりに実行させることはできます)

Answer 2

それぞれのwhile trueのナカミを関数にしてしまって、その２つの関数をwhile trueでループさせればいいだけです。
がんばってやってみましょう。やった上でなにか不具合が起きたなら、またここで質問してくれればいいかと

Comments

[KRSugar]

できました！
言われた方法になっているか分かりませんが、以下の通りやったらできました

```
import threading
import RPi.GPIO as GPIO
import sys
import os
import shutil
import time
from time import sleep
import serial
import micropyGPS
import threading
import codecs

LED1 = 16
PIR = 11

GPIO.setmode(GPIO.BOARD)
GPIO.setup(LED1, GPIO.OUT)
GPIO.setup(PIR, GPIO.IN)

gps = micropyGPS.MicropyGPS(9, 'dd')

def gpspython():
while True:
rungps()
gps()
ledmain()

def rungps(): # GPSモジュールを読み、GPSオブジェクトを更新する
s = serial.Serial('/dev/serial0', 9600, timeout=10)
s.readline() # 最初の1行は中途半端なデーターが読めることがあるので、捨てる
while True:
sentence = s.readline().decode('utf-8', errors='ignore') # GPSデーターを読み、文字列に変換する
for x in sentence: # 読んだ文字列を解析してGPSオブジェクトにデーターを追加、更新する
gps.update(x)

gpsthread = threading.Thread(target=rungps, args=()) # 上の関数を実行するスレッドを生成
gpsthread.daemon = True
gpsthread.start() # スレッドを起動

def ledmain():
global stopstr1
stopstr1 = " "
led()

def led():
while True:
sleep(1)
if ( stopstr1 != " "):
GPIO.cleanup()
sys.exit()

#LED ON OFF
if ( GPIO.input(PIR) == GPIO.HIGH):
led01()

#INTERRUPT Thread
def stopinput():
global stopstr1

#led01
def led01():
GPIO.output(LED1, True)
time.sleep(0.5)
GPIO.output(LED1, False)
time.sleep(0.5)

#SET Thread
th=threading.Thread(target=stopinput)
th.start()
th2=threading.Thread(target=ledmain)
th2.start()

while True:
if gps.clean_sentences > 5: # ちゃんとしたデーターがある程度たまったら出力する
h = gps.timestamp[0] if gps.timestamp[0] < 24 else gps.timestamp[0] - 24
with open('gps.txt', "a", encoding='utf-8')as f:
print(('%2d:%02d:%04.1f' % (h, gps.timestamp[1], gps.timestamp[2])), file=f)
print(('海抜: %f' % (gps.altitude)), file=f)
print(('緯度経度: %2.8f, %2.8f' % (gps.latitude[0], gps.longitude[0])), file=f)
print((''), file=f)
time.sleep(5.0)
```

「もっとココこうしたほうがいい」等ありましたら、よろしくお願いします。

Answer 3

上のコードのwhile True:以下も関数にまとめてスレッドとして実行するようにして、あとひとつにまとめちゃえば良いんじゃないですかね。