在 C++ 类中修改指向 std::string 的指针

Question

我刚开始学习C++，一方面使用gnu 编译器，另一方面使用Visual C++ 和Intel compiler，遇到了不一致。下面的例子定义了一个类Person，它带有一个指向std::string Name的指针。在方法Person::set 中，字符串是按值分配的。我确信更好的方法是使用指针，但这不是这里的问题。

#include <iostream>
#include <string>

class Person
{
   std::string *Name;
public:
   Person(std::string *n);  //Constructor
   void print();
   void set(std::string n);
};

Person::Person(std::string *n) : Name(n) //Implementation of Constructor
{
}

// This method prints data of person
void Person::print()
{
    std::cout << *Name << std::endl;
}


void Person::set(std::string n)
{
    Name = &n;
}

int main()
{
    std::string n("Me");
    std::string n2("You");
    Person Who(&n);

    Who.print();
    Who.set(n2);
    Who.print();


    return 0;
}

gnu 编译器如我所料给出以下结果：

Me
You

但是Visual C++ 和Intel 编译器会导致未定义的行为。我猜问题是Person::set 中复制变量n 的生命周期。为什么使用gnu编译器完成Person::set后仍然可用，而使用Visual C++和Intel编译器则不可用？

Answer 1

您的 Set 方法设置了未定义的行为，因为您正在获取局部变量的地址，然后在另一个范围内使用它：

void Person::set(std::string n)
{
    Name = &n; // n is a local variable
}

任何在Person::set 之外取消引用Name 的尝试，就像您在Person::print() 中所做的那样，都是未定义的行为。

您尝试过的所有编译器的行为都与未定义的行为兼容，因为一切都是。

Answer 2

该代码已完全损坏且无法使用。在Person::set 内部，n 是一个局部变量，所以它的地址&n 在该函数之外是没有意义的。存储它是没有意义的，以后使用它是未定义的行为。

---

这是用现代 C++ 编写课程的正确方法：

class Person
{
   std::string Name;

public:
   explicit Person(std::string n) : Name(std::move(n)) { }

   void set(std::string n) { Name = std::move(n); }

   void print() const { std::cout << Name << '\n'; }
};

Answer 3

“更好的方法”并不是总是在类 C 语言中使用指针。触摸。

即使你正确地传递了一个指向 set 的指针：

void Person::set(std::string* n)
{
    Name = n; // Correctly saves
}

您不知道n 何时会被释放，因为它存储在Person class 之外。更好的方法是在Person 类中保留一份Name，并通过引用传入新字符串：

class Person
{
   std::string Name;
public:
   Person(const std::string &n);  //Constructor
   void print();
   void set(const std::string &n);
};

Person::Person(const std::string &n) : Name(n) //Implementation of Constructor
{
}

void Person::set(const std::string &n)
{
    Name = n; // Person class keeps its own copy of n
}