{"id":106362,"date":"2022-03-18T07:00:00","date_gmt":"2022-03-18T14:00:00","guid":{"rendered":"https:\/\/devblogs.microsoft.com\/oldnewthing\/?p=106362"},"modified":"2023-11-12T17:32:27","modified_gmt":"2023-11-13T01:32:27","slug":"20220318-00","status":"publish","type":"post","link":"https:\/\/devblogs.microsoft.com\/oldnewthing\/20220318-00\/?p=106362","title":{"rendered":"Making our multiple-interface query more C++-like, part 2"},"content":{"rendered":"

Last time, we wrote a C++-style wrapper around Co\u00adCreat\u00adInstance\u00adEx<\/code><\/a> that treats all of the interfaces as optional. But probably, you are issuing the multiple-interface query in the case where some or perhaps even all of the interfaces are required.<\/p>\n

auto [widget, objectWithSite, persistFile] =\r\n    CreateInstanceMultiQI<IWidget, IObjectWithSite, IPersistFile>\r\n        (CLSID_Widget, nullptr, CLSCTX_LOCAL_SERVER);\r\n<\/pre>\n
The IWidget<\/code> is required. Without that, you have nothing. But maybe the IObjectWithSite<\/code> is optional: If the widget doesn’t support IObjectWithSite<\/code>, we just won’t set a site on it.<\/p>\n
We need a way to tell our Create\u00adInstance\u00adMulti\u00adQI<\/code> function that some of the interfaces are required and others are optional.<\/p>\n
I’m going to pull a dirty trick. and instead of creating my own marker, I’m going to reuse an existing template from the standard library, since it has such a great name: std::optional<\/code>. I’m going to say that all of the interfaces are required by default, but you can wrap them inside a std::optional<\/code> to say that they’re optional.<\/p>\n
auto [widget, objectWithSite, persistFile] =\r\n    CreateInstanceMultiQI<\r\n        IWidget,\r\n        std::optional<IObjectWithSite><\/span>,\r\n        IPersistFile>\r\n        (CLSID_Widget, nullptr, CLSCTX_LOCAL_SERVER);\r\n<\/pre>\n
Most of the stuff we wrote last time still works. We just need to add a validation step that verifies that all of the required interfaces were successfully obtained.<\/p>\n
template<typename Interface>\r\nstruct multiqi_traits\r\n{\r\n    using type = Interface;\r\n    static constexpr bool is_required = true;\r\n};\r\n\r\ntemplate<typename Interface>\r\nstruct multiqi_traits<std::optional<Interface>>\r\n{\r\n    using type = Interface;\r\n    static constexpr bool is_required = false;\r\n};\r\n\r\ntemplate<typename Interface>\r\nusing multiqi_traits_com_ptr =\r\n    wil::com_ptr<typename multiqi_trauts<Interface>::type>;\r\n<\/pre>\n
The multiqi_traits<\/code> template traits type assumes that every interface is required. The specialization unwraps any interface that is wrapped inside a std::optional<\/code> and remembers that it is not required. For convenience, we also define a multiqi_traits_com_ptr<\/code> that represents the final com_ptr<\/code> we want to return.<\/p>\n
We can use this traits type to modify our existing function to throw if any required interface was not obtained:<\/p>\n
template<typename... Interfaces, std::size_t... Ints>\r\nauto CreateInstanceMultiQIWorker(\r\n    REFCLSID clsid, IUnknown* punkOuter,\r\n    DWORD clsctx, MULTI_QI* mqi,\r\n    std::index_sequence<Ints...>)\r\n{\r\n    THROW_IF_FAILED(\r\n        CoCreateInstanceEx(clsid, punkOuter, clsctx,\r\n                           sizeof...(Interfaces), mqi));\r\n\r\n    std::tuple<multiqi_traits_com_ptr<Interfaces><\/span>...> t;\r\n    ((std::get<Ints>(t).attach(\r\n        static_cast<typename multiqi_traits<Interfaces>::type<\/span>*>\r\n        (mqi[Ints].pItf))), ...);\r\n\r\n    ([&] {                                                      <\/span>\r\n        if constexpr (multiqi_traits<Interfaces>::is_required) {<\/span>\r\n            THROW_IF_FAILED(mqi[Ints].hr);                      <\/span>\r\n        }                                                       <\/span>\r\n    }(), ...);                                                  <\/span>\r\n\r\n    return t;\r\n}\r\n\r\ntemplate<typename... Interfaces>\r\nstd::tuple<multiqi_traits_com_ptr<Interfaces><\/span>...>\r\nCreateInstanceMultiQI(\r\n    REFCLSID clsid, IUnknown* punkOuter,\r\n    DWORD clsctx)\r\n{\r\n    MULTI_QI mqi[] = {\r\n        MULTI_QI{ &__uuidof(typename multiqi_traits<Interfaces>::type<\/span>),\r\n          nullptr, 0 }...\r\n    };\r\n\r\n    return CreateInstanceMultiQIWorker<Interfaces...>\r\n        (clsid, punkOuter, clsctx, mqi,\r\n         std::index_sequence_for<Interfaces...>{});\r\n}\r\n<\/pre>\n
The first change is mechanical: We have to use multiqi_traits<\/code> to unwrap the elements of Interfaces...<\/code>, because some of them may be std::optional<T><\/code>, and in those cases, we want to reach inside the std::optional<\/code> and extract the T<\/code>.<\/p>\n
The new part is the template parameter pack expansion of a lambda invocation. Template parameter pack expansions can expand expressions, but we need to expand an if<\/code> statement. No problem: We wrap the if<\/code> statement in a lambda, and then evaluate the lambda immediately, and then take the lambda evaluation (now an expression!) and make the evaluation the thing that is given to the template parameter pack expansion.<\/p>\n
Note that we perform the validation as a separate step after transferring the raw interface pointers into the tuple. This attempted optimization would be incorrect:<\/p>\n
    ([&] {\r\n        std::get<Ints>(t).attach(\r\n        static_cast<typename multiqi_traits<Interfaces>::type*>\r\n        (mqi[Ints].pItf));\r\n\r\n        if constexpr (multiqi_traits<Interfaces>::is_required) {\r\n            THROW_IF_FAILED(mqi[Ints].hr);\r\n        }\r\n    }(), ...);\r\n<\/pre>\n
If any of the required interfaces were not found, then the above incorrect version throws an exception immediately, leaking all of the interface pointers that hadn’t yet been processed. We need to move the raw pointers into smart pointers first, so they don’t leak, and only then can we start throwing exceptions.<\/p>\n
Bonus chatter<\/b>: Note that this code allows you to call Create\u00adInstance\u00adMulti\u00adQI<\/code> and specify that all of the interfaces are optional. That’s intentional, because you might have this:<\/p>\n
auto [widget, doodad] =\r\n    CoCreateInstanceMultiQI<\r\n        std::optional<IWidget>,\r\n        std::optional<IDoodad>\r\n    >(clsid, nullptr, CLSCTX_ANY);\r\n<\/pre>\n
In this case, both interfaces are tagged as optional. You don’t care whether the object is a widget or doodad, but it needs to be one of the two. (Because Co\u00adCreate\u00adInstance\u00adEx<\/code> will fail if none<\/i> of the interfaces is supported.)<\/p>\n
If you want to create the object, with the possibility that it is neither<\/i> a widget nor a doodad, you can specify IUnknown<\/code> as a required interface. Every COM object supports IUnknown<\/code>, so you know that will succeed if the object can be created at all.<\/p>\n
Bonus bonus chatter<\/b>: I put the required checks inside the Create\u00adInstance\u00adMulti\u00adQI\u00adWorker<\/code>, which means that you get a separate version of Create\u00adInstance\u00adMulti\u00adQI\u00adWorker<\/code> for each arity, as well for each pattern of required\/optional. To improve code sharing, we could factor out the part of the worker that is independent of the required\/optional pattern, so that function could be shared among all uses with the same number of interfaces.<\/p>\n","protected":false},"excerpt":{"rendered":"
Letting you specify which interfaces are required and which are optional.<\/p>\n","protected":false},"author":1069,"featured_media":111744,"comment_status":"open","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"_acf_changed":false,"footnotes":""},"categories":[1],"tags":[25],"class_list":["post-106362","post","type-post","status-publish","format-standard","has-post-thumbnail","hentry","category-oldnewthing","tag-code"],"acf":[],"blog_post_summary":"
Letting you specify which interfaces are required and which are optional.<\/p>\n","_links":{"self":[{"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/posts\/106362","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/users\/1069"}],"replies":[{"embeddable":true,"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/comments?post=106362"}],"version-history":[{"count":0,"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/posts\/106362\/revisions"}],"wp:featuredmedia":[{"embeddable":true,"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/media\/111744"}],"wp:attachment":[{"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/media?parent=106362"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/categories?post=106362"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/devblogs.microsoft.com\/oldnewthing\/wp-json\/wp\/v2\/tags?post=106362"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}