【民航精品课件 软件质量】代码审查(教师)

18) #define max(a, b) ((a > b) ? a : b) /*宏定义函数*/ /*错误，
宏单数应该用括号围起来*/
19) void calculate(Triangle* pTriangle) /*判断三角形的类型*/ 20) {
21)
/*缺少初始化语句 pTriangle->type = INVALIDATE;*/
/******************************************************* **计算 3 个三角形的周长，打印出最大周长和最小周长， 和每个三角形的类型（普通，非法，等腰，等边**
*******************************************************/ 1) #include <stdlib.h> 2) #include <stdio.h>
22) if( pTriangle->x>0 && pTriangle->y>0 &&
pTriangle->z>0)
23) {
24)
if( pTriangle->z > fabs(pTriangle->x-pTriangle->y)
25)
&& pTriangle->z < pTriangle->x-pTriangle->y ) /*判
14) short girth; /*周长*/
15) int type;/*类型*/ 16) }Triangle;
17) #define min(a, b) ((a < b) ? a : b) /*宏定义函数*/ /*错误，宏
单数应该用括号围起来。举例#define s(x) x*x ，执
行 s(1+2)结果为 5 ，而不是 9*/
8) typedef struct tagTriangle /*属于公共耦合，建议改成更低级
耦合。如无直接耦合、数据耦合、标记耦合、控制
耦合、外部耦合。可能改成标记耦合。*/
9) { 10) char name[10];
11) int x; /*边长*/
12) int y; /*边长*/
13) int z; /*边长*/
pTriangle->type = NORMAL;
பைடு நூலகம்
40) }
41) }
42) girth = pTriangle->x+pTriangle->y+pTriangle->z;
/*计算三
角形周长*/ /*错误：int 转 short,可能损失精度*/
43) pTriangle->girth = girth; 44) }
printf("%s: 等腰三角形\n", pTriangle->name);
53)
else if(pTriangle->type=EQUILATERAL) /*错误：=应
为==*/ +1
54)
printf("%s: 等边三角形\n", ); /*错
误：.应为->*/
55) } 56) void main(void) 57) { 58) int i; 59) double girth_min, girth_max;
45) void print_triangle(Triangle* pTriangle)
/*将
46) { 47)
运算结果打印出来*/ if(pTriangle->type=NORMAL) /*错误：=应为==*/
48)
printf("%s: 普通三角形\n", pTriangle->name);/*没有
60) Triangle* pTriangle = malloc( 3*sizeof(Triangle) ); /*分配
一块缓存存放三个三角形的数据信息*/ /*缺陷：没
有判断 malloc 是否成功就进行下面的操作*/
61) strcpy( pTriangle[0].name, "triangle01" ); /*将字符串
拷贝到一个字符数组中*/ /*错误，数组越界*/
62) pTriangle[1].x = 3; 63) pTriangle[1].y = 5; 64) pTriangle[1].z = -1; 65) strcpy( pTriangle[1].name, "triangle02" ); 66) pTriangle[2].x = 3; 67) pTriangle[2].y = 3; 68) pTriangle[2].z = 3; 69) strcpy( pTriangle[2].name, "triangle03" );/**/ 70) pTriangle[3].x = 4; 71) pTriangle[3].y = 4; 72) pTriangle[3].z = 4.2;
3) #include <malloc.h> /*缺少了 math.h*/
4) #define NORMAL 1 /*普通三角形*/
5) #define INVALIDATE 2
/*非法三角形*/
6) #define ISOSCELES 3 /*等腰三角形*/
7) #define EQUILATERAL 4 /*等边三角形*/
加括号*/
49)
else if(pTriangle->type=INVALIDATE) /*错误：=应为
==*/
50)
printf("%s: 非法三角形\n", pTriangle->name);
51)
else if(pTriangle->type=ISOSCELES) /*错误：=应为
==*/
52)
断是否构成一个三角形*/ /*算式中应为+号*/
26) {
/*判
断三角形的各种类型*/
27)
if(pTriangle->x == pTriangle->y
28)
&& pTriangle->y==pTriangle->z)
29)
{
30)
pTriangle->type = EQUILATERAL;
31)
}
32)
else if(pTriangle->x==pTriangle->y
33)
|| pTriangle->x==pTriangle->z
34)
|| pTriangle->y==pTriangle->z)
35)
{
36)
pTriangle->type = ISOSCELES;
37)
}
38)
else
39)