垃圾代码评析——关于《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) );
}
}
