Проблема с утечкой оперативной памяти

[User90]

При использовании данной программы для создание паролей при больших количестве вариантов переборов около 2*10 в 13 степени происходит непрерывный прирост оперативной памяти , что в конечном итоге приводит к аварийному завершению программы:

#include <iostream>
#include <algorithm>
#include <time.h>
#include <stdlib.h>
#include <fstream>

using std::cout;
using std::cin;
using std::endl;


class PassGen 
{

public:
    void displayMessage()
    {
        unsigned long long passLenght;
        unsigned long long numOfPasswords;
        char * filename = new char;

        cout << "Введите длину пароля для генерации: ";
        cin >> passLenght;
        cout << "Введите количество паролей для генерации: ";
        cin >> numOfPasswords;
        cout << "Будет сгенерировано паролей: " << numOfPasswords << "." << endl;
        cout << endl;
        cout << "Введите имя файла для записи: ";
        cin >> filename;

        std:: ofstream outFile(filename);

        for (unsigned long long k = 0; k < numOfPasswords; k++) 
		{
            for (unsigned long long i = 0; i < passLenght; ++i) 
			{    
                numOfchars(passLenght);
                passGenerator(passLenght);
                outFile << password [i];
            }
            outFile << endl;
			
        }
        outFile.close();

        cout << "Пароли успешно сгенерированы и записаны в файл " << filename << "" << endl;
		cin.get ();
		cin.get ();
    }

    void passGenerator(unsigned long long passLenght)
    {
        char* password = new char [passLenght];

        for (unsigned long long i = 0; i < numOfNumbers; ++i) 
		{
            password [i] = char(rand() % 10 + 48);
        }
        for (unsigned long long i = numOfNumbers; i < numOfNumbers + numOfBigchars; ++i) 
		{
            password [i] = char(rand() % 26 + 65);
        }
        for (unsigned long long i = numOfNumbers + numOfBigchars; i < passLenght; ++i) 
		{
            password [i] = char(rand() % 26 + 97);
        }
         std::random_shuffle(password, password + passLenght);
		 
    }

    void numOfchars(unsigned long long passLenght)
    {
        numOfSmallchars = rand() % passLenght;
        unsigned long long charRandEnd = passLenght - numOfSmallchars;
        numOfBigchars = rand() % charRandEnd;
        numOfNumbers = passLenght - numOfSmallchars - numOfBigchars;
		
    }

private:
    unsigned long long numOfSmallchars;
    unsigned long long numOfBigchars;
    unsigned long long numOfNumbers;
    

};

int main()
{
    setlocale(LC_ALL, "Russian");
    time_t srand(time(NULL));
    PassGen * pass = new PassGen;
    pass->displayMessage();
    delete [] pass;
    return 0;
}

Подскажите как лучше оптимизировать код и где и как необходимо еще очистить динамическую память , что б это не повлияло на ее работу?

[Dmitry_B]

Непонятно, что с именем файла: резервируется только 1 символ? А где освобождается использованная память?

[User90]

вообще это не мой исходный код, я его немного изменил, delete [] pass но этого не достаточно....но потери памяти в есть...

[Rififi]

User90

вообще это не мой исходный код

тогда надо вырвать руки тому, кто этот код написал.
с таким подходом C++ противопоказан.

не говоря о том что класс - одна большая фикция. Ничего полезного не далет, а то что делает - криво и через (_|_) неправильно.

[User90]

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

User90

вообще это не мой исходный код

тогда надо вырвать руки тому, кто этот код написал.
с таким подходом C++ противопоказан.

не говоря о том что класс - одна большая фикция. Ничего полезного не далет, а то что делает - криво и через (_|_) неправильно.

как убрать потери в памяти ?

[Rififi]

User90

переписать код так, чтобы там не было ни одного new/delete в явном виде.
К примеру, зачем ты создаешь инстанс PassGen динамически?
бездуменое применение возможностей C++ - верный путь к провалу (:

[Dmitry_B]

Ещё раз: где возвращается вот эта память:
char * filename = new char; ?
Надо в деструкторе освободить.

[Ezhik Kihze]

Гораздо интересней, где овобождается вот эта память:

char* password = new char [passLenght];

[Smitt&Wesson]

Да не парьтесь Вы с этим (если у Вас не 386-я машина)
Утечку Вашей программы, устранит супервизор Винды, после закрытия приложения (или деструктор по-умолчанию).

[Ezhik Kihze]

Угу, а потом смотришь на прогу - интрефейс г, не х не делает, а как "поработает" несколько минут... так и 8 Гб оперативы съест и еще свопит.