函数返回 std::wstring = NULL;

Question

我正在尝试为 winapi 函数 GetWindowText 制作包装器。 函数返回 std::wstring 但我不知道如何处理发生错误的位置。我返回 NULL 但我知道这是错误的。

std::wstring GetWindowText(HWND handle)
{
    const int size = 1024;
    TCHAR wnd_text[size] = {0};

    HRESULT hr = ::GetWindowText(handle,
                    wnd_text, size);
    if(SUCCEEDED(hr))
        return std::wstring(wnd_text);
    else
        return NULL;    
}

Answer 1

根据您的应用，有几种合适的 解决方案。第一个是在发生异常的情况下抛出异常 错误：如果你走这条路，你应该定义一个 WindowsError 异常（源自标准异常之一），其中 包括GetLastError 提供的所有信息， 和可能的附加信息（函数的名称 失败等），以易于解析的格式。另一种是 返回某种Fallible；在这种情况下，您可能想要 扩展经典的Fallible 成语，使其可以包含 有关错误原因的附加信息。仍然 另一种可能性是通过 out 参数返回值， 并使用返回码（同样，可能带有额外的 信息，并可能添加代码以确保它具有 在被破坏之前经过测试）。

Answer 2

首先 GetWindowText() 不返回 HRESULT 所以你的代码在这方面是错误的。

其次，GetWindowTextW 对任何错误或字符数返回 0（如果确定）。 所以只返回一个空字符串：

std::wstring GetWindowText(HWND handle)
{
    const int size = 1024;
    TCHAR wnd_text[size] = {0};

    INT n = ::GetWindowTextW(handle,
                    wnd_text, size);
    if(n > 0)
        return std::wstring(wnd_text,n);
    else
        return std::wstring();
}

Answer 3

作为异常的替代方法，您还可以通过参数列表中的引用返回字符串，并通过返回 true 或 false 来指示成功，即

bool GetWindowText(HWND handle, std::wstring& windowText)
{
    const int size = 1024;
    TCHAR wnd_text[size] = {0};

    HRESULT hr = ::GetWindowText(handle,
                    wnd_text, size);
    if(SUCCEEDED(hr))
    {
        windowText = wnd_text;
        return true;
    }
    else
        return false;    
}

另一个避免引用参数的替代方法是返回一个类的实例，该实例包装了一个值，但也让你知道一个值是否存在，例如

class ValueWrapper
{
public:
    ValueWrapper() : present( false ) {}
    ValueWrapper( const std::wstring& s ) : value( s ), present( true ) {}

    bool isPresent() const { return present; }
    const std::wstring& getValue() const { return value; }

private:
    std::wstring value;
    bool present;
};

请注意，您可以很容易地制作此包装器的模板。那么你的功能就是

ValueWrapper GetWindowText(HWND handle)
{
    const int size = 1024;
    TCHAR wnd_text[size] = {0};

    HRESULT hr = ::GetWindowText(handle,
                    wnd_text, size);
    if(SUCCEEDED(hr))
        return ValueWrapper( wnd_text );
    else
        return ValueWrapper();
}

Answer 4

另一种解决方案（不抛出异常）：使用Boost.Optional 库。

Answer 5

WinApi 设计为永远不会抛出异常。并且要获得某些函数返回失败的原因，在大多数情况下，您必须通过GetLastError() 获取最后一个错误。据我了解，您的功能将成为 WinApi 的一些补充，易于使用。所以我建议保留他们的设计。即如果失败返回空字符串，如果你的函数返回它，检查最后一个错误是什么。

Answer 6

NULL 对于字符串绝对是不可以的，特别不允许将空指针传递给字符串构造函数。

如果不想抛出异常，可以返回空字符串return std::wstring();。

Answer 7

改为使用exception。

std::wstring GetWindowText(HWND handle)
{
    const int size = 1024;
    TCHAR wnd_text[size] = {0};

    HRESULT hr = ::GetWindowText(handle,
                    wnd_text, size);
    if(SUCCEEDED(hr))
        return std::wstring(wnd_text);
    else
        throw std::runtime_error("insert error message here");    
}