学习 重构--改善既有的代码

    读书笔记，记录自己的点点滴滴！学无止境，记录自己的每一个脚步！

 今天看了书的第一小章，就让我感觉自己井底之蛙了，自己感觉已近挺不错的代码，在真正的程序员里居然那么的糟糕！

     首先说起书中的第一个例子：

                  这是一个影片出租店用的程序，计算每一位顾客的消费金额并打印报表（statement）。操作者告诉程序：顾客租了哪些影片、租期多长，程序便根据租赁时间和影片类型算出费用。影片分为三类：普通片、儿童片和新片。除了计算费用，还要为常客计算点数；点数会随着「租片种类是否为新片」而有不同。

  

public class Movie {
 
  public static final int  CHILDRENS = 2;
  public static final int  REGULAR = 0;
  public static final int  NEW_RELEASE = 1;
 
  private String _title;        //名称
  private int _priceCode;        //价格（代号）
 
  public Movie(String title, int priceCode) {
      _title = title;
      _priceCode = priceCode;
  }
 
                                       
  public int getPriceCode() {
      return _priceCode;
  }
 
  public void setPriceCode(int arg) {
    _priceCode = arg;
  }
 
  public String getTitle (){
      return _title;
  };

Rental（租赁） 
Rental class 表示「某个顾客租了一部影片」
class Rental {
    private Movie _movie;        //影片
    private int _daysRented;        //租期
    public Rental(Movie movie, int daysRented) {
      _movie = movie;
      _daysRented = daysRented;
    }
    public int getDaysRented() {
      return _daysRented;
    }
    public Movie getMovie() {
      return _movie;
    } 
}
Customer（顾客）
 
Customer class 用来表示顾客。就像其他classes 一样，它也拥有数据和相应的访问函数（accessor）：
 
class Customer {
  private String _name;                        //姓名
  private Vector _rentals = new Vector();        //租借记。
 
  public Customer (String name){
      _name = name;
  };
 
  public void addRental(Rental arg) {
    _rentals.addElement(arg);
  }
  public String getName (){
      return _name;
  };
 
Customer「还提供了一个用以制造报表的函数（method），图1.2显示这个函数带来的交互过程（interactions ）。完整代码显示于下一页。
 

public String statement() {
       double totalAmount = 0;        //总消费金。
       int frequentRenterPoints = 0;        //常客积点
       Enumeration rentals = _rentals.elements();
       String result = "Rental Record for " + getName() + "\n";
       while (rentals.hasMoreElements()) {
           double thisAmount = 0;
           Rental each = (Rental) rentals.nextElement();        //取得一笔租借记。
 
           //determine amounts for each line
           switch (each.getMovie().getPriceCode()) {        //取得影片出租价格
               case Movie.REGULAR:                        //普通片
                   thisAmount += 2;
                   if (each.getDaysRented() > 2)
                       thisAmount += (each.getDaysRented() - 2) * 1.5;
                   break;
               case Movie.NEW_RELEASE:                //新片
                   thisAmount += each.getDaysRented() * 3;
                   break;
               case Movie.CHILDRENS:                //儿童。
                   thisAmount += 1.5;
                   if (each.getDaysRented() > 3)
                       thisAmount += (each.getDaysRented() - 3) * 1.5;
                   break;
 
           }
 
           // add frequent renter points        （累计常客积点。
           frequentRenterPoints ++;
           // add bonus for a two day new release rental
           if ((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) &&
each.getDaysRented() > 1) frequentRenterPoints ++;
 
           //show figures for this rental（显示此笔租借记录）
           result += "\t" + each.getMovie().getTitle()+ "\t" +
String.valueOf(thisAmount) + "\n";
           totalAmount += thisAmount;
 
       }
       //add footer lines（结尾打印）
       result +=  "Amount owed is " + String.valueOf(totalAmount) + "\n";
       result += "You earned " + String.valueOf(frequentRenterPoints) +
" frequent renter points";
       return result;
 

       这个程序你的第一印象是什么，好还是差 ！ 给我的感觉是已近不错了！功能都实现的了！ 但是看了书的评论：（这个起始程序给你留下什么印象？我会说它设计得不好，而且很明显不符合面向对象精神。对于这样一个小程序，这些缺点其实没有什么关系。快速而随性（quick and dirty ）地设计一个简单的程序并没有错。但如果这是复杂系统中具有代表性的一段， 那么我就真的要对这个程序信心动摇了。Customer 里头那个长长的statement()  做的事情实在太多了，它做了很多原本应该由其他完成的事情。）  自己真的还差很多！ 待续。。。