为什么这个while循环卡在无限循环中?

时间:2010-10-15 20:40:36

标签: java data-structures infinite-loop

我正在研究一个混合数据结构,它是一个有序的双链表,其中每个节点都包含一个SIZE数组[]。我的困难是添加方法。

使用教授提供的单元测试,测试字符串被分解为单个字符,并使用提供的比较器添加到列表中。默认测试为abcdefghijklmnopqrstuvwxyzaeiou

我编写的add方法添加了第一个项目,但它没有超过该字符。由于测试适用于班上的其他学生,因此必须将我的代码搞砸了。

我提出的代码是

boolean add(E item){
    Chunk<E> currentNode= head; //set lookup node to the head

    if (size==0) {
        //new chunk, connect it with head
        Chunk<E> newNode= new Chunk<E>(); 
        newNode.next= head;
        newNode.prev=head;
        head.next= newNode;
        head.prev= newNode;

        //add item and update counters
        ll.insertItem(newNode, item);
        size++;

        //System.out.println(currentNode.items[0]);
    }

    else {
        if (currentNode.next== head) {
            ll.insertItem(currentNode, item);
        }

        while (currentNode.next != head) {
            currentNode= currentNode.next;
            System.out.println(currentNode.items[0]);

            //if the add item is less than first item, move to previous node
            if (comp.compare(item, currentNode.items[0])<0) 
                currentNode= currentNode.prev;

            //if item fits within a chunk
            if (comp.compare(item, currentNode.items[0])> 0 && 
                    comp.compare(item, currentNode.items[currentNode.numUsed-1])<0) {
                ll.insertItem(currentNode, item);


                //otherwise, move search onto next node
            } else {
                currentNode= currentNode.next;
            }
        } 
    }
    return true;
}

ll.insertItem是嵌套类中的辅助方法,它确定数组中要插入项的位置,如果数组已满,则将节点拆分为两个节点,将旧节点的后半部分复制到new,然后将项添加到适当的节点。我将其实现为

public void insertItem(Chunk<E> node, E item) {
        if(node.numUsed==0) {
            node.items[0]=item;
            node.numUsed++;

        }else if (node.numUsed<chunkSize) {
            for (int i=0; i<=node.numUsed; i++) {
                if (comp.compare(item, node.items[i])>0) {
                    for (int j= node.numUsed; j>i; j--) {
                        node.items[j+1]= node.items[j];
                    }

                    node.items[i]= item;
                    node.numUsed++;
                }
            }
        } else {

            //make new chunk, determine which chunk item should be added
            Chunk<E> newChunk= newChunk(node);
            size++;

            //if item fits in new node
            if (comp.compare(item, newChunk.items[0])>0) {
                insertItem(newChunk, item);
            } else {
                insertItem(node, item); //item fits in old node
            }
        }
    }

我没有得到的是为什么这会陷入无限循环,特别是在测试字符串的第一个字符上。由于if (size==0)条件执行,为什么代码重复添加一个字符?

Addenum- RD要求

System.out.println("insertion order: "+order);
    Comparator<String> comp = new StringCmp();
    ((CmpCnt)comp).resetCmpCnt();            // reset the counter inside the comparator
    ChunkList<String> clist = new ChunkList<String>(comp);
    for (int i = 0; i < order.length(); i++){
        String s = order.substring(i, i+1);
        clist.add(s);
    }

1 个答案:

答案 0 :(得分:2)

添加第一项

后,您没有增加numUsed

这样:

if(node.numUsed==0) {
            node.items[0]=item;

        }

应该是:

if(node.numUsed==0) {
            node.items[0]=item;
            node.numUsed++;
        }