質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.13%

Pythonでの割り込みで時間取得をする。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,377

nagog

score 14

前提・実現したいこと

Raspberry 3 でpython を使っています。
GPIOの外部信号で立ち上がりを検出し、そのときの時間を計測し、
これを繰り返してルーチンの時間を出そうとしています。

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

下記エラーが発生しました。
エラーメッセージ
UnboundLocalError: local variable 'today_now' referenced before assignment

該当のソースコード

python2.7
'#!/usr/bin/python
'# -*- coding: utf-8 -*-

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

def sw_on(channel17):
today_before = today_now
today_now = datetime.datetime.today()
timedelta = today_now - today_before
sig = timedelta

today_now = datetime.datetime.today()
today_before = today_now

channel17= 17
standard_time = 600
counter = 0

GPIO.setmode(GPIO.BCM)
GPIO.setup(4, GPIO.IN,pull_up_down=GPIO.PUD_UP)
GPIO.setup(channel17,GPIO.IN)
GPIO.setup(21, GPIO.OUT)
GPIO.add_event_detect(channel17 , GPIO.RISING, callback=sw_on, bouncetime=300) 

ser= serial.Serial('/dev/ttyUSB0', 115200)
serial.PARITY_EVEN
serial.EIGHTBITS
serial.STOPBITS_TWO

sig1 = 'a2234567890123'
sig2 = 'a3876543210987'

time.sleep(2)
while True:

g4 = GPIO.input(4)
if  g4 == 0:
ser.write(sig)
else:
ser.write(sig2 )
time.sleep(2)
GPIO.cleanup()
ser.close()

上記エラーの後、対策として ローカルの中をmain.today_now に変更しましたが別のエラーがでました。
NameError: global name 'main' is not defined

このプログラムはどこを直すべきなのかお分かりの方よろしくお願いいたします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

today_before = today_now


today_nowという変数は、ここで初めて出てきます。当然、値は設定されていません。
なのに、today_beforeに代入するためにtoday_nowの値を参照しようとしています。

これが問題点。

上記のコードより前に、today_nowに何らかの値を代入すればエラーは消えます。

<参考>
UnboundLocalError: local variable 'today_now' referenced before assignment
を訳すと
未代入ローカル変数のエラー:ローカル変数'today_now'は、値が代入される前に参照されている

そんなに難しい英語ではないので、エラーメッセージを読んでみるほうが解決が早いです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/08/26 06:56 編集

    nagogさん
    時間の差分をとっていきたい、ということと思います。

    coco_bauerさんがおっしゃるように、定義されていない値を定義前に読みだしてこけているので、解決策は「(変数も中身も)先に定義しておくことでエラーを避ける」「(変数が定義されていて)中身が空っぽならすっ飛ばす」「とりあえずやってみて、エラーが起きるならすっ飛ばす」どれかになると思います。

    # ifで分岐させる例
    # この場合はグローバル変数か引数で先に today_beforeを定義しておかないと、
    # 値が空っぽかどうか判定する処理でこけます。
    if today_before =="":
    空だった時の処理(比較しないなど)
    else:
    今まで通りの処理
    today_before = today_now

    ---
    大元のコードがインデントがなくて何とも言えませんが、today_beforeとtoday_nowはグローバル変数にするか、引数で別の関数から値をもらえるようにしないとdef sw_on(channel17)内で常に同じ処理になってしまいます。

    キャンセル

  • 2017/08/28 10:59

    大変助かりました。
    if文での分岐で実施できました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.13%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる