Arduinoプログラムについての質問

Question

前提・実現したいこと

ボタンを押すと、LEDが点灯するArduinoのプログラムを作ろうとしています。

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

ボタンを押したとき、点灯しないことがあります。

エラーメッセージ

該当のソースコード

const int LED=13;
const int BUTTON=7;
int val=0;
int old_val=0;
int state=0;

void setup(){
pinMode(LED,OUTPUT);
pinMode(BUTTON,INPUT);
}
void loop(){
val=digitalRead(BUTTON);
if((val==HIGH)&&(old_val==LOW)){
state=1-state;
delay(10);
}

old_val=val;

if(state==1){
digitalWrite(LED,HIGH);
}else{
digitalWrite(LED,LOW);
}
}

試したこと

配線とボタンはすべて交換したためプログラムの問題だと思われます。

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

ここにより詳細な情報を記載してください。

Comments

[ikadzuchi]

点灯しない時、一瞬たりとも点灯しませんか?
チャタリングが原因ならdelay(10)の間点灯するのが見えると思います。

Answer 1

チャタリングは、離した時にも起こります。
delay(10);
はループ最後（つまり、サンプリング間隔を10msにする）に移動すれば直ると思います。

Answer 2

「点灯しないことがある」というのは「点灯することもある」と理解します

ということは、配線やスイッチのロジックの問題では無いと理解します

y_waiwaiさんが仰っている通り、チャタリングの問題かと思いました

スイッチによっては 20～30msecの間ばたつくものもあるようなので、もう少しdelayを長めに取ってみるのも一手かと思います

あと、このコードだと HIGH と LOW を入れ替えた後、またすぐにスイッチの検出に行ってしまうような気がします

そうすると、長めに押してしまった場合、せっかく HIGH になってもまたすぐに LOW になることもあるかもしれません

一旦、LEDを切り替えたら2～3secくらい待って（delay）、上のチャタリングを解決するために長くしたdelay分、気持ち長めにスイッチを押してみてはいかがでしょうか？

違ったらごめんなさい

Comments

[yamamoto99]

回答ありがとうございます。教えていただいたとおりにdelayを伸ばし長めにスイッチを押す方法を試してみましたが、点灯しない割合はあまり変わりませんでした。

[token]

delayの後でもう一度if文を入れてスイッチの検出をしないと、折角待ったのに意味がないかもしれません
２度目のifの後でstateを変えてみてはいかがでしょうか

また、蛇足ですが、LEDをHIGHに切り替えた後、すぐにループに戻ってスイッチの検出をしてしまうので、スイッチを押している時間が長いと、一旦HIGHになっても、またすぐLOW に戻っちゃいますので、結果として人の目には点灯したことがわかりません

なのでLEDの切り替えをしたら、そこでも少し待ってみてはいかがでしょうか

Answer 3

スイッチにはチャタリングというものがあります
ONからOFF、OFFからONになった場合に短時間のあいだにON/OFFを繰り返す、というもんです。
おそらくそこらへんのせいで、OFF-ONのエッジが２回カウントされたんでしょう

この場合、スイッチの状態を見る場合、１０ｍｓ程度の間隔で２回スイッチの状態を見て、違う場合にはその結果を捨て、同一の場合のみ、その状態を確定として扱うようにします

---

って、ちょっと気がついたけど、スイッチのとこに抵抗つけてます？ｗ

Comments

[yamamoto99]

回答ありがとうございます。
10kΩの抵抗をつけています。