Question

这是我用于实现单链表的完整Java源代码。我已经看过许多教程，他们一直在谈论在开头插入一个节点。所以，我决定在我的代码中添加一个方法insertAfterNode(int y)，我可以在特定节点之后在节点内添加数据。

/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package datastructures;

/**
 *
 * @author Administrator
 */
public class Link {

    int data;

    Link next;

    public Link(int d) {

        this.data = d;
    }

    public void display() {

        System.out.println(data);
    }


    public static class LinkedList {

        Link first;

        public void insertItemFirst(int x){

            Link newLink = new Link(x);

            newLink.next = first;
            first = newLink;


        }



        public Link deleteFirst() {

            Link temp = first;
            first = first.next;
            return temp;
        }

        public void displayList() {

            Link current = first;

            while (current != null) {

                current.display();
                current = current.next;

            }

        }

        // START Of METHOD


        public void insertAfterNode(int y) {

            // Considering LinkedList is Sorted
            Link newNode = new Link(y);

            Link current = first;

            while (current != null) {

                while (current.data < y) {

                    current = current.next;
                    newNode.next = current;
                    current = newNode;

                }

            }

        }





        //END Of METHOD

    }



    public static void main(String[] args) {

        LinkedList addelements = new LinkedList();

        addelements.insertItemFirst(20);
        addelements.insertItemFirst(30);
        addelements.insertItemFirst(40);
        addelements.insertItemFirst(50);
       addelements.insertAfterNode(44);



        addelements.displayList();

        System.out.println("After Calling Deletion Method Once ");

        addelements.deleteFirst();



        addelements.displayList();

    }

}

上面的代码继续在Netbeans中运行，我不得不停止构建以退出它。我相信我的方法实现有问题。请告诉我以下方法的错误：

 public void insertAfterNode(int y) {

            // Considering LinkedList is Sorted
            Link newNode = new Link(y);

            Link current = first;

            while (current != null) {

                while (current.data < y) {

                    current = current.next;
                    newNode.next = current;
                    current = newNode;

                }

            }

        }

代码在没有上述方法的情况下运行良好。

Answer 1

仅当current值大于y时，current.data链接引用才会更改。如果fist.data是10，而y是5，那么您的代码将永远不会终止。

您必须修改while(current != null)周期。不要使用内部while。

Link newNode = new Link(y);
// "first" requires a special treatment, as we need to replace the "first" value
if(first.data > y){
    // insert the new Link as first element
    newNode.next = first;
    first = newNode;
} else {
    // we need the previous Link, because previous.next will be set to the newNode
    Link previous = first;
    Link current = first.next;

    while (current != null) {
        if(current.data < y) {
            previous = current;
            current = current.next;
        } else {
            // we insert newNode between previous and current
            // btw. what should happen if current.data == y?
            previous.next = newNode;
            newNode.next = current;
            break; // we are done, quit the cycle
        }
    }
    // if the newNode.next is null at this point, means we have not inserted it yet.
    // insert it as the last element in the list. previous is pointing at it.
    if(newNode.next == null){
        previous.next = newNode;
    }
} // end of else

P.S。我希望它有效，我还没有测试过代码：）

在链接列表中的特定节点之后插入

1 个答案: