Arduinoでデジタルセンサで値を取得した入力に対して、閾値内に出力がHIGHならその閾値内の出力をすべてHIGHにするプログラムを書きたい

arduinoでデジタル式のモーションセンサの作動を行っています。しかし、このセンサだと人がセンサの計測範囲内にいる場合でも人が動かなければセンサの値がLOWになってしまいます。そこで、配列を用いたプログラムを書きましたが、いまいち改善はしていない状況です。どのようにしたらよろしいでしょうか。

該当のソースコード

define PPR1 3
float time;
int ss[5][5];

void setup() {
pinMode(PPR1, INPUT);
Serial.begin(9600);
while (!Serial);
Serial.println ("センサが安定するまで少々お待ちください(30s)");
delay(30000);
}

void loop() {
int i;
int State1 = digitalRead(PPR1);
ss[5][5] = State1;
for(i=0;i<NULL;i++){
if(ss[i][i]==HIGH&&ss[i+8][i+9]==HIGH){
ss[i][i+1]==HIGH;
ss[i+1][i+2]==HIGH;
ss[i+2][i+3]==HIGH;
ss[i+3][i+4]==HIGH;
ss[i+4][i+5]==HIGH;
ss[i+5][i+6]==HIGH;
ss[i+6][i+7]==HIGH;
ss[i+7][i+8]==HIGH;
}
}

time = (millis() - 30000) * 0.001;

Serial.print(time, 1);
Serial.print(" ");
Serial.print(ss[5][5]);
Serial.println(" ");
}

試したこと

配列を用いてセンサ値を配列に格納し、条件文によって一つ目と10つ目の値に反応が見られたらその間のセンサ値をすべて反応している状態にするプログラムを書きました。

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

配列を使う方法でも構わないです。またチャタリングの原因もあるかと思いますのでそちらの改善方法も教えていただきたいです。よろしくお願いいたします。

jimbe

2021/08/18 06:11

arduino 特有の表現なのか分からないのですが、「閾値内に出力がHIGHならその閾値内の出力をすべてHIGHにする」の閾値内とは何のことでしょう。

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

回答3件

ベストアンサー

arduino をやったことが無いので、見つけたリファレンスを見ながら試しに書いてみました。
本当に書いただけでコンパイルも試していないのでエラーが多いかもしれません。スミマセン。
というか仕様が合っているのかさえ分からないのですが…。
setup 等は省略です。

c
1void loop() {
2  int state1 = digitalNoiselessRead(PPR1, 100, 5000); //PPR1 ピンが 0.1秒間 HIGH になっていたら,その後 5秒間 HIGH が返る
3
4  //表示
5  float t = (millis() - 30000) * 0.001; //loop開始からの経過秒
6  Serial.print(t,1);
7  Serial.print("  ");
8  Serial.println(state1);
9}
10
11int lastState = LOW;
12unsigned long risingEdgeTime = 0;
13int stage = 0; //0=初期, 1=監視期間, 2=HIGH期間
14
15// pin からの入力が, 立ち上がりから monitoringPeriod の間 HIGH になった場合のみ 
16// highPeriod の間 強制的に HIGH を返す.
17// ( monitoringPeriod の間は LOW を返す. )
18// パラメータ:
19//   pin ピン番号
20//   monitoringPeriod 監視期間[ms]
21//   highPeriod HIGH期間[ms]
22// 返り値:
23//   HIGH または LOW
24int digitalNoiselessRead(int pin, long monitoringPeriod, long highPeriod) {
25
26  int state = digitalRead(pin);
27  unsigned long now = millis();
28  if(stage == 0 && lastState == LOW && state == HIGH) { //立ち上がり検出
29    risingEdgeTime = now;
30    stage = 1; //監視期間開始
31  }
32  lastState = state;
33
34  if(stage == 1) { //監視期間中
35    if(state == LOW) { //ノイズだった
36      stage = 0;
37    } else if(now-risingEdgeTime >= monitoringPeriod) {
38      stage = 2; //HIGH 期間へ移行
39    } else {
40      state = LOW; //監視期間中は LOW
41    }
42  } else if(stage == 2) { //HIGH期間中
43    if(now-risingEdgeTime <= monitoringPeriod+highPeriod) {
44      state = HIGH;  //HIGH期間中は HIGH
45    } else {
46      stage = 0; //HIGH期間終了
47    }
48  }
49  return state;
50}

投稿2021/08/18 12:15

編集2021/08/18 12:29

jimbe

総合スコア12648

ソースを見るだけだと、具体的にどう処理しようとしてるのか見えてきません。
何をどうしたいコードなのか、コメントをつけてくださいね。
少なくとも、二次元配列ssの各行、各列がどういう意味になっているのか、for文の中身では何をどうしようとしているのか、その辺りのコメントが必要です。

あと、編集時にエディタのメニューに <code> があると思います。それを使って、コードを見やすいようにしてください。インデント等が消えているので大変に読みづらいです。

ssがセンサで得た情報なのかな？
ss[5][5] （<-これ範囲外じゃない？）にStateを代入することで配列内に全データ入れたことにしている？
for (i=0;i<NULL;i++){ <- 回らなくない？
if(ss[i][i]==HIGH&&ss[i+8][i+9]==HIGH){ これ、iが>=0でないと前半部で領域外参照、i<=-4でないと後半部で領域外参照してない？
ss[i][i+1]==HIGH; y_waiwaiさんの指摘通り

投稿2021/08/18 05:33

編集2021/08/18 09:25