图书馆仿真的重构（三）------类的职责讨论与对象文件的封装内容

      再一次的重构，这次就是讨论类的职责和对象文件的封装内容。类的职责，指的是我们一个类到底应该做什么？做多少？以我的图书馆仿真为例，显然，主类的职责之一就是显示结果，就是图书馆图书列表。但是，在图书馆列表这个部分，需要实现很多方法，而这些方法是否都需要在主类里实现呢？就像前面的例子那样，这次我们依然是有三个类：RatingBook,BookDepot和main，那么，它们各自应该承担的职责是什么呢？我们可以将目光先放在那些涉及到RatingBook的方法上。首先，RatingBook就像所有的对象文件一样，应该有设置数据属性和返回数据属性的方法,当然还有构造器。一般，一开始设计类的时候应该遵守的原则就是，职责尽可能简单，这样可读性才会高。

      接着就是BookDepot。这个类的最大职责就是产生Book，但是由于我自己单方面的设定是在一个String数组里随机产生，为了贴近真实情况，需要将一些重复的去除掉。本来这部分的代码是放在主类里，但是后来我将它放在BookDepot中，因为我觉得，这个类最主要的功能就是能够返回一个BookList，那么，作为附加的要求，返回一个不重复的BookList就能减少代码的逻辑冗余（所谓的逻辑冗余，就是一段代码中包含太多不相关的逻辑），这并没有不妥。而且，我这里将它作为静态方法，是因为我需要在两个地方得到bookList，但是如果我要在两个地方都new一个BookDepot，所以我将它变成静态方法，这样就无需每次 使用时都需创建一个根本没有用到的对象。

     最后就是主类。当然，经过上面这样的修改，主类的逻辑已经简洁很多，它最主要的工作就是对List进行加工，这些加工是可以放在主类里的，如果将它们放在BookDepot里，那么这个对象文件承担的职责就过多了，而且有些并不是我们这个对象产生文件应该去处理的，它最主要的功能就是产生我们想要的不重复List，至于这个List的加工，应该交给其他类处理，当然，如果我们想要写个加工类也是可以的，但是这样主类就只是启动加工类而已，这种处理我觉得是将类的职责过分的细分，类的职责的确需要细分，这是重构的很重要的原则，但是盲目的细分并不是一个正确的选择，所以，主类还是必须承担一定的职责。

      综合上述，我认为，类应该承担多少职责，这个问题应该与我们的逻辑分类有关，就像我的例子那样，所有的逻辑可以分为BookList的产生，RatingBook的设置，以及对BookList的加工这三部分，那么，就有三个职责类，然后由这三个职责类来处理这些逻辑。这对于对象文件来说同样适用，对象文件封装的逻辑应该是与其自身相关的逻辑，但是，必须要意识到一点，就是对象文件的逻辑最好不要包含有其他对象的信息，尤其是对象的创建。

       最后，就是想要说一下方法名的命名问题，我认为，可读性高的代码，并不是一定要有详尽的注释，事实上，方法名和参数列表就已经是最好的注释了，只要方法名得当，我们完全可以将方法名当做一个注释来看，所以，方法名是很重要的，当然，方法名是要根据方法的功能和参数列表来决定，这个真的是需要很高的素养才能做到，像是我的例子，就真的是很难达到要求，这需要我们在实践中不断摸索。

      这次的重构，再次暴露了我编程素养极低的弱点，甚至连一些基本的常识都不知道，比如说，一般名字前带有m的是类的成员变量，带有s的是static成员，但是我都不知道，一直以来都是随便命名，可见没有多看别人代码的坏处是如此明显！而且，我还发现，如果在代码创建这个类只是为了调用它里面的一个方法，就没有其他用途，那么，请把这个方法设为static，因为这样就不用每次都创建无用的类实例。

      这个例子远远没有完成，因为它里面还有很多我需要去思考的东西，直到我认为它已经几乎没有问题。

     

public class Library {
    public static void main(String[] args) throws InterruptedException,
            IOException {
        searchBookAndGetResultOfSearch();
    }
    private static void searchBookAndGetResultOfSearch(String name)
            throws IOException {
        String name = enterAndGetName();
        List<RatingBook> list = getBookListFromZhongZhuanTaiByName(name);
        List<RatingBook> depotList = BookDepot.createBookListFromDepot(10, 1);
        List<RatingBook> bookList = getListContainsName(name, depotList);
        for (RatingBook libraryBook : bookList) {
            for (RatingBook zhongZhuanTaiBook : list) {
                if ((libraryBook.getName())
                        .equals((zhongZhuanTaiBook.getName()))) {
                      libraryBook.setRating(zhongZhuanTaiBook.getRating());
                }
            }
        }
        showList("图书馆里的图书列表：", bookList);
    }
   
    private static String enterAndGetName() throws IOException {
       
 BufferedReader name = new BufferedReader(new InputStreamReader(System.in));
  String entername = name.readLine().toLowerCase();
       return entername;
   }

    private static List<RatingBook> getBookListFromZhongZhuanTaiByName(
            String name) {
        List<RatingBook> list = BookDepot.createBookListFromDepot(10, 6);
        List<RatingBook> containsBook = getListContainsName(name, list);
        showList("中转台图书列表:", list);
        showList("中转台符合条件的图书：", containsBook);
        return list;
    }

    private static List<RatingBook> getListContainsName(String name,
            List<RatingBook> list) {
        List<RatingBook> bookList = new ArrayList<RatingBook>();
        for (RatingBook ratingBook : list) {
            String book = ratingBook.getName().toLowerCase();
            if (book.contains(name)) {
                bookList.add(ratingBook);
            }
        }
        return bookList;
    }

    private static void showList(String string, List<RatingBook> list) {
        sortBookByRating(list);
        System.out.println(string);
        for (RatingBook book : list) {
            System.out.println(book.getName() + "评分:" + book.getRating());
        }
    }

    private static void sortBookByRating(List<RatingBook> list) {
        for (int i = 0; i < list.size() - 1; i++) {
            for (int j = i + 1; j < list.size(); j++) {
                if ((list.get(i).getRating()) <= (list.get(j).getRating())) {
                    String temp = list.get(i).getName();
                    int rating = list.get(i).getRating();
                    list.get(i).setName(list.get(j).getName());
                    list.get(i).setRating(list.get(j).getRating());
                    list.get(i).setName(temp);
                    list.get(i).setRating(rating);
                }
            }
        }
    }
}

public class RatingBook {
    private int mRating = 0;
    private String mName = "";

    public RatingBook() {}

    public RatingBook(String name, int rating) {
        mName = name;
        mRating = rating;
    }

    public void setRating(int rating) {
        mRating = rating;
    }

    public void setName(String name) {
        mName = name;
    }

    public String getName() {
        return mName;
    }

    public int getRating() {
        return mRating;
    }
}

public class BookDepot {
    static String[] sBookDepot = { "java and javascript", "java and C++",
            "java and c", "sql", "CEHNJINSHENG", "java", "C++", "javascript",
            "Java", "java and C", "java and SQL", "chenjinsehng", "int" };

    static List<RatingBook> createBookListFromDepot(int num, int rating) {
        List<RatingBook> list = new ArrayList<RatingBook>();
        int len = new Random().nextInt(num);
        for (int i = 0; i < len; i++) {
            RatingBook book = new RatingBook(sBookDepot[len],
                    returnRandom(rating));
            list.add(book);
        }
        List<RatingBook> newList = removeSameBookFromList(list);
        return newList;
    }

    private static List<RatingBook> removeSameBookFromList(List<RatingBook> list) {
        Map<String, Integer> map = createMapWithSingalKeyOfBookName(list);
        List<RatingBook> bookList = new ArrayList<RatingBook>();
        @SuppressWarnings("rawtypes")
        Set entries = map.entrySet();
        if (entries != null) {
            @SuppressWarnings("rawtypes")
            Iterator iterator = entries.iterator();
            while (iterator.hasNext()) {
                @SuppressWarnings("rawtypes")
                Map.Entry entry = (Entry) iterator.next();
                String key = (String) entry.getKey();
                Integer value = (Integer) entry.getValue();
                RatingBook book = new RatingBook();
                book.setBook(key, value);
                bookList.add(book);
            }
        }
        return bookList;
    }

    private static Map<String, Integer> createMapWithSingalKeyOfBookName(
            List<RatingBook> list) {
        Map<String, Integer> map = new HashMap<String, Integer>();
        for (RatingBook book : list) {
            map.put(book.getName(), book.getRating());
        }
        return map;
    }
}