LED点滅制御によるカウント不定

前提・実現したいこと

初めて質問させて頂きます。
PICの初心者で、様々な所からご教示頂き勉強中です。
よろしくお願いいたします。

PIC16F1503を使用して下記動作のプログラミングを製作しています。
①PB1を押すとLED1が10秒間点滅しBZがON、LED1が点滅中にPB2を押すとキャンセルされ最初に戻る。
②10秒後LED1が消灯、BZがOFF、LED2が点滅
③PB2を押すとすべてキャンセルされ①に戻る

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

動作はしているが
①PB1を押した後の、10秒カウントが17秒位ある。
②LED2の点滅が不安定で点滅周期が長かったり、短かったりする。
また、PB2を押さない限りLED2はずっと点滅するはずが、途中から点灯に代わってしまう。
③そもそもこのコードは最適なのか？

該当のソースコード

#define _XTAL_FREQ 2000000

void main(void) {
    //マイコン設定
    OSCCON = 0b01100000; //内部クロック周波数を2MHzに設定
    ANSELA = 0b00000000; //PortA全てのピンをデジタルモードに設定
    ANSELC = 0b00000000; //PortC全てのピンをデジタルモードに設定
    TRISA  = 0b00000000; //PortA全てのピンを入力モードに設定
    TRISC  = 0b00000000; //PortC全てのピンを出力モードに設定
    
//初期処理：電源ON後、全ての出力を一度OFFにする。    
    LATC0=0; //LED1
    LATC1=0; //LED2
    LATC2=0; //BZ

//RA5:PB1、RA4:PB2  
    while(1){
        if(RA5==1){
            int cnt=0; //PB1がONの時、最初にｶｳﾝﾀに0を代入
            while(cnt<10000 && RA4==0){ //ｶｳﾝﾄ値が30秒以下、PB2 OFFの時、下を実行
                LATC2=1; //BZ ON
                if(cnt%500<250)LATC0=1; //LED1 ON
                else LATC0=0; //LED1 OFF
                __delay_ms(1);
                cnt++;
            }            
            while(cnt==10000){ //ｶｳﾝﾄ値が10秒に達したら下を実行
              LATC0=0; //LED1 OFF
              LATC2=0; //BZ OFF
             while(RA4==0){ //PB2を押さない限り下を実行
                if(cnt%1000<500)LATC1=1; //LED2 ON
                else LATC1=0; //LED2 OFF
                __delay_ms(1);
                cnt++;
                }
            }
            LATC0=0; //LED1 OFF
            LATC1=0; //LED2 OFF
            LATC2=0; //BZ OFF
        }
    }
    return;
}


```### 試したこと
①の10秒カウントについて
・点滅コードが原因かと思い、LED1の点滅をやめて10間点灯に変更すると正常に10後LED2が点滅した。
・点滅コードを
```ここに言語を入力
for(timer=0;teimer<10;timer++){
LATC0=1;
__delay_ms(500);
LATC1=0;
__delay_ms(500);

```にすると、PB2を押してもキャンセルされない時がある。
②の点滅が点灯に代わってしまう事について
カウント上限？になってしまい、計算結果が真のままになってしまって点灯しっぱなしになっているのが原因かと思い、cnt++の下に
if(cnt==2000){
cnt=0;
}
を追加したが関係なかった。
### 補足情報（FW/ツールのバージョンなど）
MPLABX IDE v5.30を使用

dodox86

2020/04/30 01:08

見づらいですので、プログラムのソースコードはマークダウン記法で記載してください。（読みづらいと、それだけで回答を得る機会が減ります） https://teratail.com/help#about-markdown C言語のソースコードであれば ```C int main() { printf("hello\n"); return 0; } ``` などと書きます。

ikadzuchi

2020/04/30 01:18

文中にPB1・PB2・LED1・LED2とありますが、コード中には作動PB・停止PB・作動灯・不具合灯とあります。合わせてください。また、LEDがHiとLowのどちらで点灯するか分かりません。

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

前提・実現したいこと

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

該当のソースコード

コメントに関して追記 5/1

関連した質問