我的指向 std::string 对象的指针列表无法正常工作，我不明白为什么

Question

你能帮我弄清楚这段代码吗： onCreate(xMsg) 由 WindowProcedure 在 WM_NCCREATE 消息上调用。问题出在迭代器解引用内。我似乎无法将迭代器设置为列表的 begin() ，然后获取迭代器的内容，它应该是指向可以与 MessageBox 中的 ->c_str() 一起使用的字符串的指针。

class MyController
{
public:
    MyController() { }
    ~MyController() { }
    void onCreate(xMessage *msg);

protected:
    std::list<std::string *> m_stringlist;
    std::list<std::string *>::iterator m_stringlistiter;
};

void onCreate(xMessage *msg)
{
    std::string *first_string = new std::string("Hello world");
    m_stringlist.push_back(first_string);
    // This is where my iterator comes into play and it doesn't seem to do the trick.

    m_stringlistiter = m_stringlist.begin();
    MessageBox(NULL, (*m_stringlistiter)->c_str(), "Title", MB_OK);
}

Answer 1

我们没有给出完整的示例，但我假设您在迭代器的创建和使用之间进行其他操作。

在对容器进行各种操作后，不保证迭代器有效。您不应该保存它们。

我敢打赌这会起作用吗？

class MyController
{
public:
    MyController() : title("Hello World") {}
    void onCreate(xMessage *msg);
    {
         MessageBox(NULL, title.c_str(), "Title", MB_OK);
    }
protected:
    std::string title;  // generic lists of strings with 
                        // magic indexes are hard to maintain.
};

这也更容易阅读和调试，其成本比软件开发的任何其他部分都要高得多。

大多数情况下，您应该将迭代器用作临时对象。它们很少扩展到函数/方法的范围之外。

struct some_struct {

  some_struct() {
    gui_text.push_back( std::string("Hello World!") );
  }
  void some_function() {
    std::list<std::string>::iterator iter = gui_text.begin();
    for (; iter != gui_text.end(); iter++) {
      std::cout << *iter << std::endl;
    }
  }

  std::list<std::string> gui_text;
};

从容器中添加、删除等操作将使迭代器失效。因此，只需在每个点重新创建迭代器即可。字符串和迭代器非常轻量级，因此您无需担心以这种方式进行优化。真的。