在java中使用单链表和冒泡排序进行编码

Question

我的代码有问题,我已经创建了一个单独链接列表类,您可以在其中添加,删除,修改,合并等...但是,我正在尝试一个简单的冒泡排序,并遇到了列表是的问题没有正确排序.这里有一些注意事项:

• 它是链表的自定义实现
• 单链表的节点包含两件事:一个CustomerFile对象,包含客户的所有数据,一个'next'节点指向列表中的下一个项目
• 列表按存储在每个节点的客户文件中的姓氏以升序(AZ)排序
• 添加记录功能将节点插入列表中的正确位置,以便最初不需要对列表进行排序 - 但是如果更改姓氏,作为程序的一部分,则需要再次对列表进行排序
• 我宁愿不创建一个新列表并重新使用该列表上的插入记录来创建一个新列表,因为这是内存密集型的,我的任务是尽可能高效
• 链接列表的结构不能改变 - 它已经决定,我太过分了,无法改变像数组这样的东西
• 该列表有一个头节点,它有下一个项目,但没有尾节点.它有一个指定的NULL下一个指针,用于指示列表的结尾

代码

public static void sortList()
{
    if (isEmpty() == true)
    {
        System.out.println("Cannot sort - the list is empty");
    }
    else if (getHead().getNext() == null)
    {
        System.out.println("List sorted");
    }
    else
    {
        Node current = getHead().getNext();
        CustomerFile tempDat;
        boolean swapDone = true;
        while (swapDone)
        {
            current = getHead().getNext();
            swapDone = false;
            while (current != null)
            {
                if (current.getNext() != null &&
                    current.getData().getSurname().compareTo(
                        current.getNext().getData().getSurname()) >0)
                {
                    tempDat = current.getData();
                    current.setData(current.getNext().getData());
                    current.getNext().setData(tempDat);
                    swapDone = true;
                }
                current = current.getNext();
            }
        }
        if (getHead().getData().getSurname().compareTo(
            getHead().getNext().getData().getSurname()) >0)
        {
            current = getHead().getNext();
            getHead().setNext(current.getNext());
            setHead(current);
        }
    }
}

我很感激反馈

Answer 1

您的代码是一个奇怪的混合物，主要是交换数据，但特别对待头部，试图通过切换链接将其与下一个交换。

正如另一个答案中所指出的，最后一个交换没有正确实现，但如果您通过交换数据来执行操作，实际上没有理由特殊对待头部。

您所要做的就是在外循环的头部开始每次遍历列表。

这产生了一个可行的解决方案：

public void sortList()
{
    if (isEmpty())
    {
        System.out.println("An empty list is already sorted");
    }
    else if (getHead().getNext() == null)
    {
        System.out.println("A one-element list is already sorted");
    }
    else
    {
        Node current = getHead();
        boolean swapDone = true;
        while (swapDone)
        {
            swapDone = false;
            while (current != null)
            {
                if (current.getNext() != null && current.getData().getSurname().compareTo(current.getNext().getData().getSurname()) >0)
                {
                    CustomerFile tempDat = current.getData();
                    current.setData(current.getNext().getData());
                    current.getNext().setData(tempDat);
                    swapDone = true;
                }
                current = current.getNext();
            }
            current = getHead();
        }
    }
}

我已将其设为非静态，因为正如评论中所述，我不清楚您的静态如何工作。您也许可以在您的上下文中将其设为静态。

我还改变了其他一些小事情。从来没有必要通过像这样的代码来检查条件

if (isEmpty() == true)

在 Java 中，这完全等同于

if (isEmpty())

并且tempDat不需要在使用它的地方之外声明。