Ошибка при работе деструктора

@shukaka · Регистрация: 09.03.2014

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

C++

#include<iostream>
#include<time.h>
#include<Windows.h>
 #include <stdio.h>
#include <conio.h>
#include <fstream>
#include <iomanip>
using namespace std;
class game
{
    char*packmen;
    char*kassper;
    char **map;
public:
    game();
    ~game();
    game(const game &B);
    void start_game();
    void continue_game();
    void save_game();
    int move_packmen();
    int move_kassper();
    int build_map();
    void show_map();
};
game::game()
{
    map=new char*[20];
    for(int i=0;i<20;i++)
        map[i]=new char[30];
    packmen=new char[1];
    kassper=new char[1];
}
game::game(const game &B)
{
    strcpy(packmen,B.packmen);
    strcpy(kassper,B.kassper);
    for(int i=0;i<20;i++)
          strcpy(map[i],B.map[i]);
}
game::~game()
{
    delete[]kassper;
    delete[]packmen;
    for(int i=0;i<20;i++)
    delete[]map[i];
}
int game::build_map()
{
    
    for(int i=0;i<20;i++)
    {
        if(i==0||i==19)
        {
        for(int j=0;j<30;j++)
            map[i][j]='=';
        }
        else
            for(int j=0;j<29;j++)
                map[i][j]=' ';
 
    }
    for(int j=0;j<20;j++)
            {
                map[j][29]='=';
            }
    int k=0,p=0;
    for(int i=1;i<30;i++)
        map[k=rand()%18+1][p=rand()%28+1]='|';
    packmen[0]='o';
    for(int i=1;i<19;i++)
    {
        for(int j=1;j<29;j++)
        {
    if(map[i][j]!='|')
    {
        map[i][j]=packmen[0]; 
        i=29;
    }
    else
          continue;
        }
        }
    kassper[0]='$';
    int calc=0;
    while(true)
    {
    for(int i=1;i<19;i++)
    {
        for(int j=0;j<29;j++)
        {
    if(map[i][j]==' ')
    {
        map[ i=rand()%17+1][j=rand()%27+1]=kassper[0]; 
        calc++;
        if(calc==3)
            return 0;
    }
    else
          continue;
        }
        }
    }
}
void game::show_map()
{
    Sleep(0);
    system("cls");
    for(int i=0;i<20;i<i++)
    {
        for(int j=0;j<30;j++)
            cout<<map[i][j];
    cout<<endl;
    }
}
int game::move_packmen()
{
    char*location=new char[1];
    while(true)
    {
        int move=move_kassper();
    if(move==119)
        for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(map[i-1][j]!='='&&map[i-1][j]!='|')
                    {
                    map[i][j]=' ';
                    map[i-1][j]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
            if(move==115)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(map[i+1][j]!='='&&map[i+1][j]!='|')
                    {
                    map[i][j]=' ';
                    map[i+1][j]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
                if(move==97)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(j-1>=0&&map[i][j-1]!='|')
                    {
                    map[i][j]=' ';
                    map[i][j-1]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
                if(move==100)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(map[i][j+1]!='='&&map[i][j+1]!='|')
                    {
                    map[i][j]=' ';
                    map[i][j+1]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
                if(move==27)
                {
                    save_game();
                    return 0;
                }
 
    }
}
void game::save_game()
{
    ofstream outF("game.txt",ios::out);
    for(int i=0;i<20;i++)
    {
        for(int j=0;j<30;j++)
        {
    outF<<map[i][j];
        }
        outF<<endl;
    }
    outF.close();
}
int game::move_kassper()
{
    while(!kbhit())
    {
        Sleep(100);
        int move=rand()%4;
        if(move==0)
        for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(map[i-1][j]!='='&&map[i-1][j]!='|'&&map[i-1][j]!='$')
                    {
                    map[i][j]=' ';
                    map[i-1][j]='$';
                    show_map();
                    j=j++;
                    i=i++;
                    }
                    else break;
                }
            if(move==1)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(map[i+1][j]!='='&&map[i+1][j]!='|'&&map[i+1][j]!='$')
                    {
                    map[i][j]=' ';
                    map[i+1][j]='$';
                    show_map();
                    j++;
                    i++;
                    }
                    else break;
                }
                if(move==2)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(j-1>=0&&map[i][j-1]!='|'&&map[i][j-1]!='$')
                    {
                    map[i][j]=' ';
                    map[i][j-1]='$';
                    show_map();
                    j++;
                    i++;
                    }
                    else break;
                }
                if(move==3)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(map[i][j+1]!='='&&map[i][j+1]!='|'&&map[i][j+1]!='$')
                    {
                    map[i][j]=' ';
                    map[i][j+1]='$';
                    show_map();
                    j++;
                    i++;
                    }
                    else break;
                }
    };
    int move=getch();
    return move;
}
void game::start_game()
{
    build_map();
    move_packmen();
    
}
void game::continue_game()
{
    ifstream inF("game.txt",ios::in| ios::_Nocreate);
        for(int i=0;i<1;i++)
    {
        for(int j=0;j<30;j++)
        {
    inF>>map[i][j];
        }
        //cout<<endl;
    }
    inF.close();
}
 
void main()
{
    //srand(time(NULL));
    game A;
    //A.start_game();
    A.build_map();
    A.show_map();
    //A.save_game();
    //A.continue_game();
    //A.show_map();
    system("pause");
}

я так понимаю конструктор копирования должен убрать ошибку но чего то не выходит(

@zss · 12.02.2015, 21:51

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

C++

class game
{
    char packmen[1]; // Вы уверены, что массив должен быть из одного элемента. Для строки надо минимум 2.
// если все же один, то предлагаю поменять на  char packmen;
    char kassper[1];
    char map[30][20];
public:
    game();
    ~game();
    game(const game &B);
    void start_game();
    void continue_game();
    void save_game();
    int move_packmen();
    int move_kassper();
    int build_map();
    void show_map();
};
game::game()
{
   // ничего делать не надо
}

@shukaka · 12.02.2015, 22:51 **[ТС]**

C++

#include<iostream>
#include<time.h>
#include<Windows.h>
 #include <stdio.h>
#include <conio.h>
#include <fstream>
#include <iomanip>
using namespace std;
class game
{
    char*packmen; 
    char*kassper;
    char **map;
public:
    game();
    ~game();
    game(const game &B);
    void start_game();
    void continue_game();
    void save_game();
    int move_packmen();
    int move_kassper();
    int build_map();
    void show_map();
};
game::game()
{
    map=new char*[20];
    for(int i=0;i<20;i++)
        map[i]=new char[30];
    packmen=new char[1];
    kassper=new char[1];
}
game::game(const game &B)
{
    packmen=new char[1];
    kassper=new char[1];
    map=new char*[20];
    for(int i=0;i<20;i++)
    map[i]=new char[30];
    strcpy(packmen,B.packmen);
    strcpy(kassper,B.kassper);
    for(int i=0;i<20;i++)
          strcpy(map[i],B.map[i]);
}
game::~game()
{
    delete[]kassper;
    delete[]packmen;
    for(int i=0;i<20;i++)
    delete[]map[i];
}
int game::build_map()
{
    
    for(int i=0;i<20;i++)
    {
        if(i==0||i==19)
        {
        for(int j=0;j<30;j++)
            map[i][j]='=';
        }
        else
            for(int j=0;j<29;j++)
                map[i][j]=' ';
 
    }
    for(int j=0;j<20;j++)
            {
                map[j][29]='=';
            }
    int k=0,p=0;
    for(int i=1;i<30;i++)
        map[k=rand()%18+1][p=rand()%28+1]='|';
    packmen[0]='o';
    for(int i=1;i<19;i++)
    {
        for(int j=1;j<29;j++)
        {
    if(map[i][j]!='|')
    {
        map[i][j]=packmen[0]; 
        i=29;
    }
    else
          continue;
        }
        }
    kassper[0]='$';
    int calc=0;
    while(true)
    {
    for(int i=1;i<19;i++)
    {
        for(int j=0;j<29;j++)
        {
    if(map[i][j]==' ')
    {
        map[ i=rand()%17+1][j=rand()%27+1]=kassper[0]; 
        calc++;
        if(calc==3)
            return 0;
    }
    else
          continue;
        }
        }
    }
}
void game::show_map()
{
    Sleep(0);
    system("cls");
    for(int i=0;i<20;i<i++)
    {
        for(int j=0;j<30;j++)
            cout<<map[i][j];
    cout<<endl;
    }
}
int game::move_packmen()
{
    char*location=new char[1];
    while(true)
    {
        int move=move_kassper();
    if(move==119)
        for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(map[i-1][j]!='='&&map[i-1][j]!='|')
                    {
                    map[i][j]=' ';
                    map[i-1][j]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
            if(move==115)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(map[i+1][j]!='='&&map[i+1][j]!='|')
                    {
                    map[i][j]=' ';
                    map[i+1][j]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
                if(move==97)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(j-1>=0&&map[i][j-1]!='|')
                    {
                    map[i][j]=' ';
                    map[i][j-1]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
                if(move==100)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='o')
                {
                    if(map[i][j+1]!='='&&map[i][j+1]!='|')
                    {
                    map[i][j]=' ';
                    map[i][j+1]='o';
                    show_map();
                    j=30;
                    i=29;
                    }
                    else continue;
                }
                else continue;
                if(move==27)
                {
                    save_game();
                    return 0;
                }
 
    }
}
void game::save_game()
{
    ofstream outF("game.txt",ios::out);
    for(int i=0;i<20;i++)
    {
        for(int j=0;j<30;j++)
        {
    outF<<map[i][j];
        }
        outF<<endl;
    }
    outF.close();
}
int game::move_kassper()
{
    while(!kbhit())
    {
        Sleep(100);
        int move=rand()%4;
        if(move==0)
        for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(map[i-1][j]!='='&&map[i-1][j]!='|'&&map[i-1][j]!='$')
                    {
                    map[i][j]=' ';
                    map[i-1][j]='$';
                    show_map();
                    j=j++;
                    i=i++;
                    }
                    else break;
                }
            if(move==1)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(map[i+1][j]!='='&&map[i+1][j]!='|'&&map[i+1][j]!='$')
                    {
                    map[i][j]=' ';
                    map[i+1][j]='$';
                    show_map();
                    j++;
                    i++;
                    }
                    else break;
                }
                if(move==2)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(j-1>=0&&map[i][j-1]!='|'&&map[i][j-1]!='$')
                    {
                    map[i][j]=' ';
                    map[i][j-1]='$';
                    show_map();
                    j++;
                    i++;
                    }
                    else break;
                }
                if(move==3)
                for(int i=0;i<20;i++)
            for(int j=0;j<30;j++)
                if(map[i][j]=='$')
                {
                    if(map[i][j+1]!='='&&map[i][j+1]!='|'&&map[i][j+1]!='$')
                    {
                    map[i][j]=' ';
                    map[i][j+1]='$';
                    show_map();
                    j++;
                    i++;
                    }
                    else break;
                }
    };
    int move=getch();
    return move;
}
void game::start_game()
{
    build_map();
    move_packmen();
    
}
void game::continue_game()
{
    ifstream inF("game.txt",ios::in| ios::_Nocreate);
        for(int i=0;i<1;i++)
    {
        for(int j=0;j<30;j++)
        {
    inF>>map[i][j];
        }
        //cout<<endl;
    }
    inF.close();
}
 
void main()
{
    //srand(time(NULL));
    game A;
    //A.start_game();
    A.build_map();
    A.show_map();
    //A.save_game();
    //A.continue_game();
    //A.show_map();
    system("pause");
}

все равно не пашет( даже после выделение памяти(

GREGOR_812 · 13.02.2015, 00:30

в каком месте деструктор вызывается? при выходе из main()? Не вижу тут проблемы, связанной с копированием, т.к. создан только один объект

GREGOR_812 28 / 28 / 5 Регистрация: 23.04.2014 Сообщений: 130
	13.02.2015, 00:30	4
	в каком месте деструктор вызывается? при выходе из main()? Не вижу тут проблемы, связанной с копированием, т.к. создан только один объект 0