Разработать класс матрица, конструктор по умолчанию, вывод на экран

@Pari · Регистрация: 24.11.2013

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

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

C++

 #include "stdafx.h"
#include <iostream>
#include <ctime>
using namespace std;
 
class Matrix
{
private:
    int rows;
    int column;
    int** matrix;
    
public:
    Matrix():rows(0), column(0)
    {}
    void get_data()
    {
        cout << "Введите количество строк а затем столбцов" << endl;
        cin >> rows;
        cin >> column;
    }
    void random ()
    {
    int** matrix = new int* [rows]; //создаем двумерный динамический массив
     for (int i = 0; i<rows; i++)
         matrix[i] = new int [column];
 
        for (int i = 0; i<rows; i++) //заполняем массив случ. знач
            for (int i = 0; i<column; i++)
                matrix[rows][column] = 1+rand()%9;
    }
    void display()
    {
        for (int i = 0; i<rows; i++) //заполняем массив случ. знач
        {
            for (int i = 0; i<column; i++)
            {   cout << matrix[rows][column] << " "; }
            cout << endl;
        }
    }
 
 
    
};
 
 
void main()
{
    setlocale (0, "");
    Matrix matrix;
    matrix.get_data();
    matrix.random();
    matrix.display();
}

@jurok_85 · 10.09.2014, 19:06

вот тут сразу видно непорядок

C++

1
2
3

for (int i = 0; i<rows; i++) //заполняем массив случ. знач
            for (int i = 0; i<column; i++)
                matrix[rows][column] = 1+rand()%9;

скорее так:

C++

1
2
3

 for (int i = 0; i<rows; i++) //заполняем массив случ. знач
            for (int j = 0; j<column; j++)
                matrix[i][j] = 1+rand()%9;

@Firework · 10.09.2014, 19:18

C++

#include <iostream>
#include <ctime>
using namespace std;
 
class Matrix
{
private:
    int rows;
    int column;
    int** matrix;
    
public:
    Matrix():rows(0), column(0)
    {}
    void get_data()
    {
        cout << "Введите количество строк а затем столбцов" << endl;
        cin >> rows;
        cin >> column;
    }
    void random ()
    {
        matrix = new int* [rows]; //ИСПРАВИЛ!!!!!!!!!!!
        for (int i = 0; i<rows; i++)
            matrix[i] = new int [column];
 
        for (int i = 0; i<rows; i++) //заполняем массив случ. знач
            for (int j = 0; j<column; j++)
                matrix[i][j] = 1+rand()%9;
    }
    void display()
    {
        for (int i = 0; i<rows; i++) //заполняем массив случ. знач
        {
            for (int j = 0; j<column; j++)
            { 
                cout << matrix[i][j] << " ";
            }
            cout << endl;
        }
    }
 
 
    
};
 
 
void main()
{
    setlocale (0, "");
    Matrix matrix;
    matrix.get_data();
    matrix.random();
    matrix.display();
    system("pause");
}

Не нужно в методе random() ещё объявлять указатель **matrix с именем члена класса.

@Pari · 10.09.2014, 20:03 **[ТС]**

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

Не нужно в методе random() ещё объявлять указатель **matrix с именем члена класса.

спасибо, не до конца с указателями разобралась

Добавлено через 2 минуты
и кстати, не работает по прежнему, пишет что есть какие-то неразрешенные внешние элементы, 1 штука

@Firework · 10.09.2014, 20:05

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

спасибо, не до конца с указателями разобралась

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

Добавлено через 58 секунд
Попробуйте это убрать #include "stdafx.h"

@Pari · 10.09.2014, 20:05 **[ТС]**

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

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

окей, осмыслила

но не работает по прежнему

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

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

Попробуйте это убрать #include "stdafx.h"

без него не компилируется

@Firework · 10.09.2014, 20:06

А простой "хеллоу ворд" компилируется?

@Pari · 10.09.2014, 20:08 **[ТС]**

о, все, заработало)

Добавлено через 49 секунд
кстати без #include "stdafx.h" у меня вообще ничего не компилируется. это нормально? потому что я не вижу в примерах кода этого заголовка никогда

@Dani · 10.09.2014, 20:09

Нормально.
Просто когда в создавался проект, в его настройках было указано "Use precompiled headers"

@Firework · 10.09.2014, 20:10

Вы когда создаёте проект, убирайте галочку с "Предварительно скомпилированный заголовок".

@Pari · 10.09.2014, 20:14 **[ТС]**

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

Вы когда создаёте проект, убирайте галочку с "Предварительно скомпилированный заголовок".

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

C++

void transposition()
    {
        int temp;
        temp = rows;
        rows = column;
        column = temp;
    }

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

@Firework · 10.09.2014, 20:20

C++

if (rows != column) {
    throw Matrix::Exсeption("matrix is not square");
}
for(int i = 0; i < rows; ++i)
    {
        for(int j = i; j < column; ++j)
        {
            t = matrix[i][j];
            matrix[i][j] = matrix[j][i];
            matrix[j][i] = t;
        }
    }

@Pari · 10.09.2014, 20:23 **[ТС]**

Firework, а если она таки не квадратная будет?

@Firework · 10.09.2014, 20:25

C++

1
2
3

if (rows != column) {
    throw Matrix::Exсeption("matrix is not square");
}

Читайте внимательнее код

@Pari · 10.09.2014, 20:27 **[ТС]**

и ваш метод не срабатывает, я видела здесь такой погуглив по форуму, уже пробовала его, программа вылетает

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

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

Читайте внимательнее код

я не об этом, я понимаю что можно задать ограничения, но если так и надо, чтобы не квадратная матрица проэекспонировалась?

@Firework · 10.09.2014, 20:31

Всё работает

C++

#include <iostream>
#include <ctime>
using namespace std;
 
class Matrix
{
private:
    int rows;
    int column;
    int** matrix;
    
public:
    Matrix():rows(0), column(0)
    {}
    void get_data()
    {
        cout << "Введите количество строк а затем столбцов" << endl;
        cin >> rows;
        cin >> column;
    }
    void random ()
    {
        matrix = new int* [rows]; //ИСПРАВИЛ!!!!!!!!!!!
        for (int i = 0; i<rows; i++)
            matrix[i] = new int [column];
 
        for (int i = 0; i<rows; i++) //заполняем массив случ. знач
            for (int j = 0; j<column; j++)
                matrix[i][j] = 1+rand()%9;
    }
    void display()
    {
        for (int i = 0; i<rows; i++) //заполняем массив случ. знач
        {
            for (int j = 0; j<column; j++)
            { 
                cout << matrix[i][j] << " ";
            }
            cout << endl;
        }
    }
 
    void transposition()
    {
        if (rows != column) {
            throw ("matrix is not square");
        }
        for(int i = 0; i < rows; ++i)
        {
            for(int j = i; j < column; ++j)
            {
                int t = matrix[i][j];
                matrix[i][j] = matrix[j][i];
                matrix[j][i] = t;
            }
        }
    }
 
 
    
};
 
 
void main()
{
    setlocale (0, "");
    Matrix matrix;
    matrix.get_data();
    matrix.random();
    matrix.display();
    matrix.transposition();
    cout << endl;
    matrix.display();
    system("pause");
}

@Pari 41 / 4 / 0 Регистрация: 24.11.2013 Сообщений: 139
	10.09.2014, 20:27 [ТС]	15
	и ваш метод не срабатывает, я видела здесь такой погуглив по форуму, уже пробовала его, программа вылетает Добавлено через 52 секунды Сообщение от Firework Читайте внимательнее код я не об этом, я понимаю что можно задать ограничения, но если так и надо, чтобы не квадратная матрица проэекспонировалась? 0

@Pari 41 / 4 / 0 Регистрация: 24.11.2013 Сообщений: 139
	10.09.2014, 20:03 [ТС]	4
	Сообщение от Firework Не нужно в методе random() ещё объявлять указатель *matrix с именем члена класса. спасибо, не до конца с указателями разобралась Добавлено через 2 минуты* и кстати, не работает по прежнему, пишет что есть какие-то неразрешенные внешние элементы, 1 штука 0

@Firework 102 / 93 / 36 Регистрация: 01.04.2013 Сообщений: 654
	10.09.2014, 20:05	5
	Сообщение от Pari спасибо, не до конца с указателями разобралась Да тут даже не в указателях дело. Просто вы выделили память для локального указателя объявленного в методе random(). А не для указателя, являющегося членом вашего класса. Поэтому, когда вы пытались вывести матрицу на экран, то вы обращались к невыделенной области памяти. Поэтому возникала ошибка. Добавлено через 58 секунд Попробуйте это убрать #include "stdafx.h" 0

@Pari 41 / 4 / 0 Регистрация: 24.11.2013 Сообщений: 139
	10.09.2014, 20:05 [ТС]	6
	Сообщение от Firework Да тут даже не в указателях дело. Просто вы выделили память для локального указателя объявленного в методе random(). А не для указателя, являющегося членом вашего класса. Поэтому, когда вы пытались вывести матрицу на экран, то вы обращались к невыделенной области памяти. Поэтому возникала ошибка. окей, осмыслила но не работает по прежнему Добавлено через 14 секунд Сообщение от Firework Попробуйте это убрать #include "stdafx.h" без него не компилируется 0

@Firework 102 / 93 / 36 Регистрация: 01.04.2013 Сообщений: 654
	10.09.2014, 20:06	7
	А простой "хеллоу ворд" компилируется? 0

@Pari 41 / 4 / 0 Регистрация: 24.11.2013 Сообщений: 139
	10.09.2014, 20:08 [ТС]	8
	о, все, заработало) Добавлено через 49 секунд кстати без #include "stdafx.h" у меня вообще ничего не компилируется. это нормально? потому что я не вижу в примерах кода этого заголовка никогда 0

@Dani 1405 / 647 / 135 Регистрация: 11.08.2011 Сообщений: 2,299 Записей в блоге: 2
	10.09.2014, 20:09	9
	Нормально. Просто когда в создавался проект, в его настройках было указано "Use precompiled headers" 0

@Firework 102 / 93 / 36 Регистрация: 01.04.2013 Сообщений: 654
	10.09.2014, 20:10	10
	Вы когда создаёте проект, убирайте галочку с "Предварительно скомпилированный заголовок". 0

@Pari 41 / 4 / 0 Регистрация: 24.11.2013 Сообщений: 139
	10.09.2014, 20:23 [ТС]	13
	Firework, а если она таки не квадратная будет? 0

Разработать класс матрица, конструктор по умолчанию, вывод на экран

Решение