1

我已经阅读了许多与此问题类似的问答，但尚未找到任何答案：我必须对fillbyte函数中的动态字节数组进行一些分配，如下所示：

int Error;
result = fillbyte (&Error);

如果我评论下面显示的行，一切正常。但是如果该行被执行，则第二次调用此函数时，将引发访问冲突异常，但是第一次代码正常运行并且一切正常。我似乎找不到这行代码或用密码字节填充数组的其他方法的问题。

Bool fillbyte(int *Error)
{
    byte BCC;
    byte *Packet1 = new byte;
    *Packet1 = 0x01;
    *(Packet1+1) = 'P';
    *(Packet1+2) = '1';
    *(Packet1+3) = STX;
    *(Packet1+4) = '(';
    int add = sizeof(readingprops.password)*2;
    for(int i=0;i<add;i++)
    {
        *(Packet1+(5+i)) = readingprops.password[i];     //this line raises the problem
    }
    *(Packet1+add+5) = ')';
    *(Packet1+add+6) = ETX;
    BCC = calc.CalcBCC(Packet1,add+7);
    *(Packet1+add+7) = BCC;
    SerialPort.Write(Packet1,add+8);
    delete Packet1;
    return true;
}

任何帮助，将不胜感激

4

2 回答 2

3

我看不出它是如何工作的。您在堆上分配一个字节，但将其视为多个字节：

byte *Packet1 = new byte;
*Packet1 = 0x01;
*(Packet1+1) = 'P';  // !!!
*(Packet1+2) = '1';  // !!!
*(Packet1+3) = STX;  // !!!
*(Packet1+4) = '(';  // !!!
于 2013-06-26T12:54:09.313 回答
3

在这里你只分配一个字节

byte *Packet1 = new byte;

然后使用超出分配内存的指针

*(Packet1+1) = 'P';
*(Packet1+2) = '1';
*(Packet1+3) = STX;
*(Packet1+4) = '(';

这会导致未定义的行为，有时它可能会起作用。所以你想要类似的东西

byte Packet1 = new byte[size]

其中 size 适合您的需要（可能是add + 8，因为这是您在该函数中写入的字节数）。然后用 删除它delete[]。您也可以使用堆栈分配，或者std::vector<byte>因为这是 C++。

于 2013-06-26T12:57:54.947 回答