Question

我希望我的函数返回一个BYTE数组。功能如下。

BYTE sendRecieveData(BYTE control, unsigned int value){

//Open connection to LAC
HANDLE LACOutpipe;
HANDLE LACInpipe;
LACOutpipe=openConnection(MP_WRITE);
LACInpipe=openConnection(MP_READ);

//declare variables
BYTE bufDataOut[3];
BYTE bufDataIn[3];
DWORD bufInProcess;
DWORD bufOutProcess;


//sets CONTROL
bufDataOut[0]=control;

//sets DATA to be sent to LAC
BYTE low_byte = 0xff & value;
BYTE high_byte = value >> 8;
bufDataOut[1]=low_byte;
bufDataOut[2]=high_byte;

MPUSBWrite(LACOutpipe,bufDataOut,3,&bufOutProcess,1000);
MPUSBRead(LACInpipe,bufDataIn,3,&bufInProcess,1000);
MPUSBClose(LACOutpipe);
MPUSBClose(LACInpipe);

return bufDataIn[3];
}

它不返回字节数组，当我将BYTE更改为BYTE[]或BYTE[3]时，它会给我一个错误。

Answer 1

return bufDataIn[3];表示“返回bufDataIn数组的第4个元素”，在这种情况下，由于此数组的大小为3，因此会导致未定义的行为。

您可以在函数体中为这个新数组分配内存，并返回指向其第一个元素的指针：

BYTE* createArray(...)
{
    BYTE* bufDataOut = new BYTE[3];
    ....
    return bufDataOut;
}

完成后不要忘记delete：

{
    BYTE* myArray = createArray(...);
    ...
    delete[] myArray;
}

更好的是，使用std::vector<BYTE>并使用它来摆脱这种丑陋的内存管理;）这可以确保在任何返回路径上正确释放内存，即使抛出异常也是如此。

Answer 2

由于你的数组相对较小，我建议将缓冲区作为函数参数传递。

void sendRecieveData(BYTE control, unsigned int value, BYTE (&buffdataIn)[3] ).

现在，人们将按如下方式使用此功能：

BYTE result[3] = {0};
sendRecieveData(3, 0, result);

这样您可以避免使用动态内存分配并使代码更安全。

函数返回BYTE数组

2 个答案: