这几天要做一个ADC采集功能相关的任务,调试过程当中犯了个几个低级错误,记录一下,希望大家也能避免。
判断范围
问题描述
我用了它原来的接口,ADC转换出来的数值是0-512,原接口如下。
#define NORMAL_V_L 60
#define NORMAL_V_H 110
#define SHORT_V_L 200
#define SHORT_V_H 340
uint8_t Get_XXX_Status()
{
uint8_t tmp_Status = STATUS_NORMAL;
uint16_t tmp_AD = 0;
tmp_AD = ADC_XXX_Value();
if((tmp_AD >= NORMAL_V_L) && (tmp_AD <= NORMAL_V_H))
{
tmp_Status = STATUS_NORMAL;
}
else if((tmp_AD >= SHORT_V_L) && (tmp_AD <= SHORT_V_H))
{
tmp_Status = STATUS_SHORT;
}
return tmp_Status;
}
里面只判断值从60-120和200-340之间的,所以导致输出的状态很多时候都是没经过判断直接输出的STATUS_NORMAL,这个STATUS_NORMAL还跟判断过后的STATUS_NORMAL是一样的,看不出来是判断过的还是没判断过的。
修改方法
1、设置一个默认状态,区分开默认状态和判断后的状态,接口外面如果获取到默认状态就不进行处理。
2、设置的AD值范围覆盖到全部。
连续采集
以下这段代码,Get_ADC已经是滤波之后的了。
if(Num != Get_ADC()){
Num = Get_ADC();
……
printf("Get_ADC:%d\r\n",Get_ADC());
}
里面有两个问题。一个是每次都调用获取ADC的接口去获取底层的值,下面又要采集又要转换的,非常耗时间。另一个是由于时间差,获取到的信号变化快,判断时候获取到的值、赋值给Num的值、打印出来的值,都不一样。
修改方法
1、使用一个临时buff去记忆,每次只获取一次ADC的值。
2、对比、赋值、打印,都用这个临时buff。