Java敏捷开发技巧之消除代码异味

发表于:2011-9-28 09:33

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

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

#
java
分享:

  异味这个词,可能有点抽象,我们先看一下下面的例子。

  这是一个CAD系统。现在,它已经可以画三种形状了:线条,长方形和圆。先认真的看一下下面的代码:

class Shape {  
final static int TYPELINE = 0;  
final static int TYPERECTANGLE = 1;
final static int TYPECIRCLE = 2;
int shapeType;  
//线条的开始点
//长方形左下角的点
//圆心
Point p1;
//线条的结束点
//长方形的右上角的点
//如果是圆的话,这个属性不用
Point p2;
int radius; 
}  
class CADApp { 
void drawShapes(Graphics graphics, Shape shapes[]) {
for (int i = 0; i < shapes.length; i++) {
switch (shapes[i].getType()) {  
case Shape.TYPELINE: 
graphics.drawLine(shapes[i].getP1(), shapes[i].getP2()); 
break;
case Shape.TYPERECTANGLE:
//画四条边
graphics.drawLine(...);  
graphics.drawLine(...);  
graphics.drawLine(...);  
graphics.drawLine(...);  
break;
case Shape.TYPECIRCLE:
graphics.drawCircle(shapes[i].getP1(), shapes[i].getRadius());
break;
}
}
}
}

  代码都是一直在改变的,而这也是上面的代码会碰到的一个问题。

  现在我们有一个问题:如果我们需要支持更多的形状(比如三角形),那么肯定要改动Shape这个类,CADApp里面的drawShapes这个方法也要改。

  好,改为如下的样子:

class Shape { 
final static int TYPELINE = 0;
final static int TYPERECTANGLE = 1;
final static int TYPECIRCLE = 2;
final static int TYPETRIANGLE = 3;
int shapeType; 
Point p1;  
Point p2;  
//三角形的第三个点.
Point p3;  
int radius;

class CADApp {
void drawShapes(Graphics graphics, Shape shapes[]) {
for (int i = 0; i < shapes.length; i++) {
switch (shapes[i].getType()) {
case Shape.TYPELINE:
graphics.drawLine(shapes[i].getP1(), shapes[i].getP2());
break;
case Shape.TYPERECTANGLE:
//画四条边.
graphics.drawLine(...);
graphics.drawLine(...);
graphics.drawLine(...);
graphics.drawLine(...);
break;
case Shape.TYPECIRCLE:
graphics.drawCircle(shapes[i].getP1(), shapes[i].getRadius());
break;
case Shape.TYPETRIANGLE:
graphics.drawLine(shapes[i].getP1(), shapes[i].getP2());
graphics.drawLine(shapes[i].getP2(), shapes[i].getP3());
graphics.drawLine(shapes[i].getP3(), shapes[i].getP1());
break;
}  
}  
}  
}

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

关注51Testing

联系我们

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

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

沪ICP备05003035号

沪公网安备 31010102002173号