ラズパイのシグナルハンドラ、タクトスイッチON、OFFの入力を反映させたい

Question

実現したいこと
ラズパイにて
タクトスイッチオン→LEDとブザーがON
もう一度タクトスイッチを押す→LEDとブザーがOF
システム終了をしない限りは、上記の繰り返しを行うシステムを実現させたいです

現状ではタクトスイッチを押した際にLEDとブザーは反応するのですが
再度タクトスイッチを押下してもLEDとブザーが反応したままになっています。
また、シグナルハンドラを呼び出した際にもLEDとブザーが反応したままなので、消したいです。

  #include<stdio.h>
  #include<string.h>
  #include<wiringPi.h>
  #include<signal.h>
  #define GPIO_LED 21
  #define GPIO_TCT 20
  #define GPIO_BUZ 5

 void motion_on(int flag2);
 void motion_off(void);
 void Signal (int sig_name);
 void SetSignal(int sig_name);
 void tct_callbak(void);
 volatile sig_atomic_t flag = 0;
 
 int main(void){
 
      if(wiringPiSetupGpio()==-1){
        return 1;
       }
      pinMode(GPIO_LED,OUTPUT);
      pinMode(GPIO_TCT,INPUT);
      pinMode(GPIO_BUZ,OUTPUT);
      pullUpDnControl(GPIO_TCT,PUD_DOWN);
      pullUpDnControl(GPIO_LED,PUD_DOWN);
      pullUpDnControl(GPIO_BUZ,PUD_DOWN);
      signal(SIGINT,Signal);
      wiringPiISR(GPIO_TCT,INT_EDGE_BOTH,tct_callbak);
 
      while(flag==0){
      };
      return 0;
 }
 
  void tct_callbak(void){
      if(digitalRead(GPIO_TCT)==HIGH){
          motion_on(1);
      }else if(digitalRead(GPIO_TCT)==LOW){
          motion_off() ;
      }
  }
 
  void motion_on(int flag2){
 
      digitalWrite(GPIO_BUZ,HIGH);
 
      for(int i=0; i<10;i++){
      digitalWrite(GPIO_LED,HIGH);
      delay(500);
      digitalWrite(GPIO_LED,LOW);
      delay(500);
      }
 
      flag2=1;
      while(flag2==1){
      digitalWrite(GPIO_LED,HIGH);
      digitalWrite(GPIO_BUZ,HIGH);
      }
  }
 
  void motion_off(void){
      printf("OFFのよびだし");
      digitalWrite(GPIO_BUZ,LOW);
      digitalWrite(GPIO_LED,LOW);
      motion_on(0);
  }
 
  void SetSignal(int sig_name){
 
      f(signal(sig_name , Signal)==SIG_ERR){
      printf("シグナル設定が出来ませんでした");
      }
 }
 
  void Signal (int sig_name){
 
      char str[3];
      char str2[]="YES";
      printf("システムを終了しますか。YES/NO\n");
      scanf("%s",str);
 
  if(strcmp (str,str2)==0){
      flag=1;
  }else{
      flag=0;
 }
 
  }

Comments

[y_waiwai]

このままではコードが見づらいので、質門を編集し、<code>ボタンで、出てくる’’’の枠の中にコードを貼り付けてください

[y_waiwai]

行番号抜きましょうよ。。

Answer 1

「システム終了」というと普通はRaspberryPiをシャットダウンするようなことをいうと思うけど...このプログラムだけの話なら「プログラム終了」とか「アプリケーション終了」とか。

それはともかく。flagとかflag2とか、体を表さない名前で混乱している面もあるのではないでしょうか。

ついでに言えば、singalとSignalとか、数文字の大文字小文字で別のシンボルになるような命名も混乱の元です。

  void motion_on(int flag2){

      digitalWrite(GPIO_BUZ,HIGH);

      for(int i=0; i<10;i++){
      digitalWrite(GPIO_LED,HIGH);
      delay(500);
      digitalWrite(GPIO_LED,LOW);
      delay(500);
      }

      flag2=1; //<--flag2を1にしているのだから...
      while(flag2==1){ //ここでは常にflag2==1は真なので無限ループ
      digitalWrite(GPIO_LED,HIGH);
      digitalWrite(GPIO_BUZ,HIGH);
      }
  }

motion_on()は一度入ると絶対に抜けられないブラックホールのような関数になっています。その中で、LEDやブザーをONしているのですから、それ以外の動作はできません。

シグナルハンドラを呼び出した際にもLEDとブザーが反応したまま

シグナルハンドラによって変数が書き換えられたらメインのプログラムを終了する動作をしているのですから、その終了動作にLEDとブザーをOFFにする処理を加えれば止まります。

Comments

[mkm]

回答ありがとうございます
命名については変えてみます。
void motion_on(int flag2)のところに「０」が入ってきても「flag2=1」で常に「１」
になってしまう、ということですかね

[thkana]

「なってしまう」というか、あなたがプログラムで「そうしろ」と書いているからなのですけれど。そもそもどういう意図だったのでしょう?