水を検知するプログラムを書いていて直前の値と今の値を用いてif,elseを使いたいと思っているのですがうまく作動しません。

Question

###前提・実現したいこと
Arduinoで水センサーを用いて水を検出した時のみ光らせるプログラムを作りたいと思っています。前回水を検出せず、今回検出したら光るというプログラムです。
私自身ついこないだarduinoやプログラムに触れ始めたばかりで初心者です。どこで質問していいのかもわからないのでここで質問させていただきます。

###発生している問題・エラーメッセージ
if,elseを用いて書いてみたのですが常にelse側になってしまいうまく作動しません。エラーメッセージなどはないです。

###該当のソースコード

#define Moi_detect 12
#define led 13

int val = 0;
int old_val = 0;


void setup() {
  // put your setup code here, to run once:
pinMode(Moi_detect,INPUT);
pinMode(led,OUTPUT);
}

void loop() {




  val = !digitalRead(Moi_detect);
  if((val = HIGH) && (old_val = LOW)){
    digitalWrite(led,HIGH);
    delay(1000);
  }else{
    digitalWrite(led,LOW);
    delay(1000);
  }

 
  val = old_val;
}

###試したこと
初心者ながら

val = old_val
```がうまく置き換わってないと思います

###補足情報(言語/FW/ツール等のバージョンなど)
使っている個体はarduino yunで次は水を検知したらBlynkを通じてスマホに通知させようと思っています。

Answer 1

if((val = HIGH) && (old_val = LOW)){
を
if((val == HIGH) && (old_val == LOW)){
にしてみましょう。
= ではなく== ですよ。

Answer 2

まずロジックの確認。
12番Pinに水分センサーを接続している。
水分センサーは恐らく水分が無い時はLOWで水分が存在したらHIGHになると仮定して、水分が無いから水分を検出したらLEDを光らす。水分を検出して以降はLEDを消す？でいいのでしょうか？
ならば単純に前回値がLOWからHIGHになった時だけLEDが点滅してloopで次に前回値が'HIGHで今回値もHIGH`であるからLEDを消す？もし持続してLEDを光らすならば前回値は必要なくて単純に今回値だけで点灯、消灯にするのでは？
仮に瞬間だけ光らすならば以下の単純なソースになります。

#define Moi_detect 12
#define led 13

//int val = 0;
int old_val = LOW;
int val;
void setup() {
  // put your setup code here, to run once:
pinMode(Moi_detect,INPUT);
pinMode(led,OUTPUT);
}

void loop() {
  val = digitalRead(Moi_detect);
  if((val == HIGH) && (old_val == LOW)){
    digitalWrite(led,HIGH);
    delay(1000);
  }else{
    digitalWrite(led,LOW);
    delay(1000);
  }
  old_val = val;
// old_val = val;
}

Comments

[ananasie]

moidetect_v2:14: error: 'val' was not declared in this scope
val = digitalRead(Moi_detect);
^
exit status 1
'val' was not declared in this scope

このようなエラーが出てしまいました。

[MasahikoHirata]

失礼しました。`val`と`old_val`の定義を見落としておりました。
int val;とint old_val；を記載しましょう。回答を修正いたします。