Arduinoにて割り込み機能で連続してパルスを受信し、そのパルス幅を把握する方法

お世話になります。
表題の通りなのですが、
30ms間隔で送信されるパルスのみを受信し,その数をカウントしたいです。
稀に,1ms間隔程度のノイズが発生するので,そのパルスが29~31ms程度に収まっていることを確認してカウントしたいです。

下記のコードを試してみたのですが、millis()が割り込みで正しく動作していないせいか、全く変数（totalPulseAmount）が動きません。

アドバイスを頂戴出来れば幸いです。

宜しくお願い致します。

#define PulsePin 2 
unsigned long Time[100] ;
unsigned long Timing = 0;

unsigned long receivedPulseAmount = 0;

void setup() {
digitalPinToInterrupt(PulsePin),samplePulse,CHANGE);
}

void loop(){
while(capturedTiming >=2){
  checkSampledPulse();
  }
}

void samplePulse(){
  Timing += 1;
  Time[Timing] = millis();
}

void checkSampledPulse(){
  if(Timing >= 2){
    unsigned long pulseWidth =Time[Timing]-Time[Timing-1];
    if(pulseWidth > 29){
    if(pulseWidth < 31){
      receivedPulseAmount += 1;
      }}
    Time[Timing] = 0;
    Time[Timing-1] = 0;
    Timing -= 2;
    }
  }

thkana

2020/04/27 12:12

C#は関係ないですね?

thkana

2020/04/27 13:05

質問のコードはコンパイラを通りません。どういうものと考えたらよいのでしょう?

thkana

2020/04/27 13:08

カウントする、ということは30ms毎のパルスが欠損する可能性もある、ということですか?　(欠損がないならただ時間を測って30msで割ればいいだけの話ですね)

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

回答2件

要点をまとめると、1ms程度のノイズと避けて、30ms程度のパルス数をカウントしたい、という事ですよね？

やり方は色々あるでしょう。
そのスケッチで言えば、ある点は駄目で、ある点はちょっと無駄が多いです。

・CHANGEじゃなくて、片側で充分
・一瞬で終わる処理を割り込みとloop関数で分ける必要も無い
・下記書式
×　digitalPinToInterrupt(PulsePin),samplePulse,CHANGE);
○　attachInterrupt(0, samplePulse, CHANGE);//0とはint0ピン＝D2ピンの事
もしくは
○　attachInterrupt(digitalPinToInterrupt(PulsePin), samplePulse, CHANGE);
・capturedTimingとは？
・割り込み内で操作する関数にはvolatileを付ける
等々。
つまり、全体的な構成は良いとしても、1つずつ気を付けなければならない点を押さえていないように思います。

ぶっちゃけ、ハードで、ローパスフィルターでも付けて、単にカウント++させた方が良いかも。

あるいは、もっと簡単に、↓程度で良いと思うよ

byte pulse_pin=2;//どこでも

byte pulse_state;//8個分保存
uint32_t count=0;

void setup(){}

void loop(){
pulse_state<<=1;//シフトさせてLSBを空ける
pulse_state+=digitalRead(pulse_pin);//最新を取得
if((pulse_state&7)==5) pulse_state|=7;//ノイス消去
else if((pulse_state&7)==2)pulse_state&=~7;
if((pulse_state==0xF)count++;//0b00001111となった時だけ採用
delay(1);
}

投稿2020/04/28 07:49