Сортировка вставок в Java

Question

Сортировка вставок в Java

1

Я пытаюсь сам научить себя Введение в алгоритмы с помощью CLRS, и я только что сделал, пытаясь запрограммировать алгоритм сортировки вставки. Алгоритм берется непосредственно из книги, но я не слишком уверен в своем Java-коде. Сортировочная часть неверна. Если кто-то может указать на мою ошибку, это было бы фантастически!

public static void main(String[] args){
    int[] A = {5,6,8,9,1,2,3};
    System.out.println(Arrays.toString(A));
    InsertionSort(A);
    System.out.println(Arrays.toString(A));
}

public static void InsertionSort(int[] A){
    for(int j = 1; j < A.length; j++){
        int key = A[j];
        int i = j - 1;
        while(i > 1 && A[i] > key){
            A[i + 1] = A[i];
            i = i - 1;
        }
        A[i + 1] = key;
    }
}

user3754974 05 авг. 2014, в 06:33

Источник

0

Значит, сортировка вашего алгоритма сортировки неверна?
Sotirios Delimanolis 05 авг. 2014, в 04:26
0

Теперь я в замешательстве. Алгоритм имеет смысл на бумаге.
user3754974 05 авг. 2014, в 04:29
1

@PhamTrung Я знаю, что позиция индекса в Java начинается с 0, но для алгоритма сортировки вставки вы начинаете со второго элемента массива, который в данном случае равен 1.
user3754974 05 авг. 2014, в 04:40
0

@PhamTrung Они делают более быструю версию, которая не поменяется внутри цикла. И элемент 0 не учитывается, так как вам нужно два элемента для учета свопа, поэтому он начинается с 1 и должен сравниваться с A[i-1]
weston 05 авг. 2014, в 04:46
0

@weston Weston Спасибо :)
Pham Trung 05 авг. 2014, в 05:31

Показать ещё 3 комментария

Теги:

java

algorithm

sorting

1 ответ

Ещё вопросы

Значит, сортировка вашего алгоритма сортировки неверна?
Теперь я в замешательстве. Алгоритм имеет смысл на бумаге.
@PhamTrung Я знаю, что позиция индекса в Java начинается с 0, но для алгоритма сортировки вставки вы начинаете со второго элемента массива, который в данном случае равен 1.
@PhamTrung Они делают более быструю версию, которая не поменяется внутри цикла. И элемент 0 не учитывается, так как вам нужно два элемента для учета свопа, поэтому он начинается с 1 и должен сравниваться с A[i-1]

weston · Accepted Answer · 2014-08-05T02-08-00.000Z

public static void insertionSort(int[] A){
    for(int j = 1; j < A.length; j++){
        int key = A[j];
        int i = j; //change - don't -1
        while(i > 0 && A[i-1] > key){ //change here, must go from 0 and note change to index
            A[i] = A[i - 1]; //change here
            i = i - 1;
        }
        A[i] = key; //change here
    }
}

Для массива {5,6,8,9,1,2,3} я получаю [5, 1, 2, 3, 6, 8, 9]. Это какая-то ошибочная ошибка?
Да, но это не мое! : p Попробуйте скопировать мой код точно.
Могу поспорить, что вы оставили, while(i > 1 там
Ты понял. Маленькая глупая ошибка.
Кстати, я сослался на sudo-код в статье вики, чтобы исправить ваш: en.wikipedia.org/wiki/Insertion_sort. Возможно, вы найдете его полезным.