Конструктор

@DrSMERTb · Регистрация: 12.11.2010

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

Всем доброго времени суток. Пишу программу, которая вычисляет ранг матрицы и выводит транспонированную. Вот код:
matr.h:

C++

class matr
{
public: 
matr(int, int);
//matr();
~matr();
void out() const;
void trans();
float det();
private:
int n;
int m;
float x[50][50];
};

matr.cpp:

C++

#include <iostream.h>
#include "matr.h"
 
//void matr::matr()
//{
//n=2;
//m=2;
//int i,j;
//for(i=0;i<n;i++)
//for(j=0;j<m;j++)
//{
//x[i][j]=0;
//}
//}
 
//----------------------------
void matr::matr(int k, int l)
{
n=k; m=l;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=rand()%10;
}
}
//--------------------
void matr::out()const
{
int i,j; 
for(i=0;i<n;i++)
cout<<endl;
for(j=0;j<m;j++)
{
cout<<x[i][j]<<" ";
}
}
//------------------------------
void matr::trans()
{
if(m==n)
{ 
int i,j,a[50][50];
for(i=0;i<n;i++)
for(j=0;j<m;j++)
a[j][i]=x[i][j];
for(i=0;i<n;i++)
cout<<endl;
for(j=0;j<m;j++)
{
cout<<a[i][j]<<" ";
}}
else cout<<"No trans!!!";
}
 
//--------------------------------
float matr::det()
{
float det=0;
if((m==3)&&(n==3))
{
det=x[1][1]*x[2][2]*x[3][3]+x[1][2]*x[2][3]*x[3][1]+x[1][3]*x[2][1]*x[3][2]-x[1][3]*x[2][2]*x[3][1]-x[1][2]*x[2][1]*x[3][3]-x[1][1]*x[2][3]*x[3][2];
return det;
}
}

ispoln.cpp:

C++

#include <iostream.h>
#include <string.h>
#include <conio.h>
#include <stdio.h>
#include <math.h>
#include "matr.cpp"
int main()
{
matr m1,m2;
int i,j,b,c;
float det;
//m1.matr();
m1.out();
cin>>b>>c;
//m2.matr(b,c);
cout<<"Matrix"<<endl<<endl;
m2.out();
cout<<endl<<endl<<"Trans Matrix"<<endl<<endl;
m2.trans();
m2.det();
if (det==0)
cout<<"No Det!!! Your Matrix - invalid";
else
cout<<endl<<endl<<"Det="<<det;
}

Ругается на ошибки в файле "matr.cpp" посмотрите код, подскажите, как его довести до работоспособного состояния.

@~~Bers~~ · 07.12.2011, 23:07

1. Работает, но делает не то что надо. (описание неисправности)

2. Крошится в рантайме, под ругательства системы. (скриншот, или примерное описание текста сообщение системы)

3. Не компилируется. (Точный текст ошибки компилятора. Желательно - с указанием точных строчек кода, на которые ругнулся компилятор)

4. Не компонуется. (Точный текст ошибки компоновщика.)

Телепаты в отпуске.

@DrSMERTb · 07.12.2011, 23:26 **[ТС]**

18 строка : C:...Matr\matr.cpp return type specification for constructor invalid
9 строка : 9 C:\...\Matr\isz.cpp no matching function for call to `matr::matr()'
и вот ещё ошибки:
note C:\....Matr\matr.h:2 candidates are: matr::matr(const matr&)
note C:\...\Matr\matr.h:2 matr::matr(int, int)
note C:\...\matr.h:2 matr::matr(int, int)
note C:\...\Matr\matr.h:2 candidates are: matr::matr(const matr&)

@~~Bers~~ · 07.12.2011, 23:31

Обратите внимание:

в файле ispoln.cpp:

C++

int main()
{
matr m1,m2; //здесь вы создаете 
                //два объекта класса матрицы
...

Так как для объектов не указаны явные конструкторы, с помощью которых объекты будут построены, то они будут конструироваться конструктором по умолчанию.

Покажите мне в коде матрицы, где находится конструктор по умолчанию?

@DrSMERTb · 07.12.2011, 23:42 **[ТС]**

Как я понимаю matr();-это конструктор по умолчанию, а

C++

void matr::matr()
{
n=2;
m=2;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=0;
}
}

это его заполнение, там закаменчено просто было.

@~~Bers~~ · 07.12.2011, 23:44

DrSMERTb, То есть, ты за комментировал код, а потом удивляешься, что компилятор его не нашёл?

@DrSMERTb · 07.12.2011, 23:51 **[ТС]**

Нет) Просто с этим кодом он ругался ещё больше...

Добавлено через 36 секунд
Да, а что по остальным ошибкам?

@~~Bers~~ · 07.12.2011, 23:53

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

Нет) Просто с этим кодом он ругался ещё больше...

То есть, вы преднамеренно создаете ошибку, что бы до других ошибок компиляция даже не дошла? Что бы процесс компиляции программы загнулся ещё раньше?

Это у вас такой способ борьбы с ошибками: не исправлять их, а создавать ещё более фатальные?

Добавлено через 43 секунды

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

Да, а что по остальным ошибкам?

Вы исправили первую ошибку? Вы изменили код?
Теперь компилируйте заново. И если есть ещё какие то ошибки, то:

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

1. Работает, но делает не то что надо. (описание неисправности)
2. Крошится в рантайме, под ругательства системы. (скриншот, или примерное описание текста сообщение системы)
3. Не компилируется. (Точный текст ошибки компилятора. Желательно - с указанием точных строчек кода, на которые ругнулся компилятор)
4. Не компонуется. (Точный текст ошибки компоновщика.)
Телепаты в отпуске.

@DrSMERTb · 08.12.2011, 00:23 **[ТС]**

4 C:\...\Matr\matr.h matr::matr(int, int)
18 C:\....\Matr\matr.cpp return type specification for constructor invalid
error C:\...\Matr\matr.h:2 candidates are: matr::matr(const matr&)
5 C:\...\Matr\matr.cpp prototype for `matr::matr()' does not match any in class `matr'

@~~Bers~~ · 08.12.2011, 00:29

DrSMERTb, а текст исправленный где?
Ты думаешь, я вижу, чего ты там наисправлял?

@DrSMERTb · 08.12.2011, 00:36 **[ТС]**

раскоментил

C++

//void matr::matr()
{
n=2;
m=2;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=0;
}
}

и в matr.h:
matr();

@~~Bers~~ · 08.12.2011, 00:45

Обрати внимание:

в файле matr.h:

C++

class matr
{
public: 
matr(int, int);//здесь ты объявляешь конструктор 
                   //с параметрами
....

А вот здесь:

в файле matr.cpp:

C++

void matr::matr(int k, int l)
{
n=k; m=l;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=rand()%10;
}
}

У тебя должна быть реализация этого конструктора.
А теперь объясни мне, почему твой конструктор возвращает тип void?

@DrSMERTb · 08.12.2011, 00:50 **[ТС]**

Получается его через int прописать?

@~~Bers~~ · 08.12.2011, 00:53

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

Получается его через int прописать?

Открывай учебник. Раздел "классы". Читай, как грамотно оформляются конструкторы класса.

@DrSMERTb · 08.12.2011, 01:01 **[ТС]**

Ха вот переклинило, точно тут же просто пишется matr::matr(int k, int l)...

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

C++

 matr::matr()
{
n=2;
m=2;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=0;
}
}
 
//----------------------------
matr::matr(int k, int l)
{
n=k; m=l;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=rand()%10;
}
}

остались ощибки
4 C:\...\Matr\matr.h matr::matr(int, int)
5 C:\...Matr\matr.cpp prototype for `matr::matr()' does not match any in class `matr'

@~~Bers~~ · 08.12.2011, 01:02

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

остались ощибки
4 C:\...\Matr\matr.h matr::matr(int, int)
5 C:\...Matr\matr.cpp prototype for `matr::matr()' does not match any in class `matr'

Показывай исправленный код

@DrSMERTb · 08.12.2011, 01:04 **[ТС]**

matr.h

C++

class matr
{
public: 
matr(int, int);
matr();
~matr();
void out() const;
void trans();
float det();
private:
int n;
int m;
float x[50][50];
};

matr.cpp

C++

#include <iostream.h>
#include "matr.h"
 
 matr::matr()
{
n=2;
m=2;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=0;
}
}
 
//----------------------------
matr::matr(int k, int l)
{
n=k; m=l;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=rand()%10;
}
}
//--------------------
void matr::out()const
{
int i,j; 
for(i=0;i<n;i++)
cout<<endl;
for(j=0;j<m;j++)
{
cout<<x[i][j]<<" ";
}
}
//------------------------------
void matr::trans()
{
if(m==n)
{ 
int i,j,a[50][50];
for(i=0;i<n;i++)
for(j=0;j<m;j++)
a[j][i]=x[i][j];
for(i=0;i<n;i++)
cout<<endl;
for(j=0;j<m;j++)
{
cout<<a[i][j]<<" ";
}}
else cout<<"No trans!!!";
}
 
//--------------------------------
float matr::det()
{
float det=0;
if((m==3)&&(n==3))
{
det=x[1][1]*x[2][2]*x[3][3]+x[1][2]*x[2][3]*x[3][1]+x[1][3]*x[2][1]*x[3][2]-x[1][3]*x[2][2]*x[3][1]-x[1][2]*x[2][1]*x[3][3]-x[1][1]*x[2][3]*x[3][2];
return det;
}
}

ispoln.cpp

C++

#include <iostream.h>
#include <string.h>
#include <conio.h>
#include <stdio.h>
#include <math.h>
#include "matr.cpp"
int main()
{
matr m1,m2;
int i,j,b,c;
float det;
//m1.matr();
m1.out();
cin>>b>>c;
//m2.matr(b,c);
cout<<"Matrix"<<endl<<endl;
m2.out();
cout<<endl<<endl<<"Trans Matrix"<<endl<<endl;
m2.trans();
m2.det();
if (det==0)
cout<<"No Det!!! Your Matrix - invalid";
else
cout<<endl<<endl<<"Det="<<det;
}

@~~Bers~~ · 08.12.2011, 01:12

DrSMERTb, обрати внимание:

C++

class matr
{
public: 
matr(int, int);
matr();
~matr();  //ты знаешь что это такое?
....

А теперь покажи в коде, где находится реализация диструктора.

@DrSMERTb · 08.12.2011, 01:21 **[ТС]**

Он пока не вызывается, но я как то думал что не обязательно обрабатывать все заявленные функции..
Переделал вот что получил
matr.cpp

C++

#include <iostream.h>
#include "matr.h"
 
 matr::matr()
{
n=2;
m=2;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=0;
}
}
 
//----------------------------
matr::matr(int k, int l)
{
n=k; m=l;
int i,j;
for(i=0;i<n;i++)
for(j=0;j<m;j++)
{
x[i][j]=rand()%10;
}
}
//--------------------
 //деструктор
  matr::~matr() 
  {
  cout<<"DESTRUCTOR"<<endl;
  } 
//--------------------
void matr::out()const
{
int i,j; 
for(i=0;i<n;i++)
cout<<endl;
for(j=0;j<m;j++)
{
cout<<x[i][j]<<" ";
}
}
//------------------------------
void matr::trans()
{
if(m==n)
{ 
int i,j,a[50][50];
for(i=0;i<n;i++)
for(j=0;j<m;j++)
a[j][i]=x[i][j];
for(i=0;i<n;i++)
cout<<endl;
for(j=0;j<m;j++)
{
cout<<a[i][j]<<" ";
}}
else cout<<"No trans!!!";
}
 
//--------------------------------
float matr::det()
{
float det=0;
if((m==3)&&(n==3))
{
det=x[1][1]*x[2][2]*x[3][3]+x[1][2]*x[2][3]*x[3][1]+x[1][3]*x[2][1]*x[3][2]-x[1][3]*x[2][2]*x[3][1]-x[1][2]*x[2][1]*x[3][3]-x[1][1]*x[2][3]*x[3][2];
return det;
}
}

ispoln.cpp

C++

#include <iostream.h>
#include <string.h>
#include <conio.h>
#include <stdio.h>
#include <math.h>
#include "matr.cpp"
int main()
{
matr m1,m2;
int i,j,b,c;
float det;
//m1.matr();
m1.out();
cin>>b>>c;
//m2.matr(b,c);
cout<<"Matrix"<<endl<<endl;
m2.out();
cout<<endl<<endl<<"Trans Matrix"<<endl<<endl;
m2.trans();
m2.det();
if (det==0)
cout<<"No Det!!! Your Matrix - invalid";
else
cout<<endl<<endl<<"Det="<<det;
matr();//вызов дестуктора
//в предыдущей проге пытался вызвать типа такого ~matr() выдавал ошибки без тильды заработал
}

Добавлено через 1 минуту
Ошибки не исчезли и не изменились.

@~~Bers~~ · 08.12.2011, 01:24

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

matr();//вызов дестуктора

закомментируй

@DrSMERTb 64 / 40 / 10 Регистрация: 12.11.2010 Сообщений: 841
	08.12.2011, 00:50 [ТС]	13
	Получается его через int прописать? 0

@~~Bers~~ Заблокирован
	08.12.2011, 00:53	14
	Сообщение от DrSMERTb Получается его через int прописать? Открывай учебник. Раздел "классы". Читай, как грамотно оформляются конструкторы класса. 0

@~~Bers~~ Заблокирован
	08.12.2011, 01:02	16
	Сообщение от DrSMERTb остались ощибки 4 C:\...\Matr\matr.h matr::matr(int, int) 5 C:\...Matr\matr.cpp prototype for `matr::matr()' does not match any in class `matr' Показывай исправленный код 0

@~~Bers~~ Заблокирован
	07.12.2011, 23:07	2
	1. Работает, но делает не то что надо. (описание неисправности) 2. Крошится в рантайме, под ругательства системы. (скриншот, или примерное описание текста сообщение системы) 3. Не компилируется. (Точный текст ошибки компилятора. Желательно - с указанием точных строчек кода, на которые ругнулся компилятор) 4. Не компонуется. (Точный текст ошибки компоновщика.) Телепаты в отпуске. 1

@DrSMERTb 64 / 40 / 10 Регистрация: 12.11.2010 Сообщений: 841
	07.12.2011, 23:26 [ТС]	3
	18 строка : C:...Matr\matr.cpp return type specification for constructor invalid 9 строка : 9 C:\...\Matr\isz.cpp no matching function for call to `matr::matr()' и вот ещё ошибки: note C:\....Matr\matr.h:2 candidates are: matr::matr(const matr&) note C:\...\Matr\matr.h:2 matr::matr(int, int) note C:\...\matr.h:2 matr::matr(int, int) note C:\...\Matr\matr.h:2 candidates are: matr::matr(const matr&) 0

@~~Bers~~ Заблокирован
	07.12.2011, 23:44	6
	DrSMERTb, То есть, ты за комментировал код, а потом удивляешься, что компилятор его не нашёл? 0

@DrSMERTb 64 / 40 / 10 Регистрация: 12.11.2010 Сообщений: 841
	07.12.2011, 23:51 [ТС]	7
	Нет) Просто с этим кодом он ругался ещё больше... Добавлено через 36 секунд Да, а что по остальным ошибкам? 0

@~~Bers~~ Заблокирован
	07.12.2011, 23:53	8
	Сообщение от DrSMERTb Нет) Просто с этим кодом он ругался ещё больше... То есть, вы преднамеренно создаете ошибку, что бы до других ошибок компиляция даже не дошла? Что бы процесс компиляции программы загнулся ещё раньше? Это у вас такой способ борьбы с ошибками: не исправлять их, а создавать ещё более фатальные? Добавлено через 43 секунды Сообщение от DrSMERTb Да, а что по остальным ошибкам? Вы исправили первую ошибку? Вы изменили код? Теперь компилируйте заново. И если есть ещё какие то ошибки, то: Сообщение от Bers 1. Работает, но делает не то что надо. (описание неисправности) 2. Крошится в рантайме, под ругательства системы. (скриншот, или примерное описание текста сообщение системы) 3. Не компилируется. (Точный текст ошибки компилятора. Желательно - с указанием точных строчек кода, на которые ругнулся компилятор) 4. Не компонуется. (Точный текст ошибки компоновщика.) Телепаты в отпуске. 0

@DrSMERTb 64 / 40 / 10 Регистрация: 12.11.2010 Сообщений: 841
	08.12.2011, 00:23 [ТС]	9
	4 C:\...\Matr\matr.h matr::matr(int, int) 18 C:\....\Matr\matr.cpp return type specification for constructor invalid error C:\...\Matr\matr.h:2 candidates are: matr::matr(const matr&) 5 C:\...\Matr\matr.cpp prototype for `matr::matr()' does not match any in class `matr' 0

@~~Bers~~ Заблокирован
	08.12.2011, 00:29	10
	DrSMERTb, а текст исправленный где? Ты думаешь, я вижу, чего ты там наисправлял? 0

@~~Bers~~ Заблокирован
	08.12.2011, 01:24	20
	Сообщение от DrSMERTb matr();//вызов дестуктора закомментируй 0