函数返回 BYTE 数组答案

【问题标题】：Function returns BYTE array函数返回 BYTE 数组
【发布时间】：2012-03-17 05:25:18
【问题描述】：

我希望我的函数返回一个 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] 时，它给了我一个错误。

【问题讨论】：

标签： c++ visual-c++ return bytearray return-value

【解决方案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> 并用它摆脱这种丑陋的内存管理；）这样可以确保在任何返回路径上正确释放内存，即使抛出异常也是如此。

【讨论】：

请把“你应该”换成“你可以”……总体来说答案非常好。 +1

【解决方案2】：

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

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

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

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

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

【讨论】：