Как объединить строки и присвоить значение переменной в классе?

Question

[Сергей Сова]

У меня есть класс

class Tampler
{
public:
Tampler();
private:
const char* m_sMyString;
};

И есть набор строк

#define TLDR_EXT ".tldr"
const char* GetCurrentDir();
const char *filename = "filename";

Мне необходимо в конструкторе объединить полученные строки в одну и значение присвоить m_sMyString;
Как это сделать?

Answer 1

[EXL]

Простите, а в чём проблема? В зависимости от ситуации можете использовать как С++-подход:

Tampler()
{
        std::string str(GetCurrentDir());
        str = str + filename + TLDR_EXT;
        m_sMyString = strdup(str.c_str());
}

Так и C-подход:

Tampler()
{
        char *tempstr = (char *)malloc(strlen(GetCurrentDir()) + strlen(filename) + strlen(TLDR_EXT) + 1);
        strcpy(tempstr, GetCurrentDir());
        strcat(tempstr, filename);
        strcat(tempstr, TLDR_EXT);
        m_sMyString = tempstr;
}

Если я, конечно, правильно вас понял.

Comments

[jcmvbkbc]

@EXL в ваших примерах всё прекрасно, и инициализация указателя строкой, которая будет освобождена при выходе из конструктора, и переполнение буфера копированием строк в хвост строки, полученной strdup.

[AxisPod]

В 1 примере для последней строки юзать именно strdup. А во 2м, да, надо посчитать сначала размер, выделить, а уже только потом strcpy и пара strcat.

[EXL]

@jcmvbkbc @AxisPod

Поправил. Посмотрите пожалуйста, так правильнее?

[jcmvbkbc]

@EXL во втором примере внутри malloc нужно добавить '+ 1' для нуля в конце строки, а вместо m_sMyString = strdup(tempstr); free(tempstr); можно сделать просто m_sMyString = tempstr;

[EXL]

@jcmvbkbc
Большое спасибо за пояснения!

[Сергей Сова]

@EXL при выходе из конструктора tempstr не будет уничтожен?
В m_sMyString будет именно нужный текст?

[EXL]

@LestaD Не будет, так память выделена с помощью malloc.

[Сергей Сова]

@EXL огромное спасибо. Совсем забыл про malloc.

Answer 2

[AxisPod]

Поменять const char* на std::string и не париться.

Comments

[Сергей Сова]

при выходе из конструктора, std::string будет уничтожен