赞同楼主的说法，特别是第一条，第二条，尽量简化代码，不要在一行中有太多的表达式，不要在一个代码段实现太多的功能。多用函数分解代码。

不过楼主的最后一条则不太认同，使用宏来定义常量确实是C语言中常用的方法，不过现在的C语言规范里面通常是建议const来定义一个受限变量，这样可以使代码更清晰。我个人决定，宏定义还是少用的好。