memset 清除它不应该的东西

Question

我有一个服务器 - 客户端程序，我可以添加或删除用户。当达到最大用户数（10）时，删除一个用户时出现错误。

这是删除用户的部分功能：

int i;
int confirmacao = 0;
msg_t msg;
char cliname[10];
if (strlen(estrutura.id) == 3) {
    for (i = 0; memoria->x[i].id[0] != '\0'; ++i) {
        if (strcmp(estrutura.id, memoria->x[i].id) == 0) {
            confirmacao = 1;
            for (; memoria->x[i].id[0] != '\0'; ++i) {
                memoria->x[i] = memoria->x[i + 1];
            }
            // I think the problem is with this memset
            memset(memoria->x[i].id, '\0', sizeof(memoria->x[i].id));
            printf("IA antes de decrementar: %i\n", memoria->ia);
            memoria->ia--;
            printf("Ia depois: %i\n", memoria->ia);
            break;
        }
    }
} else {
    ...
}

Here is the print from gdb

memoria->ia 应该减一，我可以弄清楚为什么 memset 将其设置为零。

memoria 是这个结构的全局指针：

typedef struct mmap_uti_s {
    uti_t x[NUTI];
    int ia;
} mmap_uti_t;

而uti_t就是这个结构：

typedef struct uti_s {
    char id[NDIG + 1];
    char nome[NDIM + 1];
    char portas[NPOR + 1];
} uti_t;

Answer 1

复制前一个元素后，您未能重置最后一个元素，因为您在测试数组末尾之前增加了i。此外，正如您正确指出的那样，您必须始终在到达数组末尾之前停止复制。

您应该将嵌套 for 循环中的测试更改为：

for (; i + 1 < 10 && memoria->x[i + 1].id[0] != '\0'; i++)

如果数组已满，您还应该更改主循环中的测试以在数组末尾停止扫描。

虽然不是绝对必要，但最好在复制阶段使用不同的索引，以避免修改当前循环的索引。

清除整个结构，而不仅仅是 id 成员，似乎也更可取。

这里是修改后的代码：

int i, j;
int confirmacao = 0;
msg_t msg;
char cliname[10];
if (strlen(estrutura.id) == 3) {
    for (i = 0; i < 10 && memoria->x[i].id[0] != '\0'; ++i) {
        if (strcmp(estrutura.id, memoria->x[i].id) == 0) {
            confirmacao = 1;
            for (j = i; j + 1 < 10 && memoria->x[j].id[0] != '\0'; ++j) {
                memoria->x[j] = memoria->x[j + 1];
            }
            // I think the problem is with this memset
            memset(&memoria->x[j], 0, sizeof(memoria->x[j]));
            printf("IA antes de decrementar: %i\n", memoria->ia);
            memoria->ia--;
            printf("Ia depois: %i\n", memoria->ia);
            break;
        }
    }
} else {
    ...
}