[REVIEW] Добавяне на reverse и sort(mergesort) функции в списъците #11
Conversation
Alexxx2207
force-pushed
the
feature/in-place-reverse-and-mergesort
branch
from
2f54bb2 to
67f8093
Compare
| @@ -100,6 +101,71 @@ class DoubleLinkedList { | |||
| deleteFirst(front->data); | |||
| } | |||
|
|
|||
| void split(I& list, I& left, I& right) { | |||
There was a problem hiding this comment.
Тази функция може да е публична и да унищожава оригиналния списък и да създав два нови със съответните вериги в тях.
There was a problem hiding this comment.
Ако приемем да работим със списъци вместо с итератори, първият параметър може да се пропусне и да се използва this.
| slow.ptr->next = nullptr; | ||
| } | ||
|
|
||
| I merge(I left, I right) { |
There was a problem hiding this comment.
Тази функция може да е публична, да приема два списъка с препратка, след което да им „краде“ веригите като ги слива. Ако резултатът е текущият списък, първо трябва да се изтрие текущото му съдържание,ако има такова. Ако резултатът е нов списък, тогава функцията би следвало да е статична.
| return left; | ||
|
|
||
| if(left.get() <= right.get()) { | ||
| I recResult = merge(left.next(), right); |
There was a problem hiding this comment.
Рекурсията е излишна. Идеята на функцията е да стане с константна допълнителна памет, а така става с линейна допълнителна памет заради стековите рамки.
| } | ||
| } | ||
|
|
||
| I mergeSort(I list) { |
There was a problem hiding this comment.
Използва се като статична функция, макар че не е. Ако следваме идеята горните функции също да не са статични, то тогава би трябвало:
splitда запълва два подадени празни списъка, като изпразва текущия.mergeда слива два подадени списъка в текущия, като изпразва попадените.mergeSortби трябвало да ползва два временни списъка и да работи върху списъка, за който се извиква метода. Това ще стане по естествен начин,тъй като така ще работи извикването наmerge.
| void sort() { | ||
| front = mergeSort(begin()).ptr; | ||
|
|
||
| I it = begin(); |
There was a problem hiding this comment.
Частта със задаването на back ще отпадне, ако merge работи директно със списъци.
| @@ -94,6 +95,62 @@ class LinkedList { | |||
| deleteFirst(front->data); | |||
| } | |||
|
|
|||
| void split(I list, I& left, I& right) { | |||
There was a problem hiding this comment.
Виж коментарите за двойносвързания списък.
| list.reverse(); | ||
|
|
||
| CHECK(!list.empty()); | ||
| CHECK_EQ(list.begin().get(), val); |
There was a problem hiding this comment.
Да се провери, че списъкът е само от един елемент.
No description provided.