-6

我编写的代码编译时没有错误，因此我不知道发生错误的原因。

#pragma once
class Macierz
{
public:
class Wiersz
{
public:
    int *tab;
    int dlugosc;
    Wiersz(int d);
};

int x;
int y;
Wiersz *wiersze;
public:
Macierz(int m, int n);
Macierz(Macierz &m);

Macierz & operator+(Macierz &m);
Macierz & operator-(Macierz &m);
Macierz & operator*(Macierz &m);
Macierz & operator=(Macierz &m);
int& operator()(int x, int y);
string to_String();

};

#include "stdafx.h"
#include "Macierz.h"


Macierz::Macierz(int _x, int _y) :x(_x), y(_y)
{
Wiersz **tab = new Wiersz*[_x];
for (int i = 0; i < x; i++)
    tab[i] = new Wiersz(_y);
wiersze = *tab;
}


Macierz::Macierz(Macierz &m) :Macierz(m.x, m.y)
{
for (int i = 0; i < x; i++)
    for (int j = 0; j < y; j++)
        wiersze[i].tab[j] = m.wiersze[i].tab[j];
}




Macierz & Macierz::operator+(Macierz &m)
{
if (x != m.x || y != m.y) throw "Macierze maja rozne rozmiary";
Macierz *temp = new Macierz(m);

for (int i = 0; i < x; i++)
    for (int j = 0; j < y; j++)
        (*temp).wiersze[i].tab[j] += wiersze[i].tab[j];

return *temp;
}


Macierz & Macierz::operator-(Macierz &m)
{
if (x != m.x || y != m.y) throw "Macierze maja rozne rozmiary";
Macierz *temp = new Macierz(m);

for (int i = 0; i < x; i++)
    for (int j = 0; j < y; j++)
        (*temp).wiersze[i].tab[j] -= wiersze[i].tab[j];

return *temp;
}


Macierz & Macierz::operator*(Macierz &m)
{
Macierz *temp;
if (x == m.y)
{
    temp = new Macierz(m.x, y);
    for (int i = 0; i < (*temp).x; i++)
        for (int j = 0; j < (*temp).y; j++)
            for (int l = 0; l < m.y; l++)
                (*temp).wiersze[i].tab[j] += wiersze[i].tab[l] * m.wiersze[l].tab[j];

}
else if (y == m.x)
{
    temp = new Macierz(x, m.y);
    for (int i = 0; i < (*temp).x; i++)
        for (int j = 0; j < (*temp).y; j++)
            for (int l = 0; l < y; l++)
                (*temp).wiersze[i].tab[j] += wiersze[i].tab[l] * m.wiersze[l].tab[j];
}
else throw "Rozmiary macierzy sie nie zgadzaja";

return *temp;
}


Macierz & Macierz::operator=(Macierz &m)
{
for (int i = 0; i < x; i++)
    for (int j = 0; j < y; j++)
        wiersze[i].tab[j] = m.wiersze[i].tab[j];

return *this;
}


int& Macierz::operator()(int x, int y)
{
return wiersze[x].tab[y];
}


Macierz::Wiersz::Wiersz(int i)
{
dlugosc = i;
tab = new int[dlugosc];
}


string Macierz::to_String()
{

std::ostringstream str;
str << "|";
for (int i = 0; i < x; i++)
{
    for (int j = 0; j < y; j++)
        str << wiersze[j].tab[i] << ",";
    str << "\b|\n";
}

return str.str();
}

在 main 中使用构造函数不会使程序崩溃：

Macierz m1(1,1);

但是，访问某些数据字段似乎存在问题，例如

 m1(0,0) = 1;

（至少这是一些黄色箭头指向的代码行）

程序用于矩阵（+，-，*）的运算。我以类似的方式为欧几里德向量运算编写了一个程序，它运行良好。

提前感谢您的帮助。

4

2 回答 2

1

您的代码中有很多问题会导致问题：

  1. 你的构造函数Macierz(int _x, int _y)不像你想象的那样工作。仅wiersze[0]在完成后才有效，并且任何非零索引都将访问未知内存。问题实际上在于您对 2D 矩阵类的设计。我会摆脱手动管理的内存，只使用 2D 向量vector<vector<int>>甚至只是一个平面 1D 向量并手动管理索引。
  2. 您的复制构造函数Macierz(Macierz &m)不会为新矩阵分配内存，因此您正在写入未知内存。
  3. 您所有的二元运算符（+、-、*）分配一个新对象并返回它，这将导致内存泄漏。规范的方法通常是修改对象并且仅return *this;适用于此类运算符。有关运算符重载的更多详细信息，请参阅此问题。
  4. 您的复制操作员Macierz::operator=(Macierz &m)不会检查矩阵是否相同大小。如果它的大小不同，则需要销毁并重新分配矩阵。
  5. 您Macierz::operator()(int x, int y)不会检查可能是故意的输入索引，但您可能会考虑像使用其他方法一样抛出异常。
  6. 您没有会导致内存泄漏的析构函数，尽管如果您使用 a 则vector<>不需要它。

如果您真的需要/想要从头开始实现 2D 矩阵类，您可以在线搜索大量现有代码，例如这个，作为示例。

于 2015-01-06T14:08:35.963 回答
0

由于有很多指针算法，我敢打赌这是一个内存访问问题。
无论如何，当您使用 C++ 时，您应该利用矩阵类而不是重新发明轮子……
例如，带有矩阵类的基本线性代数库是一个顶级参考。 使用这样的类，在调试模式下编译时，会检查内存访问并帮助您快速错误索引。

于 2015-01-06T12:59:06.797 回答