Std:: unique_ptr с пользовательским удалением для win32 LocalFree

У меня есть win32 API CommandLineToArgvW, который возвращает LPWSTR* и предупреждает меня, что

CommandLineToArgvW выделяет блок смежной памяти для указатели на строки аргументов, а для строк аргументов самих себя; вызывающее приложение должно освободить память, используемую список аргументов, когда он больше не нужен. Чтобы освободить память, используйте один вызов функции LocalFree.

См http://msdn.microsoft.com/en-us/library/windows/desktop/bb776391(v=vs.85).aspx

Что такое С++ идиоматический способ освобождения памяти в вышеуказанном случае?

Я думал о std::unique_ptr с пользовательским делетером, что-то вроде этого:

#include <Windows.h>
#include <memory>
#include <iostream>

template< class T >
struct Local_Del
{
   void operator()(T*p){::LocalFree(p);}
};

int main(int argc, char* argv[])
{
   {
      int n = 0;
      std::unique_ptr< LPWSTR, Local_Del< LPWSTR > > p( ::CommandLineToArgvW(L"cmd.exe p1 p2 p3",&n) );
      for ( int i = 0; i < n; i++ ) {
         std::wcout << p.get()[i] << L"\n";
      }
   }

    return 0;
}

Есть ли какие-либо проблемы в приведенном выше коде?

Ответы

Ответ 1

Это выглядит правильно для меня. Вы можете сделать это немного более кратким, указав unique_ptr deleter inline, а не создавая для него функтор.

std::unique_ptr<LPWSTR, HLOCAL(__stdcall *)(HLOCAL)> 
      p( ::CommandLineToArgvW( L"cmd.exe p1 p2 p3", &n ), ::LocalFree );

Или, если вы не хотите связываться с подписями LocalFree и вызовами, вы можете использовать лямбда для удаления.

std::unique_ptr<LPWSTR, void(*)(LPWSTR *)> 
      p( ::CommandLineToArgvW( L"cmd.exe p1 p2 p3", &n ), 
         [](LPWSTR *ptr){ ::LocalFree( ptr ); } );

Примечание. В то время, когда этот ответ был впервые написан, VS2010 был выпущенной версией VS. Он не поддерживает преобразование неактивных lambdas в указатели функций, поэтому вам придется использовать std::function во втором примере

std::unique_ptr<LPWSTR, std::function<void(LPWSTR *)>> 
      p( ::CommandLineToArgvW( L"cmd.exe p1 p2 p3", &n ), 
         [](LPWSTR *ptr){ ::LocalFree( ptr ); } );

Ответ 2

Объявление пользовательского делетера не так красиво, использование decltype() выполняется быстрее. std::shared_ptr является альтернативой, но она больше, чем std::unique_ptr. Если вы не хотите делиться указателем, возьмите unique_ptr.

std::unique_ptr<LPWSTR, decltype(::LocalFree)> 
     p( ::CommandLineToArgvW( L"cmd.exe p1 p2 p3", &n ), ::LocalFree );

Ответ 3

Я нахожу shared_ptr немного полезнее в качестве защитника общих ресурсов. Это не требует, чтобы deleter был частью аргументов шаблона и как таковой можно легко обойти.

std::shared_ptr<LPWSTR> p(
    ::CommandLineToArgvW(L"cmd.exe p1 p2 p3", &n),
    ::LocalFree);