xmx0632

永久域名 http://xmx0632.javaeye.com

Re: 这样的TDD实践方式有问题？请教大家的 ... | html里的disabled误区

2007-07-13

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

关键字: 开发

今天又看到这样的代码：

java 代码

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

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

13:17
浏览 (3386)
评论 (14)
进入论坛
相关推荐


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"     
        )      
        ;        
         }

楼主电信行业的?

rainy14f 2007-07-13

注释！注释！

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"  
        )   
        ;     
         }

weiqingfei 2007-07-13

这样才彻底,给钱吧。

yiding_he 写道

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

luyunjian 2007-07-13

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

yiding_he 2007-07-13

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

weiqingfei 2007-07-13

这样写，钱更更多？

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

ddandyy 2007-07-13

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

zhangzhaofeng 2007-07-13

.....

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);  
        }