утечка памяти

@The_bolT · Регистрация: 22.01.2013

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

Может кто-то проверить есть ли здесь утечка памяти? Мне почему-то кажется что есть. В задачи нужно когда пользователь сделал операции первый раз, программа спросила хочить он проводить операции над теми же дани или нет. Если пользователь введет нет, соответственно нужно удалить старую память и выделить новую. Я это и сделал, но что-то сомневаюсь правильно ли. Проверты кто-то пожалуйста)

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
#include <iostream>
#include "house.h"
 
using namespace std;
 
int menu();
 
int main()
{
    setlocale(0,"");
    int n; bool B = true, for_menu = true;
    cout<<"Введiть кiлькiсть квартир: "; cin>>n;
    House* Obj = new House[n];
    while(1){
        if(for_menu == false){
            delete [] Obj;
            cout<<"Введiть кiлькiсть квартир: "; cin>>n;
            House* Obj = new House[n];
        }
    int answer = menu();
 
    if(answer == 1){
    cout<<"Введiть кiлькiсть кiмнат: "; int num; cin>>num;
    for(int i = 0; i < n; i++)
        Obj[i].Show_komnatu(num,B);
    if(B == true) cout<<"Нiчого не знайдено!"<<endl;
    B = true;
    }
 
    if(answer == 2){
        cout<<"Введiть кiлькiсть квартир i дiапазон поверхiв: "; int k,up,down; cin>>k>>up>>down;
    for(int i = 0; i < n; ++i)
        Obj[i].Show_komnatu_plocha_and_powerh(k,up,down,B);
    if(B == true) cout<<"Нiчого не знайдено!"<<endl;
    B = true;
    }
 
    if(answer == 3){
    cout<<"Введiть площу: "; int p; cin>>p;
    for(int i = 0; i < n; ++i)
        Obj[i].Show_komnatu_plocha(p,B);
    if(B == true) cout<<"Нiчого не знайдено!"<<endl;
    }
 
    cout<<"Продовжити? y/n: "; char an; cin>>an;
    if(an == 'y'){ 
        cout<<"Продовжити зi старими даними? y/n: ";
        cin>>an;
        if(an == 'y') {for_menu = true; continue;}
        else
            for_menu = false;
    } 
    else
        break;
    }
    delete [] Obj;
    system("pause>0");
    return 0;
}
 
int menu()
{
    cout<<"1 - Вивести список квартир, що мають задане число кiмнат;"<<endl
        <<"2 - Вивести список квартир, що мають задане число кiмнат, i розташованих на         поверсi, який знаходиться в заданому промiжку;"<<endl
        <<"3 - Вивести список квартир, якi мають площу, яка перевищує задану."<<endl;
    int key;
    cin>>key;
    return key;
}

Вот библиотеки:

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
#include "house.h"
#include <iostream>
 
using namespace std;
 
House::House()
{
    setAddres();
    setPowerh();
    set_number_kimnat();
    setPlocha();
    cout<<endl;
}
 
void House::Show_all()
{
    cout<<"Адреса: "<<addres<<endl
        <<"Поверх: "<<powerh<<endl
        <<"Кiлькiсть кiмнат: "<<number_kimnat<<endl
        <<"Площа квартири: "<<plocha<<endl;
}
 
void House::setAddres()
{
    cout<<"Введiть адресу: "; cin>>addres;
}
 
void House::setPowerh()
{
    cout<<"Введiть поверх: "; cin>>powerh;
}
 
void House::set_number_kimnat()
{
    cout<<"Введiть кiлькiсть кiмнат: "; cin>>number_kimnat;
}
 
void House::setPlocha()
{
    cout<<"Введiть площу квартири: "; cin>>plocha;
}
 
void House::Show_komnatu(int num,bool &B)
{
    if(number_kimnat == num){
        Show_all();
        B = false;
    }
}
 
void House::Show_komnatu_plocha(int p,bool &B)
{
    if(plocha > p){
        Show_all();
        B = false;
    }
}
 
void House::Show_komnatu_plocha_and_powerh(int numb_komnatu,int powerh_up,int powerh_down,bool &B)
{
    if((numb_komnatu == number_kimnat) && (powerh >= powerh_up) && (powerh <= powerh_down) ){
        Show_all();
        B = false;
    }
}

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#ifndef house_h
#define house_h
 
class House
{
public:
    House();
    void Show_komnatu(int,bool&);
    void Show_komnatu_plocha(int,bool&);
    void Show_komnatu_plocha_and_powerh(int,int,int,bool&);
    void setAddres();
    void setPowerh();
    void set_number_kimnat();
    void setPlocha();
    void Show_all();
private:
    char addres[10];
    int powerh;
    int number_kimnat;
    int plocha;
    };
 
#endif

@instagib · 06.03.2013, 18:14

The_bolT,

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

C++
1
cout<<"Введiть кiлькiсть квартир: "; cin>>n; House* Obj = new House[n];

я думаю лучше сперва так:

C++
1
House* Obj;

И выделяй память когда уже точно нужно.

Jupiter · 06.03.2013, 18:20

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

Я это и сделал, но что-то сомневаюсь правильно ли. Проверты кто-то пожалуйста)

для начала перепишите main чтоб он вмещался в 20 строк максимум и при этом не надо делать вот так

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

C++
1
2
int n; bool B = true, for_menu = true;
cout<<"Введiть кiлькiсть квартир: "; cin>>n;

Croessmah · 06.03.2013, 18:21

C++
1
2
3
4
5
6
7
8
9
    House* Obj = new House[n];
    while(1){
        if(for_menu == false){
            delete [] Obj;
            cout<<"Введiть кiлькiсть квартир: "; cin>>n;
            House* Obj = new House[n];//Вот тут без House - Obj=new House[n];
//Иначе получается объявление локальной переменной с именем Obj и её инициализация.
//Хотя, смотря что Вы хотели этим сказать
        }

@The_bolT · 06.03.2013, 18:33 **[ТС]**

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

House* Obj = new House[n];
* * while(1){
* * * * if(for_menu == false){
* * * * * * delete [] Obj;
* * * * * * cout<<"Введiть кiлькiсть квартир: "; cin>>n;
* * * * * * House* Obj = new House[n];//Вот тут без House - Obj=new House[n];
//Иначе получается объявление локальной переменной с именем Obj и её инициализация.
//Хотя, смотря что Вы хотели этим сказать
* * * * }

Если без этого значит пользователь не сможет ввести больше квартир чем он ввел с первого раза! А мне нужно чтобы пользователь мог менять размер массива! Алгоритм такой: высвобождаем старую память, создаем новый массив для хранения n квартир. Как это сделать?

Добавлено через 5 минут

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
int main()
{
    setlocale(0,"");
    int n; 
    bool B = true, for_menu = true;
    cout<<"Введiть кiлькiсть квартир: "; cin>>n;
    House* Obj = new House[n];
    while(1){
        if(for_menu == false){ //здесь повино все происходить, удаление памяти и создание нового массива. 
            delete [] Obj;   //Но как уже сказалось это локальная переменная, поэтому надо что-то другое придумать!
            cout<<"Введiть кiлькiсть квартир: "; cin>>n;
            House* Obj = new House[n];
        }
 
    //какие-то действия
 
    cout<<"Продовжити? y/n: "; 
    char an; 
    cin>>an;
    if(an == 'y'){ 
        cout<<"Продовжити зi старими даними? y/n: ";
        cin>>an;
        if(an == 'y') {for_menu = true; continue;} //Если да, то повторяем цикл и for_menu = true
        else
            for_menu = false;
    } 
    else
        break;
    }
    delete [] Obj;
    system("pause>0");
    return 0;
}

Croessmah · 06.03.2013, 18:35

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

Как это сделать?

Ву уже сделали. Но зачем Вам второй указатель с именем Obj?

@The_bolT · 06.03.2013, 18:37 **[ТС]**

Ааааа. Я понял) То можно записать так?

C++
1
2
3
4
5
if(for_menu == false){ 
            delete [] Obj;   
            cout<<"Введiть кiлькiсть квартир: "; cin>>n;
            Obj = new House[n];
        }

Новые блоги и статьи Все статьи Все блоги /
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .