18 / 18 / 8
Регистрация: 21.10.2011
Сообщений: 115
1

Аварийное завершение программы, ошибка в функции вывода.

21.10.2011, 07:08. Показов 2032. Ответов 12
Метки нет (Все метки)

Author24 — интернет-сервис помощи студентам
Здравствуйте все. Пишу программу для сложения, вычитания матриц. Использую перегрузку операторов. При выводе результатов возникает ошибка и программа аварийно завершается. Я, наверное, неправильно работаю с выводом динамического массива, так как со статическим массивом вывод (void print() ) работает хорошо, ошибок не возникает. Если у кого - нибудь есть мысли по устранению ошибки, буду бесконечно благодарен.


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
#include <conio.h>
#include <iostream>
 
using namespace std;
 
class Matrix
{
private:
    int**a;
    int n,m;
public:
    Matrix(){}
    void getData();
    void print();
    Matrix Matrix::operator+(const Matrix&);
};
////////////////////////////////////////
 
Matrix Matrix::operator+(const Matrix &p)
{
    Matrix result(*this);
 
    for (size_t i = 0; i < n; ++i)
        for ( size_t j = 0; j < m; ++j)
            result.a[i][j] += p.a[i][j];
 
    return result;
}
 
//////////////в этой функции что - то не так как надо работает: программа 
//аварийно завершается на ее исполнении//////////////////////////
void Matrix::print()
{
 
for(int i=0;i<n;i++)
    for (int j=0;j<m;j++)
        cout<<a[i][j]<<" "<<endl;
}
 
////////////////////////////////////////
void Matrix::getData()
{
 
cout<<"\ninput n,m: \n";cin>>n>>m;
int**a=new int*[n];
for (int i=0;i<n;i++)
a[i]=new int [m];
 
cout<<"input matrix' elements: \n";
for(i=0;i<n;i++)
    for (int j=0;j<m;j++)
        cin>>a[i][j];
}
 
void main(void)
{
 
Matrix a_1;
Matrix a_2;
Matrix a_buf;
 
a_1.getData();
a_1.print(); 
a_2.getData();
a_2.print();
 
a_buf = a_1 + a_2;
a_buf.print();   
}
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
21.10.2011, 07:08
Ответы с готовыми решениями:

Ошибка. Аварийное завершение программы.
Товарищи,есть функция.. cor find(cor start) { cor buf=start; cor pv=NULL; char adr; int...

Аварийное завершение программы
Программа завершается аварийно, но в файл записывает требуемую информацию. Укажите косяки. В...

Аварийное завершение программы
#include &quot;B.h&quot; #include &quot;D1.h&quot; #include &quot;D2.h&quot; #include &lt;iostream&gt; #include &lt;typeinfo&gt; ...

Аварийное завершение работы программы
1)программа работает правильно но когда нажимаю ентер (после этого она должна завершить работу)...

12
45 / 45 / 9
Регистрация: 11.04.2010
Сообщений: 223
21.10.2011, 07:23 2
А память под массив кто выделять будет? Пардон, прогнал...
0
18 / 18 / 8
Регистрация: 21.10.2011
Сообщений: 115
21.10.2011, 07:25  [ТС] 3
А что не так делаю? Можно подробнее, а то не понимаю.
0
45 / 45 / 9
Регистрация: 11.04.2010
Сообщений: 223
21.10.2011, 07:36 4
Во первых память выделять принято в конструкторе, во вторых в строке 45 вы создаете локальную переменную a, и храните адрес выделенной памяти в ней... Вот рабочий код:
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
#include <iostream>
 
using namespace std;
 
class Matrix
{
private:
        int**a;
        int n,m;
public:
        Matrix(){}
        void getData();
        void print();
        Matrix operator+(const Matrix&);
};
////////////////////////////////////////
 
Matrix Matrix::operator+(const Matrix &p)
{
    Matrix result(*this);
 
    for (int i = 0; i < n; ++i)
        for ( int j = 0; j < m; ++j)
            result.a[i][j] += p.a[i][j];
 
    return result;
}
 
//////////////в этой функции что - то не так как надо работает: программа 
//аварийно завершается на ее исполнении//////////////////////////
void Matrix::print()
{
 
for(int i=0;i<n;i++)
        for (int j=0;j<m;j++)
                cout<<a[i][j]<<" "<<endl;
}
 
////////////////////////////////////////
void Matrix::getData()
{
 
cout<<"\ninput n,m: \n";cin>>n>>m;
a=new int*[n];
for (int i=0;i<n;i++)
a[i]=new int [m];
 
cout<<"input matrix' elements: \n";
for(int i=0;i<n;i++)
        for (int j=0;j<m;j++)
                cin>>a[i][j];
}
 
int main(void)
{
 
Matrix a_1;
Matrix a_2;
Matrix a_buf;
 
a_1.getData();
a_1.print(); 
a_2.getData();
a_2.print();
 
a_buf = a_1 + a_2;
a_buf.print();   
return 0;
}
Добавлено через 1 минуту
А еще у вас нет проверки размерности матриц в операторе +.
1
Заблокирован
21.10.2011, 07:41 5
Сейчас протестируем-с...
*через минутку*

по поводу ошибок:
в функции void Matrix::getData()
error C2065: i: необъявленный идентификатор

Добавлено через 2 минуты
Matrix Matrix::operator+(const Matrix &p)

warning C4018: <: несоответствие типов со знаком и без знака

в строчке size_t i = 0; i < n; ++i

Без знаковый сравнивается со знаковым. Предупреждение следует понимать так: мина замедленного действия.
1
18 / 18 / 8
Регистрация: 21.10.2011
Сообщений: 115
21.10.2011, 07:44  [ТС] 6
Большое спасибо. Про выделение памяти в конструкторе я забыл...Сейчас исправлю.
А как лучше реализовать проверку размерности матриц?
0
45 / 45 / 9
Регистрация: 11.04.2010
Сообщений: 223
21.10.2011, 07:48 7
C++
1
2
3
4
5
6
7
8
9
10
11
Matrix Matrix::operator+(const Matrix &p)
{
    Matrix result(*this);
    if(result.m != p.m || result.n!=p.n)
//    действие в случае несовпадения
    for (int i = 0; i < n; ++i)
        for ( int j = 0; j < m; ++j)
            result.a[i][j] += p.a[i][j];
 
    return result;
}
0
18 / 18 / 8
Регистрация: 21.10.2011
Сообщений: 115
21.10.2011, 07:49  [ТС] 8
Да, что - то я тут намудрил. А я компиллирую на Visual C++ 6.0 и никаких ошибок не возникает. Наверное, компиллятор древний.
0
Заблокирован
21.10.2011, 07:56 9
Цитата Сообщение от romex Посмотреть сообщение
Во первых память выделять принято в конструкторе
Память принято выделять тогда, когда это требуется по задаче. И далеко не всегда, это происходит в конструкторе.

Добавлено через 3 минуты
нашел ошибку.

В функции void Matrix::getData()

int**a=new int*[n]; //объявление локальной переменной.

Вам не нужно ничего объявлять. У вас итак есть переменная класса.

Адрес примет локальная переменная, чей срок жизни быстро закончится. А вот настоящая переменная а, с которой работает метод Print() ничего не знает ни о каких выделялках.


Правильно будит вот так:

//int**a=new int*[n]; //ОШИБКА! НЕ НУЖНО СОЗДАВАТЬ ЛОКАЛЬНУЮ ПЕРЕМЕННУЮ!
a=new int*[n]; //мы сразу присваиваем уже объявленной переменной значение

Добавлено через 2 минуты
Цитата Сообщение от kirdenoff Посмотреть сообщение
Да, что - то я тут намудрил. А я компиллирую на Visual C++ 6.0 и никаких ошибок не возникает. Наверное, компиллятор древний.

Это точно) Компилятор древний. Рекомендую вам его заменить на более новые аналоги. Он уже не вполне соответствует стандарту языка.
1
18 / 18 / 8
Регистрация: 21.10.2011
Сообщений: 115
21.10.2011, 08:02  [ТС] 10
Цитата Сообщение от Bers Посмотреть сообщение
Это точно) Компилятор древний. Рекомендую вам его заменить на более новые аналоги. Он уже не вполне соответствует стандарту языка.
Да просто в универе учусь, а там условие: разработка на VC++6.0 или Borland 5.0.
Вот и приходится мучиться.
0
Заблокирован
21.10.2011, 08:09 11
Цитата Сообщение от kirdenoff Посмотреть сообщение
Вот и приходится мучиться.
В таких случаях лучше писать безопасно:

int i;
for ( i = 0; i < n; ++i) {}


То есть, объявляйте счетчики вне цикла. Это прожуют и старые и новые компиляторы.
Но вообще задумайтесь - почему у вас такие ограничения. Вас там что, сознательно обучают уже устаревшим стандартам? Вам же потом после такого обучения заново переучиваться придётся.

Я бы на вашем месте призадумался - зачем тратить время и деньги на заведомо устаревшие стандарты? Не лучше ли договорится с преподавателем, объяснить ему, что я буду сидеть в новенькой студии, и писать в соответствии с новеньким стандартом.

Если преподавателю нужен рабочий код, и знания - да пожалуйста. Но я не гарантирую совместимости с древними компиляторами.
1
18 / 18 / 8
Регистрация: 21.10.2011
Сообщений: 115
21.10.2011, 08:15  [ТС] 12
Цитата Сообщение от Bers Посмотреть сообщение


То есть, объявляйте счетчики вне цикла. Это прожуют и старые и новые компиляторы.
Но вообще задумайтесь - почему у вас такие ограничения. Вас там что, сознательно обучают уже устаревшим стандартам? Вам же потом после такого обучения заново переучиваться придётся.

Я бы на вашем месте призадумался - зачем тратить время и деньги на заведомо устаревшие стандарты? Не лучше ли договорится с преподавателем, объяснить ему, что я буду сидеть в новенькой студии, и писать в соответствии с новеньким стандартом.

Если преподавателю нужен рабочий код, и знания - да пожалуйста. Но я не гарантирую совместимости с древними компиляторами.
Я в Беларуси живу. Тут ничего никому не докажешь. Совдепия сплошная.
0
Заблокирован
21.10.2011, 08:18 13
Цитата Сообщение от kirdenoff Посмотреть сообщение
Я в Беларуси живу. Тут ничего никому не докажешь. Совдепия сплошная.
Ставьте себе дома студию поновее. И пишите код одновременно в и с шестерке, и в новой.
Если не хотите потом ходить по граблям и переучиваться.
0
21.10.2011, 08:18
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
21.10.2011, 08:18
Помогаю со студенческими работами здесь

Рекурсивная функция/аварийное завершение программы
Всем привет. Задание такое: есть вырожение Нужно найти такое х, с погрешность 1e-7 или больше,...

Аварийное завершение программы (Прата, глава 12, string1)
Доброго времени суток. Не могу понять, из-за чего происходит аварийное завершение программы. VS...

Abnormal program termination (аварийное завершение программы)
Из за чего программа может выдать abnormal program termination (аварийное завершение программы),...

Аварийное завершение программы (Process terminated with status -1073741510)
Всем доброго времени суток. У меня следующая проблема. Вот код(из учебника, не мой): #include...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
13
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru