Форум программистов, компьютерный форум CyberForum.ru
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Greeezly
6 / 6 / 1
Регистрация: 31.08.2012
Сообщений: 230
#1

Быстрая сортировка, укажите на ошибки - C++

21.09.2013, 13:33. Просмотров 330. Ответов 5
Метки нет (Все метки)

Здравствуйте.

Писал быструю сортировку.

Брал информацию от сюда

и из книги Седжвик Р. Фундаментальные алгоритмы на C++. Часть 1-4. DiaSoft.2001


В общем, код, приведенный у человека ( ссылка выше) у меня не работал.

Доделывал сам. Кое как сделал, но большие проблемы с памятью, компилятор ругается на переполнение стека.



Входные данные(формально): (массив, первый элемент, конечный элемент, размер массива)


C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
void apmath_easyfunc::sort_quick(int *sf_mass,int sf_input, int sf_output, int n)
{
    int a,b,p,mid, size;
    size = n;
    if ((sf_output - sf_input + 1) <= 1)
        return;
    else
    {
        a = sf_input;
        b = sf_output-1;
        if ((sf_output - sf_input + 1) == 2)
        {
            mid = sf_output;
        }
 
        else
        {
            if (sf_mass[sf_output] < sf_mass[sf_output-1])
            {
                mid = sf_output-1;
                b = sf_output;
            }
            else
            mid = sf_output;
        }
            p = sf_mass[mid];
        do
        {
            while (sf_mass[a] < p) a++;
            while (sf_mass[b] > p) b--;
            if (a<=b)
            {
 
                swap_int(&(sf_mass[a]),&(sf_mass[b]));
                a++;
                
                if (b != 1)
                    b--;
 
                std::cout<<" "<<std::endl;
 
                //Красивый вывод сортировки
                    // if (sf_input > 1)
                    // {
                    //  for (int count1 = 1;count1 <= sf_input-1;count1++)
                    //  {
                    //      std::cout<<"  ";
                    //  }
                    // }
                    // for(int i = sf_input; i <= sf_output; i++)
                    // {
                    //  std::cout<<sf_mass[i]<<" ";
                    // }
                    // if (sf_output > 0)
                    // {
                    //  for (int count2 = 1;count2 <= sf_output;count2++)
                    //  {
                    //      std::cout<<"  ";
                    //  }
                    // }
                //Красивый вывод сортировки
 
                std::cout<<" "<<std::endl;
            }
        }   while (a<=b);
    if ( b > sf_input ) sort_quick(sf_mass, sf_input, b, size);
        if (a != size) sort_quick(sf_mass, a, size, size);
            if (size <= 1) sort_quick(sf_mass, 1, size, size-a);
 
    }
}
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
21.09.2013, 13:33
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Быстрая сортировка, укажите на ошибки (C++):

Быстрая сортировка. Помогите найти ошибки - C++
Совсем мало знакома с с++, посмотрите пожалуйста код, какие есть ошибки? using namespace std; void quickSort(vector&lt;int&gt; a, int l,...

Быстрая сортировка(сортировка Хоара). Отсортировать фрагмент массива - C++
Мне нужно отсортировать фрагмент массива, расположенный между первым и последним отрицательным элементом. Немогу понять как устоновить...

Быстрая сортировка (сортировка Хоара) для связных списков - C++
есть у кого готовый алгоритм? или подскажите как реализовать

Укажите на ошибки - C++
#include &lt;iostream&gt; using namespace std; int cmd(int cmd); int cmd() { int command; cout &lt;&lt; &quot;Команды: 1 - Информация о...

Укажите на ошибки - C++
Найти все простые числа не превосходящие заданного числа N, десятичная запись которых содержит совпадающие цифры #include&lt;iostream.h&gt; ...

Укажите на ошибки в программе - C++
В программе я создаю класс с двумя методами. Ввод матрицы, а потом замена строк на столбцы. Программа запускается, но вылетает, в общем...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Greeezly
6 / 6 / 1
Регистрация: 31.08.2012
Сообщений: 230
23.09.2013, 09:57  [ТС] #2
Немного переделал код.

В общем теперь проблемы в том, что часть массива перебирается несколько раз.
Это видно на примере массива из 7 элементов: 3 8 1 5 0 4 9 или массива: 3 8 1 4 5 0 9

Не знаю как от этого избавиться + это базовая версия сортировки, она не поддерживает дублирующихся значений. Также возникают трудности при малом N, в этом случае не удается использовать преимущество быстрой сортировки ( 2я рекурсия N-i элементов), в результате лишние просмотры массива, при этом случае я вызываю просто сортировку пузырьком.

Собственно, само тело сортировки.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
void quick_sort_helper(int *sf_mass,int sf_input, int sf_output, int n)
{
    int a,b,p,mid, size;
    size = n;
    if ((sf_output - sf_input + 1) <= 1)
        return;
    if ((sf_output - sf_input + 1) == 2)
    {
        if (sf_mass[sf_input] > sf_mass[sf_output])
            apmath_easyfunc::swap_int(&(sf_mass[sf_input]),&(sf_mass[sf_output]));
        a = sf_output;
        b = 0;
        //Красивый вывод сортировки
            std::cout<<" "<<std::endl;
            if (sf_input > 1)
            {
                for (int count1 = 1;count1 <= sf_input-1;count1++)
                {
                    std::cout<<"  ";
                }
            }
            for(int i = sf_input; i <= sf_output; i++)
            {
                std::cout<<sf_mass[i]<<" ";
            }
            if (sf_output > 0)
            {
                for (int count2 = 1;count2 <= sf_output;count2++)
                {
                    std::cout<<"  ";
                }
            }
            std::cout<<" "<<std::endl;
        //Красивый вывод сортировки
    }
    
    else
    {
        mid = sf_output;
        a = sf_input;
        b = sf_output-1;
        if ((sf_output - sf_input + 1) == 3)
        {
            mid = sf_input + 1;
            b = sf_output;
        }
 
        p = sf_mass[mid];
        do
        {
            while (sf_mass[a] < p) a++;
            while (sf_mass[b] > p) b--;
            
            if (a<=b)
            {
 
                apmath_easyfunc::swap_int(&(sf_mass[a]),&(sf_mass[b]));
                //a++;
                
                if (b != sf_input) b--;
 
                //Красивый вывод сортировки
                    std::cout<<" "<<std::endl;
                    if (sf_input > 1)
                    {
                        for (int count1 = 1;count1 <= sf_input-1;count1++)
                        {
                            std::cout<<"  ";
                        }
                    }
                    for(int i = sf_input; i <= sf_output; i++)
                    {
                        std::cout<<sf_mass[i]<<" ";
                    }
                    if ((size - sf_output) > 0)
                    {
                        for (int count2 = 1;count2 <= (size - sf_output);count2++)
                        {
                            std::cout<<"  ";
                        }
                    }
                    std::cout<<" "<<std::endl;
                //Красивый вывод сортировки
 
                
            }
        }   while (a<=b);
        if (sf_mass[a] > sf_mass[mid]) //Необходимое условие для случая p = max, b = min.
        apmath_easyfunc::swap_int(&(sf_mass[a]),&(sf_mass[mid]));
    }
    if ( b > sf_input ) quick_sort_helper(sf_mass, sf_input, b, size);
        if (a != size) quick_sort_helper(sf_mass, a, size, size);
}
Обработчик функции:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void apmath_easyfunc::sort_quick(int *sf_mass,int sf_input, int sf_output, int n)
{
    if ((sf_output - sf_input + 1) <= 1)
        return;
    if ((sf_output - sf_input + 1) == 2)
    {
        if (sf_mass[sf_input] > sf_mass[sf_output])
            swap_int(&(sf_mass[sf_input]),&(sf_mass[sf_output]));
    }
    if ((sf_output - sf_input + 1) < 4)
    {
        sort_puzirek(sf_mass,n);
    }
    else
    {
        quick_sort_helper(sf_mass,sf_input,sf_output,n);
    }
}
0
Greeezly
6 / 6 / 1
Регистрация: 31.08.2012
Сообщений: 230
27.09.2013, 13:14  [ТС] #3
тема все еще актуальна
0
Kuzia domovenok
1891 / 1746 / 118
Регистрация: 25.03.2012
Сообщений: 5,925
Записей в блоге: 1
27.09.2013, 14:56 #4
Цитата Сообщение от Greeezly Посмотреть сообщение
тема все еще актуальна
как она может быть актуальна, если её решили полвека назад? Загляни сюда Алгоритмы сортировок Что-то я там не вижу гигантских реализаций на 100 строк. Советую взять оттуда и поразмыслить над примером. Потому что вникать в 100 строк баянистого алгоритма, полагаю лень не только мне.

Добавлено через 4 минуты
а хотя, я начинаю понимать, откуда так много кода. В сортировке зачем-то присутствует ещё и вывод на экран! Убери его.
А впрочем... если тебе так необходимо чего-то выводить во время сортировки.

вынеси это в отдельную функцию! Т.к. ты мусоришь в основном алгоритме лишними инструкциями и выходит говнокод на 100 строк!

Это первейший принцип программирования! Одна функция должна быть сосредоточена на одном действии.
0
Greeezly
6 / 6 / 1
Регистрация: 31.08.2012
Сообщений: 230
27.09.2013, 15:16  [ТС] #5
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
как она может быть актуальна, если её решили полвека назад?
Для непонятливых объясню: она актуальна для меня, а не для программирования в целом.

Добавлено через 1 минуту
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
В сортировке зачем-то присутствует ещё и вывод на экран!
про отладку слышали?
0
Kuzia domovenok
1891 / 1746 / 118
Регистрация: 25.03.2012
Сообщений: 5,925
Записей в блоге: 1
27.09.2013, 15:21 #6
Цитата Сообщение от Greeezly Посмотреть сообщение
про отладку слышали?
именно для этого случая я написал жирным шрифтом.
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
вынеси это в отдельную функцию!
иначе читать невозможно.
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
27.09.2013, 15:21
Привет! Вот еще темы с ответами:

Укажите на ошибки в алгоритме программы - C++
Доброго времени суток. Прошу указать на ошибки алгоритма. После ввода всех требуемых данных ничего не происходит, программа не вычисляет...

C/C++ FAQ :: Быстрая сортировка (сортировка Хоара) - C++
Вопрос, скорее академический, по мотивам реализации. Вот в faq приведена реализация этого метода сортировки на C++. В коде есть следующий...

Быстрая сортировка (сортировка методом Хоара) - C++
Ввести массив x1,x2,...,x20 в диапазоне . Требуется расположить отрицательные элементы в порядке убывания. Вывести массивы до и после...

Сортировка расчёской и быстрая сортировка - C++
В файле in.txt записана последовательность целых чисел. Заданными методами отсортировать числа и записать в файлы out1.txt и out2.txt....


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
27.09.2013, 15:21
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru