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

@Greeezly · Регистрация: 31.08.2012

Студворк — интернет-сервис помощи студентам

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

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

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

и из книги Седжвик Р. Фундаментальные алгоритмы на 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);
 
    }
}

@Greeezly · 23.09.2013, 09:57 **[ТС]**

Немного переделал код.

В общем теперь проблемы в том, что часть массива перебирается несколько раз.
Это видно на примере массива из 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 · 27.09.2013, 13:14 **[ТС]**

тема все еще актуальна

@Kuzia domovenok · 27.09.2013, 14:56

Сообщение от Greeezly

тема все еще актуальна

как она может быть актуальна, если её решили полвека назад? Загляни сюда Алгоритмы сортировок Что-то я там не вижу гигантских реализаций на 100 строк. Советую взять оттуда и поразмыслить над примером. Потому что вникать в 100 строк баянистого алгоритма, полагаю лень не только мне.

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

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

Это первейший принцип программирования! Одна функция должна быть сосредоточена на одном действии.

@Greeezly · 27.09.2013, 15:16 **[ТС]**

Сообщение от Kuzia domovenok

как она может быть актуальна, если её решили полвека назад?

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

Добавлено через 1 минуту

Сообщение от Kuzia domovenok

В сортировке зачем-то присутствует ещё и вывод на экран!

про отладку слышали?

@Kuzia domovenok · 27.09.2013, 15:21

Сообщение от Greeezly

про отладку слышали?

именно для этого случая я написал жирным шрифтом.

Сообщение от Kuzia domovenok

вынеси это в отдельную функцию!

иначе читать невозможно.

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .
Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .

@Greeezly 6 / 6 / 1 Регистрация: 31.08.2012 Сообщений: 230
	27.09.2013, 13:14 [ТС]
	тема все еще актуальна 0