if else的嵌套问题

只看该作者 · 2014-3-3 14:02

if(ReadTempFlag==1)
{
  ReadTempFlag=0;
  temp=ReadTemperature();
  if(temp&0x8000)
   {
   TempData[0]=0x40;//负号标志
   temp=~temp;  // 取反加1
   temp +=1;
   }
  else
   TempData[0]=0;

   TempH=temp>>4;
   TempL=temp&0x0F;
   TempL=TempL*6/10;//小数近似处理

if(TempH/100==0)
   TempData[1]=0;
else
   TempData[1]=dofly_DuanMa[TempH/100]; //十位温度
if((TempH/100==0)&&((TempH%100)/10==0))//消隐
   TempData[2]=0;
else
   TempData[2]=dofly_DuanMa[(TempH%100)/10]; //十位温度
   TempData[3]=dofly_DuanMa[(TempH%100)%10]|0x80; //个位温度,带小数点
   TempData[4]=dofly_DuanMa[TempL];
   TempData[6]=0x39;       //显示C符号
  }
}
}
这个事DS18B20的程序，有两处问题，其中1.if else 看晕了，2.小数近似处理那句程序看不明白。3.消隐那里不明白。先谢谢

1万 · 2014-3-3 15:49

if else不够规范，哪怕一句话，也要加括号

只看该作者 · 2014-3-3 16:53

1.小数处理是为了避免浮点运算；
2.消隐是想不显示不必要的0吧，比如012只显示12

只看该作者 · 2014-3-3 23:18

ayb_ice 发表于 2014-3-3 15:49
if else不够规范，哪怕一句话，也要加括号

开发板配的程序，很多都要看得很费劲。现在这儿看了一下午才看明白。

只看该作者 · 2014-3-3 23:21

弹弓发表于 2014-3-3 23:18
开发板配的程序，很多都要看得很费劲。现在这儿看了一下午才看明白。

嗯，那个小数处理，他是怎样处理的？我看不明白原理，你能看明白吗？对了，还麻烦你帮我看一下 TempData[1]=dofly_DuanMa[TempH/100]; //十位温度
if((TempH/100==0)&&((TempH%100)/10==0))//消隐
那里的备注//十位温度，我觉得应该是百位才对。谢谢你了

只看该作者 · 2014-3-3 23:31

写程序一定要注意缩进清晰，各级的对应关系就会一目了然。像你这个段落那样，代码堆在一起，谁也看不明白。

只看该作者 · 2014-3-3 23:47

香水橙发表于 2014-3-3 23:31
写程序一定要注意缩进清晰，各级的对应关系就会一目了然。像你这个段落那样，代码堆在一起，谁也看不明白。 ...

嗯。学习了，我写的时候会注意的。这个是开发板配的程序。

只看该作者 · 2014-3-4 09:19

建议lz：
1、使用专门的源代码编辑软件来查看及修改源码，行业内流行的是 UltraEdit，和SourceInsight，这比PIC编译工具自带的源码编辑器方便得多。
专门的源码编辑器，可将成对的花括号收缩起来，理解起代码的逻辑很是方便。

2、if－else这样的语句，紧接着判断句的后面，需要加1对花括号，那怕是一个语句也要加上，这样逻辑清晰，便于理解。

例如
if（条件）
｛
A
｝

else
｛
B
｝