Chinaunix首页 | 论坛 | 博客
  • 博客访问: 136415
  • 博文数量: 22
  • 博客积分: 1326
  • 博客等级: 中尉
  • 技术积分: 258
  • 用 户 组: 普通用户
  • 注册时间: 2010-03-02 00:52
文章分类
文章存档

2012年(1)

2011年(21)

分类: C/C++

2011-11-11 20:17:01

几乎每次在代码中发现flag变量,我总是能嗅到一股馊味。不管你闻没闻到,反正我闻到了。

在代码中,flag通常作为标志变量的名字。但问题在于,不是所有的问题或代码都需要使用这种标志变量,更不是使用标志变量一定要用flag这个名字。需要使用标志变量的问题只有一小部分而已,况且即使使用标志变量,取flag这个名字也往往是一种草率的下策。

然而由于基础不扎实及某些书籍的误导等缘故,不少初学者往往特别善于滥用flag。几乎在任何场合他们都有能力把这个flag使用上且用得出神入化般的别扭,就如同用if语句和goto语句构造循环结构一样,甚至比这更坏。这种情况是由于不熟悉基本的代码结构所致。

另一种滥用flag的情况则是,由于最初的设计千疮百孔漏洞百出,等到发现时用flag打上丑陋的补丁。

下面开始阅读代码,请自备口罩。

样本代码1

  1. #include <stdio.h>
  2. #define N 10
  3. int main()
  4. {
  5.  int num [N],number,flag=1,c;
  6.  char name[N][8];
  7.  /*……*/
  8.  while(flag==1)
  9.     {printf("\ninput number to look for:");
  10.      scanf("%d",&number);
  11.      search(number,num,name);
  12.      printf("continue ot not(Y/N)?");
  13.      getchar();
  14.      c=getchar();
  15.      if(c=='N'||c=='n')
  16.        flag=0;
  17.     }
  18.   /*……*/
  19.   return 0;
  20. }

——谭浩强 ,《C程序设计(第四版)学习辅导》,清华大学出版社,20107月,p92

这段代码中的while语句是明显的缺陷,因为从这个程序段的功能来说使用do-while才是自然的表达。

  1. #include <stdio.h>
  2. #define N 10
  3. int main( void )
  4. {
  5.    int num [N],number,flag=1,c;
  6.    char name[N][8];
  7.    /*……*/
  8.    do{
  9.        printf("\ninput number to look for:");
  10.        scanf("%d",&number);
  11.        search(number,num,name);
  12.        printf("continue ot not(Y/N)?");
  13.        getchar();
  14.        c=getchar();
  15.        if(c=='N'||c=='n')
  16.          flag=0;
  17.        }
  18.    while(flag==1);
  19.     /*……*/
  20.    return 0;
  21. }

其次flag标志变量根本没有必要,代码完全可以写成

  1. #include <stdio.h>
  2. #define N 10
  3. int main( void )
  4. {
  5.    int num [N],number,flag=1,c;
  6.    char name[N][8];
  7.    /*……*/
  8.    do{
  9.       printf("\ninput number to look for:");
  10.       scanf("%d",&number);
  11.       search(number,num,name);
  12.       printf("continue ot not(Y/N)?");
  13.       getchar();
  14.       c = getchar() ;
  15.    }
  16.    while(!(c=='N'||c=='n'));
  17.     /*……*/
  18.    return 0;
  19. }

最后,getchar();也用得十分蹩脚,它的用意是读入前面调用scanf("%d",&number)输入一个十进制整数后输入的(回车换行)。但实际上如果程序使用者输入十进制整数后又顺手按了一个空格再回车,这个“getchar();”就会因为读入了空格而导致后面的“c = getchar() ;”读取NnYy这几个文字字符的企图遭到失败。更好的写法应该是

  1. #include <stdio.h>
  2. #define N 10
  3. int main( void )
  4. {
  5.    int num [N],number,flag=1,c;
  6.    char name[N][8];
  7.    /*……*/
  8.    do{
  9.        printf("\ninput number to look for:");
  10.        scanf("%d",&number);
  11.        search(number,num,name);
  12.        printf("continue ot not(Y/N)?");
  13.        scanf("\n%c",&c );
  14.    }
  15.    while(!(c=='N'||c=='n'));
  16.     /*……*/
  17.    return 0;
  18. }

与样本代码1对比一下,不难发现样本代码1中的flag有多么拙劣。

样本代码2

  1. #define N 10
  2. char str[N];
  3. int main()
  4. {
  5. /*……*/
  6. int i,flag;
  7. for(flag=1;flag==1;)
  8.   {scanf("%s",&str);
  9.    if(strlen(str)>N)
  10.     printf("string too long,input again!");
  11.    else
  12.     flag=0;
  13.   }
  14. /*……*/
  15. }

——谭浩强 ,《C程序设计(第四版)学习辅导》,清华大学出版社,20107月,p85

这段代码的flag用得更加可惊可愕,它甚至不如写成功能等价的

  1. #define N 10
  2. char str[N];
  3. int main(void)
  4. {
  5.    /*……*/
  6.    int i,flag;
  7.    do {
  8.       scanf("%s",&str);
  9.       if(strlen(str)>N)
  10.         printf("string too long,input again!");
  11.       else
  12.         break;
  13.    }
  14.    while(1);
  15.    /*……*/
  16. }

由此不难看出那个flag有多么多余。然而这个写法依旧啰嗦,更简洁的写法是:

  1. #define N 10
  2. char str[N];
  3. int main(void)
  4. {
  5.    /*……*/
  6.    do {
  7.       scanf("%s",&str);
  8.    }
  9.    while(strlen(str)>N && printf("string too long,input again!"));
  10.    /*……*/
  11. }

这写法虽然简洁,然而依然带着样本代码中原来的毛病,这个毛病就是scanf("%s",&str);中的&str。因为str的定义是 char str[10];,而与转换格式%s相对应的参数应该是char *类型的指针,所以这个调用应该写为scanf("%s",str);

即便如此,代码依然是错误的。原代码企图在strlen(str)>N的情况下重新输入,这是幼稚的一厢情愿,而且有着双重的错误。从逻辑来说,strlen(str) 等于N时就已经是越界使用数组了。所以strlen(str)>N这个关系表达式不应该使用“>”运算。从实践的角度来说strlen(str)>=N时的越界已然对内存中的数据存储造成了破坏,程序以后的行为是建立在错误的基础上,最后的结果必然是错误的,至少是不可靠的。而结果不可靠,对于程序来说就是一种错误。

那么,这段代码应该怎样写呢?由于样本代码2要解决的问题是对十个字符排序,这里的要求是输入10个字符。这根本不用写一个循环语句反复折腾,只需简单地

  1. #define N 10
  2. char str[N];
  3. int main(void)
  4. {
  5.    /*……*/
  6.    int i;
  7.    scanf("%10c" , str ) ;
  8.    /*……*/
  9. }

即可。

当然,也许有人会指责scanf("%10c" , str )没有用上N这个宏,这个指责确实有道理。其实用N这个宏来写scanf("%10c" , str )也并非什么难事,代码可以这样写:

      #define N 10
  1. #define FMT(x) ("%"NUM(x)"c")
  2. #define NUM(x) #x
  3. char str[N];
  4. int main(void)
  5. {
  6.    /*……*/
  7.    int i;
  8.    scanf( FMT(N) , str ) ;
  9.    /*……*/
  10. }

阅读(3756) | 评论(0) | 转发(0) |
给主人留下些什么吧!~~