конструкторы

@zhenya.ya · Регистрация: 29.11.2009

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

Есть 2 проблемы:
1) Сказали сделать 2 конструктора: один по умолчанию, а другой-с параметрами. Какие должны быть параметры в моем конструкторе?(я так понимаю нужно как-то избавиться от private:i,j,n,m?)
2) Почему при данном выделении памяти( через malloc нужно по условию) "ложится" метод print();
Может для этого нужен конструктор копирования?

C++

class matrix
{   
private:
 
        int m,n,i,j;
        int **matrica;
public: 
        matrix();
        matrix(параметры); //констр. с параметрами
        ~matrix(); 
        int get(int ,int);
        void print();
        void setSize();
};
 
matrix::matrix()
{ 
}
 
matrix::matrix( параметры)
{
    setSize();
    int**matrica=(int**)malloc(n*sizeof(int*));
    for(i=0;i<n;i++)
        matrica[i]=(int*)malloc(m*sizeof(int));
 
        for ( i = 0; i < n; i++)
               for ( j = 0; j < m; j++)
                        matrica[i][j] = rand() % 10;
}
 
 void matrix::setSize()
{
    int first;
    int second;
    cout<<"Enter first size: ";
    cin>>first;
    cout<<"Enter second size: ";
    cin>>second;
    n=first;
    m=second;
    system("cls");
 }
void matrix::print()
{
     for( i = 0; i < n; i++)
     {
         for(j = 0; j < m; j++)
               cout<<matrica[i][j]<<' ';
         cout<<"\n";
     }
}
 matrix::~matrix()
{
     for (i = 0; i < n; i++)
     free(matrica[i]);
free(matrica);
}
int matrix::get(int i,int j)
{
         if(i>=n||j>=m)
            return 0;
         cout<<matrica[i][j]<<'\n';
}
 
int main ()
{  
        system("cls");
        matrix object;
        object.print();
        int i, j;
        cout<<"\n get (i.j):"<<endl;
        cout<<"Enter i: ";
        cin>>i;
        cout<<"Enter j: ";
        cin>>j;
        if(object.get(i,j)==0)
           cout<<"i or j more or equal than n or m!\n";
        system("PAUSE");
        return 0;
}

Jupiter · 14.09.2010, 19:56

Сказали сделать 2 конструктора: один по умолчанию, а другой-с параметрами. Какие должны быть параметры в моем конструкторе?(я так понимаю нужно как-то избавиться от private:i,j,n,m?)

к примеру, размер создаваемой матрицы может быть параметром, а i,j вроде как в классе и не нужны

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

@Crudelis · 14.09.2010, 23:03

C++

1 2	matrix(); //простой конструктор matrix(int a,char a,float a и т.д.); //констр. с параметрами

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

Сообщение от zhenya.ya

понимаю нужно как-то избавиться от private:i,j,n,m?)

ошибочное мнение

@zhenya.ya · 14.09.2010, 23:19 **[ТС]**

Не, я имею в виду конкретный свой случай.
Т.е. я создаю конструктор по умолчанию(простой), а потом с параметром.
Но при запуске, видя один объект(matrix object), он запустит консторуктор по умолчанию(простой), а как же сделать так, чтобы полсе запуска простого, запустился конструктор с параметром,который вводит матрицу???

C++

class matrix
{ 
private:
        int m,n;
        int **matrica;
public: 
         matrix();
         matrix(int);
        ~matrix(); 
        matrix(const matrix &A1);
        int get(int ,int);
        void print();
        void setSize();
};
matrix::matrix()
{
    int a;
    n=m=a=0;
}
void matrix::setSize()
{
    int first;
    int second;
    cout<<"Enter first size: ";
    cin>>first;
    cout<<"Enter second size: ";
    cin>>second;
    n=first;
    m=second;
    system("cls");
 }
 
matrix::matrix(int b)
{  
    setSize();
   
    int **matrica=(int**)malloc(n*sizeof(int*));
    for(int i=0;i<n;i++)
        matrica[i]=(int*)malloc(m*sizeof(int));
 
        for ( int i = 0; i < n; i++)
               for (int j = 0; j < m; j++)
                        matrica[i][j] = rand() % 10;
 }
void matrix::print()
{
     for(int i = 0; i < n; i++)
     {
         for(int j = 0; j < m; j++)
               cout<<matrica[i][j]<<' ';
         cout<<"\n";
     }
}
 matrix::~matrix()
{
     for (int i = 0; i < n; i++)
     free(matrica[i]);
free(matrica);
}
int matrix::get(int i,int j)
{
         if(i>=n||j>=m)
            return 0;
         cout<<matrica[i][j]<<'\n';
    
}
 
matrix::matrix(const matrix &A1)
{
    n=A1.n;
    int **matrica=(int**)malloc(n*sizeof(int*));
    for(int i=0;i<n;i++)
        matrica[i]=(int*)malloc(m*sizeof(int));
 
    for(int i=0;i<n;i++)
        for(int j=0;j<n;j++)
            matrica[i][j]=A1.matrica[i][j];
}
int main ()
{  
        system("cls");
        matrix object;
        object.print();
        int i, j;
        cout<<"\n get (i.j):"<<endl;
        cout<<"Enter i: ";
        cin>>i;
        cout<<"Enter j: ";
        cin>>j;
        if(object.get(i,j)==0)
           cout<<"i or j more or equal than n or m!\n";
        system("PAUSE");
        return 0;
}

@ForEveR · 14.09.2010, 23:21

zhenya.ya,

C++

1	matrix Ob(//параметры. допустим m,n);

В месте вызова. Полагаю что в мейне

Jupiter · 14.09.2010, 23:21

C++

1	matrix object(5);

либо

C++

1	matrix object = 5;

@Crudelis · 14.09.2010, 23:22

вот пример использования перегруженного конструктора:

C++

void main()
{
matrix A;           //здесь сработает конструктор по умолчанию matrix();
matrix A(50);           //здесь сработает перегруженный конструктор к примеру matrix(int a);
}

а ваш конструктор копирования matrix(const matrix &A1); сработает при

C++

1 2	matrix A,B; A=B;

@zhenya.ya · 15.09.2010, 00:16 **[ТС]**

спс за примеры, но теперь ругается на метод print(), а точнее на строку: cout<<matrica[i][j]<<' ';
Что не так?
И кстати, я не понял как сработает конструктор копирования ( непонятно как можно прировнять оъекты, если мой копирующий конструктор написан верно,а я там выделяю память для матрицы, то какие объекты мне нужно приравнять?)

@Crudelis · 15.09.2010, 01:41

добавьте в конструктор копирования проверку, если есть матрица matrica то её нужно удалить и создать новую, вообще при копировании объектов копируются все поля класса.

@CyBOSSeR · 15.09.2010, 09:50

Сообщение от zhenya.ya

И кстати, я не понял как сработает конструктор копирования ( непонятно как можно прировнять оъекты, если мой копирующий конструктор написан верно,а я там выделяю память для матрицы, то какие объекты мне нужно приравнять?)

Конструктор копирования срабатывает при инициализации объекта другим обектом того же типа. Примеры:

C++

class Foo {
public:
  Foo(const Foo& other) {
    // ...
  }
};
 
// ...
Foo f1;
Foo f2(f1);  // Здесь буде вызван конструктор копирования
Foo f3 = f1; // И здесь
// ...

Советую ознакомится с FAQ Конструкторы и деструкторы.

@Chevit · 15.09.2010, 12:28

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

добавьте в конструктор копирования проверку, если есть матрица matrica то её нужно удалить и создать новую, вообще при копировании объектов копируются все поля класса.

А зачем? Если используется конструктор копирования, то при это просто создается новый объект со значениями того объекта, которому приравнивается, и matrica по определению не существует...
Проверка нужна при перегрузке operator=...

@Crudelis · 15.09.2010, 12:53

Если вы используете в классе динамические массивы или работаете с динамической памятью и перегружаете оператор = то обязательно нужен конструктор копирование в котором реализована проверка есть ли такой динамический массив или нет. Представьте что вы приравняли 2 объекта друг другу и у обоих объектов динамический массив с одним и тем же адресом, что будет когда начнет работать деструктор как вы думаете?

@Chevit · 15.09.2010, 13:41

В случае с оператором = безусловно надо делать проверку, но вот в случае с конструктором копирования я не пойму зачем...
Ввиду того, что это конструктор, ранее объект, который инициализируется другим объектом, не существовал, следовательно члены класса не имели никаких значений и проверка всегда будет отрицательной.

@Andrew_Lvov · 15.09.2010, 14:08

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

В случае с оператором = безусловно надо делать проверку, но вот в случае с конструктором копирования я не пойму зачем...
Ввиду того, что это конструктор, ранее объект, который инициализируется другим объектом, не существовал, следовательно члены класса не имели никаких значений и проверка всегда будет отрицательной.

Всё именно так.

@zhenya.ya · 15.09.2010, 20:49 **[ТС]**

Мне ничего не нужно перегружать.
Почему ругается на строку cout<<matrica[i][j]<<' '; в методе-выводе(print()

? И как заставить мой конструктор копирования работать?

@Andrew_Lvov · 15.09.2010, 20:55

Сообщение от zhenya.ya

Мне ничего не нужно перегружать.
Почему ругается на строку cout<<matrica[i][j]<<' '; в методе-выводе(print() ? И как заставить мой конструктор копирования работать?

Потому, что член класса int** matrica не инициализирован.

@zhenya.ya · 15.09.2010, 21:03 **[ТС]**

Все, исправил инициализацию. Но теперь ,по завершению работы ,кидает ошибку. В чем проблема?

C++

class matrix
{ 
private:
        int m,n;
        int **matrica;
public: 
         matrix();
         matrix(int);
        ~matrix(); 
        matrix(const matrix &A1);
        int get(int ,int);
        void print();
        void setSize();
};
matrix::matrix()
{
    int a;
    n=m=a=0;
}
void matrix::setSize()
{
    int first;
    int second;
    cout<<"Enter first size: ";
    cin>>first;
    cout<<"Enter second size: ";
    cin>>second;
    n=first;
    m=second;
    system("cls");
 }
 
matrix::matrix(int b)
{  
    setSize();
 
int **matrica2=(int**)malloc(n*sizeof(int*));
for(int i=0;i<n;i++)
    matrica2[i]=(int*)malloc(m*sizeof(int));
 
        for (int i = 0; i < n; i++)
               for (int j = 0; j < m; j++)
                        matrica2[i][j] = rand() % 10;
matrica=matrica2;
 }
void matrix::print()
{
     for(int i = 0; i < n; i++)
     {
         for(int j = 0; j < m; j++)
               cout<<matrica[i][j]<<' ';
         cout<<"\n";
     }
}
 matrix::~matrix()
{
     for (int i = 0; i < n; i++)
     free(matrica[i]);
free(matrica);
}
int matrix::get(int i,int j)
{
         if(i>=n||j>=m)
            return 0;
         cout<<matrica[i][j]<<'\n';
    
}
 
matrix::matrix(const matrix &A1)
{
    n=A1.n;
    int **matrica=(int**)malloc(n*sizeof(int*));
    for(int i=0;i<n;i++)
        matrica[i]=(int*)malloc(m*sizeof(int));
 
    for(int i=0;i<n;i++)
        for(int j=0;j<n;j++)
            matrica[i][j]=A1.matrica[i][j];
}
int main ()
{  
        system("cls");
        matrix object;
        matrix ob(5);
        object=ob;
        object.print();
        int i, j;
        cout<<"\n get (i.j):"<<endl;
        cout<<"Enter i: ";
        cin>>i;
        cout<<"Enter j: ";
        cin>>j;
        if(object.get(i,j)==0)
           cout<<"i or j more or equal than n or m!\n";
        system("PAUSE");
        return 0;
}

@Andrew_Lvov · 15.09.2010, 21:20

Сообщение от zhenya.ya

Все, исправил инициализацию

В одном из случаев matrica не инициализируется.
Кроме того, при копировании обектов вызывается matrix& operator =, а не конструктор копирования.

@zhenya.ya · 15.09.2010, 21:57 **[ТС]**

Как-то я не могу найти этот случай.

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

Кроме того, при копировании обектов вызывается matrix& operator =, а не конструктор копирования.

Мне пока еще нельзя вызывать эту штуку. А разьве нельзя применить тут констр. копирования? В задании было сказано сделать конструктор копирования, т.к без него не может работать код.

@Andrew_Lvov · 15.09.2010, 22:03

Сообщение от zhenya.ya

Мне пока еще нельзя вызывать эту штуку. А разьве нельзя применить тут констр. копирования? В задании было сказано сделать конструктор копирования, т.к без него не может работать код.

Конструктор копирования будет вызван только в случае инициализации объектом такого же типа.
То есть,

C++

1 2	matrix ob(5); matrix object(ob); // matrix( const matrix&);

В то же время

C++

1
2
3

matrix ob(5);
matrix object;
object = ob;  // matrix& operator[]( const matrix& );

Сообщение от zhenya.ya

Как-то я не могу найти этот случай.

В конструкторе по-умолчанию переменная matrica не инициализируется.
Во время удаления этого объекта вызывается free для указателя, который указывает неизвестно куда.

@Andrew_Lvov 261 / 191 / 10 Регистрация: 19.08.2010 Сообщений: 760 Записей в блоге: 1
	15.09.2010, 21:20	18
	Сообщение от zhenya.ya Все, исправил инициализацию В одном из случаев matrica не инициализируется. Кроме того, при копировании обектов вызывается matrix& operator =, а не конструктор копирования. 0

Jupiter Каратель 6609 / 4028 / 401 Регистрация: 26.03.2010 Сообщений: 9,273 Записей в блоге: 1
	14.09.2010, 19:56	2
	Сказали сделать 2 конструктора: один по умолчанию, а другой-с параметрами. Какие должны быть параметры в моем конструкторе?(я так понимаю нужно как-то избавиться от private:i,j,n,m?) к примеру, размер создаваемой матрицы может быть параметром, а i,j вроде как в классе и не нужны Добавлено через 3 минуты вот пример 0

@zhenya.ya 2 / 2 / 2 Регистрация: 29.11.2009 Сообщений: 320
	15.09.2010, 00:16 [ТС]	8
	спс за примеры, но теперь ругается на метод print(), а точнее на строку: cout<<matrica[i][j]<<' '; Что не так? И кстати, я не понял как сработает конструктор копирования ( непонятно как можно прировнять оъекты, если мой копирующий конструктор написан верно,а я там выделяю память для матрицы, то какие объекты мне нужно приравнять?) 0

@Crudelis Шаровик затейник 696 / 445 / 78 Регистрация: 06.05.2010 Сообщений: 1,109
	15.09.2010, 01:41	9
	добавьте в конструктор копирования проверку, если есть матрица matrica то её нужно удалить и создать новую, вообще при копировании объектов копируются все поля класса. 0

@Chevit 3 / 3 / 0 Регистрация: 06.07.2010 Сообщений: 18
	15.09.2010, 12:28	11
	Сообщение от Crudelis добавьте в конструктор копирования проверку, если есть матрица matrica то её нужно удалить и создать новую, вообще при копировании объектов копируются все поля класса. А зачем? Если используется конструктор копирования, то при это просто создается новый объект со значениями того объекта, которому приравнивается, и matrica по определению не существует... Проверка нужна при перегрузке operator=... 0

@Crudelis Шаровик затейник 696 / 445 / 78 Регистрация: 06.05.2010 Сообщений: 1,109
	15.09.2010, 12:53	12
	Если вы используете в классе динамические массивы или работаете с динамической памятью и перегружаете оператор = то обязательно нужен конструктор копирование в котором реализована проверка есть ли такой динамический массив или нет. Представьте что вы приравняли 2 объекта друг другу и у обоих объектов динамический массив с одним и тем же адресом, что будет когда начнет работать деструктор как вы думаете? 0

@Chevit 3 / 3 / 0 Регистрация: 06.07.2010 Сообщений: 18
	15.09.2010, 13:41	13
	В случае с оператором = безусловно надо делать проверку, но вот в случае с конструктором копирования я не пойму зачем... Ввиду того, что это конструктор, ранее объект, который инициализируется другим объектом, не существовал, следовательно члены класса не имели никаких значений и проверка всегда будет отрицательной. 0

@Andrew_Lvov 261 / 191 / 10 Регистрация: 19.08.2010 Сообщений: 760 Записей в блоге: 1
	15.09.2010, 14:08	14
	Сообщение от Chevit В случае с оператором = безусловно надо делать проверку, но вот в случае с конструктором копирования я не пойму зачем... Ввиду того, что это конструктор, ранее объект, который инициализируется другим объектом, не существовал, следовательно члены класса не имели никаких значений и проверка всегда будет отрицательной. Всё именно так. 0

@zhenya.ya 2 / 2 / 2 Регистрация: 29.11.2009 Сообщений: 320
	15.09.2010, 20:49 [ТС]	15
	Мне ничего не нужно перегружать. Почему ругается на строку cout<<matrica[i][j]<<' '; в методе-выводе(print() ? И как заставить мой конструктор копирования работать? 0

@Andrew_Lvov 261 / 191 / 10 Регистрация: 19.08.2010 Сообщений: 760 Записей в блоге: 1
	15.09.2010, 20:55	16
	Сообщение от zhenya.ya Мне ничего не нужно перегружать. Почему ругается на строку cout<<matrica[i][j]<<' '; в методе-выводе(print() ? И как заставить мой конструктор копирования работать? Потому, что член класса int** matrica не инициализирован. 0

@zhenya.ya 2 / 2 / 2 Регистрация: 29.11.2009 Сообщений: 320
	15.09.2010, 21:57 [ТС]	19
	Как-то я не могу найти этот случай. Сообщение от Andrew_Lvov Кроме того, при копировании обектов вызывается matrix& operator =, а не конструктор копирования. Мне пока еще нельзя вызывать эту штуку. А разьве нельзя применить тут констр. копирования? В задании было сказано сделать конструктор копирования, т.к без него не может работать код. 0