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

class CADApp {  
void drawShapes(Graphics graphics, Shape shapes[]) { 
for (int i = 0; i < shapes.length; i++) {
if (shapes[i] instanceof Line) { 
画出形状;   
} else if (shapes[i] instanceof Rectangle) { 
画出形状;   
} else if (shapes[i] instanceof Circle) {
画出形状;   
}
}
}
} 

class CADApp {  
void drawShapes(Graphics graphics, Shape shapes[]) {
for (int i = 0; i < shapes.length; i++) {
画出形状;
}
}
}  

class CADApp {
void drawShapes(Graphics graphics, Shape shapes[]) {
for (int i = 0; i < shapes.length; i++) {
shapes[i].draw(graphics);
}
}
} 

abstract class Shape {
abstract void draw(Graphics graphics);
}   
class Line extends Shape {
Point startPoint; 
Point endPoint; 
void draw(Graphics graphics) {
graphics.drawLine(getStartPoint(), getEndPoint());
}
}   
class Rectangle extends Shape {
Point lowerLeftCorner; 
Point upperRightCorner; 
void draw(Graphics graphics) {
graphics.drawLine(...);
graphics.drawLine(...);
graphics.drawLine(...);
graphics.drawLine(...);
}
}   
class Circle extends Shape {
Point center; 
int radius;
void draw(Graphics graphics) {
graphics.drawCircle(getCenter(), getRadius());
}
}

interface Shape {   
void draw(Graphics graphics);
}   
class Line implements Shape {   
...  
}   
class Rectangle implements Shape {  
...  
}   
class Circle implements Shape { 
...  
}   

interface Shape {   
void draw(Graphics graphics);
}   
class Line implements Shape {   
Point startPoint;
Point endPoint;  
void draw(Graphics graphics) {   
graphics.drawLine(getStartPoint(), getEndPoint());   
}
}   
class Rectangle implements Shape {  
Point lowerLeftCorner;   
Point upperRightCorner;  
void draw(Graphics graphics) {   
graphics.drawLine(...);  
graphics.drawLine(...);  
graphics.drawLine(...);  
graphics.drawLine(...);  
}
}   
class Circle implements Shape { 
Point center;
int radius;  
void draw(Graphics graphics) {   
graphics.drawCircle(getCenter(), getRadius());   
}
}   
class CADApp {  
void drawShapes(Graphics graphics, Shape shapes[]) { 
for (int i = 0; i < shapes.length; i++) {
shapes[i].draw(graphics);
}
}
}