сортировка связанных списков путем изменения ссылок

Question

сортировка связанных списков путем изменения ссылок

0

Я пытаюсь сортировать связанный список, изменяя ссылки, не меняя данные. Я использую select sort.i не знаю, где я иду не так. Я начинаю, помогите мне.

struct node
{
    int   data;
    node* link;
};

node* p;    

void sort()
{
    node* temp = p;
    node* save;
    node* prev;
    node* tprev;
    node* push = new node;

    tprev = NULL;
    for (; temp != NULL; temp = temp->link)
    {
        push = temp->link;
        for (; push != NULL; push = push->link)
        {
            if (push->data<temp->data)
            {
                save->link  = temp->link;
                temp->link  = push->link;
                push->link  = save->link;
                prev->link  = temp;
                tprev->link = push;
            }

            prev = push;
        }

        tprev = temp;
    }
}

saimadan 05 фев. 2014, в 18:58

Источник

1

Не могли бы вы последовательно отформатировать код, это выглядит плохо. Чистый код, это первый шаг к рабочему коду. Кроме того, поместите пробелы между операторами.
meaning-matters 05 фев. 2014, в 17:16
0

я очень новичок здесь, я не знаю, как отформатировать это было бы полезно, если вы скажете, как я могу сделать
saimadan 05 фев. 2014, в 17:18
0

Это: save->link=... не выглядит так жарко. save точки на что именно? И если это должно быть сделано без перемещения значения (только настройка ссылки), слово new должно появляться в этом коде ровно ноль раз, поэтому колеса оторвались еще до того, как вы дойдете до своего первого цикла.
WhozCraig 05 фев. 2014, в 17:19
0

@GrijeshChauhan не могли бы вы указать, где я иду не так
saimadan 05 фев. 2014, в 17:19
1

@saimadan логика (код) перестановки узлов неверна, вы не рассматриваете ссылку родительского узла. Чтение узлов подкачки в односвязном списке
Grijesh Chauhan 05 фев. 2014, в 17:21
2

Смотрите мое форматирование.
meaning-matters 05 фев. 2014, в 17:21
0

@ Имея значение, я собирался сказать очень хорошее форматирование, просто добавьте пробел в temp=p; и если сравнение
Grijesh Chauhan 05 фев. 2014, в 17:23
0

@ смысл-большое спасибо от моего следующего вопроса, я постараюсь отформатировать так, как вы сделали
saimadan 05 фев. 2014, в 17:24
0

Что делает p ?
meaning-matters 05 фев. 2014, в 17:24
0

@ смысл-вопросы р является главой связанного списка
saimadan 05 фев. 2014, в 17:25
0

@GrijeshChauhan я снова посмотрел код, я не могу найти ошибку, пожалуйста, напишите ответ
saimadan 05 фев. 2014, в 17:27
0

Жаль, что вы выбрали сортировку выбора. по крайней мере, у пузыря есть шанс стать O (n), если список на самом деле отсортирован. Тем не менее вам нужно потратить некоторое время на отладчик, а также на обзор работы с указателями. В этом коде есть как минимум два неопределенных указателя, которые разыменовываются и вызывают неопределенное поведение. И единственный (ненужный) node* push = new node; динамического выделения node* push = new node; почти сразу утечка памяти в циклы: push = temp->link;
WhozCraig 05 фев. 2014, в 17:28
1

@saimadan Я хотел бы ответить, но есть логика сортировки, и я разместил свой комментарий, касающийся перестановки узлов. Я узнаю, как правильно поменять код, вы можете прочитать мой ответ, который я связал.
Grijesh Chauhan 05 фев. 2014, в 17:30
0

@GrijeshChauhan очень хороший пост
saimadan 05 фев. 2014, в 17:46
0

@saimadan спасибо Вы нашли, почему ваш обмен не является правильным? Читать Связанный список Bubble Sort¶
Grijesh Chauhan 05 фев. 2014, в 17:52
0

@GrijeshChauhan я понял пост, который вы связали, но я не понял ошибку в моем
saimadan 05 фев. 2014, в 17:59
1

@saimadan Хорошо, я не обещаю, но позвольте мне попытаться исправить.
Grijesh Chauhan 05 фев. 2014, в 18:01
0

Ссылка @WhozCraig save-> указывает на следующий узел temp (temp - первый из двух узлов, подлежащих обмену), и не могли бы вы сказать мне, почему я не должен использовать новый
saimadan 05 фев. 2014, в 18:10
0

@saimadan Я понял / я думаю, что требуются дополнительные изменения, извините, но я могу сделать это немедленно, если вы не получите ответ, я отвечу @ вечером на выходных.
Grijesh Chauhan 05 фев. 2014, в 18:15
0

@GrijeshChauhan хорошо, я буду ждать
saimadan 05 фев. 2014, в 18:16
0

@saimadan вы можете связать остальную часть вашего кода? поместите это в codepad и отправьте мне комментарий здесь.
Grijesh Chauhan 05 фев. 2014, в 18:20
0

@ Саймадан Нет, это не так. Где в опубликованном коде всегда есть save = ... для назначения действительного адреса в указатель save . Любая операция разыменования ( *save or save-> ) или даже оценка адреса в save является неопределенным поведением . И prev имеет ту же проблему на начальном проходе внутреннего цикла. Как я уже сказал, вам нужно пересмотреть работу вашего указателя. А что касается использования new для этого алгоритма, потому что C ++ не Java. p = new Type; p = somethine-else; это прямая утечка памяти, опять же связанная со знакомством с использованием указателя языка.
WhozCraig 05 фев. 2014, в 18:20
0

@GrijeshChauhan codepad.org/o9JFImJy
saimadan 05 фев. 2014, в 18:28
0

@WhozCraig, честно говоря, я не понимаю технических терминов, которые вы произносите как утечка памяти. Я новичок. было бы полезно, если бы вы предоставили некоторый источник, где я могу выучить указатели
saimadan 05 фев. 2014, в 18:33
0

@ Саймадан Это в некоторой степени моя точка зрения. Без достаточных практических знаний о том, как работают указатели на языке, любой опубликованный код решения не будет иметь большого значения. Полотно имеет массу примеров из самых простых до самых эзотерических относительно того, как указатели работы. Подобное решение , хотя и правильное, не принесет пользы без основ, и я думаю, что именно на этом вам следует сосредоточиться. Такое решение (связанное) - это такая вещь, которая требует удобного знания работы с указателями для понимания.
WhozCraig 05 фев. 2014, в 18:45
0

@GrijeshChauhan поправьте мое решение как можно раньше, пожалуйста
saimadan 05 фев. 2014, в 19:15
0

@saimadan ниже BLUEPIXY уже исправлено
Grijesh Chauhan 06 фев. 2014, в 04:27

Показать ещё 25 комментариев

Теги:

c++

c

linked-list

1 ответ

Ещё вопросы

Не могли бы вы последовательно отформатировать код, это выглядит плохо. Чистый код, это первый шаг к рабочему коду. Кроме того, поместите пробелы между операторами.
я очень новичок здесь, я не знаю, как отформатировать это было бы полезно, если вы скажете, как я могу сделать
Это: save->link=... не выглядит так жарко. save точки на что именно? И если это должно быть сделано без перемещения значения (только настройка ссылки), слово new должно появляться в этом коде ровно ноль раз, поэтому колеса оторвались еще до того, как вы дойдете до своего первого цикла.
@GrijeshChauhan не могли бы вы указать, где я иду не так
@saimadan логика (код) перестановки узлов неверна, вы не рассматриваете ссылку родительского узла. Чтение узлов подкачки в односвязном списке
@ Имея значение, я собирался сказать очень хорошее форматирование, просто добавьте пробел в temp=p; и если сравнение
@ смысл-большое спасибо от моего следующего вопроса, я постараюсь отформатировать так, как вы сделали
@ смысл-вопросы р является главой связанного списка
@GrijeshChauhan я снова посмотрел код, я не могу найти ошибку, пожалуйста, напишите ответ
Жаль, что вы выбрали сортировку выбора. по крайней мере, у пузыря есть шанс стать O (n), если список на самом деле отсортирован. Тем не менее вам нужно потратить некоторое время на отладчик, а также на обзор работы с указателями. В этом коде есть как минимум два неопределенных указателя, которые разыменовываются и вызывают неопределенное поведение. И единственный (ненужный) node* push = new node; динамического выделения node* push = new node; почти сразу утечка памяти в циклы: push = temp->link;
@saimadan Я хотел бы ответить, но есть логика сортировки, и я разместил свой комментарий, касающийся перестановки узлов. Я узнаю, как правильно поменять код, вы можете прочитать мой ответ, который я связал.
@saimadan спасибо Вы нашли, почему ваш обмен не является правильным? Читать Связанный список Bubble Sort¶
@GrijeshChauhan я понял пост, который вы связали, но я не понял ошибку в моем
@saimadan Хорошо, я не обещаю, но позвольте мне попытаться исправить.
Ссылка @WhozCraig save-> указывает на следующий узел temp (temp - первый из двух узлов, подлежащих обмену), и не могли бы вы сказать мне, почему я не должен использовать новый
@saimadan Я понял / я думаю, что требуются дополнительные изменения, извините, но я могу сделать это немедленно, если вы не получите ответ, я отвечу @ вечером на выходных.
@saimadan вы можете связать остальную часть вашего кода? поместите это в codepad и отправьте мне комментарий здесь.
@ Саймадан Нет, это не так. Где в опубликованном коде всегда есть save = ... для назначения действительного адреса в указатель save . Любая операция разыменования ( *save or save-> ) или даже оценка адреса в save является неопределенным поведением . И prev имеет ту же проблему на начальном проходе внутреннего цикла. Как я уже сказал, вам нужно пересмотреть работу вашего указателя. А что касается использования new для этого алгоритма, потому что C ++ не Java. p = new Type; p = somethine-else; это прямая утечка памяти, опять же связанная со знакомством с использованием указателя языка.
@WhozCraig, честно говоря, я не понимаю технических терминов, которые вы произносите как утечка памяти. Я новичок. было бы полезно, если бы вы предоставили некоторый источник, где я могу выучить указатели
@ Саймадан Это в некоторой степени моя точка зрения. Без достаточных практических знаний о том, как работают указатели на языке, любой опубликованный код решения не будет иметь большого значения. Полотно имеет массу примеров из самых простых до самых эзотерических относительно того, как указатели работы. Подобное решение , хотя и правильное, не принесет пользы без основ, и я думаю, что именно на этом вам следует сосредоточиться. Такое решение (связанное) - это такая вещь, которая требует удобного знания работы с указателями для понимания.
@GrijeshChauhan поправьте мое решение как можно раньше, пожалуйста

BLUEPIXY · Accepted Answer · 2014-02-05T23-24-00.000Z

void sort(){
    node *temp, *push;
    node *tprev, *pprev;
    node *save;

    tprev = NULL;
    for (temp = p; temp != NULL; temp = temp->link){
        pprev = temp;
        for (push=temp->link; push != NULL; push = push->link){
            if (push->data < temp->data){
                save = temp->link;
                temp->link = push->link;
                if(save == push)
                    push->link = temp;
                else
                    push->link = save;
                if(pprev != temp)
                    pprev->link = temp;
                if(tprev)
                    tprev->link = push;
                else
                    p = push;
                save = temp;
                temp = push;
                push = save;
            }
            pprev = push;
        }
        tprev = temp;
    }
}