В чем моя ошибка?(ООП, классы)

@roo3 · Регистрация: 14.10.2015

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

Что изменить в файле functionss.h? Заранее спасибо!!!

Задание: Определить класс «матрица». В класс включить два конструктора: 1) создание матрицы по количеству столбцов и строк; 2) конструктор-копию. Определить функции члены: вывод на экран элементов матрицы. Перегрузить оператор: ++ нахождение суммы положительных элементов главной диагонали.

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
Файл: mainfuction.cpp
#include <cstdlib>
#include <iostream>
#include <fstream> 
#include <string> 
#include <clocale> 
#include <string.h>
#include "class.h"
#include "functionss.h"
 
using namespace std;
 
int main()
{   
    int a,b;
    cout<<"Vveite kolichestvo strok"<<endl;
    cin>>a;
    cout<<"Vvedite kolichestvo stolbcov"<<endl;
    cin>>b;
    cout<<endl;
    Matrix M(a,b);
    M.Random();
    M.Display();
    cout<<"___________________________"<<endl;
    Matrix A = M;
    cout<<"___________________________"<<endl;
    A.Display();
    //int Kol=(M.Search());
    cout<<++A;
    cout<<endl;
    //cout<<" Kolichestvo elementov = "<<Kol<<endl;
    return 0;
}
 
Файл: class.h
using namespace std;
 
class Matrix
{
    private:
        int *Matr;
        int m;
        int n;
 
    public:
        // конструкторы и деструктор
         Matrix(int i,int j);
         Matrix(const Matrix &M);
         ~Matrix();
         
         
        void Random();
        //int Search();
        int operator++();
        void Display();
};
Файл: functionss.h
Matrix::Matrix(int i,int j): m(i), n(j) 
         { 
            Matr = new int*[m];
            for (int i=0; i<m; i++)
                Matr[i] = new int[n]; 
         }
Matrix::Matrix(const Matrix &M)
         {
          cout<<"copyed"<<endl;
          Matr = M.Matr;
          m = M.m;
          n = M.n;
         }
int Matrix::operator++()
         {
             int k=0;
          for (int i=0; i<m; i++)if(Matr[i][i]>0)k+=Matr[i][i];
          return k;
         }
Matrix::~Matrix()
        {
        cout<<"deleted"<<endl;
        }  
void Matrix::Random()
{
         for(int i=0; i<m; i++)
            {
                for(int j=0; j<n; j++)
                {
                    Matr[i][j]=rand() % 200 - 100;
                }   
        }
};
 
 
void Matrix::Display()
{
     for(int i=0; i<m; i++)
            {
                for(int j=0; j<n; j++)
                {
                    cout.width(4);
                    cout << Matr[i][j];
                }
                cout << endl;
            }

@zss · 14.10.2015, 10:46

Копиконструктор

C++
1
2
3
4
5
6
7
8
9
10
11
Matrix::Matrix(const Matrix &M)
{
      m = M.m;
      n = M.n;
      Matr = new int*[m];
      for (int i=0; i<m; i++)
         Matr[i] = new int[n];          
      for (int i=0; i<m; i++)
        for(int j=0;j<n;j++)
              Matr[i][j]=M.Matr[i][j];
}

Для поддержки ПРАВИЛА ТРЕХ обязателен operator=

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
Matrix& operator=(const Matrix &M)
{
  if(this!=&M)
  {      
 
      for (int i=0; i<m; i++)
           delete[] Matr[i];
      delete[] Matr;
      m = M.m;
      n = M.n;
      Matr = new int*[m];
      for (int i=0; i<m; i++)
         Matr[i] = new int[n];          
      for (int i=0; i<m; i++)
        for(int j=0;j<n;j++)
              Matr[i][j]=M.Matr[i][j];
  }
  return *this;
}

@EfesXZC · 14.10.2015, 10:46

roo3, а где высвобождение памяти, отводимой под матрицы?

@Kerry_Jr · 14.10.2015, 10:52

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

C++
1
if(this!=&M)

желательно это условие и в конструктор копии вставить.

@zss · 14.10.2015, 10:52

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

Файл: functionss.h

Этот файл надо переобозвать как functionss.cpp и подключить к проекту в раздел "Файлы исходного кода".
И в конструкторе Matrix::Matrix(int i,int j) имеет смысл присвоить значение созданным элементам массива Matr

@Kerry_Jr · 14.10.2015, 10:54

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

C++
1
terurn *this;

все конечно поняли, что Вы хотели написать return, но все же

@EfesXZC · 14.10.2015, 10:55

C++
1
2
3
4
5
6
7
Matrix::~Matrix()
        {
            for (int i=0; i<m; i++)
                 delete[] Matr[i];
            delete[] Matr;
            cout<<"deleted"<<endl;
        }

@zss · 14.10.2015, 10:59

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

желательно это условие и в конструктор копии вставить

Приведите пример, когда это может быть?
Копиконструктор вызывается при СОЗДАНИИ объекта и, следовательно, условие if(this!=&M) никогда не будет истинным.

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

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

все конечно поняли, что Вы хотели написать return

Пардон, левая рука всегда печатает медленнее правой

@roo3 · 16.10.2015, 16:02 **[ТС]**

zss,

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
Matrix::Matrix(int i,int j): m(i), n(j) 
         { 
            Matr = new int*[m];
            for (int i=0; i<m; i++)
                Matr[i] = new int[n]; 
         }
Matrix::Matrix(const Matrix &M)
{
      m = M.m;
      n = M.n;
      Matr = new int*[m];
      for (int i=0; i<m; i++)
         Matr[i] = new int[n];          
      for (int i=0; i<m; i++)
        for(int j=0;j<n;j++)
              Matr[i][j]=M.Matr[i][j];
}
Matrix & operator = (const Matrix &M)
{
  if(this!=&M)
  {      
 
      for (int i=0; i<m; i++)
           delete[] Matr[i];
      delete[] Matr;
      m = M.m;
      n = M.n;
      Matr = new int*[m];
      for (int i=0; i<m; i++)
         Matr[i] = new int[n];          
      for (int i=0; i<m; i++)
        for(int j=0;j<n;j++)
              Matr[i][j]=M.Matr[i][j];
  }
  return *this;//присвоение самому себе
}
 
int Matrix::operator++()
         {
             int k=0;
          for (int i=0; i<m; i++)if(Matr[i][i]>0)k+=Matr[i][i];
          return k;
         }
Matrix::~Matrix()
        {
            for (int i=0; i<m; i++)
                 delete[] Matr[i];
            delete[] Matr;
            cout<<"deleted"<<endl;
        }   
void Matrix::Random()
{
         for(int i=0; i<m; i++)
            {
                for(int j=0; j<n; j++)
                {
                    Matr[i][j]=rand() % 200 - 100;
                }   
        }
};
 
 
void Matrix::Display()
{
     for(int i=0; i<m; i++)
            {
                for(int j=0; j<n; j++)
                {
                    cout.width(4);
                    cout << Matr[i][j];
                }
                cout << endl;
            }
};

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

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
Matrix::Matrix(int i,int j): m(i), n(j) 
         { 
            Matr = new int*[m];
            for (int i=0; i<m; i++)
                Matr[i] = new int[n]; 
         }
Matrix::Matrix(const Matrix &M)
{
      m = M.m;
      n = M.n;
      Matr = new int*[m];
      for (int i=0; i<m; i++)
         Matr[i] = new int[n];          
      for (int i=0; i<m; i++)
        for(int j=0;j<n;j++)
              Matr[i][j]=M.Matr[i][j];
}
Matrix & Matrix::operator = (const Matrix &M)
{
  if(this!=&M)
  {      
 
      for (int i=0; i<m; i++)
           delete[] Matr[i];
      delete[] Matr;
      m = M.m;
      n = M.n;
      Matr = new int*[m];
      for (int i=0; i<m; i++)
         Matr[i] = new int[n];          
      for (int i=0; i<m; i++)
        for(int j=0;j<n;j++)
              Matr[i][j]=M.Matr[i][j];
  }
  return *this;//присвоение самому себе
}
 
int Matrix::operator++()
         {
             int k=0;
          for (int i=0; i<m; i++)if(Matr[i][i]>0)k+=Matr[i][i];
          return k;
         }
Matrix::~Matrix()
        {
            for (int i=0; i<m; i++)
                 delete[] Matr[i];
            delete[] Matr;
            cout<<"deleted"<<endl;
        }   
void Matrix::Random()
{
         for(int i=0; i<m; i++)
            {
                for(int j=0; j<n; j++)
                {
                    Matr[i][j]=rand() % 200 - 100;
                }   
        }
};
 
 
void Matrix::Display()
{
     for(int i=0; i<m; i++)
            {
                for(int j=0; j<n; j++)
                {
                    cout.width(4);
                    cout << Matr[i][j];
                }
                cout << endl;
            }
};

Все, сообразил

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@EfesXZC 2 / 2 / 2 Регистрация: 07.08.2015 Сообщений: 72
	14.10.2015, 10:46
	roo3, а где высвобождение памяти, отводимой под матрицы? 0

В чем моя ошибка?(ООП, классы)

Решение