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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
fuelcs
12 / 12 / 0
Регистрация: 23.01.2013
Сообщений: 143
#1

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

11.02.2013, 14:07. Просмотров 331. Ответов 3
Метки нет (Все метки)

Добрый день.
Написал заготовку под программу. Возник вопрос: правильно ли выделяю-освобождаю память?
Программа работает корректно, но выдает лишь один ворнинг: 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;
}
Как вообще проверить правильность работы с памятью?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
11.02.2013, 14:07
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Выделение-освобождение памяти (C++):

Освобождение и выделение памяти - C++
Помогите разобраться с программой, где здесь выделение памяти (как я понял это new) и где освобождение памяти. #include &lt;iostream&gt; ...

Выделение и освобождение памяти - C++
Вот встретился с вопросом какой вариант является более правильным и почему. char *Buffer1 = (char *)malloc(65536) char *Buffer2 =...

выделение и освобождение памяти - C++
В моей программе есть следующий код: char* lexems; // массив указателей на символ char* lexem; // вспомогательная...

Выделение и освобождение памяти - C++
Доброго времени суток. Пишу класс по ООП, и назрел один вопрос, связанный с моментом выделения и освобождения памяти. class SOK { ...

Выделение и освобождение памяти в c++ - C++
Не давно наткнулся на такую замечательность : system(&quot;PAUSE&quot;);//1 раз int *p = new int; // память system(&quot;PAUSE&quot;);//2 раз ...

Освобождение и выделение динамической памяти - C++
Привет народ, такой вопрос: Допустим в main, указателю выделил память, всё разумеется нормально, читаю пишу всё ок, потом передал этот...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
vua72
415 / 415 / 85
Регистрация: 28.11.2010
Сообщений: 1,183
Завершенные тесты: 1
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
415 / 415 / 85
Регистрация: 28.11.2010
Сообщений: 1,183
Завершенные тесты: 1
11.02.2013, 16:50 #4
То на что ругается компилятор, рассматривается по ссылке. Мы никого не обманываем, просто правильно прописываем альтернативу и все, плюс у вас в объявляется шаблон, а память вы выделяете под дабл, попробуйте в основной программе поменять тип данных и посмотрите что у вас выйдет.
Контролировать можно, например valgrind, в Windows есть свои инструменты. Но хотя бы нужно для каждого new иметь парный delete и следить за правильным использованием указателей.
Можно косвенным путем, запускаете диспетчер задач и несколько раз память, если после запуска память не освобождается, а все время растет- значит есть проблемы. Можно прогнать программу через анализаторы, типа cppcheck, а также использовать стандартные контейнеры, которые создаются и удаляются
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
11.02.2013, 16:50
Привет! Вот еще темы с ответами:

выделение памяти и никогда не выполняете* ее освобождение - C++
Как это можно сделать написать программу с бесконечным циклом» в котором постоянно происходит выделение памяти и никогда не выполняете*...

Указатели на строки. Выделение и освобождение памяти - C++
Всем добрый день. Есть два указателя на char. Для первого выделяется память а у второго просто делаем копию с первого. Для 1-го...

Выделение и освобождение памяти в конструкторе и деструкторе - C++
Есть программа с классом содержащим обьекты векторы. Однако по кончанию программы выбивате ошибку _block_type_is_vali(phead-&gt;nblockuse)....

Выделение и освобождение памяти для локальных переменных функций - C++
Привет Столкнулся со следующей проблемой. Написал простенькое приложение: читаем 1 строку из файла на диске, записываем 1 строку в файл...


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
11.02.2013, 16:50
Ответ Создать тему
Опции темы

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