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

Question

[marlaaa]

Как написать лучше и так чтоб работало правильно?

#include <iostream>
#include <fstream>
#include <vector>
#include <ctime> 
using namespace std;

void CreatMatr(int n, int m, int l) {
    ofstream file("matriza.bin", ios::app);
    cout << "Ввести строки первой матрицы";
    cin >> n;
    cout << "Ввести столбцы для первой матрицы и строки для второй";
    cin >> m;
    int** M;
    M = new int* [n];
    for (int i = 0; i < n; i++)
        M[i] = new int[m];

    for (int i = 0; i < n; i++)
        for (int j = 0; j < m; j++)
            M[i][j] = rand() % 13, 5;


    cout << "Ввести столбцы второй матрицы";
    cin >> l;
    int** M2;
    M2 = new int* [m];
    for (int i = 0; i < m; i++)
        M2[i] = new int[l];

    for (int i = 0; i < m; i++)
        for (int j = 0; j < l; j++)
            M2[i][j] = rand() % 13, 5;



    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++) {
            file << M[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < l; j++) {
            file << M2[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    file.close();
}

vector<vector<int>> ReadMatrix(ifstream& is, int n, int m) {
    vector<vector<int>> result(n, vector<int>(m));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < m; ++j) {
            is >> result[i][j];
        }
    }
    return result;
}

void WriteMatrix(ofstream& os, const vector<vector<int>>& m1) {
    os << m1.size() << " " << m1[0].size() << "\n";
    for (int i = 0; i < m1.size(); ++i) {
        for (int j = 0; j < m1[0].size(); ++j) {
            os << m1[i][j] << " ";
        }
        os << "\n";
    }
}

vector<vector<int>> MatrixProduct(const vector<vector<int>>& m1, const vector<vector<int>>& m2) {
    int n = m1.size(), m = m2.size(), l = m2[0].size();
    vector<vector<int>> result(n, vector<int>(l));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < l; ++j) {
            int sum = 0;
            for (int k = 0; k < m; ++k) {
                sum += m1[i][k] * m2[k][j];
            }
            result[i][j] = sum;
        }
    }
    return result;
}
int main() {
    setlocale(LC_ALL, "Rus");
    int m = 1, n = 1, l=1;
    int k = 3;
    int r = 0;
    while (r < k) {
        CreatMatr(n, m, l);
        r++;
    }

    ifstream file("matriza.bin", ios::in);
    ofstream out("result.bin", ios::out);
    if (!file.is_open() || !out.is_open()) {
        cout << "Файл не открыт\n";
        return 1;
    }

    for (int i = 0; i < k; i++) {
        file >> n >> m >> l;

        vector<vector<int>> m1 = ReadMatrix(file, n, m);
        vector<vector<int>> m2 = ReadMatrix(file, m, l);

        WriteMatrix(out, MatrixProduct(m1, m2));

    }

    out.close();
    file.close();
    return 0;
}

Answer 1

[res2001]

Хотелось бы больше конкретики, т.к. чтение у вас в двух функциях CreatMatr() и ReadMatrix().
В CreatMatr() вы не освобождаете динамически выделенную память - это утечка. Используйте вместо "сырого" массива вектор.
Так же не понятно, зачем вы вызываете CreatMatr() 3 раза?

Предположу, что проблема в том, что в CreatMatr() вы не возвращаете вызывающему коду считанные значения n, m, l, поэтому в main эти значения никак не изменяются и равны 1.

Comments

[marlaaa]

k-это количество произведений матриц которые я записываю в файл. Я решила сделать его 3. CreatrMatr - создаю файл и записываю в него матрицы. ReadMatr-читаю матрицу из файла в вектор.

#include <iostream>
#include <fstream>
#include <vector>
#include <ctime> 
using namespace std;

int CreatMatr(int n, int m, int l) {
    ofstream file("matriza.bin", ios::app);
    cout << "Ввести строки первой матрицы";
    cin >> n;
    cout << "Ввести столбцы для первой матрицы и строки для второй";
    cin >> m;
    vector <std::vector <int>> myVect(n, std::vector <int>(m));
    for (size_t i = 0; i < n; i++)
    {
        std::vector <int> tmp(m);
        for (size_t j = 0; j < m; j++)
            tmp[j] = rand() % 100;
        myVect.push_back(tmp);
    }
    cout << "Ввести столбцы второй матрицы";
    cin >> l;
    vector <std::vector <int>> myVect1(n, std::vector <int>(l));
    for (size_t i = 0; i < n; i++)
    {
        std::vector <int> tmp(l);
        for (size_t j = 0; j < l; j++)
            tmp[j] = rand() % 100;
        myVect1.push_back(tmp);
    }


    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++) {
            file << myVect[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < l; j++) {
            file << myVect1[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    file.close();

    return n, m, l;
}

vector<vector<int>> ReadMatrix(ifstream& is, int n, int m) {
    vector<vector<int>> result(n, vector<int>(m));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < m; ++j) {
            is >> result[i][j];
        }
    }
    return result;
}

void WriteMatrix(ofstream& os, const vector<vector<int>>& m1) {
    os << m1.size() << " " << m1[0].size() << "\n";
    for (int i = 0; i < m1.size(); ++i) {
        for (int j = 0; j < m1[0].size(); ++j) {
            os << m1[i][j] << " ";
        }
        os << "\n";
    }
}

vector<vector<int>> MatrixProduct(const vector<vector<int>>& m1, const vector<vector<int>>& m2) {
    int n = m1.size(), m = m2.size(), l = m2[0].size();
    vector<vector<int>> result(n, vector<int>(l));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < l; ++j) {
            int sum = 0;
            for (int k = 0; k < m; ++k) {
                sum += m1[i][k] * m2[k][j];
            }
            result[i][j] = sum;
        }
    }
    return result;
}
int main() {
    setlocale(LC_ALL, "Rus");
    int m = 1, n = 1, l=1;
    int k = 3;
    int r = 0;
    while (r < k) {
        CreatMatr(n, m, l);
        r++;
    }

    ifstream file("matriza.bin", ios::in);
    ofstream out("result.bin", ios::out);
    if (!file.is_open() || !out.is_open()) {
        cout << "Файл не открыт\n";
        return 1;
    }

    for (int i = 0; i < k; i++) {
        file >> n >> m >> l;

        vector<vector<int>> m1 = ReadMatrix(file, n, m);
        vector<vector<int>> m2 = ReadMatrix(file, m, l);

        WriteMatrix(out, MatrixProduct(m1, m2));

    }

    out.close();
    file.close();
    return 0;
}

Я сначала попробовала изменить создание файла и его заполнение под вектор, но оно не работало поэтому я просто поправила то что уже было:
убрала утечку памяти и в CreatMatr сделала return n,m,l

#include <iostream>
#include <fstream>
#include <vector>
#include <ctime> 
using namespace std;

int CreatMatr(int n, int m, int l) {
    ofstream file("matriza.bin", ios::app);
    cout << "Ввести строки первой матрицы";
    cin >> n;
    cout << "Ввести столбцы для первой матрицы и строки для второй";
    cin >> m;
    int** M;
    M = new int* [n];
    for (int i = 0; i < n; i++)
        M[i] = new int[m];

    for (int i = 0; i < n; i++)
        for (int j = 0; j < m; j++)
            M[i][j] = rand() % 10;


    cout << "Ввести столбцы второй матрицы";
    cin >> l;
    int** M2;
    M2 = new int* [m];
    for (int i = 0; i < m; i++)
        M2[i] = new int[l];

    for (int i = 0; i < m; i++)
        for (int j = 0; j < l; j++)
            M2[i][j] = rand() % 10;



    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++) {
            file << M[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < l; j++) {
            file << M2[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    file.close();
    delete[] M;
    delete[] M2;
    return n, m, l;
}

vector<vector<int>> ReadMatrix(ifstream& is, int n, int m) {
    vector<vector<int>> result(n, vector<int>(m));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < m; ++j) {
            is >> result[i][j];
        }
    }
    return result;
}

void WriteMatrix(ofstream& os, const vector<vector<int>>& m1) {
    os << m1.size() << " " << m1[0].size() << "\n";
    for (int i = 0; i < m1.size(); ++i) {
        for (int j = 0; j < m1[0].size(); ++j) {
            os << m1[i][j] << " ";
        }
        os << "\n";
    }
}

vector<vector<int>> MatrixProduct(const vector<vector<int>>& m1, const vector<vector<int>>& m2) {
    int n = m1.size(), m = m2.size(), l = m2[0].size();
    vector<vector<int>> result(n, vector<int>(l));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < l; ++j) {
            int sum = 0;
            for (int k = 0; k < m; ++k) {
                sum += m1[i][k] * m2[k][j];
            }
            result[i][j] = sum;
        }
    }
    return result;
}
int main() {
    setlocale(LC_ALL, "Rus");
    int m = 1, n = 1, l=1;
    int k = 3;
    int r = 0;
    while (r < k) {
        CreatMatr(n, m, l);
        r++;
    }

    ifstream file("matriza.bin", ios::in);
    ofstream out("result.bin", ios::out);
    if (!file.is_open() || !out.is_open()) {
        cout << "Файл не открыт\n";
        return 1;
    }

    for (int i = 0; i < k; i++) {
        file >> n >> m >> l;

        vector<vector<int>> m1 = ReadMatrix(file, n, m);
        vector<vector<int>> m2 = ReadMatrix(file, m, l);

        WriteMatrix(out, MatrixProduct(m1, m2));

    }

    out.close();
    file.close();
    return 0;
}

[marlaaa]

результат тот же

[res2001]

marlaaa,

убрала утечку памяти и в CreatMatr сделала return n,m,l

Не убрала и не сделала!
По утечкам: в С++ каждому вызову new должен соответствовать вызов delete. Посчитай сколько раз у тебя вызывается new и сколько delete. Так что утечки остались.

В С++ нельзя в return вернуть несколько значений через запятую. В таком случае вернется последнее перечисленное значение. Запятая - это оператор, можешь почитать в учебнике про него или тут: https://en.cppreference.com/w/cpp/language/operato...
Чтоб вернуть 3 значения (n, m, l) тебе надо упаковать их в структуру и возвращать структуру. Что-то типа такого:

struct nml_s {
 int n, m, l;
 nml_s(int _n, int _m, int _l) : n(_n), m(_m), l(_l) {}
};
...
nml_s CreatMatr(int n, int m, int l)
{
   ...
   nml_s nml(n, m, l);
   ...
   return nml;
}

Другой вариант передавать не значения n, m, l, а ссылки на них. Тогда CreatMatr будет выглядеть так:

void CreatMatr(int &n, int &m, int &l)
{
 ...
}

В таком случае возвращать ничего не нужно, т.к. изменение значения ссылки ведет к изменению значения на которое ссылается ссылка :)

Следующий момент. Предположим ты разобралась с возвратом значений из CreatMatr, но не достаточно просто их вернуть. Сейчас в CreatMatr ты как-будто их возвращаешь, но возвращенные значения нигде не сохраняются и в дальнейшем не используются (они просто пропадают).
Если будешь возвращать через ссылки, то n, m, l в main автоматически модифицируются после возврата из CreatMatr (формулировка не верна, но это не принципиально сейчас). Если возвращать через структуру, то структуру надо куда-то сохранить и затем использовать то что сохранено.

Еще один момент. Ты вызываешь CreatMatr несколько раз, но сохраняешь (пока что не сохраняешь) набор значений n, m, l только один - последний. А как же все предыдущие? Они не нужны?

[marlaaa]

res2001,
Я попробовала сделать через &n &m &l, вывод в файл поменялся но он всё равно не правильный.

Еще один момент. Ты вызываешь CreatMatr несколько раз, но сохраняешь (пока что не сохраняешь) набор значений n, m, l только один - последний. А как же все предыдущие? Они не нужны?

CreatMatr должен записывать в файл все значения в виде матриц, а потом их надо считать ReadMatr насколько я понимаю

#include <iostream>
#include <fstream>
#include <vector>
#include <ctime> 
using namespace std;

void CreatMatr(int &n, int &m, int &l) {
    ofstream file("matriza.bin", ios::app);
    cout << "Ввести строки первой матрицы";
    cin >> n;
    cout << "Ввести столбцы для первой матрицы и строки для второй";
    cin >> m;
    int** M;
    M = new int* [n];
    for (int i = 0; i < n; i++)
        M[i] = new int[m];

    for (int i = 0; i < n; i++)
        for (int j = 0; j < m; j++)
            M[i][j] = rand() % 10;


    cout << "Ввести столбцы второй матрицы";
    cin >> l;
    int** M2;
    M2 = new int* [m];
    for (int i = 0; i < m; i++)
        M2[i] = new int[l];

    for (int i = 0; i < m; i++)
        for (int j = 0; j < l; j++)
            M2[i][j] = rand() % 10;



    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++) {
            file << M[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < l; j++) {
            file << M2[i][j] << " ";
        }
        file << endl;
    }
    file << endl;
    file.close();

    delete[] M;
    delete[] M2;
}

vector<vector<int>> ReadMatrix(ifstream& is, int n, int m) {
    vector<vector<int>> result(n, vector<int>(m));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < m; ++j) {
            is >> result[i][j];
        }
    }
    return result;
}

void WriteMatrix(ofstream& os, const vector<vector<int>>& m1) {
    os << m1.size() << " " << m1[0].size() << "\n";
    for (int i = 0; i < m1.size(); ++i) {
        for (int j = 0; j < m1[0].size(); ++j) {
            os << m1[i][j] << " ";
        }
        os << "\n";
    }
}

vector<vector<int>> MatrixProduct(const vector<vector<int>>& m1, const vector<vector<int>>& m2) {
    int n = m1.size(), m = m2.size(), l = m2[0].size();
    vector<vector<int>> result(n, vector<int>(l));
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < l; ++j) {
            int sum = 0;
            for (int k = 0; k < m; ++k) {
                sum += m1[i][k] * m2[k][j];
            }
            result[i][j] = sum;
        }
    }
    return result;
}
int main() {
    setlocale(LC_ALL, "Rus");
    int m=1, n=1, l=1;
    int k = 3;
    int r = 0;
    while (r < k) {
        CreatMatr(n, m, l);
        r++;
    }

    ifstream file("matriza.bin", ios::in);
    ofstream out("result.bin", ios::out);
    if (!file.is_open() || !out.is_open()) {
        cout << "Файл не открыт\n";
        return 1;
    }

    for (int i = 0; i < k; i++) {
    

        vector<vector<int>> m1 = ReadMatrix(file, n, m);
        vector<vector<int>> m2 = ReadMatrix(file, m, l);

        WriteMatrix(out, MatrixProduct(m1, m2));

    }

    out.close();
    file.close();
    return 0;
}

[res2001]

marlaaa,

CreatMatr должен записывать в файл все значения в виде матриц, а потом их надо считать ReadMatrix

Верно, вот только в CreatMatr ты не пишешь в файл значения n, m, l, а передаещь их в ReadMatr в виде параметров. Так что ReadMatrix не может корректно прочитать файл, если у него не правильные n и m,

Тебе надо в main значения n, m, l сделать массивами размером k. И на каждой итерации вызова CreatMatr передавать ссылки на следующие элементы массива. (Я бы сделал структуру, типа той, что описывал выше, и сделал один массив структур, а не 3 массива интов.)
В следующем цикле так же используй элементы массива для передачи в ReadMatrix.

Вообще, не понятно зачем ты сначала пишешь эти файлы, а потом их читаешь. Сгенерировала матрицу в памяти и сразу отдавай ее в MatrixProduct. Можешь генерировать сразу вектор, а не сырой массив, как сейчас. Зачем эта возня с промежуточным файлом?