Вставка сортировки с использованием списка STL рекурсивно - PullRequest
Новая
       8

Вставка сортировки с использованием списка STL рекурсивно

0 голосов
/ 15 октября 2018

Итак, я пытаюсь изменить этот код векторов в списки.Я понимаю векторы, но довольно новые для списков.Это то, что я пытался до сих пор, как я могу это исправить, пожалуйста, дайте мне знать.Вот оригинальный векторный код: 

void insertion_sort(std::vector <int> &num) {
int i, j, key;
bool insertionNeeded = false;

for (j = 1; j < num.size(); j++) {
    key = num[j];
    insertionNeeded = false;
    for (i = j - 1; i >= 0; i--) { // larger values move right

        if (key < num[i]) {
            num[i + 1] = num[i];
            insertionNeeded = true;
        }
        else
            break;
    }
    if (insertionNeeded)
        num[i + 1] = key;    //Put key into its proper location
}
}

Вот код, который я пытался преобразовать в списки: при запуске я получаю сообщение об ошибке 

void insertion_sort(list<int> &li) {
int i, j, key;
bool insertionNeeded = false;
list<int>::iterator itr = li.begin();
for (j = 1; j < li.size(); j++) {
    advance(itr, j);
    key = *itr;
    insertionNeeded = false;
    for (i = j - 1; i >= 0; i--) { // larger values move right
        advance(itr, i);

        if (key < i) {
            advance(itr, i + 1);
            int temp1 = *itr;
                 advance(itr, i);
                 int temp2 = *itr;

                 temp1 = temp2;
            insertionNeeded = true;
        }
        else
            break;
    }
    if (insertionNeeded)
        advance(itr, i + 1);
        *itr = key;
}
}

1 Ответ

0 голосов
/ 15 октября 2018

Это мой быстрый ответ.Код теста здесь .

Обратите внимание, что std::advance изменяет свой аргумент.OTOH, std::next оставляет свой аргумент без изменений. 

void insertion_sort(std::list<int> &li)
{
    for (auto j = 1U; j < li.size(); ++j) 
    {
        auto itr = li.begin();
        std::advance(itr, j);
        const auto key = *itr;

        for (auto i = j; i > 0U; --i) // i is just a counter.
        {        
            std::advance(itr, -1);

            if (key < *itr) // larger values move right
            {
                const int temp = *itr;
                *itr = key;
                *std::next(itr) = temp;        
            }
            else{            
                break;
            }
        }
    }
}
...