GDI+使用异常：两种实现方式为何一种无法转换位图？

核心问题定位 #

两种实现的差异并非GDI+本身的问题，而是C++代码中的命名空间一致性、初始化状态检查缺失导致的异常：

1. 命名空间混用问题
   代码中同时出现了Gdiplus::和gdi::两种命名空间前缀，如果gdi不是Gdiplus的有效别名（比如未添加namespace gdi = Gdiplus;），会导致：

   • std::unique_ptr<gdi::Bitmap>持有Gdiplus::Bitmap*的类型不匹配，触发错误的对象析构行为；
   • gdi::GdiplusStartup未真正调用GDI+的初始化函数，导致GDI+环境未正确启动。

2. 未检查GDI+初始化结果
   两种实现都忽略了GdiplusStartup的返回值，如果第二种实现中GdiplusStartup调用失败（比如参数错误、系统资源不足），后续的Bitmap::FromFile和GetHBITMAP都会在未初始化的GDI+环境中运行，导致转换失效。

3. 第一种实现的潜在风险
   第一种实现的执行顺序存在错误：先调用GdiplusShutdown，再销毁Bitmap对象。这违反了GDI+的使用规则（必须在所有GDI+对象销毁后再调用Shutdown），只是碰巧未触发崩溃，属于未定义行为，不应依赖。

修复建议 #

1. 统一命名空间
   在代码开头添加别名定义，确保gdi与Gdiplus完全一致：

   namespace gdi = Gdiplus;
   

2. 添加初始化状态检查
   修改GDIInit类的构造函数，确保GDI+初始化成功：

   class GDIInit {
   public:
       GDIInit() {
           Gdiplus::Status status = gdi::GdiplusStartup(&token_, &input_, NULL);
           if (status != Gdiplus::Ok) {
               throw std::runtime_error("GDI+ initialization failed");
           }
       }
   
       ~GDIInit() {
           gdi::GdiplusShutdown(token_);
       }
   
   private:
       gdi::GdiplusStartupInput input_;
       ULONG_PTR token_;
   };
   

3. 验证GetHBITMAP的返回状态
   在第二种实现中添加错误日志，明确失败原因：

   HRESULT ThumbnailProvider::getWithGDI(LPCWSTR wpath, HBITMAP* hBitmap)
   {
       try {
           GDIInit gdiInit;
   
           std::unique_ptr<gdi::Bitmap> bitmap(gdi::Bitmap::FromFile(wpath));
           if (!bitmap) {
               return E_FAIL;
           }
   
           Gdiplus::Status status = bitmap->GetHBITMAP(NULL, hBitmap);
           if (status != Gdiplus::Ok) {
               // 输出status值，排查具体错误（如InvalidParameter、OutOfMemory等）
               return HRESULT_FROM_WIN32(status);
           }
   
           return S_OK;
       } catch (const std::exception& e) {
           return E_FAIL;
       }
   }
   

4. 修正第一种实现的生命周期顺序
   虽然第一种实现能运行，但需调整顺序以符合GDI+规范：

   HRESULT ThumbnailProvider::getWithGDI(LPCWSTR wpath, HBITMAP* hBitmap)
   {
       Gdiplus::GdiplusStartupInput input_;
       ULONG_PTR token_;
       Gdiplus::Status startupStatus = Gdiplus::GdiplusStartup(&token_, &input_, NULL);
       if (startupStatus != Gdiplus::Ok) {
           return E_FAIL;
       }
   
       HRESULT result = E_FAIL;
       std::unique_ptr<Gdiplus::Bitmap> bitmap(Gdiplus::Bitmap::FromFile(wpath));
       if (bitmap) {
           Gdiplus::Status status = bitmap->GetHBITMAP(NULL, hBitmap);
           if (status == Gdiplus::Ok) {
               result = S_OK;
           }
       }
   
       // 先销毁Bitmap，再调用Shutdown
       bitmap.reset();
       Gdiplus::GdiplusShutdown(token_);
   
       return result;
   }
   

内容的提问来源于stack exchange，提问作者feiniks