下面的类旨在按行索引单词的位置。引发错误的方法旨在将单独文档的索引追加到当前索引上。也就是说,如果第一个文档中有 6 行,则附加文档的第一行应索引为第 7 行。
public class DocumentIndex {
// a NavigableMap implementation to store indexed words and their locations
private TreeMap<String, ArrayList<Integer>> map = new TreeMap<String, ArrayList<Integer>>();
/**
* A method to append another index onto the main index
* @param indexAppendix the additional index to be appended onto the main index
*/
public void append(DocumentIndex indexAppendix){
if(indexAppendix == null){
throw new NullPointerException();
}
Integer docEnd = 0; // the last line recorded in the main index
Set<String> set = map.keySet(); // a Set of the key values from map
//select each key
for(Iterator<String> iter = set.iterator(); iter.hasNext();){
String key = iter.next(); // the current key value
// for each key select contents and determine the highest value
for(Iterator<Integer> iter2 = this.find(key).iterator(); iter2.hasNext();){
Integer compare = iter2.next(); // the key index current value
if(compare>docEnd){
docEnd=compare;
}
}
}
// for each key find an index value
for(Iterator<String> iter = set.iterator(); iter.hasNext();){
String key = iter.next(); // the current key value
// for each index value map that value adjusting for the length of the original document
ArrayList<Integer> toAdd = new ArrayList<Integer>();
for(Iterator<Integer> iter2 = this.find(key).iterator(); iter2.hasNext();){
Integer addIter = iter2.next();
toAdd.add(addIter); // the current index value
}
/**
*Below is the loop in which the error is thrown
*/
for(Iterator<Integer> iter3 = toAdd.iterator(); iter.hasNext();){
Integer addIter = iter3.next(); // The error is thrown on this line
map.get(key).add(addIter+docEnd);
}
}
}
我做错了什么?
Louis Wasserman已经钉住了它。
我只想指出,如果你使用"新的"Java for
循环语法,你的代码会简单得多,并且不会(不能!!)首先犯这个错误。 例如,下面是使用"new"for
语法的代码大致的样子:
...
Integer docEnd = 0;
Set<String> set = map.keySet();
for (String key : set) {
for (Integer compare : this.find(key)) {
if (compare < docEnd){
docEnd = compare;
}
}
}
for (String key : set) {
ArrayList<Integer> toAdd = new ArrayList<Integer>();
for (String add : this.find(key)) {
toAdd.add(add);
}
for (Integer add : toAdd) {
map.get(key).add(add * docEnd);
}
}
这不是更具可读性吗?
我在引号中加上"new",因为这种语法是在 2004 年发布的 Java 5 中引入的。 它应该是所有实践Java程序员的标准报告的一部分......和教官...到现在为止。
请不要复制和粘贴上面的代码。 它只是为了说明我的观点。
问题循环中的循环条件应该是iter3.hasNext()
,而不是iter.hasNext()
。