边界测试——让BUG现形

发表于:2011-12-02 16:15

字体: | 上一篇 | 下一篇 | 我要投稿

 作者:garbageMan    来源:51Testing软件测试网采编

  题目:写一个函数,输入一行字符,将此字符串中最长的单词输出。

#include <stdio.h>

#include <string.h>

int main()

{int alphabetic(char);

int longest(char []);

int i;

char line[100];

printf("input one line:\n");

gets(line);

printf("The longest word is:");

for(i=longest(line);alphabetic(line[i]);i++)

printf("%c",line[i]);

printf("\n");

return 0;

}



int alphabetic(char c)

{

if((c>='a'&&c<='z')||(c>='A'&&c<='z'))

return(1);

else

return(0);

}



int longest(char string[])

{int len=0,i,length=0,flag=1,place=0,point;

for(i=0;i<=strlen(string);i++)

if(alphabetic(string[i]))

if(flag)

{point=i;

flag=0;

}

else

len++;

else

{flag=1;

if(len>=length)

{length=len;

place=point;

len=0;

}

}

return(place);

}

  运行结果:

  input a line:            
  I am a student.          
  The longest word is : student

  题目要求“写一个函数,输入一行字符,将此字符串中最长的单词输出”,可是无论alphabetic()还是longest()函数都没有实现“输入一行字符,将此字符串中最长的单词输出”这个功能要求。疑惑很久,发现实现这个功能的函数居然是main()。这就难免让人贻笑大方了。因为按照常规的惯例,要求写一个函数实现某个功能,从来不是要求写main(),尽管不能说main()不是“一个函数”。然而如果是要求main()完成的事情,通常是作为一个完整的问题提出的,不会提出“写一个函数”这样的要求。如果硬要狡辩“写一个函数”也不排除是写main(),就牵强的近乎强词夺理了。不过设若真的有人如此嘴硬,你还真拿他没什么办法。

  既然是不见棺材不掉泪,那就不妨继续往下看。

  在代码中一眼瞄见了flag这个变量。经验表明,凡是有这个flag变量的代码,80%以上都是垃圾代码。道理很简单:首先,多数问题根本不需要设置这个别别扭扭标志变量,只有那些善于把自己的思维扭曲得如同烂麻花一样的人才喜欢时不时地祭出flag这个破烂的法宝。其次,即使需要设置标准变量,优秀的代码作者也不会使用这个含义模糊不清的名字作为标志变量名,而会用一个更贴切、意义更明确恰当更适合描述问题的名字。所以,一般来说,flag往往反映了代码的垃圾度。

31/3123>
《2023软件测试行业现状调查报告》独家发布~

关注51Testing

联系我们

快捷面板 站点地图 联系我们 广告服务 关于我们 站长统计 发展历程

法律顾问:上海兰迪律师事务所 项棋律师
版权所有 上海博为峰软件技术股份有限公司 Copyright©51testing.com 2003-2024
投诉及意见反馈:webmaster@51testing.com; 业务联系:service@51testing.com 021-64471599-8017

沪ICP备05003035号

沪公网安备 31010102002173号