Форум программистов, компьютерный форум CyberForum.ru

утечка памяти - C++

Восстановить пароль Регистрация
 
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
06.03.2013, 18:00     утечка памяти #1
Может кто-то проверить есть ли здесь утечка памяти? Мне почему-то кажется что есть. В задачи нужно когда пользователь сделал операции первый раз, программа спросила хочить он проводить операции над теми же дани или нет. Если пользователь введет нет, соответственно нужно удалить старую память и выделить новую. Я это и сделал, но что-то сомневаюсь правильно ли. Проверты кто-то пожалуйста)
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
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
06.03.2013, 18:00     утечка памяти
Посмотрите здесь:

Утечка памяти?! C++
Утечка памяти C++
C++ Утечка памяти в
C++ утечка памяти
C++ Утечка памяти
Утечка памяти C++
C++ Утечка памяти
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
instagib
122 / 85 / 3
Регистрация: 14.02.2011
Сообщений: 341
06.03.2013, 18:14     утечка памяти #2
The_bolT,
Цитата Сообщение от The_bolT Посмотреть сообщение
C++
1
cout<<"Введiть кiлькiсть квартир: "; cin>>n; House* Obj = new House[n];
я думаю лучше сперва так:
C++
1
House* Obj;
И выделяй память когда уже точно нужно.
Jupiter
Каратель
Эксперт C++
6545 / 3965 / 226
Регистрация: 26.03.2010
Сообщений: 9,273
Записей в блоге: 1
Завершенные тесты: 2
06.03.2013, 18:20     утечка памяти #3
Цитата Сообщение от The_bolT Посмотреть сообщение
Я это и сделал, но что-то сомневаюсь правильно ли. Проверты кто-то пожалуйста)
для начала перепишите main чтоб он вмещался в 20 строк максимум и при этом не надо делать вот так
Цитата Сообщение от The_bolT Посмотреть сообщение
C++
1
2
int n; bool B = true, for_menu = true;
cout<<"Введiть кiлькiсть квартир: "; cin>>n;
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
12279 / 6966 / 784
Регистрация: 27.09.2012
Сообщений: 17,274
Записей в блоге: 2
Завершенные тесты: 1
06.03.2013, 18:21     утечка памяти #4
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
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
06.03.2013, 18:33  [ТС]     утечка памяти #5
Цитата Сообщение от 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
Модератор
Эксперт С++
 Аватар для Croessmah
12279 / 6966 / 784
Регистрация: 27.09.2012
Сообщений: 17,274
Записей в блоге: 2
Завершенные тесты: 1
06.03.2013, 18:35     утечка памяти #6
Цитата Сообщение от The_bolT Посмотреть сообщение
Как это сделать?
Ву уже сделали. Но зачем Вам второй указатель с именем Obj?
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
06.03.2013, 18:37     утечка памяти
Еще ссылки по теме:

C++ утечка памяти
C++ Утечка памяти
C++ Утечка памяти?
Утечка памяти C++
Утечка памяти C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
06.03.2013, 18:37  [ТС]     утечка памяти #7
Ааааа. Я понял) То можно записать так?
C++
1
2
3
4
5
if(for_menu == false){ 
            delete [] Obj;   
            cout<<"Введiть кiлькiсть квартир: "; cin>>n;
            Obj = new House[n];
        }
Yandex
Объявления
06.03.2013, 18:37     утечка памяти
Ответ Создать тему
Опции темы

Текущее время: 09:27. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru