你会改掉所有Warning么？

system888net

原帖由 zx_wing 于 2009-12-4 10:47 发表

这个没有关系的，由编译器不同引起的warning大多都是一些对规则的解释不同，并不暗示隐藏的bug。
只要保证在本地开-Wall -Werror编译无warning即可，但代码发布的时候一定要记得去掉，以免别人使用不同版本的 ...

有道理.

system888net

1. 
   
2. int f1(int var)
   
3. {
   
4.         if(var<0)
   
5.         {
   
6.                 return(-1);
   
7.         }
   
8.         else if(var==0)
   
9.         {
   
10.                 return(0);
    
11.         }
    
12.         else if(var>0)
    
13.         {
    
14.                 return(1);
    
15.         }
    
16. }
    

复制代码

warning C4715: 'f1' : not all control paths return a value
test - 0 个错误，1 个警告
========== 全部重新生成: 1 已成功, 0 已失败, 0 已跳过 ==========

注: 是在vc 2005 环境下,lx也是.

[ 本帖最后由 system888net 于 2009-12-10 22:59 编辑 ]

system888net

1. 
   
2. int f1(int var)
   
3. {
   
4.         if(var<0)
   
5.         {
   
6.                 return(-1);
   
7.         }
   
8.         else if(var==0)
   
9.         {
   
10.                 return(0);
    
11.         }
    
12.         else if(var>0)
    
13.         {
    
14.                 return(1);
    
15.         }
    
16.         return(88);
    
17. }
    

复制代码

test - 0 个错误，0 个警告
========== 全部重新生成: 1 已成功, 0 已失败, 0 已跳过 ==========

糊精

最后一个不应当加if吧，既然自己都清楚是所有分支了，为什么 还要else if多此一举