что-то я запуталась немного,а где ошибка,кто знает?

Matrix &operator +( const Matrix &R)

    {
        Matrix **T;
        T=new Matrix*[m_width];
        for(unsigned i=0;i<m_width;i++)
        {
            T[i]=new Matrix[m_height];
            for(unsigned j=0;j<m_height;j++)
            {
                T[i][j]=m_value[i][j]+R.m_value[i][j];
            }
        } return T[m_width][m_height];???
    }

Во-первых интересно было бы узнать какая ошибка
Во-вторых в приведенно коде не видно объявления переменных m_width и m_height

Не могу почему-то сложить матрицы друг с другом, m_width и m_height  равны значениям,которые я ввожу сама в main(),это предусмотрено в конструкторе:
Matrix A(3,4),где 3-это m_width ,а 4-m_height

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

Какая конкретно ошибка? Неправильно складывает? 

void main()
{

Matrix A(3,3);
    Matrix B(3,3);
    Matrix C(3,3);
    C=A+B;вот сдесь и ошибка - вообще не работает

return T[m_width-1][m_height-1]

Это сообщение отредактировал(а) TIGERоX - 16.12.2006, 22:54

Не работает или не компилируется?

компилируется,пока не ввожу в

 main()
{
 C=A+B; вот это

Natalka, покажи весь код и скажи, что этот перегружаемый оператор должен делать

#include "stdafx.h"
#include <ctime>
using namespace std;
class Matrix
{
    int ** m_value;
   unsigned m_width,m_height;
public:
   Matrix( unsigned width = 0, unsigned height = 0 ) : 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][k]=0;
       }
   }
 void Get()
   {
       for ( unsigned i = 0; i < m_width; ++i )
       {
          for ( unsigned k = 0; k < m_height; ++k )
          {
              m_value[i][k]=rand()%10+1;
              cout<<m_value[i][k]<<" ";
          }cout<<endl;
       }
   }
 Matrix &operator +( const Matrix &R)
    {
        Matrix **T;
        T=new Matrix*[m_width];
        for(unsigned i=0;i<m_width;i++)
        {
            T[i]=new Matrix[m_height];
            for(unsigned j=0;j<m_height;j++)
            {
                T[i][j]=m_value[i][j]+R.m_value[i][j];
            }
        } return T[m_width][m_height];
    }

   ~Matrix()
   {
       for ( unsigned i = 0; i < m_width; i++ )
          delete [] m_value[i];
       delete [] m_value;       
   }
}; 
void main()
{
    //srand(time(0));
    Matrix A(3,3);
                Matrix B(3,3);
                Matrix C(3,3);
    C=A+B;
    A.Get();
    cout<<endl;
    B.Get();
    cout<<endl;
    C.Get();
    
}
Я просто должна сложить две матрицы друг с другом

Natalka, и присвоить результат третьей =) Равно тоже надо перегружать

А зачем столько объектов типа Matrix?

#include "stdafx.h"
#include <ctime>
using namespace std;
class Matrix
{
    int ** m_value;
   unsigned m_width,m_height;
public:
   Matrix( unsigned width = 0, unsigned height = 0 ) : 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][k]=0;
       }
   }
 void Get()
   {
       for ( unsigned i = 0; i < m_width; ++i )
       {
          for ( unsigned k = 0; k < m_height; ++k )
          {
              m_value[i][k]=rand()%10+1;
              cout<<m_value[i][k]<<" ";
          }cout<<endl;
       }
   }
 Matrix &operator +( const Matrix &R)
    {
        Matrix **T;
        T=new Matrix*[m_width];
        for(unsigned i=0;i<m_width;i++)
        {
            T[i]=new Matrix[m_height];
            for(unsigned j=0;j<m_height;j++)
            {
                T[i][j]=m_value[i][j]+R.m_value[i][j];
            }
        } return T[m_width][m_height];
    }
 Matrix& operator=( const Matrix &matrix )
   {
       if ( this != &matrix )
       {
          for ( unsigned i = 0; i < m_width; i++ )
             delete [] m_value[i];
          delete [] m_value;
          m_width = matrix.m_width;
          m_height = matrix.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 k = 0; k < m_height; ++k )
                m_value[i][k] = matrix.m_value[i][k];
          }          
       }
       return *this;
   }

   ~Matrix()
   {
       for ( unsigned i = 0; i < m_width; i++ )
          delete [] m_value[i];
       delete [] m_value;       
   }
}; 
void main()
{
    //srand(time(0));
    Matrix A(3,3);
                Matrix B(3,3);
                Matrix C(3,3);
    C=A+B;    ------------------------------------//а это разве не третья, равно перегрузила
    A.Get();
    cout<<endl;
    B.Get();
    cout<<endl;
    C.Get();
    
}

Зачем такой большой код

Cпасибо,конечно,но мне нужен код с перегруженным оператором и с динамическим массивом 

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


и оператор +=



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



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

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



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

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


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;
    }

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

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

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

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

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

#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();



} 
 

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

A вот так:

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

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

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

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

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

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

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

msdn, General Rules for Operator Overloading


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


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

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

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

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

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