Строковые литералы внутри функций: автоматические переменные или выделенные в куче?

Строковые литералы будут помещены компилятором в сегмент инициализированных данных или текста (кода) вашего двоичного файла, а не в (выделенной во время выполнения) памяти или стеке. Таким образом, вы должны использовать указатель, поскольку вы собираетесь ссылаться на строковый литерал, который компилятор уже создал для вас. Обратите внимание, что изменение этого параметра (которое обычно требует изменения защиты памяти) изменит все способы использования этого литерала.

Может быть, причина сбоя в том, что вы не завершили строку нулем?

Изменение строкового литерала является неопределенным поведением и, скорее всего, является причиной сбоя в вашей программе (ISO C++: 2.13.4 / 2). Стандарт допускает преобразование строкового литерала в char* для обратной совместимости с C, и вы должны иметь это преобразование в своем коде только в том случае, если оно вам абсолютно необходимо.

Если вы хотите рассматривать строковый литерал как константу, вы можете изменить тип своего члена на const char *.

Если ваш дизайн требует, чтобы s мог быть изменен, я бы рекомендовал изменить его тип на std::string.

Спасибо, Коди и Ричард.

Я нашел причину ошибки. Это произошло потому, что я выполнял удаление объекта, который уже был удален. Я делал:

if (obj != NULL) delete obj;

Я изменил его на:

if (obj != NULL)
{
    delete obj;
    obj = NULL;
}

Изучение C++ определенно весело :)

Давайте посмотрим на ваши варианты. Вам также следует сделать несколько вещей:

    /*
     * Should initialize s to NULL or a valid string in constructor */
        MemoryLeak()
        {
            store();
        }

        void store()
        {
            // This does not need to be freed because it is a string literal
            // generated by the compiler.
            s = "Storing a string"; // ONE

            // Note this is allowed for backward compatibility but the string is
            // really stored as a const char* and thus unmodifiable. If somebody
            // retrieves this C-String and tries to change any of the contents the
            // code could potentially crash as this is UNDEFINED Behavior.

            // The following does need to be free'd.
            // But given the type of s is char* this is more correct.
            s = strdup("Storing a string");

            // This makes a copy of the string on the heap.
            // Because you allocated the memory it is modifiable by anybody
            // retrieving it but you also need to explicitly de-allocate it
            // with free()
        }

Вы используете C-Strings. Их не следует путать с C++ std :: string. C++ std :: string автоматически инициализируется пустой строкой. Любая выделенная память освобождается правильно. Его можно легко вернуть как в изменяемой, так и в неизменяемой версии. Также легко манипулировать (изменение усадки увеличения т.е.). Если вы увеличиваете C-String, вам необходимо перераспределить память и скопировать строку в новую память и т. д. (Это требует очень много времени и может привести к ошибкам).

Чтобы справиться с динамическим распределением вашего объекта, я бы узнал об интеллектуальных указателях. Подробнее об интеллектуальных указателях см. В этой статье. Умные указатели или кто владеет вами, детка

std::auto_ptr<MemoryLeak> obj(new MemoryLeak());

obj->store();
std::cout << obj->retrieve() << std::endl; // TWO

// No need to delete When object goes out of scope it auto deletes the memory.