c++ - 功能结束后值被更改

我正在尝试设置需要在应用程序的另一个文件（层）中使用的数据。在调试过程中，我看到该值是第一次正确设置的。但是，当我尝试在函数后使用此set变量时，值将更改。我认为内存正在释放，导致变量重置。任何人都可以帮我做我想做的事。

void SetExpectedTabsData(_In_ PCWSTR tabUrls[], _In_ PCWSTR tabTitles[], _In_ UINT tabsCount, _In_ FakeCourier* courier)
{
    wchar_t jsonPerTab[256];
    wchar_t tabsDataJSON[c_jsonTabDataSize];
    // tabsDataJSON should be large enough to hold any data.
    StringCchPrintf(tabsDataJSON, c_jsonTabDataSize, L"\"tabs\":[");
    bool isActiveTab = true;
    // tabId starts from 1 and the tabIndex starts with 0. Manipulated the json string generation accordingly.
    for (unsigned int i = 1; i <= tabsCount; ++i)
    {
        StringCchPrintf(jsonPerTab, ARRAYSIZE(jsonPerTab), L"{\"id\":%i,\"index\":%i,\"windowId\":1,\"active\":%s,\"status\":\"complete\",\"title\":\"%s\",\"url\":\"%s\"}", i, (i - 1), isActiveTab ? L"true" : L"false", tabTitles[i - 1], tabUrls[i - 1]);
        StringCchCat(tabsDataJSON, c_jsonTabDataSize, jsonPerTab);
        isActiveTab = false;
        if (i != tabsCount)
        {
            StringCchCat(tabsDataJSON, c_jsonTabDataSize, L",");
        }
    }
    StringCchCat(tabsDataJSON, c_jsonTabDataSize, L"],");
    VERIFY_SUCCEEDED(courier->SetExpectedTabsData(tabsDataJSON));
}

需要设置数据的快递文件为

HRESULT FakeCourier::SetExpectedTabsData(_In_ wchar_t* tabsData)
{
  m_tabsData = tabsData;
  return S_OK;
}

请提出正确的做法。

最佳答案

变量tabsDataJSON在函数SetExpectedTabsData中是局部变量，因此在完成此函数后可以将其覆盖。并且由于它可能发生，所以它发生了。

调用FakeCourier::SetExpectedTabsData(tabsDataJSON)时，此函数仅记住指向该局部变量tabsDataJSON的指针。但是，在函数SetExpectedTabsData完成后，此指针立即无效。

要修复它，您需要修改函数FakeCourier::SetExpectedTabsData：将指针的复制替换为指向数据的复制。由于您的m_tabsData是wchar_t*，只需分配此字符串并将数据从tabsData复制到m_tabsData（了解有关wcsncpy和类似功能的信息）。

尝试这样的事情：

HRESULT FakeCourier::SetExpectedTabsData(_In_ wchar_t* tabsData)
{
  const size_t len = wcslen(tabsData);
  if (m_tabsData)
    delete m_tabsData;
  m_tabsData = new wchar_t[len + 1];
  wcsncpy(m_tabsData, tabsData, len + 1);
  return S_OK;
}

而且，您还需要在FakeCourier的析构函数中释放此内存（以避免内存泄漏）。并在构造函数（m_tabsData）中初始化m_tabsData = 0;