Arduinoを用いてLEDの発光時にif文を用いて動作を分けたい

前提・実現したいこと

LEDを9個接続し真ん中の9番ピンのLEDの発光時にボタンを押すと全てのLEDが発光するものを目指しています。
if文の中が間違っていると思うのですが何をやってもLEDが光るだけでボタンを押しても何も反応しません。

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

特に出ませんでした。

該当のソースコード

int i;
void setup(){
  for(i=5;i<14;i++){
    pinMode(i, OUTPUT);    //LED
  }
  pinMode(1, INPUT);      //スイッチ
  pinMode(2, INPUT);      //圧電スピーカー
}

void loop(){
  int sw = digitalRead(1);
  
  for(i=5;i<14;i++){
    digitalWrite(i, HIGH);
    delay(500);
    digitalWrite(i, LOW);
    delay(500);                  //500msでLEDの発光
  }
  
  if(sw == HIGH){                    //9番ピンのLEDが発光した時にスイッチを押すと以下のif文
    if(digitalRead(9)== HIGH){
      for(i=5;i<14;i++){
        digitalWrite(i, HIGH);
      }
      tone(2,261,100);
      delay(100);
      tone(2,261,100);
      delay(100);
      tone(2,261,100);
      delay(100);
    }
  }
}

試したこと

if文の中のif(digitalRead(9) == HIGH)をif(digitalWrite(9, HIGH))にしてみましたがエラーが出ました。エラーメッセージは
error: could not convert 'difitalWrite(9u, 1u)' from 'void' to 'bool'
でした

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

。

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

回答2件

コードを見ると、
0. スイッチ状態を読み込み
0. LEDを回転点灯
0. １で読み込んだスイッチを判定

となってます。そもそもの考え方がおかしいかと。
それに、

if(digitalRead(9)== HIGH){

ってのが意味不明です。２の実行でLEDは全消灯してます

投稿2020/07/17 01:42

y_waiwai

総合スコア88042

ベストアンサー

一番悪い点は、SWの状態の読み取りとLEDの点灯位置とが無関係になっている点です。
LEDが移動していく最中もSWの状態を見て、どこで押したのかを判断する必要があります。

例えば、下の例だと、10msでwhileの中を回しています。
その回数を数えてLEDを操作しています。
LEDが9個なので、計9秒(cntだと900)で一巡する計算になります。

つまりcntが、0,100,200…のタイミングLEDnを点灯させる
50,150,250…のタイミングで、LEDnを消灯させる
という事で、各LEDを操作します。
そして、SWを押したらwhileを抜ける→その時のcntがSWを押したときの状態になります。
で、cntが400～450なら9ピンのLEDが押した時に相当しますから、
その時に成功のイベント、それ以外の時は失敗のイベントになります。

後の注意事項は
・LEDピンは配列にした方が解り易いです。
・圧電スピーカはOUTPUT（出力）です。ただし、tone関数を使うなら出力設定は要りません。
・SWを押した時にHIGHにするのなら、『プルダウン抵抗を付けて』、PIN-SW-5Vと繋ぎます。

ARDUINO
1void setup() {
2	for(int i=5; i<14; i++) {
3		pinMode(i, OUTPUT);    //LED
4	}
5	pinMode(1, INPUT);      //スイッチ
6//	pinMode(2, OUTPUT);      //圧電スピーカー
7}
8
9void loop() {
10	word cnt=0;
11	while(1) {
12		if(cnt%100==0)digitalWrite(cnt/100+5,HIGH);
13		else if(cnt%100==50)digitalWrite(cnt/100+5,LOW);
14		
15		if(digitalRead(1)==HIGH)break;
16		
17		delay(10);
18		if(++cnt==900)cnt=0;
19	}
20	
21	if(cnt>=400 &&cnt<450) {//成功なら
22		for(byte i=5; i<14; i++)digitalWrite(i,HIGH);
23		//ファンファーレ
24		delay(4000);//音の代わり
25		for(byte i=5; i<14; i++)digitalWrite(i,LOW);
26	} else {//失敗なら
27		for(byte i=5; i<14; i++)digitalWrite(i,LOW);
28		//ﾌﾞﾌﾞー
29		delay(4000);//音の代わり
30	}
31}
32