Форум программистов, компьютерный форум CyberForum.ru

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

Восстановить пароль Регистрация
 
Lenovo_G575
5 / 5 / 0
Регистрация: 02.04.2013
Сообщений: 41
01.10.2013, 19:23     QuickSort найдите ошибку #1
Не правильно сортирует массив, подскажите в чем ошибка....
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);
    }
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
01.10.2013, 19:23     QuickSort найдите ошибку
Посмотрите здесь:

C++ Найдите ошибку
Найдите ошибку C++
найдите ошибку C++
Найдите ошибку C++
C++ найдите ошибку
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
newb_programmer
 Аватар для newb_programmer
237 / 237 / 19
Регистрация: 03.09.2011
Сообщений: 553
01.10.2013, 19:53     QuickSort найдите ошибку #2
Цитата Сообщение от Lenovo_G575 Посмотреть сообщение
Не правильно сортирует массив, подскажите в чем ошибка....
C++
1
2
3
4
5
        {
            tmp=A[i];
            A[i]=A[j];
            A[j]=tmp;
        }
что это за блок? else??
Lenovo_G575
5 / 5 / 0
Регистрация: 02.04.2013
Сообщений: 41
01.10.2013, 20:16  [ТС]     QuickSort найдите ошибку #3
Нет не else. Этот блок выполняется после предыдущего шага, и относится к циклу for.

Добавлено через 5 минут
Цитата Сообщение от newb_programmer Посмотреть сообщение
что это за блок? else??
Нет не else. Этот блок выполняется после предыдущего шага, и относится к циклу for.
Kuzia domovenok
 Аватар для Kuzia domovenok
1882 / 1737 / 116
Регистрация: 25.03.2012
Сообщений: 5,907
Записей в блоге: 1
01.10.2013, 20:35     QuickSort найдите ошибку #4
это не квиксорт это ерунда какая-то, и близко не похожая...
Вот, например, тут: i=p-1;
А далее
tmp=A[i];
A[i]=A[j]
100% выход за границы массива
Да и в общем, взглянув на алгоритм не ясно.
Что такое i?
Что такое j?
Почему i изменяется в цикле фор, а j нет?
Почему обмен происходит на каждой итерации цикла? Разве этому учит нас кусорт?

Но дело даже не в этом! Все привыкли к классическому кусорту Алгоритмы сортировок
А то, что тут происходит чёрт разберёт.
Точнее, разобрать то может и можно, но лень, когда есть известный готовый алгоритм!
Yandex
Объявления
01.10.2013, 20:35     QuickSort найдите ошибку
Ответ Создать тему
Опции темы

Текущее время: 09:29. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru