Pythonについて質問です

Question

Raspberry Pi 3で赤黄緑の三色の光が点灯、点滅したかを光センサーで検知し、
それぞれの色のLEDを光らせるということをやっています。
処理は起動から止めるまで続けていきます。

各色が点灯or点滅したとセンサーが反応したら、
反応した時間を保持させ、次に反応したときに
前回反応した時間との差が3秒以内だったら、
「異常」とみなしLEDを点滅させます。

また、前回との差が3秒以上開いていたら、
「正常」とみなしLEDを消灯させます。

この比較のさせかたができないため教えていただきたいです。

該当のソースコード

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

import RPi.GPIO as GPIO
import datetime
import time
import os

#sensor
Red_Pin = 24
Green_Pin = 25
Yellow_Pin = 23

#LED
Red = 13
Green = 18 
Yellow = 17

#停止時間
Wait = 0.25
#前回時間
Last_Time = [0, 0, 0]
GPIO.setmode(GPIO.BCM)

#各センサーが反応しているかどうか判断するためのリスト(0:検知 1:未検知)
value = [1, 1, 1]
color = ["Red","Green","Yellow"]

#前回時間比較用配列
Last_Light = [0, 0, 0]
colorPorts = [Red, Green, Yellow]
pinPorts = [Red_Pin, Green_Pin, Yellow_Pin]

#セットアップ
for colorPort in colorPorts:
    GPIO.setup(colorPort, GPIO.OUT)
for pinPort in pinPorts:
    GPIO.setup(pinPort, GPIO.IN)

#各種LEDを光らせる関数
def RED_LED(r):#赤
    GPIO.output(Red, r)
    time.sleep(Wait)

def GREEN_LED(g):#緑
    GPIO.output(Green, g)
    time.sleep(Wait)

def YELLOW_LED(y):#黄
    GPIO.output(Yellow, y)
    time.sleep(Wait)


#ログを取る予定のテキストファイル名を代入
filename = "log.txt"
#テキストファイルがないなら作成、あるなら一度削除してから再度作成
if (os.path.isfile("/home/pi/dev/log.txt")) == False:
    fp = open("/home/pi/dev/log.txt", "w")
else:
    os.remove(filename)
    fp = open("/home/pi/dev/log.txt", "w")

fp.write("-------------- Start --------------\n")
print("Start\n" + datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + "\n")
i = 0
try:
    while True:
        for pinPort in pinPorts:
            value[i] = GPIO.input(pinPort)
            if value[0] == 0: #光を検知
                #書き込み
                fp.write(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + ' Yellow\n')
                #点滅
                RED_LED(1)
                RED_LED(0)
                Last_Light[0] = datetime.datetime.now().strftime("%Y-%m-%d %H:%M:%S")
                if Last_Time[0] == 0:
                    Last_Time[0] = Last_Light[0]
                else:
                    Last_Time[0] = Last_Light[0]
                    value[0] = 0
                time.sleep(Wait)
                
            elif value[1] == 0:#光を検知
                #書き込み
                fp.write(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + ' Red\n')
                #点滅
                GREEN_LED(1)
                GREEN_LED(0)
                Last_Light[1] = datetime.datetime.now().strftime("%Y-%m-%d %H:%M:%S")
                if Last_Time[1] == 0:
                    Last_Time[1] = Last_Light[1]
                else:
                    Last_Time[1] = Last_Light[1]
                value[1] = 0
                time.sleep(Wait)
                   
            elif value[2] == 0: #光を検知
                #書き込み
                fp.write(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + ' Green\n')
                #点滅
                YELLOW_LED(1)
                YELLOW_LED(0)
                Last_Light[2] = datetime.datetime.now().strftime("%Y-%m-%d %H:%M:%S")
                if Last_Time[2] == 0:
                    Last_Time[2] = Last_Light[2]
                else:
                    Last_Time[2] = Last_Light[2]
                value[2] = 0
                time.sleep(Wait)
                
            elif value[i] == 1: #光を検知してないとき
                #書き込み
                fp.write(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + ' \n')
                GREEN_LED(0)
                RED_LED(0)
                YELLOW_LED(0)
                time.sleep(Wait)
                Last_Time[i] = Last_Light[i]
            i += 1 #カウントアップ
                
        if i == 3: #3秒間に1回でもセンサが反応したかチェック
            fp.write("-------------------------------\n")
            i = 0 #value[]の中の値を参照するため現在の位置を最初に戻す
            for i in range(3): #value[]の参照
                if value[0] == 0: #value[]の中で１つでも0があったら「異常」と表示してループを抜ける
                    print("LastTime " , Last_Time[0], "\n")
                    print(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + " Yellow 異常\n")
                    time.sleep(Wait)
                    break
                elif value[1] == 0:
                    print("LastTime " , Last_Time[1], "\n")
                    print(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + " Red 異常\n")
                    time.sleep(Wait)
                    break
                elif value[2] == 0:
                    print("LastTime " , Last_Time[2], "\n")
                    print(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + " Green 異常\n")
                    time.sleep(Wait)
                    break
                elif value[i] == 1:
                    print(datetime.datetime.today().strftime("%Y-%m-%d %H:%M:%S") + " \n")
                    i += 1
            print("--------------------------------------\n")
            i = 0
except KeyboardInterrupt:
    fp.write("--------------- End ---------------\n")
    fp.close()
    print("End")
finally:
    GPIO.cleanup()

試したこと

比較させるためにこんな感じの文を書いてみましたが
(value[0] == 0) & (Last_Light[0] - Last_Time[0]) < 3:

unsupported operand type(s) for -: 'str' and 'str'
というようなエラーが出たため訳がわからなくなりました・・・

Comments

[coco_bauer]

「unsupported operand type(s) for -: 'str' and 'str'」というエラーは、質問に書かれているコードの何処で発生するのですか？

[rgmk]

このコードには書いていませんが、前回反応した時間と
新しく反応した時間の差を比較する文を

(value[0] == 0) & (Last_Light[0] - Last_Time[0]) < 3:

と書いて実行してみたときに出てきたエラーです

Answer 1

Last_LightとLast_Timeはどういう変数なのか、なにを入れるべきなのかということをはっきりさせる必要があります

数値を入れたかと思えば文字列を入れてたりして混乱していますね

文字列が入ってると、引き算はできません（エラーはそれを指摘してます）

Comments

[rgmk]

Last_Lightは新しく反応した時間を一時的に時間を格納しておくための変数で、
Last_Timeは前回反応した時間を格納しておくための変数です。

なので(Last_Light[0] - Last_Time[0]) < 3
という比較をしていました。

エラー文の意味はなんとなくはわかりましたが、
そこから、取得した「時間」を文字から数値に変換させる
ということが思い浮かばなかったです。

[y_waiwai]

> Last_Light[0] = datetime.datetime.now().strftime("%Y-%m-%d %H:%M:%S")

現在日時をわざわざ文字列にして代入してますね
datetimeにしたいなら以下ですが、差分を求めるのはまた別の処理が必要になります
Last_Light[0] = datetime.datetime.now()

「python datetime 差分」でぐぐるとやりかたでてきます