std :: unique_ptr mit benutzerdefinierten deleter für win32 Localfree

Question

Ich habe die win32 API CommandLineToArgvW die eine LPWSTR* und gibt mir warnt davor, dass

CommandLineToArgvW für Zeiger auf die Argument Strings einen Block von zusammenhängenden Speicher zuweist, und für die Argumentketten selbst; Die aufrufende Anwendung muss den von der Argumentliste verwendeten Speicher freigeben, wenn sie nicht mehr benötigt wird. Um den Speicher freizugeben, verwenden Sie einen Einzelruf an die LocalFree-Funktion.

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

Was ist ein C++ idiomatische Weg, um den Speicher im obigen Fall zu befreien?

ich zu einem std::unique_ptr mit einem benutzerdefinierten deleter dachte, so etwas wie die:

#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; 
} 

Gibt es irgendein Problem in dem obigen Code?

Answer 1

Es sieht für mich richtig aus. Sie könnten es ein wenig prägnanter machen, indem Sie den Deleter der unique_ptr inline angeben, anstatt einen Funktor dafür zu erstellen.

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

Oder, wenn Sie mit LocalFree ‚Unterschrift zu Chaos nicht wollen und Konventionen fordern Sie eine Lambda verwenden können, das Löschen zu tun.

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

Hinweis: Zum Zeitpunkt der Antwort zuerst geschrieben wurde, wurde VS2010 die VS-Version freigegeben. Es doesn't support Umwandlung von Capture-weniger lambdas Zeiger funktionieren, so müssten Sie std::function im zweiten Beispiel

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

Answer 2

I shared_ptr nützlicher als eine generische Ressource guard etwas finden verwenden. Es erfordert nicht, dass der Löscher der Teil von Vorlagenargumenten ist und als solcher leicht weitergegeben werden kann.

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

Answer 3

von benutzerdefinierten deleter Deklarieren nicht so schön ist, die Verwendung von decltype() ist schneller. std::shared_ptr ist eine Alternative, aber es ist größer als std::unique_ptr. Wenn Sie keinen Zeiger teilen möchten, nehmen Sie eine unique_ptr.