帮我优化：int add_node(void *vaule, int len) { LIST_NODE *new_node = NULL; LIST_NODE *tmp = NULL; new_node->value = calloc(1,len); if (new_node->value == NULL){ printf("calloc failed\n"); return -1; } memcpy(new_node, vaule, len); tmp = g_list.head->next; g_list.head->next = new_node; new_node->next = tmp; g_list.node_nums++; return 0; }

在这段代码中，有几个优化的地方可以考虑：

1. 添加节点前应先检查内存分配的结果。在 `new_node` 的定义后，应该分配内存给 `new_node`，而不是在赋值时分配。这样可以避免使用未初始化的指针 `new_node`。

2. 在分配内存给 `value` 之后，应该检查 `calloc` 的返回值，以确保内存分配成功。如果分配失败，应该释放已分配的内存并返回错误。

3. 在调用 `memcpy` 之前，应该将 `vaule` 转换为正确的指针类型（例如 `void*` 转换为 `char*`），以确保正确的内存拷贝。

4. 在添加新节点时，不需要将 `g_list.head->next` 赋值给 `tmp`，然后再将 `new_node` 赋值给 `g_list.head->next`。可以直接将 `new_node` 的 `next` 指针指向 `g_list.head->next`，然后将 `new_node` 赋值给 `g_list.head->next`。

下面是优化后的代码：

int add_node(void *value, int len) {
    LIST_NODE *new_node = calloc(1, sizeof(LIST_NODE));
    if (new_node == NULL) {
        printf("calloc failed\n");
        return -1;
    }

    new_node->value = calloc(1, len);
    if (new_node->value == NULL) {
        printf("calloc failed\n");
        free(new_node);
        return -1;
    }

    memcpy(new_node->value, value, len);

    new_node->next = g_list.head->next;
    g_list.head->next = new_node;
    g_list.node_nums++;
    
    return 0;
}

请注意，我假设了存在一个名为 `g_list` 的全局变量，其中包含一个头节点 `head` 和一个节点计数器 `node_nums`。如果您的代码中不存在这些变量，请相应地调整代码。