TaterLi's LazyBlog

自言自语,不喜绕路,科学上网,远离天国.

@TaterLi2月前

01/5
22:05
技术控

注意C语言define也要考虑运算优先级.

下面两句define有什么区别,可能大家一开始也没看出来.

#define LCD_SetPoint(X,Y,Color) *((uint32_t *)LCD_VRAM_BASE + (Y) * C_GLCD_H_SIZE + (X)) = Color
#define LCD_SetPoint(X,Y,Color) *((uint32_t *)LCD_VRAM_BASE + Y * C_GLCD_H_SIZE + X) = Color

觉得结果都是一样啊,但是套用某些情况下,就变味了.比如如下函数.

/**
  * @brief  Draws a vertical line.
  * @param  Xpos: X position
  * @param  Ypos: Y position
  * @param  Length: Line length
  * @retval None
  */
void LCD_DrawVLine(uint16_t Xpos, uint16_t Ypos, uint16_t Length)
{
	uint16_t i = 0;
	for(i = 0;i<Length;i++){
		LCD_SetPoint(Xpos,Ypos + i,DrawProp.TextColor);
	}
}

如果用第二个define,那么坐标就不对了,因为展开后,Ypos+i的执行,是i*C_GLCD_H_SIZE,然后再加Ypos,与原来的思想有偏移了,这是运算优先级坑啊,编译器是不会报警告的.

注意C语言define也要考虑运算优先级.