aruduinoから受け取った情報を記録するループにおいて、キー入力でループを終了させたい。

Question

前提

aruduinoから送られてきたデータを取得し、csvファイルとして保存するループ文を作成しました。その上で、ifで条件を付け、キー入力があった時にループを抜けるような処理にしたいです。ですが、キーを押してもループが止まりません。

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

キーを押しても反応せずループが止まりません

該当のソースコード

import serial
import keyboard

while True:
  ser = serial.Serial('COM3',9600,timeout=None)
  line = ser.readline()
  line = line.decode()
  line = line.strip('\n')
  line = str(line)
  print(line)
  with open('hot.csv','a')as f:
   writer=csv.writer(f)
   writer . writerow ({line})
  ser.close()
  if keyboard.is_pressed("enter"):
      print('記録終了です')
      break
    
f.close()

試したこと

素人のため、そもそも、このやり方でループを止められるか以下のコードを書いて調べたところ、うまくいきました。
なので、自分のパソコンにおいて、ループを
if keyboard.is_pressed("enter"):
print('記録終了です')
break
で止められることを確認しました。

import keyboard

while True:
    print('fff')
    if keyboard.is_pressed("enter"):
        print('メッセージ','処理を抜けました。')
        break

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

他に必要な情報、おかしな点がありましたら、コメントで教えてください。

Comments

[y_waiwai]

とりあえず、最後のf.closeを削除してやってみよう

[kakakatokyoeki]

変化ないです

Answer 1

ちょっとPythonの動作を把握していないので推測を含みますけれど。

タイムアウトを禁止していますから、
line = ser.readline()でserial.readline()は「改行があるまで入力を貯めて改行を受け取ったら次へ」という動作をするものです。
シリアルからどういうデータを流し込んでいるか知りませんが、何文字かに一度、改行が来るまで待って改行が来たときだけ(キー検出を含む)それ以降に進むわけですから、キー検出の機会は相当少ないことになるのではないでしょうか。
行単位ではなく、serail.in_waitingを調べて受信文字があるときだけserail.read()で一文字ずつ受信して(そうしないとserail.readでブロックされる)、行に組み直したければ自前で再構成するようなことを考えるべきかと思います。

そもそもとして、そのプログラムちゃんと動くかしら...ファイルもシリアルポートも、そんなにパタパタ開いたり閉じたりするものじゃありません。openやcloseの処理ってそれなりに重かったりもするし、ポートを閉じている間にシリアルに届いたデータは多分失われるでしょう。メインのwhileループに入る前に開いて、全てが終わったら閉じればそれで十分では。

それと、withで開いたファイルはwithの範囲を出たら自動で閉じられますから、最後のf.close()は開いていないファイルを閉じようとする行為になってエラーになりませんか?

Comments

[kakakatokyoeki]

一度リストにして、まとめて保存する方式を取りました。
かなり動作が軽くなりました。ありがとうございました