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

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

Восстановить пароль Регистрация
 
Greeezly
 Аватар для Greeezly
6 / 6 / 1
Регистрация: 31.08.2012
Сообщений: 230
21.09.2013, 13:33     Быстрая сортировка, укажите на ошибки #1
Здравствуйте.

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

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

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

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

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

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

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

Текущее время: 09:07. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru