ＳＷ入力のプログラムについて

質問

ＳＷ入力のプログラムについて

プログラミング初心者です。
下記のタイマー割り込みによるＳＷ入力のプログラムで疑問ヶ所が１点あります。
添付画像の緑色のヶ所を『VolUpSw=0;』としないで、『VolUpSw=1;』としていることに何か意図があるのでしょうか？

ご教授下さい。
よろしくお願い致します。

タイマー割り込み
{
　　　　・・・・

　　　　if(!(PIND&(1<<SW_PORT_VOLUP)))
　　　　{
　　　　　　　　if(++VolUpSw==0)
　　　　　　　　　　　　VolUpSw=255;
　　　　　　　　if(VolUpSw>=20)
　　　　　　　　{
　　　　　　　　　　　　Volume-=8;
　　　　　　　　　　　　if(Volume>=0xF8)
　　　　　　　　　　　　　　　　Volume = 0x00;
　　　　　　　　　　　　VolUpSw=1;　←この部分
　　　　　　　　}
　　　　　　　　SwitchFlag|=(1<<SW_VOLUP);
　　　　}
　　　　else
　　　　{
　　　　　　　　VolUpSw=0;
　　　　}

　　　　・・・・
}

回答No.3

D-Matsu · ベストアンサー

コンパイラがちゃんとCの規格に沿っていれば、静的領域の変数は実行時に暗黙的に0初期化されます。
どちらかというと、多重割込みを許した場合に連続で割込みが入ってきてもちゃんと動作する、というのを意図しているような気がしますねぇ。

回答No.4

R32C

あんまりこのコードを追うのもどうかと思う。
たぶん、volumeが、出力データだと思うけど、結局　0にしかならない
し、正しく制御されているように思えない。

また、多重割込みを利用するにしても、volumeは一時的に
-8されてから、0に戻すというヘンテコな処理をしていますね。
多重割込みを許すなら、
if (Volume < 8 )
Volume = 0;
else
Volume -=8;
と書かないと突然変な値をとる場合があって問題です。

なので、このコードを書いた人のスキルを疑ってみてもよいと
思います。
まぁそういう悪い例の問題を出されたのかもしれませんが。

回答No.2

D-Matsu

処理自体がタイマー割り込み内ということは、コンペアマッチ辺りで定期的にスイッチ状態を見ていると思っていいですね？

まず、スイッチが押されていない状態ではVolUpSwは0です(一番外のelse)。
で、押されると中の最初のifで++VolUpSwが行われ、１になります。

更に次のifで20以上になっていたらボリュームを上げてカウンタクリア、としていますね。
押しっぱなしにしているとVolUpSwの値が上がっていき、カウンタクリア処理が走らなかった場合は最終的にオーバーフローしますが、そうなった場合には255に戻すという処理が入っています。
＃但し、普通は通りえない場所のはずです

このコードに意図があるとすれば「スイッチを押している間は、VolUpSWは0にならない」という事でしょう。

お礼

ご回答ありがとうございます。

ご指摘頂いたヶ所も少し気になっていたのですが、

(1)VolUpSwがオーバーフローした場合、２５５に戻す→VolUpSwは２０に達するとクリアされるので、オーバーフローすることはないのでは？
(2)VolUpSwが２０以上の場合→VolUpSwが２０の場合では？

(1)、(2)ともに電源投入時のVolUpSw（変数宣言時に特に値を定めていません）の不定値に対応していると解釈していたのですが、正しいでしょうか？

回答No.1

pyonmae

こんにちは。
もちろん、他人の作ったプログラムなんですよね？
はっきり言って、作った人のみぞ知る、といった所ですが・・・。

カンですが、VolUpSw変数も、キーが押されているフラグとして、どこかで使われているのではないでしょうか。
または、オートリピートの時間調整の結果かも知れませんね。（それだったら20の方を変えるか・・・？）

何にしても、コメントを書くか、せめて定数をdefine定義するとかして欲しいですね・・・。

お礼

ご回答ありがとうございます。

※記載し忘れたのですが、『Volume=0x00で最大音量、Volume=0xFFで最小音量』になります。

・コメント→ありません。
・define定義→
volatile uchar SwitchFlag;
volatile uchar Volume;
volatile uchar VolUpSw;
・VolUpSw変数→他で使用されている形跡はありません。

特別な意図は無さそうですね。