QuickSort найдите ошибку

@Lenovo_G575 · Регистрация: 02.04.2013

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

Не правильно сортирует массив, подскажите в чем ошибка....

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
#include <iostream>
using namespace std;
int partition(int A[],int p,int r);
void quicksort(int A[],int p,int r);
int main()
{
    const int r=5;
    int A[r]={-1,5,45,4,-2};
    quicksort(A,1,r);
    cout << "massiv:  ";
    for(int j=0;j<r;j++)
    {
        cout << A[j] << " ";
    }
    cin.get();
    cin.get();
    return 0;
}
int partition(int A[],int p,int r)
{
    int x,i,tmp;
    x=A[r];
    i=p-1;
    for(int j=p;j<=r-1;j++)
    {
        if(A[j]<=x)
        {
            i=i+1;
        }
        {
            tmp=A[i];
            A[i]=A[j];
            A[j]=tmp;
        }
    }
    tmp=A[i+1];
    A[i+1]=A[r];
    A[r]=tmp;
    return i+1;
}
void quicksort(int A[],int p,int r)
{
    if(p<r)
    {
        int q=partition(A,p,r);
        quicksort(A,p,q);
        quicksort(A,q+1,r);
    }
}

@newb_programmer · 01.10.2013, 19:53

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

Не правильно сортирует массив, подскажите в чем ошибка....

C++
1
2
3
4
5
        {
            tmp=A[i];
            A[i]=A[j];
            A[j]=tmp;
        }

что это за блок? else??

@Lenovo_G575 · 01.10.2013, 20:16 **[ТС]**

Нет не else. Этот блок выполняется после предыдущего шага, и относится к циклу for.

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

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

что это за блок? else??

Нет не else. Этот блок выполняется после предыдущего шага, и относится к циклу for.

@Kuzia domovenok · 01.10.2013, 20:35

это не квиксорт это ерунда какая-то, и близко не похожая...
Вот, например, тут: i=p-1;
А далее
tmp=A[i];
A[i]=A[j]
100% выход за границы массива
Да и в общем, взглянув на алгоритм не ясно.
Что такое i?
Что такое j?
Почему i изменяется в цикле фор, а j нет?
Почему обмен происходит на каждой итерации цикла? Разве этому учит нас кусорт?

Но дело даже не в этом! Все привыкли к классическому кусорту Алгоритмы сортировок
А то, что тут происходит чёрт разберёт.
Точнее, разобрать то может и можно, но лень, когда есть известный готовый алгоритм!

Новые блоги и статьи Все статьи Все блоги /
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 - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@Kuzia domovenok 4268 / 3327 / 926 Регистрация: 25.03.2012 Сообщений: 12,531 Записей в блоге: 1
	01.10.2013, 20:35
	это не квиксорт это ерунда какая-то, и близко не похожая... Вот, например, тут: i=p-1; А далее tmp=A[i]; A[i]=A[j] 100% выход за границы массива Да и в общем, взглянув на алгоритм не ясно. Что такое i? Что такое j? Почему i изменяется в цикле фор, а j нет? Почему обмен происходит на каждой итерации цикла? Разве этому учит нас кусорт? Но дело даже не в этом! Все привыкли к классическому кусорту Алгоритмы сортировок А то, что тут происходит чёрт разберёт. Точнее, разобрать то может и можно, но лень, когда есть известный готовый алгоритм! 0