Comb sort / Сортировка расчёской. Нужно усовершенствовать программу

@Airencarme · Регистрация: 30.09.2017

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

Программа рабочая , но на мой взгляд довольно громоздкая. К сожалению у меня ещё очень мало опыта в написании программ и изначальный код не мой. Я добился того лишь , что программа компилируется. Прошу укажите на изъяны в коде.

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
//Собственно , сама программа:
 
#include "stdafx.h"
#include "conio.h"
 
using namespace std;
 
 
{
 
    gap = (gap * 10) / 13;
 
    if (gap < 1)
        return 1;
    return gap;
}
 
void swap(int *i, int *j)
{
    int t = *i;
    *i = *j;
    *j = t;
}
 
{
    int gap = n;
    
    bool swapped = true;
 
    while (gap != 1 || swapped == true)
    {
        
        gap = getNextGap(gap);
 
        swapped = false;
 
        for (int i = 0; i<n - gap; i++)
        {
            if (a[i] > a[i + gap])
            {
                swap(&a[i], &a[i + gap]);
                swapped = true;
            }
        }
    }
}
 
Driver program
int main()
{
    int a[] = { 9, 8, 7, 6, 5, 4, 3, 2, 1, 0 };
    int n = sizeof(a) / sizeof(a[0]);
 
    combSort(a, n);
 
    printf("Sorted array: \n");
    for (int i = 0; i<n; i++)
        printf("%d ", a[i]);
    _getch();
    return 0;
}

@axela002 · 30.09.2017, 05:27

если это функция, то где её имя? Со 2-й функцией также.

C++
1
2
3
4
5
6
7
8
{
 
    gap = (gap * 10) / 13;
 
    if (gap < 1)
        return 1;
    return gap;
}

Статический массив всегда должен быть инициализирован количеством элементов, или константной переменной имеющие количество элементов.

C++
1
int a[] = { 9, 8, 7, 6, 5, 4, 3, 2, 1, 0 };

Зачем лишние вычисления размера массива , если у тебя размер задан уже.
Это не правильная работа с статическими массивами. Я указал выше, что размер должен быть константой.
Если хочешь динамический массив, то используй либо указатели и new(delete) либо контейнеры STL

C++
1
int n = sizeof(a) / sizeof(a[0]);

И давай имена разные , а то переменная и параметр функции одно и то же имя. Конечно ошибки нет, но программу будет сложно читать другим людям. Да и сам через месяц , два забудешь что там было, и запутаешься(если же нет, конечно, комментариев).

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .