Question

我遇到的问题是，以下函数中的 data 数组有一些糟糕的值（在我看来就像是一些内存位置）：

int
GPIO::GetValue() {
    char data[1];

    if (read(_valuefd, data, 1) < 0) {
        perror("Error on reading value fd");
        return -1;
    }

    printf("int GPIO::GetValue() %s\n", data);

    if (strcmp(data, "1") == 0) {
        return GPIO_VALUE_ON;
    }
    if (strcmp(data, "0") == 0) {
        return GPIO_VALUE_OFF;
    }

    return -1;
}

Full Source

printf 的结果：

int GPIO::GetValue() 0cx$??ݾ??˶8@l

我不知道这会出现什么问题。我在一些简单的程序中提取相同的代码，它可以正常工作。还有一些其他函数 GPIO :: GetDirection 也可以做同样的事情，也可以正常工作。我猜有一些内存，指针，分配问题。

出了什么问题？

博多

Answer 1

我认为你的结果是正确的。只需null终止字符串data。

char data[2];
data[1] = '\0';

实际上，您不需要声明数组。只需char data;即可。那么您可能需要进行以下更改：

char data;
if (read(_valuefd, &data, 1) < 0) {
        perror("Error on reading value fd");
        return -1;
    }

printf("int GPIO::GetValue() %c\n", data);

if (data == '1') {
    return GPIO_VALUE_ON;
}
else if (data == '0') {
    return GPIO_VALUE_OFF;
}
else {
    return -1;
}

Answer 2

'data'数组长1个字节。如果要将其打印为字符串，则必须以'\ 0'结束。或者，尝试使用％c而不是％s。

Answer 3

printf("int GPIO::GetValue() %s\n", data);您尝试显示char *。但是由于你的数组大小为1，printf不知道何时停止阅读，因为他找不到'\ 0'。

printf("int GPIO::GetValue() %c\n", data[0]);如果您使用1号数组

你的strcmp可能会失败，请尝试使用大小为1的strncmp

read（）缓冲区有无效数据（指针问题？）

3 个答案: