原子哥DAC 小bug的分析和我的浅见

hot008

关于这个小bug，之前有几个帖子介绍了，我分享一下我的想法。

原子哥的DAC.c文件里面DA输出程序是：
void DAC1_Set_Vol(u16 vol)
{
        double temp=vol;
        temp/=1000;
        temp=temp*4096/3.3;
    HAL_DAC_SetValue(&DAC1_Handler,DAC_CHANNEL_1,DAC_ALIGN_12B_R,temp);//12位右对齐数据格式设置DAC值
}


这里有个潜在的风险：
如果vol=3300的话，temp就是4096，十六进制表示就是0x1000。由于选择的模式是DAC_ALIGN_12B_R，此时就会将b（0000 0000 0000）写到寄存器里面，从而导致输出值为零。


我将程序改为：
void DAC1_Set_Vol(u16 vol)
{
        double temp=vol;
        if(temp>3300)temp=3300;
        if(temp<0)temp=0;
        temp/=1000;
        temp=temp*4095/3.3;
    HAL_DAC_SetValue(&DAC1_Handler,DAC_CHANNEL_1,DAC_ALIGN_12B_R,temp);//12位右对齐数据格式设置DAC值
}


这样输出就正常了，而且增强了程序的健壮性。

当然了，在原子哥的例程里面，并不会产生错误的后果，因为vol不会等于3300.

欢迎交流，谢绝攻击。