C语言嵌套if语句内层代码未执行问题求助

核心问题分析 #

对比你的C代码和可正常运行的汇编代码，发现两处关键不匹配：

1. 引脚号错误：汇编检测的是PORTA,0（即RA0引脚），但C代码里用的是PORTAbits.RA1（RA1引脚），如果实际按键接在RA0上，C代码读错引脚自然无法正确触发后续逻辑。
2. 逻辑判断矛盾+延迟时长不匹配：
   • 汇编里延迟是0.5秒，C代码里写的是1000ms（1秒），时长不一致可能导致按键状态判断时机偏差；
   • 内层if的注释说明"low=按键仍按下"，但代码判断的是RA1 == 1（高电平），逻辑完全搞反，导致短按松开的情况无法被正确识别。

修正后的C代码 #

void SwitchTest(void) 
{
    __delay_ms(3000);
    // LEDCOM = LED_ON;            // Connect Bar Common
    LED1 = LED_OFF; 
    LED2 = LED_OFF; 
    LED3 = LED_OFF; 
    LED4 = LED_OFF; 
    LED5 = LED_OFF; 
        
    if (PORTAbits.RA0 == 1)       // 按键未按下（高电平=未按）
    { 
        GoSleep();                  // 进入睡眠
    }
    else  // 按键已按下（低电平=按下）
    {
        __delay_ms(500);             // 延迟0.5秒后重测（与汇编逻辑对齐）
        if (PORTAbits.RA0 == 1)       // 按键已松开（高电平=松开，判定为短按）
        {
            LED1 = LED_ON;               // 点亮LED1
            __delay_ms(3000); 
            DisplayStep();                // 执行显示逻辑
        }
        else
        {    
            return;                       // 按键仍按下（判定为长按），返回主循环
        }
    }  
}

额外建议 #

• 嵌入式开发中，建议给引脚定义宏（比如#define KEY_PIN PORTAbits.RA0），避免直接写寄存器位，后续修改或维护更方便；
• 按键检测可加入多次采样的消抖逻辑，避免机械按键的抖动导致误判，固定延迟虽能起到简单消抖作用，但多次采样可靠性更高。

内容的提问来源于stack exchange，提问作者Ralph