1

我有一个ListElement对象的 LinkedList，我想创建一个递归方法来添加新节点，同时仍保留列表的排序顺序。

现在我有： 

public static ListElement InsertList(ListElement head, ListElement newelem) {

    if (head == null) {
        head = newelem;
        head.next = null;
    } 
    else if (head.next == null) {
        newelem.next = null;
        head.next = newelem;
    }
    else if (head.value < newelem.value) {
        newelem.next = head;
        head = newelem;
    }
    else if (head.value >= newelem.value) {
        head = head.next;
        return InsertList(head, newelem);
    }
    return head;
}

我用代码多次调用它： 

ListElement head = null;
ListElement elem;

// this block of code is repeated multiple times with values of 3, 8, 20, and 15
elem - new ListElement();
elem.value = 6;
head = InsertList( head, elem );

输出如下： 

6
6 3
8 6 3
20 8 6 3
15 8 6 3

这个输出对于前三行是正确的，但之后就变得很奇怪了。谁能改进我的算法？我觉得这个InsertList方法可以缩短很多。谢谢！

4

3 回答 3

1

第四个条件块中的head = head.next语句正在破坏head元素。我相信这应该是

else if(head.value >= newelem.value) {
    head.next = InsertList(head.next, newelem);
}
于 2013-04-11T03:09:48.623 回答
1

当您尝试插入 15 时，您输入了第 4 个条件：

// 20 > 15
else if (head.value >= newelem.value)

依次调用 InsertList 但将 8 作为头节点传递，因此进入第 3 个条件：

// 8 < 15
else if (head.value < newelem.value)

在这里，你说

newelem.next = head;

设置 15 -> 下一个 = 8

然后你说，

head = newelem;

设置 head = 15。

你看到问题了吗？使用@Zim-Zam O'Pootertoot 答案来修复您的错误。

于 2013-04-11T03:15:35.323 回答
0

感谢所有帮助和回答的家伙！
我发现了另一篇与我类似的帖子，其中一个答案似乎对我有用。
这是任何希望看到的人的链接：https ://stackoverflow.com/a/15936744/1470257

于 2013-04-11T03:15:37.893 回答