人感センサーによるdelayではなくmillisで処理したい

前提・実現したいこと

人感センサーによるLチカの処理でdelayではなくmillisで処理をしたいですが、
人感センサーによるセンサーがLOWになっても、LED側がLOWにならないです。アドバイス、解決方法あれば教えていただけますでしょうか？

該当のソースコード

#define LED_OUT 25
#define PIR_IN 14

int LedState = 0;
unsigned long previousMillis = 0;

void setup() {
pinMode(LED_OUT, OUTPUT);
pinMode(PIR_IN, INPUT);
}

void loop()
{
if(digitalRead(PIR_IN)==HIGH){
unsigned long currentMillis = millis();

if(currentMillis - previousMillis >= 100) {
if (LedState == 0) {
digitalWrite(LED_OUT, HIGH);
LedState = 1;
}
else {
digitalWrite(LED_OUT, LOW);
LedState = 0;
}
previousMillis = currentMillis;
}
}
}

行動規範の内容に同意します

回答2件

グローバル変数でタイマ変数を定義、０で初期化
そのうえで、loop関数の動作を

人感センサがOFFの場合、３に移行
LEDをONとし、millis関数で取得する値からLEDをOFFする時間を算出して、タイマ変数に設定、この時にタイマ変数が０なら＋１しておく
タイマ変数が０なら何もしないで終了
millis関数で現在時を読み出し、タイマ変数と比較してOFF時間に到達しているかを判定
OFF時間に到達してないなら何もしないで終了
LEDをOFFとして、タイマ変数を０にクリア

と、いうことで。

投稿2020/10/19 06:48

編集2020/10/19 06:59

y_waiwai

総合スコア87749

退会済みユーザー

2020/10/20 06:52

すいません。。。ちょっと理解がわからないです。。。

退会済みユーザー

2020/10/20 07:11

タイマー割り込みを使うということでしょうか？

y_waiwai

2020/10/20 08:20

いえいえ、タイマ、というより終了時間変数、ですねただのunsigned longの変数でいいです

退会済みユーザー

2020/10/20 08:52

unsigned long previousMillis = 0; これとは別に変数を追加するということですか？

y_waiwai

2020/10/20 08:55

別にそれでいいので終了時間入れましょう、ってことで。

退会済みユーザー

2020/10/22 06:44

waiwaiさんがおっしゃっている条件でも挑戦したのですが、下記と、waiwaiさんがおっしゃっている条件って一致されているのでしょうか？どこが違う部分はありますか？ #define LED_OUT 25 #define PIR_IN 14 int LedState = 0; unsigned long previousMillis = 0; void setup() { pinMode(LED_OUT, OUTPUT); pinMode(PIR_IN, INPUT); } void loop() { if(digitalRead(PIR_IN)==HIGH){ unsigned long currentMillis = millis(); if(currentMillis - previousMillis >= 100) { if (LedState == 0) { digitalWrite(LED_OUT, HIGH); LedState = 1; } else { digitalWrite(LED_OUT, LOW); LedState = 0; } previousMillis = currentMillis; } }else{ //追加 digitalWrite(LED_OUT, LOW); //追加 } }

y_waiwai

2020/10/22 06:48

if(digitalRead(PIR_IN)==HIGH){ これが成立してるときだけしか処理を行ってないようですが。そんでいいんでしょうか

退会済みユーザー

2020/10/22 07:26

下記にて修正しました。いかがでしょうか？ #define LED_OUT 4 #define PIR_IN 13 int LedState = 0; unsigned long previousMillis = 0; void setup() { pinMode(LED_OUT, OUTPUT); pinMode(PIR_IN, INPUT); } void loop() { if(digitalRead(PIR_IN)==HIGH){ unsigned long currentMillis = millis(); if(currentMillis - previousMillis >= 100) { if (LedState == 0) { digitalWrite(LED_OUT, HIGH); LedState = 1; } else { digitalWrite(LED_OUT, LOW); LedState = 0; } previousMillis = currentMillis; } }else{ digitalWrite(LED_OUT, LOW); } if(digitalRead(PIR_IN)==LOW){ //追加 digitalWrite(LED_OUT, LOW); //追加 LedState = 0; //追加 } }

y_waiwai

2020/10/22 07:28

で、そのコードで動作させてみたらどうなるんでしょうか。あなたの希望通りの動作はしますか？

退会済みユーザー

2020/10/22 07:30

希望通りの動作にはなりますが、waiwaiさんとの条件が一致しているのかは分からないです。

y_waiwai

2020/10/22 07:32

まあ、私の回答は、一つのやり方、ですんで他の方法でも実現できますあなたのコードで希望の動作ができているなら、そんでいいでしょう。＃あなたのコードを読んだわけではないのであしからず

行動規範の内容に同意します

ベストアンサー

arduino
1void loop()
2{ 
3    if(人感センサーがHIGHだったら){
4        // 諸々の処理
5    }
6}

と、人感センサーがLOWだったときの処理が何一つありません。

これがLOWだったときの処理じゃないんでしょうか？

インデント補ってみます:

arduino
1void loop()
2{
3    if(digitalRead(PIR_IN)==HIGH){ // 「人感センサーがHIGHだったら」はじまり
4        unsigned long currentMillis = millis();
5        if(currentMillis - previousMillis >= 100) {
6            if (LedState == 0) {
7                digitalWrite(LED_OUT, HIGH);
8                LedState = 1;
9            }
10            else {
11                digitalWrite(LED_OUT, LOW);
12                LedState = 0;
13            }
14            previousMillis = currentMillis;
15        }
16    } // 「人感センサーがHIGHだったら」 おわり
17}

投稿2020/10/19 06:42

編集2020/10/19 06:54

ozwk

総合スコア13521

退会済みユーザー

2020/10/19 06:46

else { digitalWrite(LED_OUT, LOW); これがLOWだったときの処理じゃないんでしょうか？

ozwk

2020/10/19 06:54

追記しました

退会済みユーザー

2020/10/19 07:06

} // 「人感センサーがHIGHだったら」おわりの後に、if(digitalRead(PIR_IN)==LOW){ の追記をすればいいのでしょうか？

ozwk

2020/10/19 07:08

もちろんelseもつけてください

退会済みユーザー

2020/10/19 08:36

#define LED_OUT 25 #define PIR_IN 14 int LedState = 0; unsigned long previousMillis = 0; void setup() { pinMode(LED_OUT, OUTPUT); pinMode(PIR_IN, INPUT); } void loop() { if(digitalRead(PIR_IN)==HIGH){ unsigned long currentMillis = millis(); if(currentMillis - previousMillis >= 100) { if (LedState == 0) { digitalWrite(LED_OUT, HIGH); LedState = 1; } else { digitalWrite(LED_OUT, LOW); LedState = 0; } previousMillis = currentMillis; } } if (digitalRead(PIR_IN) == LOW) { digitalWrite(LED_OUT, LOW); } } これで一応動いたのですが、elseの後の記述はどうすればいいんでしょうか・・？

ozwk

2020/10/19 13:10

void loop() { if(人感センサーがHIGHだったら){ // 諸々の処理 } }else{ digitalWrite(...); } という話でしたべつにそれでもいいです

ozwk

2020/10/19 13:11

ちなみにesp32でLEDを点滅させる用途ならledcを使ったほうが簡素に書けます。

退会済みユーザー

2020/10/20 06:54

#define LED_OUT 25 #define PIR_IN 14 int LedState = 0; unsigned long previousMillis = 0; void setup() { pinMode(LED_OUT, OUTPUT); pinMode(PIR_IN, INPUT); } void loop() { if(digitalRead(PIR_IN)==HIGH){ unsigned long currentMillis = millis(); if(currentMillis - previousMillis >= 100) { if (LedState == 0) { digitalWrite(LED_OUT, HIGH); LedState = 1; } else { digitalWrite(LED_OUT, LOW); LedState = 0; } previousMillis = currentMillis; } }else{ //追加 digitalWrite(LED_OUT, LOW); //追加 } } このコードでも動作できました。ありがとうございます。

行動規範の内容に同意します

あなたの回答