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

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

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

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

21.09.2013, 13:33. Просмотров 317. Ответов 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);
 
    }
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
21.09.2013, 13:33     Быстрая сортировка, укажите на ошибки
Посмотрите здесь:

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

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

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

Сортировка Хоара / Быстрая сортировка - C++
Доброго времени суток. Написал реализацию алгоритма быстрой сортировки. void SortHhoar(int *arr,int f,int l)//Хоара { int mid = (f...

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

Быстрая сортировка - C++
Здорова господа! Есть задачка: дан алгоритм быстрой сортировки. ну вообщем я ее cделал но ток без указателей. А задачка находиться в...

Быстрая сортировка - C++
Задача: пользователь задает количество элементов массива (макс. - 500 000), вводит их, затем задает количество запросов (макс. - 10000) и...

Быстрая сортировка - C++
#include &lt;iostream&gt; #include &lt;vector&gt; #include &lt;stdlib.h&gt; #include &lt;stdio.h&gt; using namespace std; int comp(const int...

Быстрая сортировка - C++
нормальный код? а то третий день парюсь, вроде сейчас получилось void quicksort (int *a, int start, int end) { int point = partition...

Быстрая сортировка - C++
void qSort(int a, int N) { int i = 0, j = N; int temp, p; p = a; do { while ( a &lt; p ) i++;


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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
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);
    }
}
Greeezly
6 / 6 / 1
Регистрация: 31.08.2012
Сообщений: 230
27.09.2013, 13:14  [ТС]     Быстрая сортировка, укажите на ошибки #3
тема все еще актуальна
Kuzia domovenok
1889 / 1744 / 117
Регистрация: 25.03.2012
Сообщений: 5,922
Записей в блоге: 1
27.09.2013, 14:56     Быстрая сортировка, укажите на ошибки #4
Цитата Сообщение от Greeezly Посмотреть сообщение
тема все еще актуальна
как она может быть актуальна, если её решили полвека назад? Загляни сюда Алгоритмы сортировок Что-то я там не вижу гигантских реализаций на 100 строк. Советую взять оттуда и поразмыслить над примером. Потому что вникать в 100 строк баянистого алгоритма, полагаю лень не только мне.

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

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

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

Добавлено через 1 минуту
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
В сортировке зачем-то присутствует ещё и вывод на экран!
про отладку слышали?
Kuzia domovenok
1889 / 1744 / 117
Регистрация: 25.03.2012
Сообщений: 5,922
Записей в блоге: 1
27.09.2013, 15:21     Быстрая сортировка, укажите на ошибки #6
Цитата Сообщение от Greeezly Посмотреть сообщение
про отладку слышали?
именно для этого случая я написал жирным шрифтом.
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
вынеси это в отдельную функцию!
иначе читать невозможно.
Yandex
Объявления
27.09.2013, 15:21     Быстрая сортировка, укажите на ошибки
Ответ Создать тему
Опции темы

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