c++ - 给初学者的快速代码建议

Question

我是初学者，我想知道这段代码有什么问题。我想计算 portA 中按钮被按下的次数。然后使用端口 C 中的 LED 显示该值。谢谢

Answer 1

如果您想将多语句 block 用作 if (或 for 或其他)语句的主体，则需要用大括号将其括起来:

else if (PORTA.RA2==1) {
    count = count+1;
    PORTC = count;
}

否则只有第一个语句是有条件的；因此，无论 if 测试的结果如何，您的代码每次都会执行 PORTC = count;。

我喜欢在所有这些 block 周围加上大括号，即使只有一个语句，所以如果以后添加更多语句，我不会忘记添加它们。

此外，main 必须返回 int 而不是 void，并且您应该更加小心地格式化代码以匹配其逻辑结构。

更新:此外，您从未初始化count，因此它具有任意浮点值。您需要一个小整数类型，因为它只能接受 0 到 16 之间的整数值，并且您需要初始化它:

char count = 0;