敏捷开发的必要技巧第3章----消除代码异味

最新推荐文章于 2025-08-10 16:19:46 发布

翻译最新推荐文章于 2025-08-10 16:19:46 发布 · 965 阅读

文章标签：

#敏捷开发 #shapes #class #图形

agile 专栏收录该内容

15 篇文章

订阅专栏

本文通过一个CAD系统的案例，探讨了如何识别并改进代码中常见的“异味”问题，即那些虽然能够运行但存在设计缺陷的代码。文章指出，当需要支持更多形状（如三角形）时，不应修改现有类，而是引入新的类来解决这一问题。

完整下载地址：
http://www.blogjava.net/Files/Wingel/第3章%20除去代码异味.rar

http://wingel.javaeye.com/topics/download/2f7b5864-fca2-42e5-ba3e-453725fcb885

               第3章除去代码异味　

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

    这是一个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这个方法也要改.
好,改为如下的样子:

......

    如果我们想要支持更多的图形（比如：三角形），上面没有一个类需要修改。我们只需要创建一个新的类Triangle就行了。

内容较长，具体的内容在下载链接里面的pdf文件里。