Ошибка при запуске программы

@assasko · Регистрация: 20.05.2011

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

Сделал алгоритм слияния, нужно проверить время сортировки от 10000 элементов до 100000 (10тыс., 20тыс., ... 100тыс.) При 10 и 20 тысячах элементов все нормально работает. Но когда пытаюсь 30тыс и больше выдает ошибку.
Вот код программы:

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
#include <cstdlib>
#include <iostream>
#include <ctime>
#include <windows.h>
 
 
using namespace std;
 
void merge(double *mas,long n,int l, int r);
 
int main(int argc, char *argv[])
{
    cout << "Zadayte chislo elementov v massive N = ";
    long n;
    cin >> n;
    double *mas=new double[n];
    srand((unsigned)time(NULL));
    for (int i=0; i<n; i++)
    { mas[i]=(int)rand()/rand()+1;
   /* cout << mas[i] <<" ";*/}
    //cout << endl;
    //cout << "==========================================="<<endl;
    double t1=clock();
    merge(mas,n,0, n-1);
    double t2=clock();
    /*for (int i=0; i<n; i++)
    {cout << mas[i] <<" ";}*/
    cout << endl;
    cout <<"t1 = "<<t1<<endl;
    cout <<"t2 = "<<t2<<endl;
    cout << "\nVremya rascheta = " <<(t2-t1)/CLOCKS_PER_SEC << endl;
    system("PAUSE");
    return EXIT_SUCCESS;
}
void merge(double *mas,long n,int l, int r) {
    if (r == l)
        return;
    //если остается 2 элемента то меняем их местами
    if (r - l == 1) { 
        if (mas[r] < mas[l])
            swap(mas[r], mas[l]);
        return;
    }
    //деление массива пополам
    int m = (r + l) / 2;
    //рекурсивный вызов функции для левой и правой части массива
    merge(mas,n,l, m);
    merge(mas,n,m + 1, r);
    //массив в котором будет происходить слияние частей
    double *buf=new double[n];
    int xl = l;
    int xr = m + 1;
    int cur = 0;
    //пока существует хотябы 1 элемент выполняится слияние
    while (r - l + 1 != cur) {
        //сравнение єлементов подмассивов, процесс слияния подмассивов
        if (xl > m)
            buf[cur++] = mas[xr++];
        else if (xr > r)
            buf[cur++] = mas[xl++];
        else if (mas[xl] > mas[xr])
            buf[cur++] = mas[xr++];
        else buf[cur++] = mas[xl++];
 
    }
    for (int i = 0; i < cur; i++)
        mas[i + l] = buf[i];
}

Помогите пожалуйста.

@ZiGSuN · 07.05.2012, 20:18

строка 18 в вашем коде сверху :

C++
1
2
for (int i=0; i<n; i++)
    { mas[i]=(int)rand()/rand()+1; }

- здесь возможно деление на ноль так как приоритет "/" выше, чем "-" ;
так что нужно
сделать так

C++
1
2
for (int i=0; i<n; i++)
    { mas[i]=(int)rand()/(rand()+1); }

@assasko · 07.05.2012, 21:26 **[ТС]**

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

строка 18 в вашем коде сверху :

C++
1
2
for (int i=0; i<n; i++)
    { mas[i]=(int)rand()/rand()+1; }

- здесь возможно деление на ноль так как приоритет "/" выше, чем "-" ;
так что нужно
сделать так

C++
1
2
for (int i=0; i<n; i++)
    { mas[i]=(int)rand()/(rand()+1); }

Сделал как ты написал, при 30 тыс выдает ошибку :"This application has requested the Runtime to terminate it in an unusual way."

@thick_int · 07.05.2012, 21:34

По всей-видимости имеет место переполнение стека.
Попробуйте строку

C++
1
merge(mas,n,0, n-1);

выполнить в блоке try/catch.
то есть так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
#include <stdexcept>
...
...
 
try
{
   merge(mas,n,0, n-1);
}
catch (runtime_error &ex)
{
   cout << ex.what();
}

@assasko · 08.05.2012, 18:50 **[ТС]**

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

По всей-видимости имеет место переполнение стека.
Попробуйте строку

C++
1
merge(mas,n,0, n-1);

выполнить в блоке try/catch.
то есть так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
#include <stdexcept>
...
...
 
try
{
   merge(mas,n,0, n-1);
}
catch (runtime_error &ex)
{
   cout << ex.what();
}

Спасибо, но оно все равно выдает ту же ошибку((

Добавлено через 6 часов 54 минуты
Не могу понять что не так(

@OstapBender · 08.05.2012, 18:57

проверь хватит ли памяти (линейной) для 30к элементов?

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

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

Не могу понять что не так(

дебаг, не?

@antoha398 · 08.05.2012, 19:10

double *mas=new double[n];

вы объявляете массив для типа double.

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

mas[i]=(int)rand()/rand()+1;

зачем вам тогда явно добиваетесь от рандома целочисленного типа?

C++
1
mas[i]=(double)rand()/rand()+1;

исправьте это и все заработает.
И второе, память динамическую вам выделили, а возвращать ее кто будет?

C++
1
2
delete [] mas; //в main
delete [] buf; // в merge

у вас память рекой льется.

@Van111 · 08.05.2012, 19:15

нашёл выход используй malloc и free

Добавлено через 1 минуту
модераторы и админы я специально не использовал теги чтоб было видно исправления
#include <cstdlib>
#include <iostream>
#include <ctime>
#include <windows.h>
#include "alloc.h"

using namespace std;

void merge(double *mas,long n,int l, int r);

int main(int argc, char *argv[])
{
cout << "Zadayte chislo elementov v massive N = ";
long n;
cin >> n;
double *mas=(double*)malloc(n*sizeof(double));
srand((unsigned)time(NULL));
for (int i=0; i<n; i++)
{ mas[i]=(int)rand()/(rand()+1);
cout << mas[i] <<" "; }
//cout << endl;
//cout << "======================================= ===="<<endl;
double t1=clock();
merge(mas,n,0, n-1);
double t2=clock();
/*for (int i=0; i<n; i++)
{cout << mas[i] <<" ";}*/
cout << endl;
cout <<"t1 = "<<t1<<endl;
cout <<"t2 = "<<t2<<endl;
cout << "\nVremya rascheta = " <<(t2-t1)/CLOCKS_PER_SEC << endl;
system("PAUSE");
free(mas);
return EXIT_SUCCESS;
}
void merge(double *mas,long n,int l, int r) {
if (r == l)
return;
//åñëè îñòàåòñÿ 2 ýëåìåíòà òî ìåíÿåì èõ ìåñòàìè
if (r - l == 1) {
if (mas[r] < mas[l])
swap(mas[r], mas[l]);
return;
}
//äåëåíèå ìàññèâà ïîïîëàì
int m = (r + l) / 2;
//ðåêóðñèâíûé âûçîâ ôóíêöèè äëÿ ëåâîé è ïðàâîé ÷àñòè ìàññèâà
merge(mas,n,l, m);
merge(mas,n,m + 1, r);
//ìàññèâ â êîòîðîì áóäåò ïðîèñõîäèòü ñëèÿíèå ÷àñòåé
double *buf=(double*)malloc(n*sizeof(double));
int xl = l;
int xr = m + 1;
int cur = 0;
//ïîêà ñóùåñòâóåò õîòÿáû 1 ýëåìåíò âûïîëíÿèòñÿ ñëèÿíèå
while (r - l + 1 != cur) {
//ñðàâíåíèå ºëåìåíòîâ ïîäìàññèâîâ, ïðîöåññ ñëèÿíèÿ ïîäìàññèâîâ
if (xl > m)
buf[cur++] = mas[xr++];
else if (xr > r)
buf[cur++] = mas[xl++];
else if (mas[xl] > mas[xr])
buf[cur++] = mas[xr++];
else buf[cur++] = mas[xl++];

}
for (int i = 0; i < cur; i++)
mas[i + l] = buf[i];
free(buf);
}

@antoha398 · 08.05.2012, 19:20

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

нашёл выход используй malloc и free

у вас какой компилятор?

я у себя пробовал их использовать, но программа все равно вылетала.

@Van111 · 08.05.2012, 19:32

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

у вас какой компилятор?
я у себя пробовал их использовать, но программа все равно вылетала.

билдер 6

Добавлено через 14 секунд

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

я у себя пробовал их использовать, но программа все равно вылетала.

на какой строке?

@antoha398 · 08.05.2012, 19:36

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

на какой строке?

как раз вот на этой

C++
1
mas[i]=(int)rand()/rand()+1;

@Van111 · 08.05.2012, 19:44

блин antoha398, вам же сказали

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

mas[i]=(int)rand()/rand()+1;

mas[i]=(int)rand()/ (rand()+1);

@antoha398 · 08.05.2012, 19:50

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

блин antoha398, вам же сказали

ну, во-первых я тут причем?! программа не моя.

во-вторых, если учесть это mas[i]=(int)rand()/(rand()+1), то программа работает и с new delete.

и все равно мне не понятно зачем автор пытается заполнить массив double целочисленными значениями.

@Van111 · 08.05.2012, 19:59

antoha398, извиняюсь перепутал

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

во-вторых, если учесть это mas[i]=(int)rand()/(rand()+1), то программа работает и с new delete

у меня вылетает при 100000 в функции marge

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

и все равно мне не понятно зачем автор пытается заполнить массив double целочисленными значениями

может просто не заметил

@assasko · 09.05.2012, 11:45 **[ТС]**

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

antoha398, извиняюсь перепутал

у меня вылетает при 100000 в функции marge

может просто не заметил

да не заметил, просто пробовал разные типы данных, и проглядел))

Добавлено через 2 минуты
antoha398 и Van111 вам огромное спасибо, просто забыл что нужно удалять)) Все работает даже при 1 млн. элементов.

Новые блоги и статьи Все статьи Все блоги /
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .	Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .

@Van111 290 / 193 / 23 Регистрация: 03.08.2011 Сообщений: 2,824 Записей в блоге: 12
	08.05.2012, 19:15
	нашёл выход используй malloc и free Добавлено через 1 минуту модераторы и админы я специально не использовал теги чтоб было видно исправления #include <cstdlib> #include <iostream> #include <ctime> #include <windows.h> #include "alloc.h" using namespace std; void merge(double mas,long n,int l, int r); int main(int argc, char argv[]) { cout << "Zadayte chislo elementov v massive N = "; long n; cin >> n; double mas=(double)malloc(nsizeof(double)); srand((unsigned)time(NULL)); for (int i=0; i<n; i++) { mas[i]=(int)rand()/(rand()+1); cout << mas[i] <<" "; } //cout << endl; //cout << "======================================= ===="<<endl; double t1=clock(); merge(mas,n,0, n-1); double t2=clock(); /for (int i=0; i<n; i++) {cout << mas[i] <<" ";}/ cout << endl; cout <<"t1 = "<<t1<<endl; cout <<"t2 = "<<t2<<endl; cout << "\nVremya rascheta = " <<(t2-t1)/CLOCKS_PER_SEC << endl; system("PAUSE"); free(mas); return EXIT_SUCCESS; } void merge(double mas,long n,int l, int r) { if (r == l) return; //åñëè îñòàåòñÿ 2 ýëåìåíòà òî ìåíÿåì èõ ìåñòàìè if (r - l == 1) { if (mas[r] < mas[l]) swap(mas[r], mas[l]); return; } //äåëåíèå ìàññèâà ïîïîëàì int m = (r + l) / 2; //ðåêóðñèâíûé âûçîâ ôóíêöèè äëÿ ëåâîé è ïðàâîé ÷àñòè ìàññèâà merge(mas,n,l, m); merge(mas,n,m + 1, r); //ìàññèâ â êîòîðîì áóäåò ïðîèñõîäèòü ñëèÿíèå ÷àñòåé double buf=(double)malloc(n*sizeof(double)); int xl = l; int xr = m + 1; int cur = 0; //ïîêà ñóùåñòâóåò õîòÿáû 1 ýëåìåíò âûïîëíÿèòñÿ ñëèÿíèå while (r - l + 1 != cur) { //ñðàâíåíèå ºëåìåíòîâ ïîäìàññèâîâ, ïðîöåññ ñëèÿíèÿ ïîäìàññèâîâ if (xl > m) buf[cur++] = mas[xr++]; else if (xr > r) buf[cur++] = mas[xl++]; else if (mas[xl] > mas[xr]) buf[cur++] = mas[xr++]; else buf[cur++] = mas[xl++]; } for (int i = 0; i < cur; i++) mas[i + l] = buf[i]; free(buf); } 1