c++ - 在函数中创建类实例并稍后使用

Question

我可以这样做吗：

static Toggle GetAutoUpdatedToggle(DWORD key, bool initialState = false)
{
    Toggle tempToggle(key, initialState);
    autoUpdateToggles.push_back(tempToggle); //This is static member - std::vector<Toggle>
    return tempToggle;
}

而且我以后也会这样使用它：

void Toggle::UpdateAllFromFactory() //This is static function
{
    for each (Toggle toggle in autoUpdateToggles)
    {
        toggle.Update();
    }
}

这是一种好方法吗？

---

更新 1 - 在您提出建议之后：

static Toggle* GetAutoUpdatedToggle(DWORD key, bool initialState = false)
{
    Toggle *pToggle = new Toggle(key, initialState);
    m_autoUpdateToggles.push_back(pToggle);
    return pToggle;
}

void Toggle::UpdateAllFromFactory()
{
    for (std::vector<Toggle*>::iterator it = m_autoUpdateToggles.begin(); it < m_autoUpdateToggles.end(); it++)
    {
        (*it)->Update();
    }
}

Answer 1

不，这不是一个好方法，因为你传递了Toggle的副本：

• GetAutoUpdatedToggle 返回它刚刚推入vector 的Toggle 的副本。这本身并不是一件错误的事情，但是调用者可能对返回的切换进行的任何操作都不会反映在您推送到 vector 上的操作上
• for 循环遍历vector 的元素，创建一个副本以在循环主体内使用。除非Toggle 本身具有类似指针的语义，否则Update() 操作不会反映在vector 内的Toggle 对象上。

要解决此问题，请让 GetAutoUpdatedToggle 返回一个 reference 到它刚刚推送到 vector 上的 Toggle 对象，并使用 vector<Toggle>::iterator 对象遍历存储的切换。这将允许您对实际对象进行操作，而不是对其副本进行操作。

Answer 2

您的静态函数返回Toggle 的副本。 .push_back 还将创建切换的副本。因此，您返回的Toggle 不在autoUpdateToggles 中，以后无法更新。

Toggle myToggle = GetAutoUpdatedToggle(key);
/* ... */

Toggle alternativToggle = myToggle;
// alternativToggle == myToggle is true
Toggle::UpdateAllFromFactory();

// alternativToggle == myToggle is still true

另请注意，如果您不使用迭代器，您当前的 Toggle::UpdateAllFromFactory(); 实现会使用 Toggles 的副本。

您必须为您的Toggle 对象提供一个句柄。这可以是一个简单的指针或任何其他在复制时不会丢失特定Toggle 标识的对象。