如何将结构分配给结构的指针数组？ [关闭]

Question

好的。所以我有以下结构：

struct pkt {
    int seqnum;
    int acknum;
    int checksum;
    char payload[20];
};

现在我有以下变量：

struct pkt packet;
struct pkt* packets[1000];

现在在对packet 进行了几次操作之后，我尝试将其分配给数组单元格，如下所示：

packets[counter++] = (pkt*)&packet;

但这会以某种方式将数组的所有单元格在新赋值时更改为新赋值的值。

我做错了什么？

我刚开始接触 C，所以我对指针和地址不是很熟悉。

EDIT：正在完成分配的函数：

void A_output(struct msg message)
{
    struct pkt packet;
    packet.acknum = 0;
    packet.seqnum = 0;
    memset(&packet.payload, '\0', 20);
    memcpy((char*)&packet.payload, (char*)&message.data, 20);
    memset(&packets[counter]->payload, '\0', 20);
    packets[counter++] = (pkt*)&packet;
    printAll();

    if(nextseq < winbase + winsize) {
        packets[nextseq]->seqnum = nextseq;
        setChecksum(packets[nextseq]);
        tolayer3(0, *packets[nextseq]);

        if(winbase == nextseq) 
            starttimer(0, increment);
        nextseq++;
    }
}

既然值是作为函数的参数来的，不是每次都有新的内存地址吗？

Answer 1

好吧，数组中的所有单元格的值都是&packet，这是一个单一的值。它们都指向同一个地址，所以当然它们都指向同一个值。

您可以动态分配内存，或者更好的是，只需将packets 设为pkt[] - 一个常规数据包数组，然后分配给其中一个单元，会将所有值复制到另一个内存，而不是复制的指针。

至于packet 是否总是具有相同地址的问题，我无法确定（我的猜测是确实如此）但您可以检查 - 每次调用函数时打印&packet 的值并查看如果它改变了。

附： packet 不是函数参数，而是局部函数变量。

Answer 2

void A_output(struct msg message)
{
    struct pkt packet;

[...]

    packets[counter++] = (pkt*)&packet;

最后一行将具有自动存储持续时间（局部变量）的变量的地址分配给具有静态存储持续时间（全局）的对象。一旦您的函数退出，具有自动存储持续时间的变量不再存在，因此指针不会指向有效位置。稍后使用此指针是未定义的行为，但很有可能您的函数每次调用它时都会为您的 struct pkt packet 重用相同的内存位置，从而导致您描述的行为。

例如，您需要为packets 中的每个元素分配内存，malloc() 或new：

struct pkt *packet = malloc(sizeof *packet); // C style
// or:
pkt *packet = new pkt(); // C++ style

[...] // fill *packet

packets[counter++] = packet;

完成后不要忘记free() (C) 或 delete (C++) 所有数据包。

附带说明，在您的原始代码中，演员 (pkt*) 是多余的。 &packet 已经具有 struct pkt * 类型。不过既然这样是错的，这里就无所谓了。

Answer 3

packets[0] = &pkt;

给予

 +-----+           +-------+
 | 0   | ========> | [pkt] |
 +-----+           +-------+

然后

packets[1] = &pkt;

给予

 +-------+              +----------+
 |    0  |  ==========> |  [pkt]   |
 +-------+         ||=> +----------+
 |    1  |  =======* 
 +-------+

有一个底层对象（pkt），它在数组中的两个位置被引用。

如果您希望它们的值不同，则将数组设为完整成员数组。

struct pkt packets[1000];

Answer 4

您似乎正在尝试创建一个结构数组。下面的代码 sn-p 创建了一个本地结构数组，每个结构都包含一个整数数组。

该方法单独作用于结构数组，为内部的整数数组分配唯一值。

// Structure definitions
#define NUM_OF_VALUES 2
#define NUM_OF_STRUCTS 5

typedef struct {
    uint value[NUM_OF_VALUES];
} Test_Struct_t;

// local object
static Test_Struct_t struct_array[NUM_OF_STRUCTS];

// Function settings values in local object.
void struct_foo(void)
{
     for (uint idx = 0; idx < NUM_OF_STRUCTS; idx++)
     {
         for (uint jdx = 0; jdx < NUM_OF_VALUES; jdx++)
         {
             struct_array[idx].value[jdx] = (idx + jdx);
         }
     }
}

既然值是作为函数的参数出现的，不是吗？ 每次都有新的内存地址吗？

函数调用时，函数参数将被压入堆栈。如果您希望正在解析的变量在对其进行操作后保留其值，请使用pointer argument instead.