static char *TCPCommand; static pthread_mutex_t mutex = PTHREAD_MUTEX_INITIALIZER; int SetUpTCPtoSendInformation(char option[], char **command) { static int isFirstCall = 1; pthread_mutex_lock(&mutex); // 加锁 if (isFirstCall && strstr(option, "set")) { if (TCPCommand != NULL) { free(TCPCommand); } TCPCommand = malloc(strlen(*command) + 1); if (TCPCommand == NULL) { printf("Failed to set and obtain TCP command variable memory allocation\n"); goto fail; } strcpy(TCPCommand, *command); printf("set:%s\n", TCPCommand); isFirstCall = 0; goto succeed; } else if (!isFirstCall && strstr(option, "get") && TCPCommand != NULL && strlen(TCPCommand)) { free(*command); *command = malloc(strlen(TCPCommand) + 1); strcpy(*command, TCPCommand); printf("get:%s\n", *command); memset(TCPCommand, '\0', strlen(TCPCommand)); free(TCPCommand); TCPCommand = NULL; isFirstCall = 1; goto succeed; } memset(*command, 0, sizeof(*command)); strcpy(*command, ""); fail: pthread_mutex_unlock(&mutex); // 解锁 return 0; succeed: pthread_mutex_unlock(&mutex); // 解锁 return 1; } char *SendString = NULL; SetUpTCPtoSendInformation("get",&SendString); if(SendString == NULL && strlen(SendString)){ }程序有误

根据你提供的代码，我注意到在调用 `SetUpTCPtoSendInformation("get", &SendString)` 后，你正在检查 `SendString` 是否为 `NULL` 并且长度是否为零。这是一个逻辑错误。如果 `SendString` 为 `NULL`，则无法访问其长度。

你可以更改代码如下：

char *SendString = NULL;
int result = SetUpTCPtoSendInformation("get", &SendString);
if (result && SendString != NULL && strlen(SendString) > 0) {
    // 在这里执行发送 SendString 的操作
    printf("SendString: %s\n", SendString);
} else {
    printf("Failed to obtain SendString\n");
}

这样，你可以在 `SetUpTCPtoSendInformation` 函数中根据条件设置 `SendString` 的值，并在返回成功时执行发送操作。请确保在使用 `SendString` 之前检查其是否为 `NULL`，以避免访问空指针错误。

相关问题

static char *TCPCommand; static pthread_mutex_t mutex = PTHREAD_MUTEX_INITIALIZER; int SetUpTCPtoSendInformation(char option[], char *command) { pthread_mutex_lock(&mutex); // 加锁 if (strstr(option, "set")) { TCPCommand = malloc(strlen(command) + 1); if (TCPCommand == NULL) { CON_LOG("Failed to set and obtain TCP command variable memory allocation"); goto fail; } strcpy(TCPCommand, command); goto succeed; } else if (strstr(option, "get") && TCPCommand != NULL && strlen(TCPCommand)) { strcpy(command, TCPCommand); memset(TCPCommand, '\0', strlen(TCPCommand)); free(TCPCommand); pthread_mutex_unlock(&mutex); // 解锁 goto succeed; } memset(command, 0, sizeof(command)); strcpy(command, ""); fail: pthread_mutex_unlock(&mutex); // 解锁 return 0; succeed: pthread_mutex_unlock(&mutex); // 解锁 return 1; }

在给定的代码中，锁的使用是正确的。使用 `goto` 语句是为了避免在不同的分支中重复写解锁代码。无论是 `fail` 还是 `succeed` 的标签，都会在相应的位置执行解锁操作。

在 `fail` 标签处，如果内存分配失败，则会执行解锁操作并返回 0，表示设置TCP命令失败。

在 `succeed` 标签处，如果是获取TCP命令并且成功获取到了命令，则会执行解锁操作并返回 1，表示设置TCP命令成功。

需要注意的是，在 `get` 分支中，使用了 `free(TCPCommand)` 来释放内存。这意味着在获取TCP命令后，需要确保在使用完命令后进行内存的释放操作。

总体而言，给定的代码使用互斥锁来确保对共享资源的访问是线程安全的，并且通过 `goto` 语句来简化代码并避免重复解锁的问题。

int SetUpTCPtoSendInformation(char option[], char *command) { static char *TCPCommand; static pthread_mutex_t mutex = PTHREAD_MUTEX_INITIALIZER; static int isFirstCall = 1; pthread_mutex_lock(&mutex); // 加锁 if (isFirstCall && strstr(option, "set")) { TCPCommand = malloc(strlen(command) + 1); if (TCPCommand == NULL) { CON_LOG("Failed to set and obtain TCP command variable memory allocation"); goto fail; } strcpy(TCPCommand, command); isFirstCall = 0; goto succeed; } else if (!isFirstCall && strstr(option, "get") && TCPCommand != NULL && strlen(TCPCommand)) { strcpy(command, TCPCommand); memset(TCPCommand, '\0', strlen(TCPCommand)); free(TCPCommand); isFirstCall = 1; goto succeed; } memset(command, 0, sizeof(command)); strcpy(command, ""); fail: pthread_mutex_unlock(&mutex); // 解锁 return 0; succeed: pthread_mutex_unlock(&mutex); // 解锁 return 1; } 和static char *TCPCommand; static pthread_mutex_t mutex = PTHREAD_MUTEX_INITIALIZER; int SetUpTCPtoSendInformation(char option[], char *command) { static int isFirstCall = 1; pthread_mutex_lock(&mutex); // 加锁 if (isFirstCall && strstr(option, "set")) { TCPCommand = malloc(strlen(command) + 1); if (TCPCommand == NULL) { CON_LOG("Failed to set and obtain TCP command variable memory allocation"); goto fail; } strcpy(TCPCommand, command); isFirstCall = 0; goto succeed; } else if (!isFirstCall && strstr(option, "get") && TCPCommand != NULL && strlen(TCPCommand)) { strcpy(command, TCPCommand); memset(TCPCommand, '\0', strlen(TCPCommand)); free(TCPCommand); isFirstCall = 1; goto succeed; } memset(command, 0, sizeof(command)); strcpy(command, ""); fail: pthread_mutex_unlock(&mutex); // 解锁 return 0; succeed: pthread_mutex_unlock(&mutex); // 解锁 return 1; }区别

这两段代码的区别在于第一段代码中的变量`TCPCommand`和`mutex`是静态变量，而第二段代码中的变量`TCPCommand`和`mutex`是全局变量。

在第一段代码中，`TCPCommand`和`mutex`都被声明为静态变量，意味着它们的作用域仅限于`SetUpTCPtoSendInformation`函数内部。这样做的好处是，每次调用该函数时都会创建新的`TCPCommand`和`mutex`变量，避免了命名冲突和数据共享的问题。

而在第二段代码中，`TCPCommand`和`mutex`被声明为全局变量，意味着它们可以在函数外部进行访问和修改。这样做的好处是，多次调用`SetUpTCPtoSendInformation`函数时可以保持`TCPCommand`和`mutex`的状态，实现数据的持久化和共享。

需要注意的是，使用全局变量可能会增加代码的复杂性，并且需要谨慎处理多线程环境下的数据竞争问题。因此，在选择使用静态变量还是全局变量时，需要根据具体情况综合考虑。