1

我对这个程序的目标是我希望能够将 1000 位分别设置为 0 或 1，为此我使用的是 128 位整数数组。

现在的问题是当我简单地 clear_bit(3) 时，第 35 位也被清除（反之亦然）。3 和 35 总是一起清除，4 和 36、5 和 37 等也是如此。所以显然有一个模式。我只想清除 3 而其余的保持 1。

任何想法为什么会发生这种情况？谢谢！

#include <stdio.h>

__uint128_t array [(1000/128) + 1];

// Set bit to 0.
void clear_bit(int k)                
{
    array[k/128] &= ~(1 << (k%128));
}

// Find the value of bit.
int test_bit(int k)
{
    return((array[k/128] & (1 << (k%128) )) != 0);     
}

// Set bit to 1.
void set_bit(int k)
{
    array[k/128] |= 1 << (k%128);  // Set the bit at the k-th position in A[i]
}

int main (void)
{
    // Set all bits to 1.
    for(int i = 0; i < 40; i++) {
        set_bit(i);
    }

    // I want to clear bit 3, but it also clears 35 for an unknown reason.
    clear_bit(3);

    for(int i = 0; i < 40; i++) {
        printf("%d is now:%d\n", i, test_bit(i));
    }

    return (0);
}

结果

4

2 回答 2

1

OP 的代码没有使用 128 位数学1 << (k%128)。 @user2357112

确保整数数学至少使用 128 位数学。

array[k/128] &= ~(1 << (k%128));
array[k/128] &= ~((__uint128_t)1 << (k%128));

或者，unsigned以便携式方式使用unsigned的位宽应为 16,32 64, 36 等。无需依赖__uint128_t.

#include <limits.h>
#define UNS_WIDTH (sizeof(unsigned)*CHAR_BIT)

unsigned array[(1000 + UNS_WIDTH - 1)/UNS_WIDTH];

void clear_bit(int k) {
  array[k/UNS_WIDTH] &= ~(1u << (k%UNS_WIDTH));
}

最好确保1与unsigned.1u

于 2017-09-30T17:23:56.177 回答
0

我建议使用平台寄存器的本机大小。只需比较生成的代码（在此示例中为 64 位平台） - https://godbolt.org/g/Y316vU

__uint128_t array [(1000/128) + 1];
uint64_t array1[(1000/64) + 1];

void assignBit64(unsigned bit, unsigned value)
{
    array1[bit >> 6] &= ~((uint64_t)1 << (bit & 63));
    array1[bit >> 6] |= ((uint64_t)!!value) << (bit & 63);
}

void assignBit128(unsigned bit, unsigned value)
{
    array[bit >> 7] &= ~((__uint128_t)1 << (bit & 127));
    array[bit >> 7] |= ((__uint128_t)(!!value)) << (bit & 127);
}


void resetBit64(unsigned bit)
{
    array1[bit >> 6] &= ~((uint64_t)1 << (bit & 63));    
}

void resetBit128(unsigned bit)
{
    array[bit >> 7] &= ~((__uint128_t)1 << (bit & 127));
}

void setBit64(unsigned bit)
{
    array1[bit >> 6] |= ((uint64_t)1 << (bit & 63));    
}

void setBit128(unsigned bit)
{
    array[bit >> 7] |= ((__uint128_t)1 << (bit & 127));
}
于 2017-09-30T17:39:28.697 回答