将智能指针定义为类的成员变量

Question

标题：

CChristianLifeMinistryHtmlView m_pHtmlView  = nullptr;

来源：

m_pHtmlView = new CChristianLifeMinistryHtmlView();

---

尝试将其更改为使用智能指针。我可以这样做（在OnInitDialog 内）：

auto m_pHtmlView2 = std::make_unique<CChristianLifeMinistryHtmlView>;

但我不知道如何将智能指针定义为CDialog 类的成员变量。我做不到：std::unique_ptr m_pHtmlView2。

---

我看到了这个讨论 (Using smart pointers as a class member) 并基于此我在标题中尝试了这个：

//CChristianLifeMinistryHtmlView *m_pHtmlView;
std::unique_ptr<CChristianLifeMinistryHtmlView> m_pHtmlView;

但这不会编译：

6>C:\Program Files\Microsoft Visual Studio\2022\Preview\VC\Tools\MSVC\14.30.30704\include\memory(3087,1): error C2248: 'CChristianLifeMinistryHtmlView::~CChristianLifeMinistryHtmlView': cannot access protected member declared in class 'CChristianLifeMinistryHtmlView'
6>D:\My Programs\2022\MeetSchedAssist\Meeting Schedule Assistant\ChristianLifeMinistryHtmlView.h(104): message : compiler has generated 'CChristianLifeMinistryHtmlView::~CChristianLifeMinistryHtmlView' here
6>D:\My Programs\2022\MeetSchedAssist\Meeting Schedule Assistant\ChristianLifeMinistryHtmlView.h(22): message : see declaration of 'CChristianLifeMinistryHtmlView'
6>C:\Program Files\Microsoft Visual Studio\2022\Preview\VC\Tools\MSVC\14.30.30704\include\memory(3085): message : while compiling class template member function 'void std::default_delete<CChristianLifeMinistryHtmlView>::operator ()(_Ty *) noexcept const'
6>        with
6>        [
6>            _Ty=CChristianLifeMinistryHtmlView
6>        ]
6>C:\Program Files\Microsoft Visual Studio\2022\Preview\VC\Tools\MSVC\14.30.30704\include\memory(3195): message : see reference to function template instantiation 'void std::default_delete<CChristianLifeMinistryHtmlView>::operator ()(_Ty *) noexcept const' being compiled
6>        with
6>        [
6>            _Ty=CChristianLifeMinistryHtmlView
6>        ]
6>C:\Program Files\Microsoft Visual Studio\2022\Preview\VC\Tools\MSVC\14.30.30704\include\memory(3122): message : see reference to class template instantiation 'std::default_delete<CChristianLifeMinistryHtmlView>' being compiled
6>D:\My Programs\2022\MeetSchedAssist\Meeting Schedule Assistant\AvailableBrothersReportPreview.h(52): message : see reference to class template instantiation 'std::unique_ptr<CChristianLifeMinistryHtmlView,std::default_delete<CChristianLifeMinistryHtmlView>>' being compiled

---

更新

根据我现在拥有的 cmets 中的建议：

• 标题：

std::unique_ptr<CChristianLifeMinistryHtmlView> m_pHtmlView;

• 来源（OnInitDialog）：

m_pHtmlView = std::make_unique<CChristianLifeMinistryHtmlView>();
if (m_pHtmlView != nullptr)
{
    m_pHtmlView->Create(nullptr, nullptr, AFX_WS_DEFAULT_VIEW,
                            m_rctPreviewHtml, this, 0);
    m_pHtmlView->ShowWindow(SW_SHOWNORMAL);
    if(CMeetingScheduleAssistantApp::WaitForFileToBeReady(m_strTempHtmlFile))
        m_pHtmlView->Navigate2(m_strTempHtmlFile, 0, nullptr);
}

它符合并有效。我的弹出对话框显示并且CHtmlView 派生控件可见。凉爽的。但是当我单击确定关闭对话框时，我得到一个异常：

我们如何解决这个问题？

Answer 1

CHtmlView派生自CFormView -> CView，在CView::PostNcDestroy和delete this;中删除自己，所以内存已经被管理了。

将new替换为std::make_unique，并立即调用release()，因为您不希望unique_ptr再删除它。

//m_pHtmlView = new CChristianLifeMinistryHtmlView();
m_pHtmlView = std::make_unique<CChristianLifeMinistryHtmlView>().release();

如果您在原始代码中编写了new 和delete（这也意味着您至少覆盖了CMyHtmlView::PostNcDestroy），那么unique_ptr 可以用来替换new 和delete。

在这种情况下，原始代码中只有 new。你不希望unique_ptr 管理delete