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

[Сtrl]

Да, просто использую их вместо malloc.

Вопрос был не в этом. Если у вас нет ни одного указателя-члена, то варианта два:

1. Вы используете глобальные переменные. Не делайте так.
2. Указатели передаются между функциями исключительно как аргументы и возвращаемые значения. В таком случае постарайтесь, чтобы каждая пара new/delete находилась в пределах одной функции.

Есть еще одно решение, железное - использовать умные указатели. Чем вам не подходит?

[Сtrl]

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

А каким образом? Путем исследования кода или можно как ни будь посмотреть (скажем программкой какой ни будь) ?

Есть и программы специальные, гляньте статью в Википедии (ссылка), но иногда они не в состоянии отследить все возможные пути выполнения программы - тогда следует искать утечки вручную. Сначала выделять возможные действия, в ходе которых из программы утекает память, а затем пошаговым отладчиком сверять вызовы new/delete.

[YourLastSong]

Вы используете глобальные переменные

Да, в качестве глобальной переменной используется указатель.

Умные указатели использовать не получится, т.к. мне необходимо использовать данные переменные в разных функциях.

[Сtrl]

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

Да, в качестве глобальной переменной используется указатель.

Паттерн Singleton вам в помощь. Никогда не используйте глобальные переменные.

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

Умные указатели использовать не получится, т.к. мне необходимо использовать данные переменные в разных функциях.

И что? Поищите что-нибудь про boost::shared_ptr - этот класс позволяет нескольким указателям ссылаться на один объект.

[YourLastSong]

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

Паттерн Singleton вам в помощь. Никогда не используйте глобальные переменные.

Почему?

И что? Поищите что-нибудь про boost::shared_ptr - этот класс позволяет нескольким указателям ссылаться на один объект.

Передавать его в качестве аргумента всем функциям?

[Сtrl]

Почему?

Ну, для начала, из-за возможных утечек памяти. (Тут должен быть закадровый смех).

Передавать в качестве аргумента всем функциям?

Как вам угодно. Суть в том, что вы сможете вообще забыть об освобождении памяти - умный указатель все сделает за вас (при грамотном использовании).

[f.hump]

2 YourLastSong
А как диспетчер задач помогает в идентификации утечек памяти? (просто любопытно)

2 Сtrl
А что не так с глобальными переменными? Почему нельзя?

[YourLastSong]

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

Ну, для начала, из-за возможных утечек памяти. (Тут должен быть закадровый смех).

Каким образом они будут возникать? Можно подробнее, пожалуйста?

Как вам угодно. Суть в том, что вы сможете вообще забыть об освобождении памяти - умный указатель все сделает за вас (при грамотном использовании).

Передавать в функции будет очень неудобно.

А как диспетчер задач помогает в идентификации утечек памяти? (просто любопытно)

Смотрю на колонку memory. Или так делать не надо?

[Сtrl]

Сообщение от f.hump

А как диспетчер задач помогает в идентификации утечек памяти? (просто любопытно)

Он показывает количество используемой памяти. Если это самое количество стремительно увеличивается - утечка.

Сообщение от f.hump

А что не так с глобальными переменными? Почему нельзя?

Ну, слово "нельзя" - глупое. Просто глобальные переменные - очень плохая практика, как goto. Есть более надеждные альтернативы, которые дают тот же результат (как, к примеру, паттерн Singleton).

[Сtrl]

Каким образом они будут возникать? Можно подробнее, пожалуйста?

Ну у вас точно как-то возникают.

Передавать в функции будет очень неудобно.

А если заменить функции на функторы? Я, конечно, не знаю всей ситуации, но при использовании функторов есть возможность создавать переменные, общие для нескольких функций. Например, у нас три функции - A, B и C. Функции A и B вызываются только из C, и при этом используют указатель p. Код примерно такой:

int* p //глобальная переменная
void A(int x) {cout << x + *p; delete p;}
void B(float y) {cout << y - *p; delete p;}
void C(int x, float y)
{
  p = new int(3);
  A(x); B(y);
}

Заменим на функтор, получится что-то такое:

class FunctorC
{
public:
  FunctorC() : p(0) {p = new int(3);}
  FunctorC(const FunctorC&right) : p(0) {p = new int(*right.p);}
  FunctorC& operator=(const FunctorC&right) {delete p; p = new int(*right.p); return (*this);}
  ~FunctorC() {delete p;}
  void operator()(int x, float y) {A(x); B(y);}
protected:
  void A(int x) {cout << x + *p;}
  void B(float y) {cout << y - *p;}
  int* p;
};