打印

程序写法问题

[复制链接]
847|2
手机看帖
扫描二维码
随时随地手机跟帖
跳转到指定楼层
楼主
txcy|  楼主 | 2012-9-14 17:45 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式
不久前刚做了一个项目,这两天在看effective C++,结合项目代码突然想到了一个问题,假如我要写一个函数,在函数内部我一般会检查参数的正确性,对于不同的参数,可能有不同的返回值,这个时候就会遇到到底在函数哪个部分去返回,我这儿以一个求字符串长度的函数说明情况...
这是第一种,做出判断后,如果条件出错,直接return......
int mystrlen(char
*str)
{  
   
int count =
0;
   
if (str == NULL)
   {
      
return
-1;
   }  

   
if (str ==
0)
   {
      
return
0;
   }  

   
while(str !=
0 )   {      count++;
      str
++;
   }  
   
return count;
}

这是第二种,先设置一个变量,对变量赋值,只有一个return.....
int mystrlen(char
*str)
{  
   
int ret;
   
if (str == NULL)
   {
      ret
=
-1;
   }  
   
else
if (str ==
0)
   {
      ret
=
0;
   }  
   
else
   {
      ret
=
0;
      
while(str !=
0 )      {         ret++;
         str
++;
      }
   }
   
return ret;
}  

这是第三种,使用goto语句:
int mystrlen(char
*str)
{
   
int ret;
   
if (str == NULL)
   {
      ret
=
-1;
      
goto _RET;
   }

   
if (str ==
0)
   {
      ret
=
0;
      
goto _RET;
   }
      
   
   
while(str !=
0 )
   {  
      ret
++;
      str
++;
   }  

_RET:
   
return ret;
}

哪种写法更好?

相关帖子

沙发
yybj| | 2012-9-14 18:09 | 只看该作者
12都差不多,看个人喜好,3最好不要用

使用特权

评论回复
板凳
sinadz| | 2012-9-14 18:13 | 只看该作者
3最好不要用,
个人倾向1,理由:
1.变量在定义时最好初始化。
2:条件不满足立即返回,否则别人可能会花很长时间看ret = -1有什么作用,其实作用很简单就是返回-1;一目了然

使用特权

评论回复
发新帖 我要提问
您需要登录后才可以回帖 登录 | 注册

本版积分规则

274

主题

2106

帖子

0

粉丝