C功能:返回值不正确？

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);
}

如你所见,我的函数中有print语句.在_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来调试它,它不会破坏你的逻辑.:^)