Question

我编写了一个C库函数，但返回值似乎不正确，即使它在函数中是正确的。

以下是相关代码：

（在dcml_private.c中）违规函数：

dcml_status _dcml_get_status(struct dcml_device *dev)
{
    uint64_t data;
    dcml_status ret;
    int len;

    libusb_bulk_transfer(dev->handle,
                     DCML_ENDPOINT | LIBUSB_ENDPOINT_IN,
                     (unsigned char *) &data,
                     DCML_REPORT_SZ, &len, RX_TIMEOUT);

    printf("data = %ld\n", data);

    if (len != DCML_REPORT_SZ)
        printf("DCML: LIBUSB ERROR (%s)\n", libusb_error_name(len));
        return STATUS_UNKNOWN;

    ret = data & ~(1>>17);
    return (ret);
}

调用函数：

void _dcml_cmd(dcml_context *ctx, dcml_cmd cmd,
      dcml_status quit_cond, int dur)
{
    struct timeval start;
    struct timeval cur;
    uint32_t stat;

    (void)gettimeofday(&start, NULL);
    (void)gettimeofday(&cur, NULL);
    _dcml_send_cmd(ctx->active, cmd);

    while(difftimeval(cur, start) < dur) {
            sleep(POLL_PERIOD);
            stat = _dcml_get_status(ctx->active);

            printf("status (%d), quit_cond (%d)", stat, quit_cond);
            if (stat == quit_cond)
                  break;

            (void)gettimeofday(&cur, NULL);
    }

    _dcml_send_cmd(ctx->active, CMD_NONE);
}

如您所见，我的函数中有打印语句。在_dcml_cmd中，该print语句的典型输出为

status (65535), quit_cond (2048)

_dcml_get_status打印的地方：

data = 128

这意味着退出值在退出_dcml_get_status之前立即正确，但在返回调用函数后立即出现错误（此处的值总是为65535 ......）

知道“dmcl_status”是一个枚举可能会有所帮助。将返回类型切换为uint16_t无法解决问题。我认为这可能是一个溢出问题或者其他问题，但更改类型，显式强制转换和添加掩码行并不能解决问题。

有什么想法吗？

Answer 1

这是因为你有一个坏习惯，就是不要在你的if语句后面放{}总是

   if (len != DCML_REPORT_SZ) {
        printf("DCML: LIBUSB ERROR (%s)\n", libusb_error_name(len));
        return STATUS_UNKNOWN;
   }

还使用fprintf(stderr,...来转储错误。向printf发送错误是不好的做法：

   if (len != DCML_REPORT_SZ) {
        fprintf(stderr, "DCML: LIBUSB ERROR (%s)\n", libusb_error_name(len));
        return STATUS_UNKNOWN;
   }

调用函数有同样的问题：

       printf("status (%d), quit_cond (%d)", stat, quit_cond);
        if (stat == quit_cond)
              break;

使用：

       fprintf(stderr, "status (%d), quit_cond (%d)", stat, quit_cond);
        if (stat == quit_cond) {
              break;
        }

是的，它会占用一条额外的线但是在凌晨3点通过在整个地方添加fprintf来调试它时它会使它不会破坏你的逻辑。：^）

C功能：返回值不正确？

1 个答案: