Функция сортировки вставки двусвязных списков: Почему моя функция сортировки вставок не вставляет последние два целочисленных ввода в мой связанный список?

Question

Функция сортировки вставки двусвязных списков: Почему моя функция сортировки вставок не вставляет последние два целочисленных ввода в мой связанный список?

1

Моя функция сортировки вставки, похоже, не вставляет мои последние два целых ввода: 15 и 8 в мой дважды связанный список. Мой выход не учитывает 15 и 8. Что я делаю неправильно в функции сортировки вставки? Я предоставил код, ввод и вывод моих функций. Спасибо.

Вот моя функция сортировки вставки:

    public void insertionSort(int p)
   {
    Node n = new Node(p);
    Node curr = head;

    if(isEmpty())
    {
        head = n;
    }

    else
    {
        if(n.getProb() <= curr.getProb())
        {
            n.setNext(curr);
            curr.setPrev(n);
            head = n;
        }

        else if(n.getProb() > curr.getProb())
        {
            while(n.getProb() > curr.getProb() && curr.getNext() != null)
            {
                curr = curr.getNext();
            }

            if(n.getProb() < curr.getProb())
            {
                n.setNext(curr);
                n.setPrev(curr.getPrev());
                curr.setPrev(n);
            }

            else if(n.getProb() >= curr.getProb())
            {
                n.setNext(curr.getNext());
                n.setPrev(curr);
                curr.setNext(n);
            }
        }
    }
    size++;
  }

Вот моя функция печати

    public void printSortedList()
{
    Node curr = head;

    while(curr != null)
    {
        System.out.println(curr.getProb());
        curr = curr.getNext();
    }
}

Вот как я вхожу в свою основную функцию:

public static void main(String [] args)
{
    List list = new List();

    list.insertionSort(10);
    list.insertionSort(5);
    list.insertionSort(5);
    list.insertionSort(60);
    list.insertionSort(5);
    list.insertionSort(10);
    list.insertionSort(15);
    list.insertionSort(8);

    list.printSortedList();
}

Вот мой вывод:

pupup 28 янв. 2015, в 10:54

Источник

0

Какой самый простой пример, который терпит неудачу? Можете ли вы добавить модульный тест, чтобы продемонстрировать это? Что вы видите, когда шагаете по коду в своем отладчике?
Peter Lawrey 28 янв. 2015, в 09:36
0

@PeterLawrey Я изменил свои входные данные только на 3 целых числа list.insertionSort(3); list.insertionSort(100); list.insertionSort(5); И результат, который я получил, был: 3 100 Таким образом, моя функция, кажется, опускает 5 в этом случае.
pupup 28 янв. 2015, в 09:45

Теги:

java

doubly-linked-list

insertion-sort

1 ответ

Ещё вопросы

Какой самый простой пример, который терпит неудачу? Можете ли вы добавить модульный тест, чтобы продемонстрировать это? Что вы видите, когда шагаете по коду в своем отладчике?
@PeterLawrey Я изменил свои входные данные только на 3 целых числа list.insertionSort(3); list.insertionSort(100); list.insertionSort(5); И результат, который я получил, был: 3 100 Таким образом, моя функция, кажется, опускает 5 в этом случае.

Vitaly · Accepted Answer · 2015-01-28T08-46-00.000Z

У вас есть ошибка при n.getProb() <= curr.getProb() В вашем коде

n.setNext(curr);
n.setPrev(curr.getPrev());
curr.setPrev(n);

Вы

установить следующий из n в curr - ok
set prev of n to currPrev - ok
set prev of curr to n - ошибка, вы пропустили ссылку на prev of curr и по-прежнему ссылаетесь на валюту

Та же проблема в вашей части n.getProb() > curr.getProb().

Правильная установка:

           if (n.getProb() <= curr.getProb()) {
                n.setNext(curr);
                n.setPrev(curr.getPrev());
                if (curr.getPrev() != null)
                     curr.getPrev().setNext(n);//set next to curr prev
                curr.setPrev(n);
            } else if (n.getProb() > curr.getProb()) {
                n.setNext(curr.getNext());
                n.setPrev(curr);
                if (curr.getNext() != null)
                     curr.getNext().setPrev(n);//set prev to curr next
                curr.setNext(n);
            }