• 垃圾代码评析——关于《C程序设计伴侣》6.2(三)


    前文链接:http://www.cnblogs.com/pmer/archive/2012/11/05/2754784.html

    【样本】

    •  输出二维数组结果

      排序完成之后,我们对于数据的处理就已经完成了,最后的任务就是将处理后的数据输出,告诉用户数据的处理结果。自然地,要将二维数组中的数据输出,我们还是利用for语句来循环遍历实现:

    1. //
    2. 
    3. int main() 
    4. {
    5.    //
    6.    
    7.    // 输出结果数据 
    8.    for(int i = 0; i < classnum;++i)
    9.    {
    10.       printf("average score is %.2f :\n",
    11.          getaver(scores[i]));
    12.       
    13.       for(int j = 0; j < stnum; ++j)
    14.       {
    15.          if(0 == scores[i][j])
    16.             break;
    17.          printf("%4d",scores[i][j]);
    18.       }
    19.       printf("\n");
    20.    }
    21.    return 0;
    22. }

      因为此时的二维数组已经是排序完成的,我们只需要利用for语句循环遍历所有数据将其全部输出就可以了。在输出的时候,我们还顺带输出了各个班级的平均成绩,这时我们使用“scores[i]”调用getaver()函数,这也说明它其实就是各个一维数组的首地址。
        ——陈良乔 ,《C程序设计伴侣》,人民邮电出版社,2012年10月,p105~107

    【评析】

      大体上这段代码还过得去,如果是初学者的习作应该能得70分。但在书里这样写有些说不过去了。
      首先,把这么长的代码挤在main()中不妥,很业余。抽象为一个函数完成这个功能为好。
      其次,代码的

    10.       printf("average score is %.2f :\n",
    11.          getaver(scores[i]));

      居然再一次计算了各个班的平均成绩。作为专业程序员,如果没有特别过硬的理由,这绝对是一种奇耻大辱。这一方面反应了作者在思考代码时没有很好地进行数据结构的设计(整天把“算法才是王道”挂在嘴边的都爱犯这种毛病),另一方面反应了作者在算法方面也缺乏全盘考虑就开始写代码了,写到最后也顾不了那么多了,只好在这里随手打了一块难看的补丁。
      代码中还有几处很业余的地方,例如

    15.          if(0 == scores[i][j])
    16.             break;

      这种写法显然是因为作者不懂得&&运算符的运算规则的缘故。(在该书77页,作者是这样讲解&&运算的:“例如:(a>0)&&(b>0) 在计算这个逻辑表达式的值的时候,会根据小括号确定的运算次序(或者表达式的默认运算顺序),首先计算a>0和b>0这两个关系表达式的值,然后逻辑运算符“&&”会根据这两个关系表达式的值最终得出整个逻辑表达式的值。”)不懂得细节就写不出漂亮的代码,神在细节当中。
      其实那个内层循环语句应该这样写:

          for(int j = 0; ( j < stnum ) && ( 0 != scores[i][j] ); ++j)
          {
             printf("%3d ",scores[i][j]);
          }
    

      这里的另一处改动是把printf()中的实参"%4d"改成了"%3d ",这样即使scores[i][j]达到四位输出也不至于乱套。
      最后一处

    19.       printf("\n");

      还是写

           putchar ('\n');
    

     为好。当然,这只是一个小细节。

    【重构】

      绝对不要上来就写 “6个班,每个班的人数不等,但最多不超过100个” 这种傻乎乎的代码,理由很简单,根本无法测试!因此这里将这个条件改为“4个班,每个班的人数不等,但最多不超过3个”。这样便于测试,将来修改这两个常数也很容易。
      其次,既然原来的代码使用了qsort()标准函数,这里也将同样使用。
      第三,设定分数不可能为负值,输入负值表示该班级分数输入完毕。
      下面边分析边写代码。
      首先,用枚举常量描述问题的规模:

    enum {
           STNUM = 3 ,     
           CLNUM = 4 
         };
    

       使用如下的数据结构描述班级:

    typedef
    struct {
             int scores[STNUM] ;
             double average ;
           } 
    CLASS ;
    

       这样,所有的班级就构成了一个数组。

    CLASS classes[CLASS_NUM];
    

       总体思路非常简单:

    int main( void )
    {
       CLASS classes[CLASS_NUM];
       //输入成绩
       //计算平均分
       //调用qsort()函数排序
       //输出排序后各班成绩 
       return 0;
    }
    

       下面是完成之后的代码:

    #include <stdio.h>
    #include <stdlib.h>
    
    enum {
           STUDENT_NUM = 3 ,     
           CLASS_NUM   = 4 
         };
         
    typedef
    struct {
             int scores[STUDENT_NUM] ;
             double average ;
           } 
    CLASS ;    
    
    #define SIZE(A) ( sizeof(A) / sizeof((A)[0]))
    
    void input_gr( CLASS [] , size_t );
    void input_cl( int   [] , size_t );
    void cal_gr ( CLASS [] , size_t );
    double cal_cl ( int   [] , size_t ) ;
    int cmp( const void * , const void * );
    void output_gr( CLASS [] , size_t );
    void output_cl( int   [] , size_t );
    
    int main( void )
    {
       CLASS classes[CLASS_NUM];
       
       input_gr( classes , SIZE(classes) );//输入成绩
       cal_gr ( classes , SIZE(classes) );//计算平均分
       qsort( classes , SIZE(classes) , sizeof(CLASS) , cmp  );//调用qsort()函数排序
       output_gr( classes , SIZE(classes) );//输出排序后各班成绩 
       return 0;
    }
    
    void output_cl( int  scr [] , size_t n)
    {
       size_t i ;
       for( i = 0 ; (i < n)&&( scr[i] >= 0 )  ; i ++ )
          printf( "%3d " , scr[i] );
    
       putchar('\n') ;
    }
    
    void output_gr( CLASS cl[] , size_t n)
    {
       size_t i ;
       for( i = 0 ; i < n ; i ++ )
       {
          output_cl( cl[i].scores , SIZE(cl[i].scores) );
          printf( "平均分为:%f。\n" , cl[i].average ) ;
       }   
    }
    
    int cmp( const void *elem1 , const void *elem2 )
    {
       if(((CLASS *)elem1)->average > ((CLASS *)elem2)->average )
          return 1;
    
       if(((CLASS *)elem1)->average < ((CLASS *)elem2)->average )
          return -1;
    
       return 0;
    }
    
    double cal_cl ( int   scr[] , size_t n )
    {
       size_t i  ;
       int total ;
       
       if( n == 0 )
          return 0.;
       
       for( 
            i = 0 , total = 0 ; 
            ( i < n ) && ( scr[i] >= 0 ) ; 
            i ++ 
          )
       {   
          total += scr[i] ;
       }
       return (double)total/(double)i;
    }
    
    void cal_gr ( CLASS cl[] , size_t n )
    {
       size_t i ;
       for( i = 0 ; i < n ; i ++ )
          cl[i].average = cal_cl ( cl[i].scores , SIZE(cl[i].scores) ) ;
    }
    
    void input_cl( int  scr [] , size_t n)
    {
       size_t i ;
       for( i = 0 ; i < n ; i ++ )
       {
          scanf( "%d" , scr + i );
          if( scr[i] < 0 )
             return ;
       }   
    }
    
    void input_gr( CLASS cl[] , size_t n)
    {
       size_t i ;
       for( i = 0 ; i < n ; i ++ )
       {
          printf( "输入%d班成绩,输入负数表示结束:\n" , i + 1 ) ;
          input_cl( cl[i].scores , SIZE(cl[i].scores) );
       }
    }
    
  • 相关阅读:
    温故vue对vue计算属性computed的分析
    bootStrap Table 如何使用
    css 的一些知识点的整理
    css 宽高自适应的div 元素 如何居中 垂直居中
    BOM,Dom 回顾
    DOM
    字符串的一些常用方法 string
    js if for 详解 获取元素方式 及一些js 基础知识
    Java入门1
    python字符串
  • 原文地址:https://www.cnblogs.com/pmer/p/2756205.html
Copyright © 2020-2023  润新知