ボタンを押すとLEDが点灯、もう一度押すと消灯というプログラムを作りたいです

###前提・実現したいこと
C言語初心者です。ボタンを押すとLEDが点灯、もう一度押すと消灯というプログラムを作りたいです。
似たような質問もいくつかあり、その回答も参考にして考えましたがよくわからなくなりました。
よくある質問だと思いますがよろしくお願いします。

###発生している問題・エラーメッセージ
現在のプログラムだとボタンを押している間は点灯、離すと消灯、離しても点灯することがあります。

###ソースコード
#define SW_OFF 0
#define SW_ON 1
unsigned char getSW(void); //スイッチ読込関数の宣言
void LED_CTR(void); //LED制御関数の宣言
void LED(unsigned char number); //LED点灯関数の宣言

void main(void)
{
while(1){
LED_CTR(); //LED制御
}
}

void LED_CTR() //LED制御関数
{
static int sw = 0;
unsigned char a; //スイッチ入力の格納変数の宣言
a = getSW(); //スイッチ入力の読み込み
if(a == 1) { //スイッチ入力の判別
++sw;
sw &=1;
LED(sw);
}
}

unsigned char getSW(void) //スイッチ読込関数
{
unsigned char flag = 0;
if(P1.0 == SW_ON) //P1.0はスイッチポート
{
flag = 1;
}
return flag;
}

void LED(unsigned char number) //LED点灯関数
{

if(number == 1)
{
P2.1 = 1; //P2.1はLEDポート
}else{
P2.1 = 0;
}
}

###補足情報(言語/FW/ツール等のバージョンなど)
c言語

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

回答3件

ベストアンサー

こんにちは。
2点問題があるようです。

まず１点目は、LED_CTR()を見ると、スイッチがONの間、継続して++swしているため、LEDのON/OFFを高速に繰り返しているようです。そこで、スイッチがOFF->ONへ変化した時だけ++swする必要があります。

次に２点目は、機械接点式のスイッチはチャタリングがあり、ONとOFFが切り替わる瞬間、高速にON/OFFを繰り返します。ですので、チャタ取りも必要です。
良くある方法は、数mSec程度あけて2回スイッチの状態を読み取り、一致したら確定します。
ここに詳しい解説があります。

チャタ取り後に、OFF->ONチェックすると安定するようになりますよ。

【追記】
ちょっと安易すぎるのですが、チャタ取りも含めたLED_CTR()関数を作ってみました。
OFF->ON検出のところは、catsforepawさんのコードとほぼ同じです。

C
1void LED_CTR() //LED制御関数
2{
3    static int sw = 0;
4    unsigned char now;    //スイッチ入力の格納変数の宣言
5    static unsigned char last = 0; 
6    int i;
7    for (i=0; i < 100; ++i)
8    {
9        now=getKey();
10        if (now == last)
11return;
12    }
13
14    if ((last == 0) && (now != 0))
15    {
16        ++sw;
17        sw &=1;
18        LED(sw);
19    }
20    last=now;
21}

for文で100回回してます。もし、LEDのON/OFFが安定しない(キーを1回押しただけでON->OFF->ONを何回か行う)ような時は、もっと数を増やしてみてください。

なお、一般的にはgetKey()の前あたりでちゃんと時間を測って数mSecの待ちをタイマで行うべきです。
お使いのシステムのタイマが分からないので、安易にループを回してます。この辺は次の課題として頑張って下さい。

投稿2015/11/12 02:16

編集2015/11/12 12:50

Chironian

総合スコア23274

isida

2015/11/12 11:35

返信ありがとうございます。上の方もおっしゃるようにOFF->ONへ変化した時だけ++swする必要があるようです。ボタンの状態の読み取り方法でもうちょっとヒントもらえませんか？

isida

2015/11/18 00:21

上記の変更点でボタン一つで切り替えることができるようになりました。ありがとうございました。まだまだ勉強が必要なので頑張ります。

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

チャタリングというよりは、スイッチの判定をエッジで行っていないことが問題の原因だと思います。
エッジとは、立ち上がり(OFF状態→ON状態への変化)と立ち下がり(ON状態→OFF状態への変化)のことです。
ですので、読み取ったスイッチの状態もstatic変数で覚えておく必要があります。そして、たった今読み取った状態と前回読み取った状態を比較して、変化があったときだけLEDをON/OFFするという処理が必要です。

追記

チャタリング対策も場合によっては必要かもしれません。ソースを見ると、超高速でスイッチの判定をしているので、スイッチ回路のお金のかけ具合によってはチャタリングが発生します。

さらに追記

ご質問の件では押したときだけ反応させたいとのことなので、立ち上がり（OFF→ON）だけを判定すれば良いことになります。

LED_CTR関数への修正はこんな感じになります。

C
1static int sw = 0;		// 変更なし
2static unsigned char last = 0;	// スイッチの状態を覚えるためのstatic変数を追加
3// スイッチの読み込みは変更なし
4if(last == 0 && a == 1)	// スイッチ入力の判定を変更（前回OFFで今回ON）
5{
6	// この中も変更なし
7}
8last = now;		// スイッチの状態を覚えておく