我正在尝试在C中实现位图数组。
我已经阅读并复制了以下链接:What is a bitmap in C?
#include <limits.h> /* for CHAR_BIT */
#include <stdint.h> /* for uint32_t */
#include <stdio.h>
#include <stdlib.h>
typedef uint32_t word_t; // I want to change this, from uint32_t to uint64_t
enum { BITS_PER_WORD = sizeof(word_t) * CHAR_BIT };
#define WORD_OFFSET(b) ((b) / BITS_PER_WORD)
#define BIT_OFFSET(b) ((b) % BITS_PER_WORD)
void set_bit(word_t *words, int n) {
words[WORD_OFFSET(n)] |= (1 << BIT_OFFSET(n));
}
void clear_bit(word_t *words, int n) {
words[WORD_OFFSET(n)] &= ~(1 << BIT_OFFSET(n));
}
int get_bit(word_t *words, int n) {
word_t bit = words[WORD_OFFSET(n)] & (1 << BIT_OFFSET(n));
return bit != 0;
}
int main(){
printf("sizeof(word_t)=%i\n",sizeof(word_t));
printf("CHAR_BIT=%i\n",CHAR_BIT);
printf("BITS_PER_WORD=%i\n",BITS_PER_WORD);
word_t x;
set_bit(&x, 0);
printf("x=%u\n",x);
set_bit(&x, 1);
printf("x=%u\n",x);
set_bit(&x, 2);
printf("x=%u\n",x);
return 0;
}
使用uint32_t,代码运行良好。它分别输出x值:1、3和7,如下所示:
[izzatul@mycomputer latihan]$ ./a.out
sizeof(word_t)=8
CHAR_BIT=8
BITS_PER_WORD=64
x=1
x=3
x = 7
没用x值变为1295807169等,这不是我预期的。我希望它和以前一样(1、3、7)。有人可以帮我修复该代码吗?
我知道“ <但是我仍然不确定自己如何修改代码。
最佳答案
问题在于代码使用1
整数常量。所有这些整数常量都具有与变量一样的类型,并且默认情况下为int
,这可能与系统上的int32_t
相同。
将像int32_t
这样的有符号整数左移30个以上位会调用未定义的行为,因为您将数据移入符号位。根据经验,切勿将带符号变量与按位运算符一起使用。
在这种情况下,正确的解决方法是将1 << BIT_OFFSET(n)
的每个实例替换为:
(word_t)1 << BIT_OFFSET(n)
或者使用
1ull
后缀,但是在较小的系统上可能会产生不必要的慢速代码。请注意,printf的正确格式说明符是inttypes.h中的
printf("x=%"PRIu64 "\n",x);
。