Посчитать число сравнений в QuickSort

@Dzhos · Регистрация: 17.12.2010

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

приветствую всех любителей и профессионалов по С++.
Изучаю Quicksort
Мне нужно чтобы программа посчитала число сравнений сделанное при сортировке.
почему то count (счетчик) показывает ноль?

как можно посчитать ?

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
#include <iostream>
#include <cstdlib>
#include <vector>
#include <fstream>
using namespace std;
 
/*!Begin Snippet*/
template <typename T>
void quick_sort(vector<T>& v, int low, int high, int count ) {
 
    // Do not solve recursively when faced with
    // only 1 or 2 elements
    count = 0;
    if (low == high) {
        count++;
        return ;
    } else if (low + 1 == high) {
        if (v[low] > v[high]) {
        count++;
            swap(v[low], v[high]);
            return ;
        }
    }
 
    // select pivot
    int middle = (low + high) / 2;
    T pivot = v[middle];
    swap(v[middle], v[high]);
 
    // partition
    int i;
    int j;
    for (i = low, j = high - 1; ;) {
 
        while (v[i] < pivot && i < j) i++;
        while (pivot < v[j] && i < j) j--;
 
        if (i < j) {
        count++;
            swap(v[i], v[j]);
        } else {
            break;
        }
    }
 
    // place pivot in correct location
    if (i != high - 1 && j != high - 1) {
        swap( v[i], v[high]);
    }
 
    // quicksort sub-vectors
    if (i == low && j == low) {
        quick_sort(v, low + 1, high,count ;
    } else if (i == high - 1 && j == high - 1) {
        quick_sort(v, low, high - 1,count );
    } else {
        quick_sort(v, low, i - 1,count );
        quick_sort(v, i + 1, high,count );
    }
 
}
/*!End Snippet*/
 
 
int main(int argc, char* argv[]) {
 
   int count = 0;
    
 
        if (argc != 2) {
            cerr << "Usage:\n" << argv[0] << " data-file";
            return EXIT_FAILURE;
        }
 
        ifstream inf(argv[1]);
        if (! inf) {
            cerr << "Could not open " << argv[1];
            return EXIT_FAILURE;
        }
 
 
        vector<int> v;
        while (! inf.eof()) {
 
            int numbers;
            inf >> numbers;
            v.push_back(numbers);
 
            
        }
        inf.close();
 
       
       
        
    quick_sort(v, 0, v.size() - 1,count);
 
    vector<int>::iterator it;
     cout << count ;
    system("pause");
    return EXIT_SUCCESS;
}

заранее спасибо)

Добавлено через 5 минут
на 55й строчке ошибка забыл закрыть кавычку )

Добавлено через 2 часа 8 минут
смог посчитать )
мы должны просто добавить v.size()-1, запущенном в общей сложности сравнения каждый раз, когда рекурсивно запускаем подмассив длиной v.size().

C++
1
2
3
4
5
6
7
8
9
10
11
if (i == low && j == low) {
        quick_sort(v, low + 1, high);
        count += v.size()-1; 
    } else if (i == high - 1 && j == high - 1) {
        quick_sort(v, low, high - 1);
        count += v.size()-1; 
    } else {
        quick_sort(v, low, i - 1);
        quick_sort(v, i + 1, high);
        count += v.size()-1; 
    }

@Kuzia domovenok · 08.04.2012, 21:46

разберись с областью видимости переменных.
объяви свою count либо глобальной,
либо чтоб красивее статической

C
1
static int count=0;

@Dzhos · 08.04.2012, 21:57 **[ТС]**

я сделал глобальным ) Спасибо)

@DU · 08.04.2012, 22:00

в функцию quick_sort переменная count передается по значению. поэтому все изменения, которые происходят с ней внутри этой функции там же и останутся. count в функцию нужно передавать по ссылке. но в quick_sorte эта переменная в самом начале каждый раз обнуляется. это тоже неверно. если десять раз рекурсивно эта функция позовется, то десять раз и произойдет это обнуление.

@Dzhos · 08.04.2012, 22:32 **[ТС]**

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

в функцию quick_sort переменная count передается по значению. поэтому все изменения, которые происходят с ней внутри этой функции там же и останутся. count в функцию нужно передавать по ссылке. но в quick_sorte эта переменная в самом начале каждый раз обнуляется. это тоже неверно. если десять раз рекурсивно эта функция позовется, то десять раз и произойдет это обнуление.

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
#include <iostream>
#include <cstdlib>
#include <vector>
#include <fstream>
using namespace std;
 int count = 0;
/*!Begin Snippet*/
template <typename T>
void quick_sort(vector<T>& v, int low, int high) {
 
    // Do not solve recursively when faced with
    // only 1 or 2 elements
      
    if (low == high) {
    
        return ;
    } else if (low + 1 == high) {
        if (v[low] > v[high]) {
    
            swap(v[low], v[high]);
            return ;
        }
    }
 
    // select pivot
   int middle = (low+high)/2 ;
    T pivot = v[middle];
    swap(v[middle], v[high]);
 
    // partition
    int i;
    int j;
    for (i = low, j = high - 1; ;) {
 
        while (v[i] < pivot && i < j) i++;
        while (pivot < v[j] && i < j) j--;
 
        if (i < j) {
        
            swap(v[i], v[j]);
        } else {
            break;
        }
    }
 
    // place pivot in correct location
    if (i != high - 1 && j != high - 1) {
        swap( v[i], v[high]);
    }
 
    // quicksort sub-vectors
    if (i == low && j == low) {
        quick_sort(v, low + 1, high);
        count += v.size()-1; 
    } else if (i == high - 1 && j == high - 1) {
        quick_sort(v, low, high - 1);
        count += v.size()-1; 
    } else {
        quick_sort(v, low, i - 1);
        quick_sort(v, i + 1, high);
        count += v.size()-1; 
    }
 
}
/*!End Snippet*/
 
 
int main(int argc, char* argv[]) {
 
 
    
 
        if (argc != 2) {
            cerr << "Usage:\n" << argv[0] << " data-file";
            return EXIT_FAILURE;
        }
 
        ifstream inf(argv[1]);
        if (! inf) {
            cerr << "Could not open " << argv[1];
            return EXIT_FAILURE;
        }
 
 
        vector<int> v;
        while (! inf.eof()) {
 
            int numbers;
            inf >> numbers;
            v.push_back(numbers);
 
            
        }
        inf.close();
 
    quick_sort(v, 0, v.size() - 1);   
       
        
   
    cout << count ;
    system("pause");
    return EXIT_SUCCESS;
}

обнуление не произошло )
посчитал 65833416
как вы думаете ? не много ли для quick_sorta?

@Kuzia domovenok · 08.04.2012, 22:35

а размер файла какой? скорость кусорта порядка С*n*log n
... вроде бы

@Dzhos · 08.04.2012, 22:43 **[ТС]**

Сообщение от Kuzia domovenok

а размер файла какой? скорость кусорта порядка С*n*log n
... вроде бы

там с одного до 10000

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 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-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@Dzhos 6 / 6 / 0 Регистрация: 17.12.2010 Сообщений: 34
	08.04.2012, 21:57 [ТС]
	я сделал глобальным ) Спасибо) 0

@DU 1500 / 1146 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	08.04.2012, 22:00
	в функцию quick_sort переменная count передается по значению. поэтому все изменения, которые происходят с ней внутри этой функции там же и останутся. count в функцию нужно передавать по ссылке. но в quick_sorte эта переменная в самом начале каждый раз обнуляется. это тоже неверно. если десять раз рекурсивно эта функция позовется, то десять раз и произойдет это обнуление. 0

@Kuzia domovenok 4268 / 3327 / 926 Регистрация: 25.03.2012 Сообщений: 12,536 Записей в блоге: 1
	08.04.2012, 22:35
	а размер файла какой? скорость кусорта порядка Сnlog n ... вроде бы 0