51单片机移位不补0的错误与解决方案
问题
在尝试用移位寄存器控制8x8点阵LED运行流水灯时,我发现LED不止亮一行,而是包括这一行之前的所有行全都被点亮。
关键的问题代码如下:
u8 i;
for(i = 0; i < 8; i++)
{
SER = (buf << i) >> 7;
SRCLK = 0;
SRCLK = 1;
}
rCLK = 0;
rCLK = 1;
按我的思路,假如buf = 0x10
, 也就是二进制的0001 0000
。意思是第五行亮,其余行不亮。理论上代码运行过程如下:
i = 0
SER = 0001 0000 >> 7 = 0
i = 1
SER = 0010 0000 >> 7 = 0
i = 2
SER = 0100 0000 >> 7 = 0
i = 3
SER = 1000 0000 >> 7 = 1
i = 4
SER = 0000 0000 >> 7 = 0
i = 5
SER = 0000 0000 >> 7 = 0
i = 6
SER = 0000 0000 >> 7 = 0
i = 7
SER = 0000 0000 >> 7 = 0
也就是说只有当i = 3
时SER = 1
其余循环中SER均为零,正好意为意思是第五行亮,其余行不亮。但实际运行结果是i = 3, 4, 5, 6, 7
时SER都等于1。
测试
我将代码修改如下:
u8 i;
for(i = 0; i < 8; i++)
{
if((buf & (1 << (7 - i))) == 0)
{
SER = 0;
}
else
{
SER = 1;
}
SRCLK = 0;
SRCLK = 1;
}
rCLK = 0;
rCLK = 1;
问题解决。这次没有用到移位超范围补0的性质,而是直接用位运算比较每一位是否为1。于是我怀疑是位运算出了问题。
本人写了如下代码进行测试:
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
typedef unsigned char u8;
int main()
{
u8 i;
u8 SER;
u8 buf = 0x80;
char binary[9];
for(i = 0; i < 8; i++)
{
SER = (buf << i) >> 7;
itoa(SER, binary, 2);
printf("%s\n", binary);
}
printf("----------\n");
for(i = 0; i < 8; i++)
{
SER = buf << i;
itoa(SER, binary, 2);
printf("%s\t", binary);
SER = SER >> 7;
itoa(SER, binary, 2);
printf("%s\n", binary);
}
printf("----------\n");
SER = (buf << 10) >> 10;
itoa(SER, binary, 2);
printf("%s\n", binary);
printf("----------\n");
u8 tmp;
tmp = buf << 10;
SER = tmp >> 10;
itoa(tmp, binary, 2);
printf("tmp : %s\t", binary);
itoa(SER, binary, 2);
printf("SER : %s\n", binary);
}
程序输出如下:
1
10
100
1000
10000
100000
1000000
10000000
----------
10000000 1
0 0
0 0
0 0
0 0
0 0
0 0
0 0
----------
10000000
----------
tmp : 0 SER : 0
结论
我发现如果直接进行超出数据类型大小的左移,又在同一个表达式中右移回数据类型大小内,程序并不会把超范围的数据刷新成0,而是保留原有数字。
而如果先进行左移,保存结果到临时变量中,再进行右移。则超范围的部分会变为0。
通过这次bug,我们知道进行位运算时尽量不要做超数据范围的操作。还是正正规规用按位与、按位或的运算,才更稳定啊!
关于这次bug的原因,我认为是内存中移位操作可以移出该变量的空间,但并不会被置零,所以可以通过右移把变量找回来。也有可能是编译器在优化代码的时候把左移和右移抵消掉了,结果没有移位,也就没有补0。希望知道具体原因的大佬能评论一下!
完整代码
这段代码可以在普中51-实验板上直接运行。
#include "reg52.h"
#define LED_COL P0
typedef unsigned char u8;
typedef unsigned int u16;
sbit SRCLK = P3^6;
sbit rCLK = P3^5;
sbit SER = P3^4;
void delay(u16 t)
{
while(t--);
}
void hc595Write(u8 buf)
{
u8 i;
for(i = 0; i < 8; i++)
{
if((buf & (1 << (7 - i))) == 0)
{
SER = 0;
}
else
{
SER = 1;
}
//SER = (buf << i) >> 7;
SRCLK = 0;
SRCLK = 1;
}
rCLK = 0;
rCLK = 1;
}
void main()
{
u8 buf = 1;
LED_COL = 0;
while(1)
{
hc595Write(0);
hc595Write(buf);
if(buf == 0x80)
{
buf = 1;
}
else
{
buf = buf << 1;
}
delay(65535);
}
}