Question

我通常避免使用类指针，因为引用似乎更有效。但我最近被迫使用它们，因为它们提供了唯一（高效且简单）的解决方案，在windows api包装器中使用window id绑定函数。我创建了一个WinControl类的数组。在我的WinHandler类中，它处理WndProc（窗口过程）并将程序中使用的所有小部件添加到该数组中。

class WinControl   //These are not the entire classes, just the significant parts.
{
    public:
        int WinID;
        virtual void Click(void) = 0; //Pure Virtual Function.
}

class WinHandler
{ 
    WinHandler() : WCount(0) { }
    WinControl* WidgetSet[MAX_LENGTH];   // Or I can use an STL vector...
    int WCount;
    void AddWidget(Widget* w) { WCount++; WidgetSet[WCount] = w; }
}

然后我使用：

if (WidgetSet[i]->ID == LOWORD(wParam)) WidgetSet[i]->Click();

从长远来看，这种做法是否正常？由于实际存储在WidgetSet中的对象都是WinControl类的衍生物。任何人都可以提出更好的方法吗？

注意：我试图让我的问题尽可能清楚。如果您仍然无法得到我要求的内容，请发表评论，我将尝试详细说明问题。

Answer 1

您在窗口小部件集中存储的是指针。不是对象。您的设计存在的问题是，不清楚谁应该拥有该对象从而破坏它。

我会更改界面以明确所有权。

选项1：

WinHandler 不拥有小部件：

void AddWidget(Widget& w) { WCount++; WidgetSet[WCount] = &w; }

选项2：

WinHandler 取得小部件的所有权

void AddWidget(std::auto_ptr<Widget> w) { WCount++; WidgetSet[WCount].reset(w); }
std::auto_ptr<WinControl> WidgetSet[MAX_LENGTH];

选项2a：

（对于那些有较新编译器的人）
WinHandler 取得小部件的所有权

void AddWidget(std::unique_ptr<Widget>& w) { WCount++; WidgetSet[WCount] = std::move(w); }
std::unique_ptr<Widget> WidgetSet[MAX_LENGTH];

选项3：

（对于那些有较新编译器的人）
WinHandler 分享小部件的所有权

void AddWidget(const std::shared_ptr<Widget>& w) { WCount++; WidgetSet[WCount] = w; }
std::shared_ptr<Widget> WidgetSet[MAX_LENGTH];

Answer 2

解决方案没问题，但请注意内存泄漏：不要忘记删除所有内容：）

在c ++中对派生类使用基类指针有任何负面影响吗？

2 个答案:

选项1：

选项2：

选项2a：

选项3：