Runtime error 3221226356

Я начал писать простой класс для работы с матрицами, один конструктор заполняет матрицу одним числом, а другой принимает в качестве аргумента 2d массив.

    #include <iostream>

using namespace std;
int i,j;
class Matrica
{
public:
Matrica(int, int, double);
Matrica(int, int, double*);
~Matrica();
void saberi(Matrica, Matrica);
void read();
void print();
private:
int n,m;
double* matrix;
};

Matrica::Matrica(int a, int b, double broj)
{
n = a;
m = b;
matrix = new double[n*m];
for(i=0;i<n;i++) {
for(j=0;j<m;j++) {
matrix[i*n + j] = broj;
}
}
}

Matrica::Matrica(int a, int b, double* matr)
{
n = a;
m = b;
matrix = new double[n*m];
for(i=0;i<n;i++) {
for(j=0;j<m;j++) {
matrix[i*n + j] = matr[i*n +j];
}
}

}
Matrica::~Matrica() {
delete[] matrix;
}

void Matrica::read() {
double e;
for(i=0;i<n;i++) {
for(j=0;j<m;j++) {
cout<< "insert element ["<<i<<"]["<<j<<"] :" <<endl;
cin >> e;
matrix[n*i +j] = e;
cout<< endl;
}
}
}

void Matrica::print() {
for(i=0;i<n;i++) {
for(j=0;j<m;j++) {
cout << matrix[n*i + j] << " ";
}
cout << endl;
}
}
void Matrica::saberi(Matrica A, Matrica B) {
cout << "foo";
}

int main()
{

Matrica A(3,3,1);
double K[3][3] = {{1,2,3},{4,5,6},{7,8,9}};
Matrica B(3,3,&K[0][0]);
Matrica C(3,3,7);
C.saberi(A,B);
return 0;
}

Моя программа работала нормально, прежде чем я добавил пустой метод «saberi», который принимает два объекта в качестве аргументов. Если я это называю, моя программа падает с возвращаемым значением 3221226356. Что может быть не так? Заранее спасибо.

1

Как вы определили свою функцию подписи

void saberi(Matrica, Matrica);

Matrica объекты передаются по значению, но вы не предоставили правильный конструктор копирования для класса Matrica,

Самый простой способ решить вашу текущую проблему, это передать параметры по ссылке

void saberi(Matrica&, Matrica&);

или же

void saberi(const Matrica&, const Matrica&);

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

Matrica(const Matrica&);

или Fobid его (просто поместите объявление в private секция класса).

1

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Я делаю змеиную игру в консоли, но всякий раз, когда я пишу system («cls») в функции рисования, она пишет «Процесс завершен с возвращаемым значением 3221226356» на экране вывода. Я думаю, что есть некоторая утечка памяти, но я не мог найти точную причину, почему это происходит.

Вы также можете увидеть этот код по адресу: https://github.com/Lakshay-Dhingra/Snake- Игра

Вот мой код: // Скомпилировано в Dev-C ++ (Windows) с компилятором mingw std-c ++ 11

#include<iostream>
#include<conio.h>
#include <unistd.h>
#include<cstdlib>
using namespace std;

class Node
{
    int x;
    int y;
    Node *link;

    public:
    Node(int ix,int iy)
    {
        x=ix;
        y=iy;
        link=nullptr;
    }

    int getx()
    {
        return x;
    }
    void setx(int x)
    {
        this->x=x;
    }
    int gety()
    {
        return y;
    }
    void sety(int y)
    {
        this->y=y;
    }

    Node* getLink()
    {
        return link;
    }
    void setLink(Node* x)
    {
        link=x;
    }
};

class LL
{
    public:
    Node* head;
    LL()
    {
        head=nullptr;
    }

    void insert(int x,int y)
    {
        Node* mynode=new Node(x,y);
        Node* temp=head;
        head=mynode;
        head->setLink(temp);
    }

//  For Debugging:
//  void showList()
//  {
//      Node* cur=head;
//      while(cur->getLink()!=nullptr)
//      {
//          cout<<cur->getx()<<" "<<cur->gety()<<" --> ";
//          cur=cur->getLink();
//      }
//      cout<<cur->getx()<<" "<<cur->gety()<<endl;
//  }

    int* show(int l)
    {

        int* arr=new int(2*l);
        int i=0;
        Node* cur=head;
        while(cur!=nullptr)
        {
            arr[i]=cur->getx();
            arr[i+1]=cur->gety();
            i+=2;
            cur=cur->getLink();
        }
        return arr;
    }

    ~LL()
    {
        Node* temp=head;
        while(head!=nullptr)
        {
            temp=head->getLink();
            delete head;
            head=temp;
        }
    }
};

class Snake
{
    int length;
    LL* mysnake;

    public:
    Snake(int l,int x,int y)
    {
        mysnake=new LL();
        for(int i=0;i<l;i++)
        {
            mysnake->insert((x+i),y);
        }
        length=l;
    }

    int getLength()
    {
        return length;
    }

    void eatFood(int x,int y)
    {
        length+=1;
        mysnake->insert(x,y);
    }

    int* showSnake()
    {
        return mysnake->show(length);
    }

    int getHeadX()
    {
        return mysnake->head->getx();
    }
    int getHeadY()
    {
        return mysnake->head->gety();
    }
};

class Window
{
    int length;
    int hieght;
    Snake* snk;
    int* arr;

    public:
    Window(int l,int h,int posx,int posy)
    {

        length=l;
        hieght=h;
        snk=new Snake(4,posx-3,posy);
        arr=snk->showSnake();

//      rungame();
//      For Debugging:
//      for(int i=0;i<2*(sk.getLength());i++)
//      {
//          cout<<arr[i]<<" ";
//      }
    }

    void rungame()
    {
        char ch;
        while(true)
        {
            if(kbhit())
            {
            ch = _getch();
            if(int(ch)==27)
                break;
            }
            draw();
        }
    }

    void draw()
    {
        system("cls");
//      arr=snk->showSnake();
//      cout<<"world ";
        bool flag;
        for(int i=0;i<length;i++)
        {
            for(int j=0;j<hieght;j++)
            {
                flag=0;
                if(i==0||i==length-1||j==0||j==hieght-1)
                {
                    cout<<"#";
                    flag=1;
                }
                else
                {
                    for(int k=0;k<snk->getLength();k++)
                    {
                        if(i==arr[2*k]&&j==arr[2*k+1])
                        {
                            cout<<"O";
                            flag=1;
                        }
                    }
                }

                if(flag==0)
                {
                    cout<<" ";
                }
            }
            cout<<endl;
        }
//      For Debugging:
//      for(int k=0;k<snk->getLength();k++)
//          cout<<arr[2*k]<<" "<<arr[2*k+1]<<" ";
        usleep(100000);
    }
};

int main()
{
    Window win(30,50,15,25);
    win.rungame();
    return 0;
}

Ошибка 3221226356 — это 0xc0000374 в шестнадцатеричном формате, то есть STATUS_HEAP_CORRUPTION.

Я пропустил его при первом прочтении, но оно вызвано вашим выделением в show():

int* arr=new int(2*l);

Это выделяет пробелы для single int и заполняет его значением 2*l. Впоследствии вы пишете в нераспределенную / неинициализированную память: arr[1], arr[2] и т. Д.

Немедленное исправление просто:

int* arr = new int[2*l];

Это выделяет пространство для 2*l int значений.

В долгосрочной перспективе вам лучше вместо этого использовать std::vector<int> или отказаться от всей этой работы с массивами и обходить свой связанный список напрямую, когда вам нужно (например, в вашей функции draw).

unistd.h — заголовок UNIX (Linux, MacOS и т. Д.), Его не следует включать в проект Windows. Это может быть причиной.

Вызов system("clear"); в Linux SO.

Затем вам нужно заменить функции, такие как usleep(), которые принадлежат unistd.h и не могут использоваться в проекте Windows.

Я считаю, что windows.h имеет некоторые похожие функции.