1

我正在尝试用值“test”填充 BSTR 类型的 10 个索引的 SAFEARRAY，并在分配 SAFEARRAY 的每个索引以验证正确性后打印出来以控制台其值。我运行调试器并为前 5 个索引获得了以下这些值（我的 SAFEARRAY 称为 sa）。不知何故，我错误地迭代了 SAFEARRAY 或使用了错误的类型，每个索引都应该是“测试”。关于我做错了什么的任何建议？

sa[0] = "test"
sa[1] = "est"
sa[2] = "st"
sa[3] = "t"
sa[4] = ""

……

#include <iostream>
#include <string>
#include <Windows.h>
#include <atlbase.h>
#include <comutil.h>
#include <string.h>
#include <stdio.h>

using namespace std;

void fillVariant(VARIANT& varIn, BSTR &srcArray);

int main()
{

    BSTR *theArray = new BSTR[10];
    for(int i = 0 ; i < 10; i++)
    {

        theArray[i] = SysAllocString(L"test");
    }

    VARIANT variantArray;
    fillVariant(variantArray, *theArray);


    return 0;
}

void fillVariant(VARIANT& varIn, BSTR &srcArray)
{
    VARIANT *variantArray = &varIn;
    VariantInit(variantArray);
    variantArray->vt = VT_ARRAY|VT_BSTR;

    SAFEARRAY* sa;
    SAFEARRAYBOUND aDim[1]; 
    aDim[0].lLbound = 0; 
    aDim[0].cElements = 10;

    sa = SafeArrayCreate(VT_BSTR, 1, aDim);

    BSTR* dwArray = NULL;
    SafeArrayAccessData(sa, (void**)&dwArray);

    for(int i = 0; i < 10; i++)
    {
        dwArray[i] = &srcArray[i];

        BSTR tmp = (BSTR) dwArray[i];
        std::wstring ws(tmp);
        //std::wstring ws(*dwArray[i], SysStringLen(dwArray[i]));
        std::wcout << ws << endl;

    }
    SafeArrayUnaccessData(sa);
    variantArray->parray = sa;
}
4

2 回答 2

3

您没有VARIANT正确填写。试试这个：

#include <iostream>
#include <string>
#include <Windows.h>
#include <atlbase.h>
#include <comutil.h>
#include <string.h>
#include <stdio.h>

using namespace std;

void fillVariant(VARIANT& varIn, BSTR *srcArray, int srcArrayLen);

int main()
{
    BSTR *theArray = new BSTR[10];
    for(int i = 0 ; i < 10; i++)
    {
        theArray[i] = SysAllocString(L"test");
    }

    VARIANT variantArray;
    fillVariant(variantArray, theArray, 10);

    // don't forget to free memory when done!
    // note: the VARIANT owns the BSTRs, so DON'T free them!
    VariantClear(&variantArray);
    delete[] theArray;

    return 0;
}

void fillVariant(VARIANT& varIn, BSTR *srcArray, int srcArrayLen)
{
    VARIANT *variantArray = &varIn;
    VariantInit(variantArray);

    SAFEARRAYBOUND aDim[1]; 
    aDim[0].lLbound = 0; 
    aDim[0].cElements = srcArrayLen;

    SAFEARRAY* sa = SafeArrayCreate(VT_BSTR, 1, aDim);
    if (sa)
    {    
        BSTR* dwArray = NULL;
        SafeArrayAccessData(sa, (void**)&dwArray);

        for(int i = 0; i < srcArrayLen; i++)
        {
            // note: passing ownership, NOT making a copy
            dwArray[i] = srcArray[i];

            //std::wstring ws(dwArray[i], SysStringLen(dwArray[i]));
            std::wcout << dwArray[i] << endl;
        }

        SafeArrayUnaccessData(sa);

        variantArray->vt = VT_ARRAY|VT_BSTR;
        variantArray->parray = sa;
    }
}

或者：

#include <iostream>
#include <string>
#include <Windows.h>
#include <atlbase.h>
#include <comutil.h>
#include <string.h>
#include <stdio.h>

using namespace std;

void fillVariant(VARIANT& varIn, BSTR *srcArray, int srcArrayLen);

int main()
{
    BSTR *theArray = new BSTR[10];
    for(int i = 0 ; i < 10; i++)
    {
        theArray[i] = SysAllocString(L"test");
    }

    VARIANT variantArray;
    fillVariant(variantArray, theArray, 10);

    // don't forget to free memory when done!

    VariantClear(&variantArray);

    // note: the VARIANT DOES NOT own the BSTRs, so DO free them!
    for(int i = 0 ; i < 10; i++)
    {
        SysFreeString(theArray[i]);
    }
    delete[] theArray;

    return 0;
}

void fillVariant(VARIANT& varIn, BSTR *srcArray, int srcArrayLen)
{
    VARIANT *variantArray = &varIn;
    VariantInit(variantArray);

    SAFEARRAYBOUND aDim[1]; 
    aDim[0].lLbound = 0; 
    aDim[0].cElements = srcArrayLen;

    SAFEARRAY* sa = SafeArrayCreate(VT_BSTR, 1, aDim);
    if (sa)
    {    
        for(LONG i = 0; i < srcArrayLen; i++)
        {
            // note: makes a copy, DOES NOT pass ownership!
            SafeArrayPutElement(sa, &i, srcArray[i]);

            //std::wstring ws(srcArray[i], SysStringLen(srcArray[i]));
            std::wcout << srcArray[i] << endl;
        }

        variantArray->vt = VT_ARRAY|VT_BSTR;
        variantArray->parray = sa;
    }
}
于 2014-07-28T21:28:06.457 回答
2

&srcArray[i]不做你认为它做的事。srcArray是一个单一的BSTR，这是一个 typedef 为WCHAR*. 它不是s的数组BSTR。srcArray[i]指的是 中的i第 th 个字符theArray[0]，并且&srcArray[i]是那个字符的地址。这就是你得到“test”、“est”等的方式。

fillVariant将 take作为BSTR*其第二个参数，并通过theArray，而不是*theArray。

在不相关的说明中，您的程序泄漏了一堆BSTRs 和 a SAFEARRAY。

于 2014-07-28T21:21:12.507 回答