0

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

word word; // этот объект хранит в себе слово, транскрипцию и сид.

generator assembler;

int i = 0; 

do { assembler (word, ++i); } while (true); // (word &word, int seed);

Он бесконечно генерирует слова в объект word, замещая старые данные.

В диспетчере задач я обнаружил утечку памяти (~33 кб/c) и довольно высокую загруженность CPU (~16%; AMD FX-8320, 8-и ядерный).

Везде, где была выделена динамическая память, использовался delete.

Может ли утекать память от таких методов?

int foo (std::vector<std::wstring> exceptions)
{
    ...

    return 0;
}

Или нужно делать так?

int foo (std::vector<std::wstring> exceptions)
{
    ...

    delete[] exceptions;
    return 0;
}

Ещё у меня практически все методы возвращают 0 в никуда.

int foo () { return 0; }

foo (); // т.е. значение, возвращаемое функцией, никуда не записывается

А вот по поводу высокой загрузки CPU предположений нет.

  • Из-за чего утекает оперативная память?

  • Что вызывает повышенную загрузку CPU?

2
  • 2
    используйте valgrind для обнаружения утечек памяти. И в с++ принята концепции RAII - лучше используйте выделение в конструкторе, а удаление в деструкторе. 18 июл 2018 в 9:33
  • Вы бы лучше не использовали new и delete без крайней необходимости. 2018 год на дворе, у нас давно есть стандартные контейнеры и умные указатели, в 99% случаев их достаточно... С ними получить утечку намного сложнее. 18 июл 2018 в 16:30

3 ответа 3

4

У Вас в цикле выполняется условие:

do { assembler (word, ++i); } while (true); // (word &word, int seed);

Выполняется на столько быстро, на сколько это возможно используя все ресурсы одного ядра, вот отсюда и нагрузка процессора ~16% (100% / 8 ядер).

По поводу утечек в диспетчере задач: это не самый лучший способ проверять на утечки памяти, для этого нужно использовать такие инструменты как Visual Leak Detector (для MSVC) или аналоги

3

Вызывать delete[] явно имеет смысл только для того, что было создано явно же через new[]. Класс std::vector сам управляет памятью, поэтому никакие delete к нему применять нельзя. Более того delete, delete[] требуют операнда типа указатель, т.е. предложенный вариант delete[] exceptions; выглядит вовсе абсурдным.

Для того чтобы выявлять утечки памяти следует использовать специализованные программы или встроенные в IDE средства. Гуглить по "memory leaks detector". Опираться на информацию обычного диспетчера задач ОС недостаточно.

Ну, а про 16% это скорее всего говорит о том, что полностью загружено одно ядро вашего процессора из-за цикла do/while без каких-либо пауз (sleep). Хотя при 8 ядрах скорее должно быть 12.5%.

3
  • Нет, конкретно сама программа нагружает до 16%, а не на ~12%.
    – Mournehowl
    18 июл 2018 в 9:36
  • @Mournehowl это скорее всего специфика подсчёта ОС, хотя может у Вас ещё есть потоки в программе - один отъедает 12.5%, а ещё 3.5% другой.
    – αλεχολυτ
    18 июл 2018 в 9:41
  • Нет, программа работает в одном потоке.
    – Mournehowl
    18 июл 2018 в 9:46
2

std::vector сам управляет памятью для своих внутренних элементов и в случае std::vector<std::wstring> все необходимые операции освобождения памяти происходят у него в деструкторе.

В C++ вообще стоит избегать ручного управления памятью, вместо этого необходимо использовать принцип RAII, в частности - умные указатели (unique_ptr, shared_ptr, weak_ptr).

Высокую нагрузку на CPU вызывает бесконечный цикл. Кроме того, чтобы избежать копирования аргументов функций, рекомендуется передавать константные ссылками (const std::vector<std::wstring> &exceptions) и при необходимости использовать перемещение.

9
  • Т.е. вероятнее всего я забыл где-то написать delete?
    – Mournehowl
    18 июл 2018 в 9:33
  • @Mournehowl, вероятнее всего. Если в word у вас есть сырой указатель на буфер с данными (T*), то попробуйте заменить его на std::unique_prt<T>. Но все равно это сейчас гадание на кофейной гуще, будет лучше, если вы добавите в вопрос листинги word и generator
    – free_ze
    18 июл 2018 в 9:36
  • Извиняюсь за глупый вопрос, что такое листинг? Я новичок.
    – Mournehowl
    18 июл 2018 в 9:39
  • @Mournehowl, исходники, в данном случае интересует объявление типа word и имплементация generator.
    – free_ze
    18 июл 2018 в 9:41
  • Ещё раз извиняюсь, но я не знаю что такое имплементация. Если это просто код класса generator, то я не смогу его здесь вставить, там 1500 строк. Или вы про хэдер generator'а?
    – Mournehowl
    18 июл 2018 в 9:45

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.