代码重构，lucene实例提炼
?? ? ? 昨天按《lucene in Action》 书中的例子动手运行了一下，也把遇到问题及相关用法作了简单总结。今天来把代码好好的梳理了一下，并对代码进行相关的重构（refactory）。
?? ? ? XP要求我们：测试——编码——重构——测试——编码——重构，我还是没有这种逆瀑布法来作为开发指导，而是以编码——测试——重构——编码——测试——重构来指导自己的开发，并且以keep it simple为原则来设计类。下面就将自己今天的重构过程记一下流水账。
?? ? ? ?一、问题
?? ? ? 1）需求总是变化的，如何设计系统能较好的适应需求变化是软件的设计的根本。昨天实现的两个建立索引与进行搜索的类，其职则是单一的，已符合单一职责法则，但只对文本文件进行建立索引，那么当要对word文件或pdf文件进行建立索引呢？这一需求的变化带来了，要么重新设计新类，要么在原有的Indexer类中进行参数判别来实现对不同文件来建立索引。这显然不符合开闭原则（OCP）。
?? ? ?2） 复用，复用，还是复用。以最少的修改来适应需求的变化，来达到代码的复用是软件重构的目标。现在的两个类显然不能很好的适应用户新的需求，开发人员不能很快修改代码，快速部署来满足用户的新需求。
?
?? ? 从上面两个问题来说，我们需要对代码进行重构：满足用户新的需求（这里是预测到的需求变化点），重构达到代码的最佳复用。
?
?? 二、重构
?? ? ? ?复用人们已提出很好的方案：依赖接口编程或抽象类编程来解决快速适应需求变化。
?
?? ? ?接口类：对外提供统一稳定简单的功能接口；
?? ? ?具体实现类：实现具体需求所要的接口功能；
??具体的用代码来说明：
?? ? 1）接口
?
package com.goodwitkey.seargine.src;import java.io.File;import java.io.IOException;import org.apache.lucene.index.IndexWriter;import org.apache.lucene.queryParser.ParseException;import org.apache.lucene.search.Hits;/** * @author Owner * */public interface Ifileseargine {//建立索引接口public int index(File dataFilesPath,File indexFilePath)throws IOException;public void indexDirectory(IndexWriter indexwr,File dataFilesPath)throws IOException;public void indexFile(IndexWriter writer, File f)throws IOException;//提供搜索接口public Hits search(File indexFilePath,String queryStr)throws IOException,ParseException ;}?
?
?? 2）文本文件的搜索一个具体实现
???/**
* */package com.goodwitkey.seargine.src;import java.io.File;import java.io.FileReader;import java.io.IOException;import java.util.Date;import org.apache.lucene.analysis.standard.StandardAnalyzer;import org.apache.lucene.document.Document;import org.apache.lucene.document.Field;import org.apache.lucene.index.IndexWriter;import org.apache.lucene.queryParser.ParseException;import org.apache.lucene.queryParser.QueryParser;import org.apache.lucene.search.Hits;import org.apache.lucene.search.IndexSearcher;import org.apache.lucene.search.Query;import org.apache.lucene.store.Directory;import org.apache.lucene.store.FSDirectory;/** * @author Owner * */public class ItxtfilesearchgineImp implements Ifileseargine { //测试public static void main(String[] args)throws Exception{Ifileseargine itxtsearchgine=new ItxtfilesearchgineImp(); File dataFilePath = new File("C:\\data");File indexFilePath= new File("C:\\indexfiles");int indexedfileNum=itxtsearchgine.index(dataFilePath, indexFilePath);System.out.println("已索引文件个数："+indexedfileNum);itxtsearchgine.search(indexFilePath, "希拉");}/* * (non-Javadoc) * * @see com.goodwitkey.seargine.src.fileseargine#index(java.io.File, * java.io.File) */public int index(File dataFilesPath, File indexFilePath) throws IOException {// TODO Auto-generated method stubif (!dataFilesPath.exists() || !dataFilesPath.isDirectory()) {throw new IOException(dataFilesPath+ "don't exists or is not a directory");}// lucene 2.4.0已将此构造函数deprecatedSystem.out.println("****************" + indexFilePath);IndexWriter indexwr = new IndexWriter(indexFilePath,new StandardAnalyzer(), true);// 设置为true时，一定要主意建立索引文件夹不能有其它的重要文件，否则不小心会全删除掉的。// boolean create - true to create the index or overwrite the existing// one; false to append to the existing indexindexwr.setUseCompoundFile(false);indexwr.mergeFactor = 2;// 建立索引indexDirectory(indexwr, dataFilesPath);System.out.println("****************" + indexFilePath);int indexedNum = indexwr.docCount();indexwr.optimize();indexwr.close();return indexedNum;}/* * (non-Javadoc) * * @see com.goodwitkey.seargine.src.fileseargine#indexDirectory() */public void indexDirectory(IndexWriter indexwr, File dataFilesPath)throws IOException {// TODO Auto-generated method stubFile[] files = dataFilesPath.listFiles();for (int i = 0; i < files.length; i++) {File f = files[i];System.out.println(f.getName());if (f.isDirectory()) {indexDirectory(indexwr, f);// recurse} else if (f.getName().endsWith(".txt")) {indexFile(indexwr, f);}}}/* * (non-Javadoc) * * @see com.goodwitkey.seargine.src.fileseargine#indexfile() */public void indexFile(IndexWriter writer, File f) throws IOException {// TODO Auto-generated method stubif (!f.exists() || !f.canRead()) {return;}System.out.println("it gets the file now");Document doc = new Document();doc.add(Field.Text("contents", new FileReader(f)));doc.add(Field.Keyword("filename", f.getCanonicalPath()));writer.addDocument(doc);System.out.println(f.toString());}/* * (non-Javadoc) * * @see com.goodwitkey.seargine.src.fileseargine#search(java.io.File, * java.lang.String) */public Hits search(File indexFilePath, String queryStr)throws IOException, ParseException {// TODO Auto-generated method stubDirectory fsDir = FSDirectory.getDirectory(indexFilePath, false);IndexSearcher is = new IndexSearcher(fsDir);Query query = QueryParser.parse(queryStr, "contents", new StandardAnalyzer());long starttime = new Date().getTime();Hits hits = is.search(query);long endtime = new Date().getTime();System.out.println("Search the key word has elapsed "+ (endtime - starttime) + "ms");for (int i = 0; i < hits.length(); i++) {Document doc = hits.doc(i);System.out.println(doc.get("filename"));System.out.println(doc.toString());}return hits;}}?代码运行时还要符合DIP原则见具体实现类中的main()函数，word,pdf文件的索引可以再具体实现相应的类，任务与目标也就达到了。
?
?