Работа с памятью

@unylkun · Регистрация: 31.12.2016

Студворк — интернет-сервис помощи студентам

Форумчане, доброго времени суток.
Заранее извиняюсь за нубские вопросы, но должен разобраться в теме.

Суть такова: написал class String, в нем реализовал метод превращения двух строк в одну. Попутно ради практики использовал указатель this, и вроде понял, для чего он нужен - для того, чтобы компилятор не путал аргументы, передаваемые в метод и то, что находится в полях класса (поправьте, если ошибаюсь). Непосредственно вопрос: У меня в коде 2 раза выделяется память, освобождается же 1, в деструкторе прописана команда. Так и должно быть, или это не норма? Если норма - прошу объяснить почему. Так же буду крайне признателен за конструктивную критику кода и самой реализации.

C++

#include <iostream>
#include <cstring>
using namespace std;
 
 
class String
{
    private:
        char *str;
        int size;
    public:
        String();
        String(char *str);
        char* append(char *str); //Метод, объединяющий две строки в одну
        ~String();
};
 
String::String()
{
    str = NULL;
    size = 0;
    //cout << "Сработал конструктор по умолчанию." << endl;
}
 
String::String(char *str)
{
    size = strlen(str);
    this->str = new char[size]; //Выделяем память
    for(int i(0); i < size; i++)
    {
        this->str[i] = str[i];
    }
    //cout << "Сработал конструктор с параметром." << endl;
}
 
char* String::append(char *str)
{
    char *buff = new char[size];
    for(int i(0); i < size; i++)
    {
        buff[i] = this->str[i];
    }
    
    int n = size;
    size = size + strlen(str);
    this->str = new char[size]; //Выделяем память второй раз
    
    for(int i(0); i < n; i++)
    {
        this->str[i] = buff[i];
    }
    
    for(int i(0); i < size; i++)
    {
        this->str[i + n] = str[i];
    }
    
    delete []buff;
 
    return this->str;
}
 
String::~String()
{
    delete []str;   //освобождаем
    //cout << "Сработал деструктор" << endl;
}
 
int main()
{
    
    setlocale(LC_ALL,"rus");
    String str_1("Hello, ");
    cout << str_1.append("World!");
}

@likehood · 02.02.2017, 21:24

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

У меня в коде 2 раза выделяется память, освобождается же 1, в деструкторе прописана команда.

Я вижу два вызова delete: в 58 и 65 строках.

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

Так же буду крайне признателен за конструктивную критику кода и самой реализации.

На сколько я вижу, в конце строки не хранится нулевой байт. А значит, выдавать такую строку во внешний мир бессмысленно/небезопасно. В частности, в 74 строке кроме "Hello, world!" будет напечатан какой-то мусор.
Поэтому лучше потратить лишний байт и записать в него признак конца строки.

@unylkun · 02.02.2017, 23:40 **[ТС]**

Значит я не совсем правильно выразился, память выделяется еще в 38 строке, получается 3 раза. Я просто не акцентировал на этом внимание, т.к освобождается она сразу в этом методе. Интересует переменная str, для неё я выделил память в 28 и 46 строках. Потом она освобождается в деструкторе один раз. Это норма?))

За замечание спасибо.

UPD: Нулевой байт нужно добавить в вызов метода .append в 74 строке? (Присобачить его к World?) К слову, когда компилировал код, выводилась только строка, никакого мусора

@likehood · 03.02.2017, 11:27

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

память выделяется еще в 38 строке, получается 3 раза

Действительно, что-то я не заметил в первый раз, невнимательно смотрел.
Теперь заметил, что в 46 строке будет утечка памяти: мы меняем указатель this->str, не удалив память, на которую он указывал.

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

выводилась только строка, никакого мусора

Тут уж как повезёт. Если в конце программы добавить строку cout << str_1.append(" Another string.");, то у меня программа вообще падает.

Привожу свою реализацию, не меняющую интерфейс класса:

C++

#include <iostream>
#include <cstring>
 
using namespace std;
 
class String
{
private:
    char *str;
    size_t size;
 
public:
    String();
    String(char *s);
    char* append(char *s); //Метод, объединяющий две строки в одну
    ~String();
};
 
String::String() : str(nullptr), size(0)
{
}
 
String::String(char *s)
{
    size = strlen(s);
    str = new char[size+1]; //Выделяем память
    // копируем s -> str, включая нулевой байт
    memcpy(str, s, size+1);
}
 
char* String::append(char *s)
{
    size_t n1 = size, n2 = strlen(s);
    char *newStr = new char[n1+n2+1];
 
    memcpy(newStr, str, n1);
    memcpy(newStr+n1, s, n2+1);
    size = n1+n2;
 
    delete[] str;
    return str = newStr;
}
 
String::~String()
{
    delete[] str;   //освобождаем
}
 
int main()
{
    setlocale(LC_ALL,"rus");
    String str_1("Hello, ");
    cout << str_1.append("World!") << endl;
    cout << str_1.append(" Another string.") << endl;
}

@likehood 1272 / 1029 / 470 Регистрация: 25.12.2016 Сообщений: 3,333
	02.02.2017, 21:24	2
	Сообщение от unylkun У меня в коде 2 раза выделяется память, освобождается же 1, в деструкторе прописана команда. Я вижу два вызова delete: в 58 и 65 строках. Сообщение от unylkun Так же буду крайне признателен за конструктивную критику кода и самой реализации. На сколько я вижу, в конце строки не хранится нулевой байт. А значит, выдавать такую строку во внешний мир бессмысленно/небезопасно. В частности, в 74 строке кроме "Hello, world!" будет напечатан какой-то мусор. Поэтому лучше потратить лишний байт и записать в него признак конца строки. 1

@unylkun 1 / 1 / 1 Регистрация: 31.12.2016 Сообщений: 8
	02.02.2017, 23:40 [ТС]	3
	Значит я не совсем правильно выразился, память выделяется еще в 38 строке, получается 3 раза. Я просто не акцентировал на этом внимание, т.к освобождается она сразу в этом методе. Интересует переменная str, для неё я выделил память в 28 и 46 строках. Потом она освобождается в деструкторе один раз. Это норма?)) За замечание спасибо. UPD: Нулевой байт нужно добавить в вызов метода .append в 74 строке? (Присобачить его к World?) К слову, когда компилировал код, выводилась только строка, никакого мусора 0

	03.02.2017, 11:27

Опции темы

Работа с памятью

Решение