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

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

Восстановить пароль Регистрация
 
3bOONAL
0 / 0 / 0
Регистрация: 11.12.2013
Сообщений: 22
17.03.2014, 21:53     Утечка памяти #1
Помогите с кодом, вот написал реализацию стека по заданию. Преподаватель попросил проверить, верно ли я использую delete, ибо mas в классе и в stack в main выглядят по разному ^^" Пытался юзать VLD , но он не пашет, говорит не найден вход в процедуру.

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
#include <iostream>
 
using namespace std;
class Stack
{
public:
    int *mas;
    int top;
    int size;
     Stack(int size){
        top= -1;
        mas = new int[size]; 
                   }                      
     ~Stack(){
    cout << "Memory has been cleaned. Good bye." << endl;
     } 
     bool push(int value ){
        if (top == size - 1)
        return false;
    top++;
   mas[top] = value; 
   return true;
    }
 
    bool pop(){
        if (top == - 1)
        return false; 
    mas[top] = 0; 
    top--;
    return true;
      }
    int print(int x){
        
    while ( x--!=0)cout << mas[x]<<endl;
    return 1;
    }
};
 
int main() {
    int size=0;
    cout << "enter the size of the stack: ";
    cin >> size;
    Stack *stack = new Stack(size);
     cout << "enter value of the elements:";
    int ct = 0;
    while (ct!= size)
                          {
        ct++;
        int temp;
        cin >> temp;
        stack -> push(temp);
                           }
 
    stack -> print(ct);
    
    stack->pop();
    
    stack -> print(ct);
 
    delete stack;
    system("pause");
              }
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
17.03.2014, 21:53     Утечка памяти
Посмотрите здесь:

Утечка памяти C++
C++ Утечка памяти
Утечка памяти?! C++
C++ Утечка памяти
Утечка памяти C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
DrOffset
6457 / 3831 / 885
Регистрация: 30.01.2014
Сообщений: 6,627
17.03.2014, 21:56     Утечка памяти #2
Если
C++
1
mas = new int[size];
то в деструкторе надо:
C++
1
delete [] mas;
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11841 / 6820 / 771
Регистрация: 27.09.2012
Сообщений: 16,911
Записей в блоге: 2
Завершенные тесты: 1
17.03.2014, 21:57     Утечка памяти #3
C++
1
2
3
     ~Stack(){
    cout << "Memory has been cleaned. Good bye." << endl;
     }
Где освобождение памяти?
Где конструктор копий и оператор присваивания?
3bOONAL
0 / 0 / 0
Регистрация: 11.12.2013
Сообщений: 22
17.03.2014, 21:58  [ТС]     Утечка памяти #4
а то как я сделал приводит к плохим последствиям ?
DrOffset
6457 / 3831 / 885
Регистрация: 30.01.2014
Сообщений: 6,627
17.03.2014, 22:00     Утечка памяти #5
3bOONAL, приводит к вычету из зарплаты, если обнаруживается на code review
3bOONAL
0 / 0 / 0
Регистрация: 11.12.2013
Сообщений: 22
17.03.2014, 22:01  [ТС]     Утечка памяти #6
лол) учту
mustimur
268 / 222 / 57
Регистрация: 22.11.2013
Сообщений: 832
Записей в блоге: 1
17.03.2014, 22:02     Утечка памяти #7
DrOffset, сомневаюсь, у него по-моему правильно в этой строчке было (элемент то один). а вот в дисрукторе присутствует только:
C++
1
 cout << "Memory has been cleaned. Good bye." << endl;
в то время как самой очистки то нет, а вот в ней то и должно быть это
C++
1
delete [] mas;
Добавлено через 45 секунд
Croessmah, опять опередили Вы меня)))
3bOONAL
0 / 0 / 0
Регистрация: 11.12.2013
Сообщений: 22
17.03.2014, 22:03  [ТС]     Утечка памяти #8
C++
1
2
3
4
~Stack(){
         delete []mas;
    cout << "Memory has been cleaned. Good bye." << endl;
     }
то есть вот так, и все будет корректно ?
mustimur
268 / 222 / 57
Регистрация: 22.11.2013
Сообщений: 832
Записей в блоге: 1
17.03.2014, 22:04     Утечка памяти #9
Цитата Сообщение от Croessmah Посмотреть сообщение
Где конструктор копий и оператор присваивания?
я думаю не все сразу, навыки хорошего тона прививать по-степенно
Croessmah
17.03.2014, 22:05
  #10

Не по теме:

Цитата Сообщение от mustimur Посмотреть сообщение
Croessmah, опять опередили Вы меня)))
We can win the race, если верить Modern Talking

mustimur
268 / 222 / 57
Регистрация: 22.11.2013
Сообщений: 832
Записей в блоге: 1
17.03.2014, 22:05     Утечка памяти #11
Цитата Сообщение от 3bOONAL Посмотреть сообщение
то есть вот так, и все будет корректно ?
Да, это и будет освобождение памяти.
DrOffset
17.03.2014, 22:05
  #12

Не по теме:

mustimur, а я уже исправил Не разглядел сперва)

mustimur
268 / 222 / 57
Регистрация: 22.11.2013
Сообщений: 832
Записей в блоге: 1
17.03.2014, 22:07     Утечка памяти #13
Цитата Сообщение от Croessmah Посмотреть сообщение
если верить Modern Talking

Не по теме:

Приятно видеть единомыслишника со схожим музыкальным вкусом We Take a Chance - Modern Talking

Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11841 / 6820 / 771
Регистрация: 27.09.2012
Сообщений: 16,911
Записей в блоге: 2
Завершенные тесты: 1
17.03.2014, 22:07     Утечка памяти #14
Цитата Сообщение от mustimur Посмотреть сообщение
я думаю не все сразу, навыки хорошего тона прививать по-степенно
причем тут хороший тон? Это необходимая составляющая. Хотя бы запереть их в приват без реализации, ли пометить удаленными(C++11)
3bOONAL
0 / 0 / 0
Регистрация: 11.12.2013
Сообщений: 22
17.03.2014, 22:08  [ТС]     Утечка памяти #15
Спасибо, теперь буду знать , что ответить преподавателю
mustimur
268 / 222 / 57
Регистрация: 22.11.2013
Сообщений: 832
Записей в блоге: 1
17.03.2014, 22:10     Утечка памяти #16
Цитата Сообщение от Croessmah Посмотреть сообщение
причем тут хороший тон
Человек только начал знакомиться с классами (это учебная задача), и Ваш легкий наезд скорее всего не понял. Хотя согласен с Вами и рекомендую ТС обратить на это внимание.
Цитата Сообщение от DrOffset Посмотреть сообщение
Не разглядел сперва
Ну бывает, сам не безгрешен
Jupiter
17.03.2014, 23:09
  #17

Не по теме:

Цитата Сообщение от DrOffset Посмотреть сообщение
приводит к вычету из зарплаты, если обнаруживается на code review
у того кто проводил тех. собеседование

mustimur
17.03.2014, 23:13
  #18

Не по теме:

Jupiter,Насмешили... Спасибо даже не скажешь за Ваш пост, хоть так

MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
17.03.2014, 23:13     Утечка памяти
Еще ссылки по теме:

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

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

Или воспользуйтесь поиском по форуму:
DrOffset
17.03.2014, 23:13     Утечка памяти
  #19

Не по теме:

Jupiter, да этого вообще уволить надо

Yandex
Объявления
17.03.2014, 23:13     Утечка памяти
Ответ Создать тему
Опции темы

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