执行结果是什么

只看该作者 · 2016-10-14 19:19

if(G_Key_Flag==4)
         if(!G_Key2)
            delay_ms(100);
            if(!G_Key2)
               G_Counter3++;
                  if(G_Counter3==100)
                  {
                     G_Counter3=1;
                     }
1.  G_Key_Flag是一个状态量；
2.G_Key2是一个按键量；
3.G_Counter3是一个变量；
这一段的执行结果是什么还有就是为什么按键按下结果加的不是1？

2万 · 2016-10-14 19:20

真正要完成的目的是啥啊

只看该作者 · 2016-10-14 19:25

按一下G_Key2加1，一直按着G_Key2就连续加1.

2万 · 2016-10-14 20:57

想法没错，建议打断点跟踪一下，程序流程是不是按照你的设计在跑。

只看该作者 · 2016-10-14 20:59

就这里不是其他的地方都是

1万 · 2016-10-14 21:01

那你的断点看到哪些程序没执行呢？控制程序跳转流程的...

2万 · 2016-10-14 21:02

因为按键按下后，变量一直做加法（哪怕按下的时间很短），所以加的不是1。

2万 · 2016-10-14 21:04

好像忘了清0了
if(flag_1s==1)
{
G_Counter3++; //1s做一次加1。
flag_1s=0;
}

2万 · 2016-10-14 21:05

我觉得他出现连续加的问题是最后没有检测按键抬起应该检测一下就不会连加了

2万 · 2016-10-14 21:07

这段程序的功能是：
如果G_Key_Flag=4,就判断按键是否按下
如果按键按下，则变量G_Counter3自加一(一直按住，则连续加，不过要看G_Key_Flag的条件是否满足)
如果G_Counter3=100，G_Counter3归1

2万 · 2016-10-14 21:12

先判断是否有键按下，如果是，软件去抖，在判断是否有键按下，真有则执行G_Counter3++

2万 · 2016-10-14 21:14

程序的格式看不大懂。

自加使用 ++G_Counter3 代替 G_Counter3++吧，或者G_Counter3 += 1，不然G_Counter3的值是不会被修改的。

1万 · 2016-10-14 21:16

当按键被按下时，该段程序已经执行了很多遍，所以不是加1。想办法使按键被按下时，只执行一次，比如：用一个状态量，在程序执行一次后改变状态量的值，使不能再次执行该段程序。

2万 · 2016-10-14 21:18

花括号是不是打掉了些哦？另外该代码是否被循环调用？多久调用一次？如果你定时器准确，你按一秒钟的话，G_Counter3也最多增加为10次，如果你按的时间不足100ms，你的计数器不会增加的。
我猜你主要是想实现按键功能，需要消颤，按键要达到一定时长才认为是一次有效的按键动作。
代码改成如下看看，如果是楼主期望的，请散下分哦，谢谢：
if(G_Key_Flag==4)
{
   if(!G_Key2)
      {
         delay_ms(100);
         if(!G_Key2)
            {
            G_Counter3++;
            if(G_Counter3==100)
                  G_Counter3=1;
            }
         else
            G_Counter3 = 0;
}
}

只看该作者 · 2016-10-14 21:20

结贴了，多谢大家讨论这么多哈，呵呵