Поправьте, пожалуйста программу. Сортировки, рекурсии, передача параметров

@alina161 · Регистрация: 11.11.2016

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

При запуске сортировки - не считает кол-во ходов(COUNT), если создавать новый массив, то считает.
Я подозреваю, что ошибка в том, что 2е разные сортировки, и одна из них работает уже с отсортированным массивом, но (я не сплю вторые сутки...) моя голова не может переварить.
Если кто сможет, поправьте код без особых переделок.
Спасибо.

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
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
// ConsoleApplication39.cpp: определяет точку входа для консольного приложения.
//
 
#include "stdafx.h"
#include <iostream>
#include <ctime>
#include <Windows.h>
 
using namespace std;
 
 
int sortShell(int *mas, int h, int s) {
    int step, k, temp, count = 0;
    step = s / 2;//шаг
    while (step > 0)
    {
        for (h = 0; h < (s - step); h++) {
            k = h;
            while (h >= 0 && mas[h] > mas[h + step])
            {
                count++;
                temp = mas[h];
                mas[h] = mas[h + step];
                mas[h + step] = temp;
                h--;
                count++;
 
            }
        }
        step = step / 2;//уменьшили в два раза шаг
    }
    return count;
}
 
 
 
void fastSort(int *mas, int begin, int end, int check) {
 
    int temp;
    int i_begin = begin;
    int i_end = end;
 
 
 
    temp = mas[begin];
 
 
    switch (check)
    {
    case 1:
    {
        while (begin<end)
        {
            while ((mas[end] >= temp) && (begin < end))
            {
                end--;
            }
            if (begin != end)
            {
                mas[begin] = mas[end];
                begin++;
            }
            while ((mas[begin] <= temp) && (begin < end))
            {
                begin++;
            }
            if (begin != end)
            {
                mas[end] = mas[begin];
                end--;
            }
        }
 
        mas[begin] = temp;
        temp = begin;
        begin = i_begin;
        end = i_end;
 
        if (begin < temp)
        {
            fastSort(mas, begin, temp - 1, check);
        }
 
        if (end>temp)
        {
            fastSort(mas, temp + 1, end, check);
        }
        break;
    }
 
    case 2:
    {
        while (begin<end)
        {
            while ((mas[end] <= temp) && (begin < end))
            {
                end--;
            }
            if (begin != end)
            {
                mas[begin] = mas[end];
                begin++;
            }
            while ((mas[begin] >= temp) && (begin < end))
            {
                begin++;
            }
            if (begin != end)
            {
                mas[end] = mas[begin];
                end--;
            }
        }
 
        mas[begin] = temp;
        temp = begin;
        begin = i_begin;
        end = i_end;
 
        if (begin < temp)
        {
            fastSort(mas, begin, temp - 1, check);
        }
 
        if (end>temp)
        {
            fastSort(mas, temp + 1, end, check);
        }
    }
    default:
    {
        break;
    }
 
    }
 
}
 
 
 
 
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    const int n = 40;
    int arr[n];
    int i, j, point, point1, point2, count;
 
    srand(time(0));
 
 
    do
    {
        system("cls");
        cout << "Введите желаемый пункт меню:\n1.Сортировка\n2.Выход\n";
        cin >> point;
        switch (point)
        {
        case 1:
        {
            system("cls");
            cout << "Массив до сортировки :" << endl;
 
            for (i = 0; i < n; i++)
            {
                arr[i] = rand() % 90 + 10;
                cout << arr[i] << " ";
            }
 
            cout << endl;
            
 
            cout << "Введите желаемый пункт меню:\n1.Сортировка по возрастанию\n2.Сортировка по убыванию\n";
 
            cin >> point1;
 
            fastSort(arr, 0, n - 1, point1);
 
            cout << "Массив после быстрой сортировки :" << endl;
 
            for (i = 0; i < n; i++)
            {
                cout << arr[i] << " ";
            }
 
 
            cout << endl;
 
            do
            {
 
                cout << endl << "";
 
                count = sortShell(arr, i, n);
 
                cout << "Массив после сортировки Шелла:" << endl;
 
                for (i = 0; i < n; i++)
                {
                    cout << arr[i] << " ";
                }
 
                cout << "---->>>>>>" << count;
 
 
 
 
                cout << "\nХотите повторить? \n1.Да\n2.Нет\n";
                cin >> point1;
 
                if (point1 == 2) break;
 
 
                cout << "Использовать старый массив, или генерируем новый? \n1.Старый\n2.Новый\n";
                cin >> point2;
                if (point2 == 2)
                {
                    system("cls");
                    cout << "Новый массив" << endl;
                    for (i = 0; i < n; i++)
                    {
                        arr[i] = rand() % 90 + 10;
                        cout << arr[i] << " ";
                    }
                }
 
 
            } while (point1 == 1);
            break;
        }
 
 
        case 2:
        {
 
            break;
        }
 
        default: cout << "Вы выбрали неправильно! Введите заново\n";
            break;
        }
 
    } while (point != 2);
 
 
    system("pause");
    return 0;
}

@L0M · 21.10.2019, 05:17

alina161, у меня дежа вю... Этот код я уже видел. Только там была одна сортировкаю. И грабли те же самые: при сортировке уже отсортированного массива кол-во ходов (COUNT) равно 0. Что вполне ожидаемо.

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

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

Если кто сможет, поправьте код без особых переделок.

Постарался сохранить аутентичность, но говнокод пофиксил. Не удержался.

Кроме того, обучил обе сортировки без дублирования кода сортировать как по возрастанию, так и по убыванию. Для этого в функцию сортировки передаётся указатель на функцию-компаратор. В общем всё кондово, без всяких плюсовых заморочек. (Я даже обмен двух значений в массиве не переделал на std::swap() - удержался.

)

И в сортировку Шелла у вас лишний параметр передавался. Убрал из параметров в локальные переменные.

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
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
// ConsoleApplication39.cpp: определяет точку входа для консольного приложения.
//
 
#include "stdafx.h"
#include <iostream>
#include <ctime>
#include <Windows.h>
 
using namespace std;
 
typedef bool (*pfComparator)(int, int);
 
bool lt(int a, int b) {
    return a < b;
}
 
bool gt(int a, int b) {
    return a > b;
}
 
int sortShell(int* mas, int s, pfComparator pfc) {
    int h;
    int step, k, temp, count = 0;
    step = s / 2;      //шаг
    while (step > 0)
    {
        for (h = 0; h < (s - step); h++) {
            k = h;
            while (h >= 0 && pfc(mas[h], mas[h + step]))
            {
                count++;
                temp = mas[h];
                mas[h] = mas[h + step];
                mas[h + step] = temp;
                h--;
                count++;
 
            }
        }
        step = step / 2;//уменьшили в два раза шаг
    }
    return count;
}
 
 
 
void fastSort(int* mas, int begin, int end, pfComparator pfc) {
 
    int temp;
    int i_begin = begin;
    int i_end = end;
 
    temp = mas[begin];
 
 
    while (begin < end)
    {
        while (pfc(mas[end], temp) && (begin < end))
        {
            end--;
        }
        if (begin != end)
        {
            mas[begin] = mas[end];
            begin++;
        }
        while (!pfc(mas[begin], temp) && (begin < end))
        {
            begin++;
        }
        if (begin != end)
        {
            mas[end] = mas[begin];
            end--;
        }
    }
 
    mas[begin] = temp;
    temp = begin;
    begin = i_begin;
    end = i_end;
 
    if (begin < temp)
    {
        fastSort(mas, begin, temp - 1, pfc);
    }
 
    if (end > temp)
    {
        fastSort(mas, temp + 1, end, pfc);
    }
}
 
 
void arr_copy(int* dst, int* src, size_t len) {
    memcpy(dst, src, len * sizeof(int));
}
 
void arr_init(int* arr, size_t len) {
    for (size_t i = 0; i < len; i++)
    {
        arr[i] = rand() % 90 + 10;
    }
}
 
void arr_print(int* arr, size_t len) {
    for (size_t i = 0; i < len; i++)
    {
        cout << arr[i] << " ";
    }
    cout << endl;
}
 
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    const int n = 40;
    int arr[n], arr_tmp[n];
    int point, point1, point2, count;
 
    srand((unsigned int)time(NULL));
 
    do
    {
        system("cls");
        cout << "Введите желаемый пункт меню:\n1.Сортировка\n2.Выход\n";
        cin >> point;
        switch (point)
        {
            case 1:
            {
                system("cls");
 
                cout << "Массив до сортировки :" << endl;
                arr_init(arr, n);
                arr_print(arr, n);
 
                do
                {
                    cout << "Введите желаемый пункт меню:\n1.Сортировка по возрастанию\n2.Сортировка по убыванию\n";
                    cin >> point1;
 
                    arr_copy(arr_tmp, arr, n);
                    if (point1 == 1)
                        fastSort(arr_tmp, 0, n - 1, gt);
                    else
                        fastSort(arr_tmp, 0, n - 1, lt);
 
                    cout << "Массив после быстрой сортировки :" << endl;
                    arr_print(arr_tmp, n);
 
                    arr_copy(arr_tmp, arr, n);
 
                    if (point1 == 1)
                        count = sortShell(arr_tmp, n, gt);
                    else
                        count = sortShell(arr_tmp, n, lt);
 
                    cout << "Массив после сортировки Шелла:" << endl;
                    arr_print(arr_tmp, n);
                    cout << "---->>>>>>" << count;
 
                    cout << "\nХотите повторить? \n1.Да\n2.Нет\n";
                    cin >> point1;
 
                    if (point1 == 2) break;
 
                    cout << "Использовать старый массив, или генерируем новый? \n1.Старый\n2.Новый\n";
                    cin >> point2;
                    if (point2 == 2)
                    {
                        system("cls");
                        cout << "Новый массив" << endl;
                        arr_init(arr, n);
                        arr_print(arr, n);
                    }
 
                } while (point1 == 1);
                break;
            }
 
            case 2:
                break;
 
            default: cout << "Вы выбрали неправильно! Введите заново\n";
                break;
        }
 
    } while (point != 2);
 
 
    system("pause");
    return 0;
}

Новые блоги и статьи Все статьи Все блоги /
Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .