sigh,又是按照代码行数统计工作量...

今天又看到这样的代码：

java 代码

        if(msisdn==null)
        {
             return mapping.findForward("return");
        }

        if(msisdn.equals(""))
        {
             return mapping.findForward("return");
        }

是不是因为这个...工作量按照代码行数统计的原因？-_-!

评论

14 楼 killvin 2007-07-15  

垃圾公司有在招人了，你们都去应聘吧

13 楼 流浪者 2007-07-15  

没一点技术含量真是

12 楼 tysw 2007-07-14  

jlusdy 写道

各位
鼠标滚轮不是自己的
可手是自己的啊

同意

11 楼 jlusdy 2007-07-14  

各位
鼠标滚轮不是自己的
可手是自己的啊

10 楼 lordhong 2007-07-13  

final String NULL_STRING = null;
final String EMPTY_STRING = "";

if
   (      
        msisdn      
        . // 比==null多了3行, 语意更明显:)
        equals
        (
        NULL_STRING
        )    
   )        
         {        
        return       
        mapping      
        .      
        findForward      
        (      
        "r"  
        +   
        "e"  
        +   
        "t"  
        +   
        "u"  
        +   
        "r"  
        +   
        "n"  
        )      
        ;        
         }        
         if     
        (      
        msisdn      
        .      
        equals      
        (      
        EMPTY_STRING    
        )      
        )        
         {        
        return     
        mapping      
        .      
        findForward      
        (      
        "r"  
        +   
        "e"  
        +   
        "t"  
        +   
        "u"  
        +   
        "r"  
        +   
        "n"     
        )      
        ;        
         }  

楼主电信行业的?

9 楼 rainy14f 2007-07-13  

注释！注释！

8 楼 leon_a 2007-07-13  

还是这样好一些

if  
        (   
        msisdn   
        ==   
        null  
        )     
         {     
        return    
        mapping   
        .   
        findForward   
        (   
        "r"
        +
        "e"
        +
        "t"
        +
        "u"
        +
        "r"
        +
        "n"
        )   
        ;     
         }     
         if  
        (   
        msisdn   
        .   
        equals   
        (   
        ""  
        )   
        )     
         {     
        return  
        mapping   
        .   
        findForward   
        (   
        "r"
        +
        "e"
        +
        "t"
        +
        "u"
        +
        "r"
        +
        "n"  
        )   
        ;     
         }   

7 楼 weiqingfei 2007-07-13  

这样才彻底,给钱吧。

yiding_he 写道

if
(
msisdn
==
null
)  
 {  
return 
mapping
.
findForward
(
"return"
)
;  
 }  
 if
(
msisdn
.
equals
(
""
)
)  
 {  
return
mapping
.
findForward
(
"return"
)
;  
 } 

6 楼 luyunjian 2007-07-13  

...代码统计工具应该能识别的吧

5 楼 yiding_he 2007-07-13  

if
(
msisdn
==
null
) 
{ 
return
mapping
.findForward
("return"); 
} 
if
(
msisdn
.equals
("")
) 
{ 
return
mapping
.findForward
("return"); 
}

4 楼 weiqingfei 2007-07-13  

这样写，钱更更多？

       if
       (
           msisdn==null
       )    
       {    
           return mapping.findForward(forward_Return);    
       }    

3 楼 ddandyy 2007-07-13  

好像我们不是按照行数统计的  也写成这样

2 楼 zhangzhaofeng 2007-07-13  

.....

1 楼 codeutil 2007-07-13  

按行数算钱?
那应该再加两行:

String msisdnDefault = "";
String forward_Return="return";
      if(msisdn==null)  
       {  
             return mapping.findForward(forward_Return);  
         }  
           
        if(msisdnDefault.equals(msisdn))  
         {  
             return mapping.findForward(forward_Return);  
        }