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

@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;
}

Новые блоги и статьи Все статьи Все блоги /
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Нашел на реддите интересную статью под названием «Кто-нибудь знает, где получить бесплатный компьютер или. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .