敏捷开发技巧消除代码异味.docxVIP

敏捷开发技巧-消除代码异味 来源于 JavaResearch 本文通过简单通俗的例子 , 告诉我们如何判断代码的稳定性和代码中的异类, 并且如何重构此类代码. 异味这个词,可能有点抽象,我们先看一下下面的例子 这是一个 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: getP2()); graphics.drawLine(shapes[i].getP1(), shapes[i]. break; case Shape.TYPERECTANGLE: //画四条边graphics.drawLine(...); graphics.drawLine(...); graphics.drawLine(...); graphics.drawLine(...); break; case Shape.TYPECIRCLE: ].getRadius()); graphics.drawCircle(shapes[i].getP1(), shapes[i 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].ge tP2()); getRadius()); tP2()); tP3()); tP1());  break; case Shape.TYPERECTANGLE: // 画 四 条 边 . graphics.drawLine(...); graphics.drawLine(...); graphics.drawLine(...); graphics.drawLine(...); break; case Shape.TYPECIRCLE: graphics.drawCircle(shapes[i].getP1(), shapes[i]. break; case Shape.TYPETRIANGLE: graphics.drawLine(shapes[i].getP1(), shapes[i].ge graphics.drawLine(shapes[i].getP2(), shapes[i].ge graphics.drawLine(shapes[i].getP3(), shapes[i].ge break; } } } } 如果以后要支持更多的形状,这些类又要改动??，这可不是什么好事情！ 理想情况下，我们希望当一个类，一个方法或其他的代码设计完以后，就不 用再做修改了。它们应该稳定到不用修改就可以重用。现在的情况恰好相反！ 每当我们增加新的形状，都得修改 Shape 这个类，跟 CA