c - 程序不扫描数字 149，对于任何其他数字，它返回 0

Question

已关闭。这个问题是 not reproducible or was caused by typos 。目前不接受答案。

---

这个问题是由拼写错误或无法再重现的问题引起的。虽然类似的问题可能是 on-topic在这里，这个问题的解决方式不太可能帮助 future 的读者。

已关闭 4 年前。

Improve this question

我的程序应该扫描某个数字(unsigned long int)并返回其奇数“打开”索引位(来自数字的二进制表示形式)。但程序不识别任何数字，只返回 0，或者根本不响应该数字。我做错了什么？

这是函数:

int count_odd_bits(unsigned long int x) {

    int count = 0;
    while (x) {
        if ((x % 2 == 1) && (x & 1))
            count++;
        else
            x = x << 1;

    }
    return count;
}

这是主要功能:

int main() {

    unsigned long int x;
    printf("\n enter a number: \n");
    scanf("%ul", &x);
    int count_odd_bits(unsigned long int x);
    printf("\n the result is:%d \n",count_odd_bits(x));

    return 0;
}

对于数字 149，它应该返回 1(仅打开第 7 位)

Answer 1

在函数中，当 if 为 true 时，您不会更改 x。所以你最终会陷入无限循环。

int count_odd_bits(unsigned long int x) {

    int count = 0;
    while (x) {
        if ((x % 2 == 1) && (x & 1))
            count++;                     // x not changed --> endless loop!!
        else
            x = x << 1;

    }
    return count;
}

此外，您似乎需要 x = x >> 1; 而不是当前代码。

此外，您不需要同时使用 x % 2 == 1 和 x & 1，因为它们是相同的。

所以计算个数的代码可以是:

int count_odd_bits(unsigned long int x) {

    int count = 0;
    while (x) {
        if (x & 1) count++;
        x = x >> 1;
    }
    return count;
}

如果您只想计算奇数位位置，请执行

int count_odd_bits(unsigned long int x) {

    int count = 0;
    x = x >> 1;     // get rid of bit zero
    while (x) {
        if (x & 1) count++;
        x = x >> 2;    // shift by 2 to get 1->3->5->....
    }
    return count;
}