Tricky bug!!

Question

Hi there, ive been using lucence for a assignment that i am doing i was wondering if some you could have a look that this piece of code that i have been working on.
The code that i am working on this below
package Search;
import org.apache.lucene.analysis.Analyzer;
import org.apache.lucene.analysis.standard.StandardAnalyzer;
import org.apache.lucene.document.Document;
import org.apache.lucene.index.FilterIndexReader;
import org.apache.lucene.index.IndexReader;
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.search.Searcher;
import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
/** Simple command-line based search demo. */
public class searchHTML {
/** Use the norms from one field for all fields. Norms are read into memory,
* using a byte of memory per document per searched field. This can cause
* search of large collections with a large number of fields to run out of
* memory. If all of the fields contain only a single token, then the norms
* are all identical, then single norm vector may be shared. */
private static class OneNormsReader extends FilterIndexReader {
private String field;
public OneNormsReader(IndexReader in, String field) {
super(in);
this.field = field;
public byte[] norms(String field) throws IOException {
return in.norms(this.field);
public searchHTML() {}
/** Simple command-line based search demo. */
public String querySearch(String[] args,String searchValue) throws Exception
     String result = "";
String usage =
"Usage: java Lucene.SearchFiles [-index dir] [-field f] [-repeat n] [-queries file] [-raw] [-norms field]";
if (args.length > 0 && ("-h".equals(args[0]) || "-help".equals(args[0])))
System.out.println(usage);
System.exit(0);
String index = "{index-dir}";
String field = "contents";
String queries = null;
int repeat = 0;
boolean raw = false;
String normsField = null;
for (int i = 0; i < args.length; i++) {
if ("-index".equals(args)) {
index = args[i+1];
i++;
} else if ("-field".equals(args[i])) {
field = args[i+1];
i++;
} else if ("-queries".equals(args[i])) {
queries = args[i+1];
i++;
} else if ("-repeat".equals(args[i])) {
repeat = Integer.parseInt(args[i+1]);
i++;
} else if ("-raw".equals(args[i])) {
raw = true;
} else if ("-norms".equals(args[i])) {
normsField = args[i+1];
i++;
IndexReader reader = IndexReader.open(index);
if (normsField != null)
reader = new OneNormsReader(reader, normsField);
Searcher searcher = new IndexSearcher(reader);
Analyzer analyzer = new StandardAnalyzer();
BufferedReader in = null;
if (queries != null) {
in = new BufferedReader(new FileReader(queries));
} else {
in = new BufferedReader(new InputStreamReader(System.in, "UTF-8"));
QueryParser parser = new QueryParser(field, analyzer);
if(true)
     if (queries == null) // prompt the user
     System.out.println("Enter query: " + searchValue);
     String line = searchValue;
     if (line == null || line.length() == -1)
          System.out.println("test");
     //break;
     line = line.trim();
     if (line.length() == 0)
          System.out.println("test2");
     //break;
     Query query = parser.parse(line);
     System.out.println("Searching for: " + query.toString(field));
     Hits hits = searcher.search(query);
     // if (repeat > 0) {                           // repeat & time as benchmark
     // Date start = new Date();
     // for (int i = 0; i < repeat; i++) {
     // hits = searcher.search(query);
     // Date end = new Date();
     // System.out.println("Time: "+(end.getTime()-start.getTime())+"ms");
     System.out.println(hits.length() + " total matching documents");
     final int HITS_PER_PAGE = hits.length();
     int end = Math.min(hits.length(), HITS_PER_PAGE);
     for (int start = 0; start < end; start++)
     //for (int i = start; i < end; i++)
     //if (raw)
          //     {                              // output raw format
     // System.out.println("doc="+hits.id(i)+" score="+hits.score(i));
     // continue;
     Document doc = hits.doc(start);
     // System.out.println(hits.length());
     for(int i = 0; i<hits.length(); i++)
          ArrayList<String> st = new ArrayList<String>();
          st.add(doc.toString());
          // System.out.println(st);
     String path = doc.get("path");
     if (path != null)
          System.out.println((start + 1) + ". " + path);
     String title = doc.get("title");
     if (title != null)
          ArrayList<String> r = new ArrayList<String>();
          r.add(result = "Title:" + doc.get("title"));
          //System.out.println(result.length());
          //result = " Title: " + doc.get("path");
     else
     result = (start+1) + ". " + "No path for this document";
     //if (queries != null) // non-interactive
     // break;
     if (hits.length() > end)
     System.out.println("more (y/n) ? ");
     line = in.readLine();
     if (line.length() == 0 || line.charAt(0) == 'n')
     break;
     reader.close();
else
     System.out.println("finished");
return result;
At the mintue i am using JSP pages to display the result however, i am getting different results on the jsp page from what i am getting on the consle.
This is the result from the jsp page
Value Searched on:myspace
1Title:MySpace
2Title:MySpace
3Title:MySpace
4Title:MySpace
5Title:MySpace
6Title:MySpace
7Title:MySpace
8Title:MySpace
9Title:MySpace
10Title:MySpace
11Title:MySpace
12Title:MySpace
13Title:MySpace
14Title:MySpace
The title MySpace is the title of the html document
and from the console this is what i am getting : -
3 total matching documents
1. H:/Lucenetext/output4.html
2. H:/Lucenetext/output8.html
3. H:/Lucenetext/output1.html
Ive been looking at this for a couple of weeks and was wondering if i could have a fresh pair of eyes to look at it.
Any help would be very grateful
Yours
Chris

Answer 1

OK
             System.out.println("Enter query: " + searchValue); What's the point of that unless you then read some input from the user?
           if (line == null || line.length() == -1) line.length() can never be -1. This is rubbish.
          // if (repeat > 0) {                           // repeat & time as benchmark
           //  Date start = new Date();
           //  for (int i = 0; i < repeat; i++) {
            //   hits = searcher.search(query);
           //  Date end = new Date();
           //  System.out.println("Time: "+(end.getTime()-start.getTime())+"ms");
          // } Remove this junk.
           System.out.println(hits.length() + " total matching documents"); This goes to the console. A JSP doesn't have a console so you will never see it - it might be in a log file somewhere if you're lucky.
             //for (int i = start; i < end; i++) Remove all this commented-out junk.
               for(int i = 0; i<hits.length(); i++)
                    ArrayList<String> st = new ArrayList<String>();
                    st.add(doc.toString());
                   // System.out.println(st);
               } As 'st' now gets thrown away this is all junk. Remove it.
                  System.out.println((start + 1) + ". " + path); See above remarks about the console.
                 if (title != null)
                      ArrayList<String> r = new ArrayList<String>();
                      r.add(result = "Title:" + doc.get("title"));
                      //System.out.println(result.length());
                      //result = "   Title: " + doc.get("path");
                 } See above. 'r' gets thrown away. Remove this junk except for the assignment to 'result'.
               System.out.println("more (y/n) ? ");
               line = in.readLine(); See above re console. You need to think more clearly about the difference between a console application and a JSP. A JSP writes a page of HTML.