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

Выделение-освобождение памяти - C++

Восстановить пароль Регистрация
 
fuelcs
12 / 12 / 0
Регистрация: 23.01.2013
Сообщений: 143
11.02.2013, 14:07     Выделение-освобождение памяти #1
Добрый день.
Написал заготовку под программу. Возник вопрос: правильно ли выделяю-освобождаю память?
Программа работает корректно, но выдает лишь один ворнинг: warning C4715: 'equation<double>' : not all control paths return a value
Вот сам код:
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
70
71
72
73
74
75
76
77
78
#include <iostream>
 
using namespace std;
 
template <typename T>
void clear_memory (T** x)
{
    if (x!=NULL)
    {
        for (int i=0;i<2;i++)
        {
            if (x[i]!=NULL)
            {
                delete [] x[i];
            }
        }
        delete [] x;
    }
}
 
template <typename T>
T** equation (T d)
{
    if (d>0)
    {
        double** x = new double* [2];
        for (int i=0;i<2;i++)
        {
            x[i]=new double [1];
            x[i][0]=i+1;
        }
        return x;
    }
    if (d==0)
    {
        double** x = new double* [2];
        x[0]=new double [1];
        x[0][0]=1;
        x[1]=NULL;
        return x;
    }
    if (d<0)
    {
        double** x =NULL;
        return x;
    }
}//warning C4715: 'equation<double>' : not all control paths return a value
 
 
 
 
int main ()
{
    double d;
 
    cin >> d;
 
    double** x=equation(d);
 
    if (x!=NULL)
    {
        for (int i=0;i<2;i++)
        {
            if (x[i]!=NULL)
            {
                cout << x[i][0] << "\n";
            }
        }
    }
    else
    {
        cout << "Error" << "\n";
    }
    clear_memory(x);
 
 
    return 0;
}
Как вообще проверить правильность работы с памятью?
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
vua72
410 / 410 / 83
Регистрация: 28.11.2010
Сообщений: 1,158
11.02.2013, 15:25     Выделение-освобождение памяти #2
Компилятор вообще-то ругается на то, что не-void функция не возвращает значение, вернее не из всех if возвращает.

Добавлено через 8 минут
посмотрите здесь

Добавлено через 8 минут
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
template<typename T>
T** equation(T d) {
    if (d > 0) {
        double** x = new double*[2];
        for (int i = 0; i < 2; i++) {
            x[i] = new double[1];
            x[i][0] = i + 1;
        }
        return x;
    }
    else if (d == 0) {
        double** x = new double*[2];
        x[0] = new double[1];
        x[0][0] = 1;
        x[1] = NULL;
        return x;
    }
    else
        {
            double** x =NULL;
            return x;
        }
}
но правильней так
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
template<typename T>
T** equation(T d) {
    T ** x = NULL;
    if (d > 0) {
        x = new T*[2];
        for (int i = 0; i < 2; i++) {
            x[i] = new T[1];
            x[i][0] = i + 1;
        }
    }
    if (d == 0) {
        x = new T*[2];
        x[0] = new T[1];
        x[0][0] = 1;
        x[1] = NULL;
    }
    return x;
}
fuelcs
12 / 12 / 0
Регистрация: 23.01.2013
Сообщений: 143
11.02.2013, 16:43  [ТС]     Выделение-освобождение памяти #3
Как я понял вся загвоздка в том, что функция может возвращать значение NULL. И на это ругается компилятор. А то что вы написали, это мы просто обманываем компилятор?
Так что выходит, что функция не может возвращать нулевое значение?

И еще, вопрос по сути, я могу освобождать память в другой функции, не в той в которой память выделялась? Так как это я сделал в программе?

И можно как-то контролировать или мы правильно работаем с памятью?

Добавлено через 9 минут
С ворнингом кажется разобрался, проблема не в NULL, а в том что компилятор думает что функция может остаться рез ретурна.
Но вопрос с выделением-освобождением памяти остается....
vua72
410 / 410 / 83
Регистрация: 28.11.2010
Сообщений: 1,158
11.02.2013, 16:50     Выделение-освобождение памяти #4
То на что ругается компилятор, рассматривается по ссылке. Мы никого не обманываем, просто правильно прописываем альтернативу и все, плюс у вас в объявляется шаблон, а память вы выделяете под дабл, попробуйте в основной программе поменять тип данных и посмотрите что у вас выйдет.
Контролировать можно, например valgrind, в Windows есть свои инструменты. Но хотя бы нужно для каждого new иметь парный delete и следить за правильным использованием указателей.
Можно косвенным путем, запускаете диспетчер задач и несколько раз память, если после запуска память не освобождается, а все время растет- значит есть проблемы. Можно прогнать программу через анализаторы, типа cppcheck, а также использовать стандартные контейнеры, которые создаются и удаляются
Yandex
Объявления
11.02.2013, 16:50     Выделение-освобождение памяти
Ответ Создать тему
Опции темы

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