关于宏的副作用_应该多使用inline函数，少使用宏

今天写 rtc 驱动， rtc 寄存器使用了 bcd 编码， 于是写了一个宏进行转换：

#define FROMBCD8(val)   (((val >> 4)  & 0xf) * 10 + (val & 0xf))

 

然后为了获取某个寄存器，比如小时, 是这么调用的：

tm->tm_hour     = FROMBCD8(micco_read(RTC_HOUR_REG));

 

但是， micco_read 的打印语句表明该函数被调用了两次！

 

仔细研究原来这是由于宏的副作用导致的！

宏扩展后变为：

tm->tm_hour     =  (((micco_read(RTC_HOUR_REG)>> 4)  & 0xf) * 10 + (micco_read(RTC_HOUR_REG)& 0xf))

 

这导致了低效（有时是错误）的代码！micco_read 可是使用低速 i2c 在访问寄存器！

 

修改为：

u8 val;

val = micco_read(RTC_HOUR_REG);

tm->tm_hour     = FROMBCD8(val);

 

即可！

 

该实例，表明宏的缺陷！正确的方法是应该使用 inline 函数， 少使用宏。

上面的宏改为：

static u8 inline frombcd8(u8 val)

{

        return ((val >> 4)  & 0xf) * 10 + (val & 0xf);

}

 

然后， 调用改为：

tm->tm_hour     = frombcd8 (micco_read(RTC_HOUR_REG));

也没有问题， 而且代码更加简洁。