Как исправить данный код сортировки?

@OVERPOWER8 · Регистрация: 29.11.2009

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

Вот только не надо никаких анализов, вопросов, и т. д.
Конкретно мой пример.

Программа на некоторых компиляторах не компилируется. Причем проблема вовсе не в переполнении стека - присвойте SIZE любое другое значение.
Если знаете, как исправить, то, пожалуйста, предложите свое решение.

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
#include <iostream>
#include <stdlib.h>
using namespace std;
 
const int SIZE=2000000;
 
void sort(int arr[])
{
        int max=arr[0];
        
        int i;
        
        for(i=0; i<SIZE; i++)
                if(arr[i] > max)
                        max = arr[i];
        
        int* Sorted = new int[max+1];
        
        for(i=0; i<SIZE; i++)
                Sorted[arr[i]]+=1;
        
        int counter=0;
        
        for(i=0; i<=max; i++)
                while(Sorted[i])
                {
                        arr[counter++]=i;
                        Sorted[i]--;
                }
                
        delete [] Sorted;
}
 
int main()
{
        srand(time(0));
 
        int Max=2000000;
        
        int i, j;
        
        int array[SIZE];
        
        for(i=0; i<SIZE; i++)
                array[i]=rand()%Max+1;
                
        sort(array);
        
        cout << "Sorted!\n";
        
        /*
        for(i=0; i<SIZE; i++)
                cout << array[i] << " ";
        */
        
        return 0;
}

@zim22 · 22.01.2010, 12:58

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

int array[SIZE];

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

const int SIZE=2000000;

как-то неправильно на стеке создавать такую громадину.
лучше динамическую память используй ну или std::vector

@OVERPOWER8 · 22.01.2010, 13:38 **[ТС]**

>> zim22

Спасибо, использование динамического массива исправило некоторые проблемы.
Если не сложно, то проверьте этот код на вашем компиляторе. А то мой G++ компилирует, и потом работает; а codepad.org выдает segmentation fault.

И почему-то мой компилятор при использовании элементов больше 1 000 000 000 тоже пишет ахинею какую-то:

Bash
1
2
3
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
Aborted

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
#include <iostream>
#include <stdlib.h>
#include <limits>
using namespace std;
 
const int SIZE=100000000;
 
void sort(int* arr)
{
    int max=arr[0];
    
    int i;
    
    for(i=0; i<SIZE; i++)
        if(arr[i] > max)
            max = arr[i];
    
    int* Sorted = new int[max+1];
    
    for(i=0; i<SIZE; i++)
        Sorted[arr[i]]+=1;
    
    int counter=0;
    
    for(i=0; i<=max; i++)
        while(Sorted[i])
        {
            arr[counter++]=i;
            Sorted[i]--;
        }
        
    delete [] Sorted;
}
 
int main()
{
        srand(time(0));
 
        int Max=2000000;
        
        int i, j;
        
        int* array = new int[SIZE];
        
        for(i=0; i<SIZE; i++)
                array[i]=rand()%Max+1;
                
        sort(array);
        
        cout << "Sorted!\n";
        
        /*
        for(i=0; i<SIZE; i++)
                cout << array[i] << " ";
        */
        
        delete [] array;
 
        return 0;
}

@zim22 · 22.01.2010, 13:42

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

Если не сложно, то проверьте этот код на вашем компиляторе.

не вижу смысла проверять. код глючный и может работать на одном наборе данных и крэшиться на другом. напиши для него юнит-тесты.

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

const int SIZE=100000000;
int i;
for(i=0; i<SIZE; i++)

для индексации массивов используй тип size_t вместо int. а то в int может не влезть такое гигантское значение.
***
в прочем, как и в size_t

@OVERPOWER8 · 22.01.2010, 14:26 **[ТС]**

>> zim22

Это почему же код глючный? У меня работает исправно.
Не вижу смысла использовать size_t, т. к:

numeric_limits<size_t>::max() = numeric_limits<unsigned int>::max();

@zim22 · 22.01.2010, 14:35

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

И почему-то мой компилятор при использовании элементов больше 1 000 000 000 тоже пишет ахинею какую-то:

это иксэпшн. система не смогла выделить столько-то памяти.

@OVERPOWER8 · 22.01.2010, 15:51 **[ТС]**

>> zim22

это иксэпшн. система не смогла выделить столько-то памяти.

Да, я неправильно память рассчитал.

При 300 000 000 элементах используется 1.0 ... 1.1 Гб оперативки.

надо nothrow добавить.

@zim22 · 22.01.2010, 16:42

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

надо nothrow добавить.

не надо. надо просто ловить исключение.

Evg · 22.01.2010, 17:10

Я же писал причину
https://www.cyberforum.ru/cpp/... post496717

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит: токи, напряжения и их 1 и 2 производные при t = 0;. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .

@OVERPOWER8 19 / 19 / 2 Регистрация: 29.11.2009 Сообщений: 224
	22.01.2010, 14:26 [ТС]
	>> zim22 Это почему же код глючный? У меня работает исправно. Не вижу смысла использовать size_t, т. к: numeric_limits<size_t>::max() = numeric_limits<unsigned int>::max(); 0

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	22.01.2010, 17:10
	Я же писал причину https://www.cyberforum.ru/cpp/... post496717 0