VPF::[C++] Где ошибка? - Форум программистов

Rockie

Дата 17.12.2006, 03:31 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1143
Регистрация: 23.4.2006

Репутация: 13
Всего: 31

Natalka, Во-первых, у тебя нет коструктора копирования, это раз. Во-вторых, операторы с двумя операндами (x+y) не могут членами класса, их надо выносить за него(вынес все для читабельности). Если определить оператор присваивания:

Код

Matrix& Matrix::operator=(const Matrix& obj)
    {
      if(this == &obj)  return *this;

      for(unsigned i=0;i<this->m_width;i++)
      {  for(unsigned j=0;j<this->m_height;j++)
           this->m_value[i][j] = obj.m_value[i][j];
      }
      return *this;
    }

и оператор +=

Код

Matrix& Matrix::operator+=(const Matrix& obj)
{
 Matrix T(this->m_width, this->m_height);

  for (unsigned  i = 0; i < m_width; i++)
    for (unsigned  j = 0; j <m_height; j++)
      m_value[i][j] = m_value[i][j] + obj.m_value[i][j];

 return *this;
}

то код оператора сложения обьъектов вырождается в

Код

const Matrix operator+( const Matrix& lhs, const Matrix& rhs ){
  Matrix obj(lhs);
  return obj+=rhs;
}

причем последний всегда возвращает объект _по значению_. По-хорошему, в операторе сложения надо еще вставлять проверку на совпадение столбцов/колонок, ну это уже на твоей совести =) По-хорошему в программе должен быть конструктор с передаваемым значением и методы доступа к элементам матрицы. А то так как в данном случае у тебя все инициалировалось одним и тем же числом в конструкторе, мне для проверки пришлось выносить члены-данные в public и изменять их напрямую:

Код

Matrix A(3,3);
A.m_value[1][1] = 10;

Этого делать конечно нельзя. Ну и названия наверное - почему-то метод Get занимается распечаткой, логичнее imho назвать по-другому а Get как раз заставить возвращать определенный элемент матрицы. В программе еще много недочетов, я исправил только функциональность.

Код

#include <iostream>

using namespace std;

class Matrix
{
  public:
  int** m_value;
  unsigned m_width, m_height;

  void Get();
  Matrix( unsigned width = 0, unsigned height = 0);
  Matrix(const Matrix&);
  Matrix& operator=(const Matrix&);
  Matrix& operator+=(const Matrix&);
  friend const Matrix operator+ (const Matrix&, const Matrix&);
  ~Matrix();
};


const Matrix operator+( const Matrix& lhs, const Matrix& rhs ){
  Matrix obj(lhs);
  return obj+=rhs;
}

Matrix& Matrix::operator+=(const Matrix& obj)
{
 Matrix T(m_width,m_height);

  for (unsigned  i = 0; i < m_width; i++)
    for (unsigned  j = 0; j <m_height; j++)
      m_value[i][j] = m_value[i][j] + obj.m_value[i][j];

 return *this;
}


Matrix::Matrix( unsigned width, unsigned height) : m_width(width), m_height(height)
   {
     m_value = new int*[m_width];
     for ( unsigned i = 0; i < m_width; i++ )
      {
         m_value[i] = new int[m_height];
         for ( unsigned j = 0; j< m_height; j++ )
           m_value[i][j]=2;
      }
   }

   
Matrix::~Matrix()
   {
    for ( unsigned i = 0; i < m_width; i++ )
      delete [] this->m_value[i];
    delete [] this->m_value;
   }

void Matrix::Get()
   {   cout<<endl;
       for ( unsigned i = 0; i < m_width; ++i )
       {
          for ( unsigned k = 0; k < m_height; ++k )
           cout<<m_value[i][k]<<" ";
           cout<<'\n';
       }
   }


Matrix& Matrix::operator=(const Matrix& obj)
    {
      if(this == &obj)  return *this;

      for(unsigned i=0;i<this->m_width;i++)
      {  for(unsigned j=0;j<this->m_height;j++)
           this->m_value[i][j] = obj.m_value[i][j];
      }
      return *this;
    }


Matrix::Matrix(const Matrix& obj)
{
    // ïðîâåðêà ..

     m_width = obj.m_width;
     m_height = obj.m_height;
     m_value = new int*[m_width];

     for ( unsigned i = 0; i < m_width; i++ )
      {  m_value[i] = new int[m_height];
         for ( unsigned j = 0; j< m_height; j++ )
           m_value[i][j]=obj.m_value[i][j];
      }
     m_width = obj.m_width;
     m_height = obj.m_height;

}



int _tmain()
{
      Matrix A(3,3);
      A.m_value[1][1] = 10;
      A.Get();

      Matrix B=A;
      B+=A;
      B.Get();

      Matrix C(3,3);
      C=A+B+B;
      C.Get();

      cin.get();
      return 0;
}

А, ну и кнопочка Код справа и выше формы для ответа =)

--------------------

Чтобы иметь большой гардероб - надо иметь большой гардероб.

Natalka

Дата 17.12.2006, 23:31 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 21
Регистрация: 7.12.2006
Где: Днепродзержинск

Репутация: нет
Всего: нет

А если сделать так,то ,наверное, будет работать

Matrix operator +( const Matrix &R)
    {
        Matrix T(m_height,m_width);
        for(unsigned i=0;i<m_width;i++)
            for(unsigned j=0;j<m_height;j++)
            {
                T.m_value[i][j]=m_value[i][j]+R.m_value[i][j];
            }
        return T;
    }

apook

Дата 18.12.2006, 06:43 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 794
Регистрация: 12.7.2006

Репутация: 10
Всего: 23

Я не пойму как class matrix делается массивом классов Matrix

--------------------

Мои руки из дуба, голова из свинца ну и пусть ...

Rockie

Дата 18.12.2006, 08:49 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1143
Регистрация: 23.4.2006

Репутация: 13
Всего: 31

Цитата(apook @ 18.12.2006, 06:43

)

Я не пойму как class matrix делается массивом классов Matrix

apook, класс не может сделаться массивом, это разные понятия. Массив объектов класса Matrix можно создать так:

Код

Matrix a[10];

Чтобы создать массив объектов, нужен конструктор по умолчанию(он есть).

--------------------

Чтобы иметь большой гардероб - надо иметь большой гардероб.

apook

Дата 18.12.2006, 09:48 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 794
Регистрация: 12.7.2006

Репутация: 10
Всего: 23

Почему сей код не работает при matrix B;

Код


#include "stdio.h"
#include "stdlib.h"

#include "conio.h"

//using namespace std;


class matrix
{
friend matrix operator+ ( matrix &, matrix & );

public:
int **m_val;
matrix( int a, int b );
void Get();
~matrix();


protected:
int height;
int width;
};


matrix::matrix( int a, int b )
{

m_val=new int*[height];

height =a;
width =b;

for( int i=0; i <height; i++ )
{
    m_val[i] =new int[width];
    for( int j=0; j <width; j++ )
        m_val[i][j] =0;
    }
}

void matrix::Get()
{
for( int i=0; i <height; i++ )
{
    for( int j=0; j <width; j++ )
        printf( "%d ", m_val[i][j] );
    putc( '\n', stdout );
    }
}

matrix::~matrix()
{
delete [] m_val;
}

matrix operator+( matrix &arg1, matrix &arg2 )
{
matrix result( arg1 );
result =result +arg2;
return result;
}



void main()
{
int i, j;


matrix A( 5, 5 );
for( i=0; i <5; i++ )
    for( j=0; j <5; j++ )
        A.m_val[i][j] =rand() % 10;


matrix B( 5, 5 ); <--Abnormal program Termination
/*
for( i=0; i <5; i++ )
    for( j=0; j <5; j++ )
        B.m_val[i][j] =rand() % 10;

*/

A.Get();
//B.Get();


getch();



}

--------------------

Мои руки из дуба, голова из свинца ну и пусть ...

ptr

Дата 18.12.2006, 10:26 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 661
Регистрация: 31.5.2006
Где: Новосибирск

Репутация: нет
Всего: 14

Код


height =a;
m_val=new int*[height];

Это сообщение отредактировал(а) ptr - 18.12.2006, 10:27

--------------------

Единственный способ определить границы возможного - это выйти за эти границы, в невозможное.
Артур Кларк.

Natalka

Дата 18.12.2006, 10:56 (ссылка)

(нет голосов)

Загрузка ...

у меня все работает так:

#include "stdafx.h"

using namespace std;
class matrix
{
    int **m_val;
    int height;
int width;
friend matrix operator+ ( matrix &, matrix & );
public:
matrix( int a, int b );
void Get();
~matrix();
};
matrix::matrix( int a, int b )
{

height =a;
width =b;
m_val=new int*[height];//->нужно было поменять местами c  этим:height =a;width =b;

for( int i=0; i <height; i++ )
{
    m_val[i] =new int[width];
    for( int j=0; j <width; j++ )
        m_val[i][j] =0;
    }
}
void matrix::Get()
{
for( int i=0; i <height; i++ )
{
    for( int j=0; j <width; j++ )
    {
        m_val[i][j] =rand() % 10;
        printf( "%d ", m_val[i][j] );
    putc( '\n', stdout );
    }
  }
}

matrix::~matrix()
{
delete [] m_val;
}
matrix operator+( matrix &arg1, matrix &arg2 )
{
matrix result( arg1 );
result =result +arg2;
return result;
}

void main()
{
//int i, j;
matrix A( 5, 5 );
matrix B( 5, 5 ); //<--Abnormal program Termination
/*
//for( i=0; i <5; i++ )
   // for( j=0; j <5; j++ )
     //   B.m_val[i][j] =rand() % 10;//это не нужно тут писать

*/

A.Get();
  putc( '\n', stdout );
B.Get();
cin.get();

}

Anikmar

Дата 18.12.2006, 11:07 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2513
Регистрация: 26.11.2006
Где: Санкт-Петербург

Репутация: 9
Всего: 59

Замечание:
Деструктор неправильный - удалит только указатели на строки (или столбцы - что там как считается - всегда путался)

Natalka

Дата 18.12.2006, 11:16 (ссылка)

(нет голосов)

Загрузка ...

A вот так:

matrix::~matrix()
   {
       for ( int i = 0; i < width; i++ )
          delete [] m_val[i];
       delete [] m_val;
   }

Anikmar

Дата 18.12.2006, 12:02 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2513
Регистрация: 26.11.2006
Где: Санкт-Петербург

Репутация: 9
Всего: 59

Natalka, так вроде нормально

Rockie тебе практически готовый код предложил - у него там вроде все нормально сделано

JackYF

Дата 18.12.2006, 14:56 (ссылка)

(нет голосов)

Загрузка ...

полуавантюрист

Профиль
Группа: Участник
Сообщений: 5814
Регистрация: 28.8.2004
Где: страна тысячи озё р

Репутация: 2
Всего: 162

Цитата(Rockie @ 17.12.2006, 03:31

)

Во-вторых, операторы с двумя операндами (x+y) не могут членами класса,

Это еще с какой стати?

--------------------

Пожаловаться на меня как модератора можно здесь.

apook

Дата 18.12.2006, 20:25 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 794
Регистрация: 12.7.2006

Репутация: 10
Всего: 23

Почему так тщательно удалять

Код


       for ( int i = 0; i < width; i++ )
          delete [] m_val[i];
       delete [] m_val;

ведь массив независимо от его разрядности удаляется одной строкрй
delete [] m_val;
я так где-то читал

--------------------

Мои руки из дуба, голова из свинца ну и пусть ...

Rockie

Дата 18.12.2006, 22:18 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1143
Регистрация: 23.4.2006

Репутация: 13
Всего: 31

Цитата(JackYF @ 18.12.2006, 14:56

)

Цитата(Rockie @ 17.12.2006, 03:31 )Во-вторых, операторы с двумя операндами (x+y) не могут членами класса,Это еще с какой стати?

JackYF, на самом деле можно определить и как ф-ю друга, и как функцию-член класса, но imho стоит привыкать к первому. Потомучто в первом случае случае (x+y) <=> (y+x), а во втором-нет:

msdn, General Rules for Operator Overloading

Цитата

// rules_for_operator_overloading.cpp
class Point
{
public:
    Point operator<( Point & );  // Declare a member operator
                                 //  overload.
    // Declare addition operators.
    friend Point operator+( Point&, int );
    friend Point operator+( int, Point& );
};

int main()
{
}

The preceding code sample declares the less-than operator as a member function; however, the addition operators are declared as global functions that have friend access. Note that more than one implementation can be provided for a given operator. In the case of the preceding addition operator, the two implementations are provided to facilitate commutativity. It is just as likely that operators that add a Point to a Point, int to a Point, and so on, might be implemented.

Ну или Страуструп, проще но то же самое:

Цитата

Операторная функция, имеющая первым параметр основного типа,
не может быть функцией-членом. Так, если мы прибавляем комплексную
переменную aa к целому 2, то при подходящем описании функции-члена
aa+2 можно интерпретировать как aa.operator+(2), но 2+aa так
интерпретировать нельзя, поскольку не существует класса int, для
которого + определяется как 2.operator+(aa). Даже если бы это было
возможно, для интерпретации aa+2 и 2+aa пришлось иметь дело с двумя
разными функциями-членами. Этот пример тривиально записывается
с помощью функций, не являющихся членами.

Добавлено @ 22:28

Цитата(apook @ 18.12.2006, 20:25

)

ведь массив независимо от его разрядности удаляется одной строкрй delete [] m_val; я так где-то читал

так и происходит. каждому new свой delete.
В конструкторе выделяется память для массива указателей на int(первый new). Потом под каждый из этих указателей выделяется память для массива int-ов. Таким образом получаем двумерный массив. Соответственно его и удалять надо так же, как создавали. То есть в цикле удаляем память под int-ы, а потом сам массив указателей.

Это сообщение отредактировал(а) Rockie - 18.12.2006, 22:30

--------------------

Чтобы иметь большой гардероб - надо иметь большой гардероб.

sergejzr

Дата 18.12.2006, 22:58 (ссылка)

(нет голосов)

Загрузка ...

Un salsero
Group Icon

Профиль
Группа: Админ
Сообщений: 13285
Регистрация: 10.2.2004
Где: Германия г .Ганновер

Репутация: 11
Всего: 360

Для домашних заданий, курсовых, существует "Центр Помощи".

Тема перенесена!

--------------------

Me.
FB.
LinkedIn.
ORCID.
YT.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Центр помощи \| Следующая тема »